@usebruno/filestore 0.8.0 → 0.8.1

package/dist/formats/bru/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/formats/bru/index.ts"],"names":[],"mappings":"AAWA,eAAO,MAAM,eAAe,SAAU,MAAM,GAAG,GAAG,WAAU,OAAO,KAAW,GAuG7E,CAAC;AAEF,eAAO,MAAM,mBAAmB,SAAU,GAAG,KAAG,MA+G/C,CAAC;AAEF,eAAO,MAAM,kBAAkB,SAAU,MAAM,GAAG,GAAG,WAAU,OAAO,KAAW,GA4ChF,CAAC;AAEF,eAAO,MAAM,sBAAsB,SAAU,GAAG,aAAa,OAAO,KAAG,MAuCtE,CAAC;AAEF,eAAO,MAAM,mBAAmB,QAAS,MAAM,KAAG,GAejD,CAAC;AAEF,eAAO,MAAM,uBAAuB,SAAU,GAAG,KAAG,MAOnD,CAAC;AAGF,eAAO,MAAM,gBAAgB,SAAU,MAAM,GAAG,GAAG,WAAU,OAAO,eAAuB,MAAM,iBAAiB,MAAM,KAAG,GA6D1H,CAAC;AAEF,eAAO,MAAM,gBAAgB,SAAU,GAAG;;;;;;;;;;;CAsBzC,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/formats/bru/index.ts"],"names":[],"mappings":"AAWA,eAAO,MAAM,eAAe,SAAU,MAAM,GAAG,GAAG,WAAU,OAAO,KAAW,GAuG7E,CAAC;AAEF,eAAO,MAAM,mBAAmB,SAAU,GAAG,KAAG,MA+G/C,CAAC;AAEF,eAAO,MAAM,kBAAkB,SAAU,MAAM,GAAG,GAAG,WAAU,OAAO,KAAW,GA4ChF,CAAC;AAEF,eAAO,MAAM,sBAAsB,SAAU,GAAG,aAAa,OAAO,KAAG,MAuCtE,CAAC;AAEF,eAAO,MAAM,mBAAmB,QAAS,MAAM,KAAG,GAejD,CAAC;AAEF,eAAO,MAAM,uBAAuB,SAAU,GAAG,KAAG,MAOnD,CAAC;AAGF,eAAO,MAAM,gBAAgB,SAAU,MAAM,GAAG,GAAG,WAAU,OAAO,eAAuB,MAAM,iBAAiB,MAAM,KAAG,GAwE1H,CAAC;AAEF,eAAO,MAAM,gBAAgB,SAAU,GAAG;;;;;;;;;;;CAsBzC,CAAC"}

package/dist/formats/bru/tests/example-status-swap.spec.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=example-status-swap.spec.d.ts.map

package/dist/formats/bru/tests/example-status-swap.spec.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"example-status-swap.spec.d.ts","sourceRoot":"","sources":["../../../../src/formats/bru/tests/example-status-swap.spec.js"],"names":[],"mappings":""}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@usebruno/filestore",
-  "version": "0.8.0",
+  "version": "0.8.1",
   "license": "MIT",
   "main": "dist/cjs/index.js",
   "module": "dist/esm/index.js",

package/src/formats/bru/index.ts

@@ -367,6 +367,17 @@ export const bruExampleToJson = (data: string | any, parsed: boolean = false, pa
         transformedType = 'http-request';
     }
 
+    /**
+     * Backward compatibility (pre-v3.0.2 - v3.2.0): Postman imports before PR #6876 stored status/statusText swapped
+     * (code: "OK", text: "202" instead of code: 202, text: "OK"). Detect and swap back.
+     * TODO(Sid / Shubh): Remove after v5 — all collections should be migrated by then.
+     */
+    let status = _.get(json, 'response.status', '200');
+    let statusText = _.get(json, 'response.statusText', 'OK');
+    if (isNaN(Number(status)) && !isNaN(Number(statusText))) {
+      [status, statusText] = [statusText, status];
+    }
+
     // Follow the same structure as the main request, but with missing fields for examples
     const transformedJson = {
       type: transformedType,
@@ -388,8 +399,8 @@ export const bruExampleToJson = (data: string | any, parsed: boolean = false, pa
           name: header.name,
           value: header.value
         })),
