Skip to content

Conversation

@c0state
Copy link
Contributor

@c0state c0state commented Jan 3, 2026

Updates for native ESM support

Depends on:

@c0state c0state marked this pull request as ready for review January 5, 2026 21:32
{
headers: {
'Accept': 'application/json',
'Content-Type': 'application/json',
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a response header, not a request header--which is what this is matching


it('should handle files', async () => {
mockedFetch.get(
fetchMock.post(
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This test was changed to a POST as GETs shouldn't have a body

ts-jest transpilation addes newline/spacing
const value = new Bar();

expect(() => asType(value, Foo)).toThrow(
'"Bar" is not an instance of class Foo {}',
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The braces get re-formatted (newline) when run in ts-jest it looks like, so just removing that part from the asssertion.

@c0state c0state merged commit 3cec7e4 into master Jan 5, 2026
3 checks passed
@c0state c0state deleted the chore/convert_esm branch January 5, 2026 23:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants