feat(config): enforce 100% test coverage threshold

jest.config.js

@@ -24,7 +24,17 @@ module.exports = {
 
   // The directory where Jest should output its coverage files
   coverageDirectory: 'coverage',
-  // coverageThreshold: { global: { lines: 75 } },
+
+  // An object that configures minimum threshold enforcement for coverage results
+  coverageThreshold: {
+    global: {
+      branches: 100,
+      functions: 100,
+      lines: 100,
+      statements: 100
+    }
+  },
+
   // An array of regexp pattern strings used to skip coverage collection
   // coveragePathIgnorePatterns: [
   //   "/node_modules/"
@@ -41,9 +51,6 @@ module.exports = {
   //   "clover"
   // ],
 
-  // An object that configures minimum threshold enforcement for coverage results
-  // coverageThreshold: undefined,
-
   // A path to a custom dependency extractor
   // dependencyExtractor: undefined,
 

package.json

@@ -11,7 +11,8 @@
   },
   "lint-staged": {
     "**/*.js": [
-      "eslint --fix"
+      "eslint --fix",
+      "prettier --write"
     ]
   },
   "dependencies": {

src/index.js

@@ -90,11 +90,8 @@ async function onRequest(request, settings) {
 /**
  * Exports for Testing Only
  */
-try {
-  if (process?.env['NODE_DEV'] === 'TEST') {
-    module.exports = {
-      onRequest
-    };
-  }
-  // eslint-disable-next-line no-empty
-} catch (e) {}
+if (typeof process !== 'undefined' && process.env['NODE_DEV'] === 'TEST') {
+  module.exports = {
+    onRequest
+  };
+}

src/index.test.js

@@ -52,8 +52,32 @@ describe('onRequest', () => {
      * Expect `Segment` methods to have been called
      */
     expect(Segment.identify).toHaveBeenCalledTimes(1);
-    expect(Segment.track).toHaveBeenCalledTimes(1);
-    expect(Segment.track).toHaveBeenCalledTimes(1);
-    expect(Segment.track).toHaveBeenCalledTimes(1);
+    expect(Segment.group).toHaveBeenCalledTimes(1);
+    expect(Segment.page).toHaveBeenCalledTimes(1);
+    expect(Segment.screen).toHaveBeenCalledTimes(1);
+  });
+
+  it('should throw RetryError on fetch connection error', async () => {
+    fetch.mockRejectOnce(new Error('Network failure'));
+
+    await expect(onRequest(baseRequest, baseSettings)).rejects.toThrow(
+      'Network failure'
+    );
+  });
+
+  it('should throw RetryError on 500 server error', async () => {
+    fetch.mockResponseOnce('', { status: 500 });
+
+    await expect(onRequest(baseRequest, baseSettings)).rejects.toThrow(
+      'Failed with 500'
+    );
+  });
+
+  it('should throw RetryError on 429 rate limit', async () => {
+    fetch.mockResponseOnce('', { status: 429 });
+
+    await expect(onRequest(baseRequest, baseSettings)).rejects.toThrow(
+      'Failed with 429'
+    );
   });
 });