-        status: String(_.get(json, 'response.status', '200')),
-        statusText: _.get(json, 'response.statusText', 'OK'),
+        status: Number(status) || 200,
+        statusText: statusText || 'OK',
         body: {
           type: _.get(json, 'response.body.type', 'json'),
           content: _.get(json, 'response.body.content', '')

package/src/formats/bru/tests/example-status-swap.spec.js

@@ -0,0 +1,109 @@
+const { bruExampleToJson } = require('../index');
+
+describe('bruExampleToJson - status/statusText swap fix', () => {
+  it('should parse normal status and statusText correctly', () => {
+    const parsed = {
+      name: 'Normal Example',
+      description: '',
+      request: { url: 'https://api.example.com/test', method: 'get' },
+      response: {
+        headers: [],
+        status: '200',
+        statusText: 'OK',
+        body: { type: 'json', content: '{}' }
+      }
+    };
+
+    const result = bruExampleToJson(parsed, true, 'http', 'GET');
+    expect(result.response.status).toBe(200);
+    expect(result.response.statusText).toBe('OK');
+  });
+
+  it('should swap back status and statusText when they are reversed (pre-fix Postman import)', () => {
+    const parsed = {
+      name: 'Swapped Example',
+      description: '',
+      request: { url: 'https://api.example.com/test', method: 'get' },
+      response: {
+        headers: [],
+        status: 'Accepted',
+        statusText: '202',
+        body: { type: 'json', content: '{}' }
+      }
+    };
+
+    const result = bruExampleToJson(parsed, true, 'http', 'GET');
+    expect(result.response.status).toBe(202);
+    expect(result.response.statusText).toBe('Accepted');
+  });
+
+  it('should swap back status OK and statusText 200', () => {
+    const parsed = {
+      name: 'Swapped OK Example',
+      description: '',
+      request: { url: 'https://api.example.com/test', method: 'get' },
+      response: {
+        headers: [],
+        status: 'OK',
+        statusText: '200',
+        body: { type: 'json', content: '{}' }
+      }
+    };
+
+    const result = bruExampleToJson(parsed, true, 'http', 'GET');
+    expect(result.response.status).toBe(200);
+    expect(result.response.statusText).toBe('OK');
+  });
+
+  it('should swap back status Not Found and statusText 404', () => {
+    const parsed = {
+      name: 'Swapped Not Found Example',
+      description: '',
+      request: { url: 'https://api.example.com/test', method: 'get' },
+      response: {
+        headers: [],
+        status: 'Not Found',
+        statusText: '404',
+        body: { type: 'json', content: '{}' }
+      }
+    };
+
+    const result = bruExampleToJson(parsed, true, 'http', 'GET');
+    expect(result.response.status).toBe(404);
+    expect(result.response.statusText).toBe('Not Found');
+  });
+
+  it('should not swap when status is already numeric and statusText is text (correct order)', () => {
+    const parsed = {
+      name: 'Correct Order Example',
+      description: '',
+      request: { url: 'https://api.example.com/test', method: 'get' },
+      response: {
+        headers: [],
+        status: '404',
+        statusText: 'Not Found',
+        body: { type: 'json', content: '{}' }
+      }
+    };
+
+    const result = bruExampleToJson(parsed, true, 'http', 'GET');
+    expect(result.response.status).toBe(404);
+    expect(result.response.statusText).toBe('Not Found');
+  });
+
+  it('should use defaults when response has no status', () => {
+    const parsed = {
+      name: 'No Status Example',
+      description: '',
+      request: { url: 'https://api.example.com/test', method: 'get' },
+      response: {
+        headers: [],
+        body: { type: 'json', content: '{}' }
+      }
+    };
+
+    const result = bruExampleToJson(parsed, true, 'http', 'GET');
+    expect(result.response.status).toBe(200);
+    expect(result.response.statusText).toBe('OK');
+  });
+});