Fix default http client (readmeio#201)

AdrianMachado · kanadgupta · web-flow · commit 2c397e5d5089 · 2023-09-21T10:30:53.000-05:00
* Fix default http client

* test: fix snapshot

* test: add comprehensive test to check all targets

* refactor: slightly better test pattern

---------

Co-authored-by: Kanad Gupta &lt;git@kanad.dev&gt;
diff --git a/src/helpers/__snapshots__/utils.test.ts.snap b/src/helpers/__snapshots__/utils.test.ts.snap
@@ -78,7 +78,7 @@ exports[`availableTargets > returns all available targets 1`] = `
         "title": "HTTP/1.1",
       },
     ],
-    "default": "1.1",
+    "default": "http1.1",
     "key": "http",
     "title": "HTTP",
   },
diff --git a/src/helpers/utils.test.ts b/src/helpers/utils.test.ts
@@ -6,6 +6,15 @@ describe('availableTargets', () => {
   it('returns all available targets', () => {
     expect(availableTargets()).toMatchSnapshot();
   });
+
+  describe('default value check', () => {
+    it.each(availableTargets().map(target => [target.title, target]))(
+      'should match `default` value with one of the client keys (%s)',
+      (_, target) => {
+        expect(target.clients).toContainEqual(expect.objectContaining({ key: target.default }));
+      },
+    );
+  });
 });
 
 describe('extname', () => {
diff --git a/src/targets/http/target.ts b/src/targets/http/target.ts
@@ -6,7 +6,7 @@ export const http: Target = {
   info: {
     key: 'http',
     title: 'HTTP',
-    default: '1.1',
+    default: 'http1.1',
   },
   clientsById: {
     'http1.1': http11,
