Support schemas with body encoding #15

shinzui · 2023-04-11T04:32:16Z

Changes

I updated the request/response inference to support schemas whose content types have an encoding set to utf8 (application/json;charset=utf-8).

I modified the test schema to have a failing test.

drwpow · 2023-04-11T15:44:47Z

I’m OK with this change. But in the test YAML file, could you add an additional test rather than modifying the existing one? Thanks!

shinzui · 2023-04-11T16:35:05Z

II'mOK with this change. But in the test YAML file, could you add an additional test rather than modifying the existing one?

You're right. I should've added a new test. I just pushed a fix.

drwpow · 2023-04-11T16:55:37Z

src/index.test.ts

@@ -204,6 +204,28 @@ describe('post()', () => {
    expect(error).toBe(undefined);
  });

+  it('supports sepecifying utf-8 encoding', async () => {


Support setting body encoding

ed43ec8

shinzui changed the title ~~Support setting body encoding~~ Support schemas with body encoding Apr 11, 2023

drwpow approved these changes Apr 11, 2023

View reviewed changes

shinzui added 2 commits April 11, 2023 08:47

Format test schema

a18d47e

Add new test for testing schemas that sepecify utf-8 encoding

4a8ac40

drwpow reviewed Apr 11, 2023

View reviewed changes

drwpow merged commit fce1546 into drwpow:main Apr 11, 2023