AAE-30882 Simplify typing of methods

This commit is contained in:
Wojciech Duda 2025-02-18 13:38:10 +01:00
parent e50a3f04ce
commit 7f4a46f4c3
2 changed files with 8 additions and 36 deletions

View File

@ -45,20 +45,12 @@ export class SuperagentHttpClient implements HttpClient {
}
request<T = any>(url: string, options: RequestOptions, securityOptions: SecurityOptions, emitters: Emitters): Promise<T> {
const { httpMethod, queryParams, headerParams, formParams, bodyParam, contentType, accept, responseType, returnType } = options;
const { returnType } = options;
const { eventEmitter, apiClientEmitter } = emitters;
const { urlWithParams, fetchOptions } = this.buildRequest({
httpMethod,
...options,
url,
queryParams,
headerParams,
formParams,
bodyParam,
contentType,
accept,
responseType,
returnType,
securityOptions
});
@ -120,19 +112,7 @@ export class SuperagentHttpClient implements HttpClient {
responseType,
returnType,
securityOptions
}: {
httpMethod: string;
url: string;
queryParams: { [key: string]: any };
headerParams: { [key: string]: any };
formParams: { [key: string]: any };
bodyParam: string | object;
contentType: string;
accept: string;
responseType: string;
returnType: string;
securityOptions: SecurityOptions;
}) {
}: RequestOptions & { securityOptions: SecurityOptions }) {
const urlWithParams = new URL(url);
urlWithParams.search = new URLSearchParams(SuperagentHttpClient.normalizeParams(queryParams)).toString();

View File

@ -27,7 +27,7 @@ jest.mock('ofetch', () => ({
jest.mock('../src/utils', () => ({
isBrowser: jest.fn(() => true),
paramToString: (param: any) => String(param)
paramToString: (param: unknown) => String(param)
}));
describe('SuperagentHttpClient', () => {
@ -100,14 +100,10 @@ describe('SuperagentHttpClient', () => {
const securityOptions = { ...defaultSecurityOptions };
const request = client['buildRequest']({
...options,
path: '',
httpMethod,
url,
queryParams: options.queryParams,
headerParams: options.headerParams,
formParams: options.formParams,
contentType: options.contentType,
accept: options.accept,
responseType: options.responseType,
bodyParam: null,
returnType: null,
securityOptions
@ -134,14 +130,10 @@ describe('SuperagentHttpClient', () => {
(isBrowser as jest.Mock).mockReturnValue(false);
const request = client['buildRequest']({
...options,
path: '',
httpMethod,
url,
queryParams: options.queryParams,
headerParams: options.headerParams,
formParams: options.formParams,
contentType: options.contentType,
accept: options.accept,
responseType: options.responseType,
bodyParam: null,
returnType: null,
securityOptions