Skip to content

Commit d0f71cf

Browse files
committed
chore: make typescript happy
response/request object may become undefined after the changes, but in our test case we can always assert they do exists.
1 parent 2eb77fb commit d0f71cf

3 files changed

Lines changed: 14 additions & 14 deletions

File tree

packages/openapi-ts/src/plugins/@hey-api/client-fetch/__tests__/client.test.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -308,8 +308,8 @@ describe('unserialized request body handling', () => {
308308
}),
309309
);
310310

311-
await expect(result.request.text()).resolves.toEqual(textValue);
312-
expect(result.request.headers.get('Content-Type')).toEqual('text/plain');
311+
await expect(result.request!.text()).resolves.toEqual(textValue);
312+
expect(result.request!.headers.get('Content-Type')).toEqual('text/plain');
313313
});
314314
});
315315

@@ -375,8 +375,8 @@ describe('serialized request body handling', () => {
375375
}),
376376
);
377377

378-
await expect(result.request.text()).resolves.toEqual(textValue);
379-
expect(result.request.headers.get('Content-Type')).toEqual(
378+
await expect(result.request!.text()).resolves.toEqual(textValue);
379+
expect(result.request!.headers.get('Content-Type')).toEqual(
380380
expectContentHeader ? 'application/json' : null,
381381
);
382382
},

packages/openapi-ts/src/plugins/@hey-api/client-ky/__tests__/client.test.ts

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -308,8 +308,8 @@ describe('unserialized request body handling', () => {
308308
expect.any(Object),
309309
);
310310

311-
await expect(result.request.text()).resolves.toEqual(textValue);
312-
expect(result.request.headers.get('Content-Type')).toEqual('text/plain');
311+
await expect(result.request!.text()).resolves.toEqual(textValue);
312+
expect(result.request!.headers.get('Content-Type')).toEqual('text/plain');
313313
});
314314
});
315315

@@ -376,8 +376,8 @@ describe('serialized request body handling', () => {
376376
expect.any(Object),
377377
);
378378

379-
await expect(result.request.text()).resolves.toEqual(textValue);
380-
expect(result.request.headers.get('Content-Type')).toEqual(
379+
await expect(result.request!.text()).resolves.toEqual(textValue);
380+
expect(result.request!.headers.get('Content-Type')).toEqual(
381381
expectContentHeader ? 'application/json' : null,
382382
);
383383
},
@@ -499,7 +499,7 @@ describe('error handling', () => {
499499
});
500500

501501
expect(result.error).toEqual({ message: 'Not found' });
502-
expect(result.response.status).toBe(404);
502+
expect(result.response!.status).toBe(404);
503503
});
504504

505505
it('throws HTTP errors with throwOnError: true', async () => {
@@ -543,7 +543,7 @@ describe('error handling', () => {
543543
});
544544

545545
expect(result.error).toBe('Internal Server Error');
546-
expect(result.response.status).toBe(500);
546+
expect(result.response!.status).toBe(500);
547547
});
548548
});
549549

packages/openapi-ts/src/plugins/@hey-api/client-ofetch/__tests__/client.test.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -269,8 +269,8 @@ describe('unserialized request body handling', () => {
269269
url: '/test',
270270
});
271271

272-
await expect(result.request.text()).resolves.toEqual(textValue);
273-
expect(result.request.headers.get('Content-Type')).toEqual('text/plain');
272+
await expect(result.request!.text()).resolves.toEqual(textValue);
273+
expect(result.request!.headers.get('Content-Type')).toEqual('text/plain');
274274
});
275275
});
276276

@@ -331,8 +331,8 @@ describe('serialized request body handling', () => {
331331
});
332332

333333
// Ensure request captures serialized text value
334-
await expect(result.request.text()).resolves.toEqual(textValue);
335-
expect(result.request.headers.get('Content-Type')).toEqual(
334+
await expect(result.request!.text()).resolves.toEqual(textValue);
335+
expect(result.request!.headers.get('Content-Type')).toEqual(
336336
expectContentHeader ? 'application/json' : null,
337337
);
338338

0 commit comments

Comments
 (0)