@mimik/request-retry 4.0.7 → 4.0.8

package/.husky/pre-commit

@@ -1,4 +1,2 @@
 #!/bin/sh
-. "$(dirname "$0")/_/husky.sh"
-
 npm run commit-ready

package/.husky/pre-push

@@ -1,4 +1,2 @@
 #!/bin/sh
-. "$(dirname "$0")/_/husky.sh"
-
 npm run test

package/README.md

@@ -9,10 +9,11 @@ import rp from '@mimik/request-retry';
 ```
 <a name="module_request-retry..rpRetry"></a>
 
-### request-retry~rpRetry(options) ⇒ <code>Promise</code>
+### request-retry~rpRetry(options) ⇒ <code>Promise.&lt;object&gt;</code>
 Make a request with retries.
 
 **Kind**: inner method of [<code>request-retry</code>](#module_request-retry)  
+**Returns**: <code>Promise.&lt;object&gt;</code> - A bluebird promise (supports cancellation).  
 **Category**: async  
 **Throws**:
 

package/eslint.config.js

@@ -1,3 +1,4 @@
+import globals from 'globals';
 import importPlugin from 'eslint-plugin-import';
 import js from '@eslint/js';
 import processDoc from '@mimik/eslint-plugin-document-env';
@@ -10,13 +11,13 @@ const MAX_LINES_IN_FUNCTION = 150;
 const MAX_STATEMENTS_IN_FUNCTION = 45;
 const MIN_KEYS_IN_OBJECT = 10;
 const MAX_COMPLEXITY = 30;
-const ECMA_VERSION = 2022;
+const ECMA_VERSION = 'latest';
 const MAX_DEPTH = 6;
 const ALLOWED_CONSTANTS = [0, 1, -1];
 
 export default [
   {
-    ignores: ['mochawesome-report/**', 'node_modules/**', 'dist/**'],
+    ignores: ['coverage/**', 'mochawesome-report/**', 'node_modules/**', 'dist/**'],
   },
   importPlugin.flatConfigs.recommended,
   stylistic.configs.recommended,
@@ -28,10 +29,8 @@ export default [
     languageOptions: {
       ecmaVersion: ECMA_VERSION,
       globals: {
-        console: 'readonly',
-        describe: 'readonly',
-        it: 'readonly',
-        require: 'readonly',
+        ...globals.mocha,
+        ...globals.nodeBuiltin,
       },
       sourceType: 'module',
     },
@@ -39,6 +38,7 @@ export default [
       '@stylistic/brace-style': ['warn', 'stroustrup', { allowSingleLine: true }],
       '@stylistic/line-comment-position': ['off'],
       '@stylistic/max-len': ['warn', MAX_LENGTH_LINE, { ignoreComments: true, ignoreStrings: true, ignoreRegExpLiterals: true }],
+      '@stylistic/quotes': ['warn', 'single'],
       '@stylistic/semi': ['error', 'always'],
       'capitalized-comments': ['off'],
       'complexity': ['error', MAX_COMPLEXITY],
@@ -49,19 +49,20 @@ export default [
       'init-declarations': ['off'],
       'linebreak-style': ['off'],
       'max-depth': ['error', MAX_DEPTH],
+      'max-len': ['off'],
       'max-lines': ['warn', { max: MAX_LINES_IN_FILES, skipComments: true, skipBlankLines: true }],
       'max-lines-per-function': ['warn', { max: MAX_LINES_IN_FUNCTION, skipComments: true, skipBlankLines: true }],
       'max-params': ['error', MAX_FUNCTION_PARAMETERS],
       'max-statements': ['warn', MAX_STATEMENTS_IN_FUNCTION],
       'no-confusing-arrow': ['off'],
       'no-inline-comments': ['off'],
-      'no-magic-numbers': ['error', { ignore: ALLOWED_CONSTANTS, enforceConst: true, detectObjects: true }],
+      'no-magic-numbers': ['error', { ignore: ALLOWED_CONSTANTS, enforceConst: true, detectObjects: false }],
       'no-process-env': ['error'],
       'no-ternary': ['off'],
       'no-undefined': ['off'],
       'one-var': ['error', 'never'],
       'processDoc/validate-document-env': ['error'],
-      'quotes': ['warn', 'single'],
+      'quotes': ['off'],
       'sort-imports': ['error', { allowSeparatedGroups: true }],
       'sort-keys': ['error', 'asc', { caseSensitive: true, minKeys: MIN_KEYS_IN_OBJECT, natural: false, allowLineSeparatedGroups: true }],
     },

package/index.js

@@ -49,7 +49,7 @@ Promise.config({ cancellation: true });
  * The `validateStatus` option is disabled: an error is thrown for status codes outside **[200, 300)**.
  * An additional option, `resolveWithFullResponse`, controls the return shape:
  * when `true`, the full Axios response object is returned; when `false` or omitted, only `response.data` is returned.
- * @return {Promise}
+ * @return {Promise<object>} A bluebird promise (supports cancellation).
  * @fulfil {object} - The Axios response when `resolveWithFullResponse` is `true`; otherwise `response.data`.
  * @throws {Error} An error produced by `getRichError`, wrapping an
  * [Axios](https://www.npmjs.com/package/axios) error or a `TimeoutError`.

package/manual-test/retryMockMan.js

@@ -32,9 +32,9 @@ app.use(bodyParser.json());
 app.get(`${config.base}${config.path}`, (req, res) => {
   if (nbRequest === 0) message = 'Received first request';
   else {
-    const timeLap = Date.now() - time;
+    const timeLapse = Date.now() - time;
 
-    message = `Received ${nbRequest} retry with timelapse ${timeLap}`;
+    message = `Received ${nbRequest} retry with timelapse ${timeLapse}`;
   }
   console.log('----->', message);
   time = Date.now();

package/package.json

@@ -1,14 +1,14 @@
 {
   "name": "@mimik/request-retry",
-  "version": "4.0.7",
+  "version": "4.0.8",
   "description": "Request retry wrapping axios",
   "main": "./index.js",
   "type": "module",
   "scripts": {
     "lint": "eslint . --no-error-on-unmatched-pattern",
     "docs": "jsdoc2md index.js > README.md",
-    "test": "mocha --reporter mochawesome --bail --check-leaks test/",
-    "test-ci": "c8 --reporter=lcov --reporter=text npm test --exit",
+    "test": "mocha --reporter mochawesome --bail --check-leaks --exit test/",
+    "test-ci": "c8 --reporter=lcov --reporter=text npm test",
     "prepublishOnly": "npm run docs && npm run lint && npm run test-ci",
     "commit-ready": "npm run docs && npm run lint && npm run test-ci"
   },
@@ -29,23 +29,23 @@
     "url": "https://bitbucket.org/mimiktech/request-retry"
   },
   "dependencies": {
-    "@mimik/request-helper": "^2.0.3",
-    "@mimik/response-helper": "^4.0.9",
-    "@mimik/sumologic-winston-logger": "^2.1.10",
+    "@mimik/request-helper": "^2.0.5",
+    "@mimik/response-helper": "^4.0.10",
+    "@mimik/sumologic-winston-logger": "^2.1.12",
     "axios": "1.13.5",
     "bluebird": "3.7.2"
   },
   "devDependencies": {
     "@eslint/js": "9.39.2",
     "@mimik/eslint-plugin-document-env": "^2.0.8",
-    "@stylistic/eslint-plugin": "5.8.0",
-    "acorn": "8.15.0",
+    "@stylistic/eslint-plugin": "5.9.0",
     "body-parser": "2.2.2",
     "c8": "10.1.3",
     "chai": "6.2.2",
     "eslint": "9.39.2",
     "eslint-plugin-import": "2.32.0",
     "express": "5.2.1",
+    "globals": "17.3.0",
     "husky": "9.1.7",
     "jsdoc-to-markdown": "9.1.3",
     "mocha": "11.7.5",

package/test/retryMock.js

@@ -32,9 +32,9 @@ app.use(bodyParser.json());
 app.get(`${config.base}${config.path}`, (req, res) => {
   if (nbRequest === 0) message = 'Received first request';
   else {
-    const timeLap = Date.now() - time;
+    const timeLapse = Date.now() - time;
 
-    message = `Received ${nbRequest} retry with timelapse ${timeLap}`;
+    message = `Received ${nbRequest} retry with timelapse ${timeLapse}`;
   }
   console.log('----->', message);
   time = Date.now();

package/test/rpRetry.spec.js

@@ -172,7 +172,7 @@ describe('request-retry Unit Tests', () => {
         expect(callArgs[CALL_ARG_MESSAGE]).to.equal('invalid retryStrategy, ignoring');
       });
     });
-    it('should generate warning: invalid delayStrategy: "notANumber" becoming delay or default', () => {
+    it('should generate warning: invalid delayStrategy: "notAFunction" becoming delay or default', () => {
       options.retry.delayStrategy = 'notAFunction';
       return rpRetry(options).then(() => {
         assert.calledOnce(loggerSpyWarn);
@@ -252,6 +252,63 @@ describe('request-retry Unit Tests', () => {
       });
     });
   });
+  describe('rpRetry(options) request mapping and response details', () => {
+    it('should succeed with body mapped to axios data', () => rpRetry({
+      method: 'POST',
+      headers: { 'x-correlation-id': correlationId },
+      url: 'http://localhost:9070/test/retry',
+      body: { test: 'data' },
+      retry: {},
+    }).then((response) => {
+      expect(response).to.have.property('statusCode');
+    }));
+    it('should succeed with json object mapped to axios data', () => rpRetry({
+      method: 'POST',
+      headers: { 'x-correlation-id': correlationId },
+      url: 'http://localhost:9070/test/retry',
+      json: { test: 'data' },
+      retry: {},
+    }).then((response) => {
+      expect(response).to.have.property('statusCode');
+    }));
+    it('should succeed with qs mapped to axios params', () => rpRetry({
+      method: 'POST',
+      headers: { 'x-correlation-id': correlationId },
+      url: 'http://localhost:9070/test/retry',
+      json: true,
+      qs: { key: 'value' },
+      retry: {},
+    }).then((response) => {
+      expect(response).to.have.property('statusCode');
+    }));
+    it('should throw a System error on network failure', () => rpRetry({
+      method: 'GET',
+      headers: { 'x-correlation-id': correlationId },
+      url: 'http://localhost:1/unreachable',
+      retry: { retries: 0, timeout: 10 },
+    }).catch((err) => {
+      expect(err.name).to.be.a('string');
+    }));
+    it('should log response with count details', () => {
+      const loggerSpySilly = spy(logger, 'silly');
+
+      return rpRetry({
+        method: 'POST',
+        headers: { 'x-correlation-id': correlationId },
+        url: 'http://localhost:9070/test/retry',
+        json: true,
+        retry: {
+          logLevel: {
+            response: 'silly',
+            responseDetails: 'count',
+          },
+        },
+      }).then(() => {
+        assert.calledOnce(loggerSpySilly);
+        loggerSpySilly.restore();
+      });
+    });
+  });
   describe('rpRetry(options) options validation run', function Test() {
     this.timeout(TIMEOUT);
     const options = {
@@ -294,7 +351,7 @@ describe('request-retry Unit Tests', () => {
           expect(callArgs[CALL_ARG_MESSAGE]).to.equal('bad retry strategy');
         });
     });
-    it('should not generate warning: retry strategy returning a boolean', () => {
+    it('should not generate warning for retry strategy, but warn on request error', () => {
       options.retry.retryStrategy = () => false;
       options.retry.retries = RETRY_RETRIES;
       return rpRetry(options)
@@ -337,7 +394,7 @@ describe('request-retry Unit Tests', () => {
           expect(callArgs[CALL_ARG_MESSAGE]).to.equal('calculated delay out of scope: using delay or default');
         });
     });
-    it('should not generate warning: delay strategy returning an in scope number', () => {
+    it('should not generate warning for delay strategy, but warn on request error', () => {
       options.retry.delayStrategy = () => DELAY_STRATEGY_RESPONSE_2;
       options.retry.retries = RETRY_RETRIES;
       return rpRetry(options)

package/.nycrc

@@ -1,4 +0,0 @@
-{
-  "exclude": ["gulpfile.js"],
-  "reporter": ["lcov", "text"]
-}