feat(typescript): convert to typescript

update tests to tsx match types version with dep versions generic type change to main/types to exports change build to dist change target to es6 update types/react to 18 to provide modern context type switched to vitest and react-testing-library updated to stricter eslint update to react@17 install and enforce prettier during testing fixed snapshot removed timezone from setup
americanexpress · Dec 12, 2024 · b8b1a31 · b8b1a31
1 parent 95702fd
commit b8b1a31
Show file tree

Hide file tree

Showing 17 changed files with 23,614 additions and 18,280 deletions.
diff --git a/.babelrc b/.babelrc
@@ -1,5 +1,3 @@
 {
-  "presets": [
-    "amex"
-  ]
-}
+  "presets": ["amex", "@babel/preset-typescript"]
+}
diff --git a/.eslintrc.json b/.eslintrc.json
@@ -1,3 +1,54 @@
 {
-  "extends": "amex"
-}
+  "extends": ["amex", "plugin:@typescript-eslint/recommended", "prettier"],
+  "settings": { "import/resolver": { "typescript": {} } },
+  "parserOptions": {
+    "project": true,
+    "extraFileExtensions": [".json"]
+  },
+  "parser": "@typescript-eslint/parser",
+  "plugins": ["@typescript-eslint"],
+  "ignorePatterns": ["dist/**/*", "commitlint.config.js"],
+  "overrides": [
+    {
+      "files": ["__tests__/**"],
+      "extends": [
+        "amex/test",
+        "plugin:@typescript-eslint/recommended",
+        "prettier"
+      ],
+      "rules": {
+        "import/extensions": [
+          "error",
+          "ignorePackages",
+          {
+            "ts": "never",
+            "tsx": "never",
+            "js": "never",
+            "jsx": "never"
+          }
+        ]
+      }
+    },
+    {
+      "files": ["src/**"],
+      "extends": [
+        "amex",
+        "plugin:@typescript-eslint/recommended",
+        "plugin:@typescript-eslint/strict-type-checked",
+        "prettier"
+      ],
+      "rules": {
+        "import/extensions": [
+          "error",
+          "ignorePackages",
+          {
+            "ts": "never",
+            "tsx": "never",
+            "js": "never",
+            "jsx": "never"
+          }
+        ]
+      }
+    }
+  ]
+}
diff --git a/.github/workflows/health-check.yml b/.github/workflows/health-check.yml
@@ -25,6 +25,8 @@ jobs:
       run: npm ci
       env:
         NODE_ENV: development
+    - name: Build
+      run: npm run build
     - name: Run Test Script
       run: npm test
       env:

diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -26,6 +26,8 @@ jobs:
           node-version-file: ".nvmrc"
       - name: Install dependencies
         run: npm ci
+      - name: Build
+        run: npm run build
       - name: Release
         env:
           GIT_AUTHOR_EMAIL: ${{ secrets.GIT_AUTHOR_EMAIL }}

diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -25,6 +25,8 @@ jobs:
       run: npm ci
       env:
         NODE_ENV: development
+    - name: Build
+      run: npm run build
     - name: Unit Tests
       run: npm run test:unit
       env:

diff --git a/.gitignore b/.gitignore
@@ -1,3 +1,4 @@
 node_modules
 test-results
-.jest-cache
+coverage
+dist
diff --git a/__tests__/.eslintrc.json b/__tests__/.eslintrc.json
diff --git a/__tests__/__snapshots__/index.spec.jsx.snap b/__tests__/__snapshots__/index.spec.jsx.snap
diff --git a/__tests__/index.spec.jsx b/__tests__/index.spec.jsx
diff --git a/__tests__/setup.ts b/__tests__/setup.ts
@@ -0,0 +1,6 @@
+import { afterEach } from "vitest";
+import { cleanup } from "@testing-library/react";
+
+afterEach(() => {
+  cleanup();
+});
diff --git a/__tests__/unit/__snapshots__/index.spec.tsx.snap b/__tests__/unit/__snapshots__/index.spec.tsx.snap
@@ -0,0 +1,40 @@
+// Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
+
+exports[`createSharedReactContext > should not return the same context object if the key does not match 1`] = `
+<body>
+  <div>
+    <div
+      data-testid="consumerDiv"
+      id="value"
+    >
+      default
+    </div>
+  </div>
+</body>
+`;
+
+exports[`createSharedReactContext > should return the same context object if the key matches 1`] = `
+<body>
+  <div>
+    <div
+      data-testid="consumerDiv"
+      id="value"
+    >
+      context value
+    </div>
+  </div>
+</body>
+`;
+
+exports[`createSharedReactContext > should use the same default value from first call 1`] = `
+<body>
+  <div>
+    <div
+      data-testid="consumerDiv"
+      id="value"
+    >
+      default
+    </div>
+  </div>
+</body>
+`;