refactor CI test runs to run against the code in a built and packaged form instead of against the source code

Author: shtaif

.github/workflows/___.yaml

@@ -0,0 +1,30 @@
+name: CI - ___
+
+on:
+  pull_request:
+    branches: [master]
+
+jobs:
+  lint_check:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          ref: ${{ github.head_ref }}
+
+      - uses: ./.github/actions/ci-common-setup
+
+      - name: Lint check
+        run: pnpm exec eslint --cache
+
+  prettier_check:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          ref: ${{ github.head_ref }}
+
+      - uses: ./.github/actions/ci-common-setup
+
+      - name: Prettier check
+        run: pnpm exec prettier --check "./{src,spec}/**/*.{ts,tsx,js,mjs,jsx}"

.github/workflows/ci-tests.yaml

@@ -13,6 +13,21 @@ jobs:
           ref: ${{ github.head_ref }}
 
       - uses: ./.github/actions/ci-common-setup
-      
-      - name: Run tests
-        run: pnpm run test
+
+      - name: Build and pack library locally
+        id: pkg-pack
+        run: |
+          echo "pkg_pack_results<<EOF" >> $GITHUB_OUTPUT
+          pnpm pack --json >> $GITHUB_OUTPUT
+          echo "EOF" >> $GITHUB_OUTPUT
+
+      - name: Install locally packaged library
+        run: |
+          pnpm i packaged-react-async-iterators@file:./${{fromJson(steps.pkg-pack.outputs.pkg_pack_results)[0].filename}}
+
+      - name: Run tests against packaged library code
+        run: |
+          [[ -e ./src ]] && mv ./src ./src-ignored-for-packaged-testing
+          echo 'export * from "packaged-react-async-iterators";' \> ./spec/libEntrypoint.ts
+          pnpm test
+          [[ -e ./src-ignored-for-packaged-testing ]] && mv ./src-ignored-for-packaged-testing ./src

package.json

@@ -48,7 +48,7 @@
     "test-typings-check": "tsc --noEmit -p ./spec/tsconfig.json",
     "build": "rm -rf ./dist && tsc -p tsconfig.json && tsc -p tsconfig-cjs.json && node ./scripts/set-module-type-in-dist-builds.mjs",
     "build-check": "tsc --noEmit -p ./tsconfig.json",
-    "prepublishOnly": "npm run build"
+    "prepack": "npm run build"
   },
   "peerDependencies": {
     "react": ">=17"

spec/libEntrypoint.ts

@@ -0,0 +1 @@
+export * from '../src/index.js';

spec/tests/Iterate.spec.tsx

@@ -1,7 +1,7 @@
 import { it, describe, expect, afterEach, vi, type Mock } from 'vitest';
 import { gray } from 'colorette';
 import { render, cleanup as cleanupMountedReactTrees, act } from '@testing-library/react';
-import { Iterate, It, iterateFormatted, type IterationResult } from '../../src/index.js';
+import { Iterate, It, iterateFormatted, type IterationResult } from '../libEntrypoint.js';
 import { asyncIterOf } from '../utils/asyncIterOf.js';
 import { IteratorChannelTestHelper } from '../utils/IteratorChannelTestHelper.js';
 

spec/tests/IterateMulti.spec.tsx

@@ -7,7 +7,7 @@ import {
   ItMulti,
   type IterateMultiProps,
   type IterationResultSet,
-} from '../../src/index.js';
+} from '../libEntrypoint.js';
 import { pipe } from '../utils/pipe.js';
 import { asyncIterOf } from '../utils/asyncIterOf.js';
 import { IteratorChannelTestHelper } from '../utils/IteratorChannelTestHelper.js';

spec/tests/iterateFormatted.spec.tsx

@@ -1,7 +1,7 @@
 import { it, describe, expect, afterEach } from 'vitest';
 import { gray } from 'colorette';
 import { render, cleanup as cleanupMountedReactTrees, act } from '@testing-library/react';
-import { iterateFormatted, Iterate } from '../../src/index.js';
+import { iterateFormatted, Iterate } from '../libEntrypoint.js';
 import { pipe } from '../utils/pipe.js';
 import { asyncIterOf } from '../utils/asyncIterOf.js';
 import { asyncIterToArray } from '../utils/asyncIterToArray.js';

spec/tests/useAsyncIter.spec.ts

@@ -1,7 +1,7 @@
 import { it, describe, expect, afterEach, vi } from 'vitest';
 import { gray } from 'colorette';
 import { cleanup as cleanupMountedReactTrees, act, renderHook } from '@testing-library/react';
-import { useAsyncIter, iterateFormatted } from '../../src/index.js';
+import { useAsyncIter, iterateFormatted } from '../libEntrypoint.js';
 import { asyncIterOf } from '../utils/asyncIterOf.js';
 import { IteratorChannelTestHelper } from '../utils/IteratorChannelTestHelper.js';
 

spec/tests/useAsyncIterMulti.spec.ts

@@ -1,7 +1,7 @@
 import { it, describe, expect, afterEach, vi } from 'vitest';
 import { gray } from 'colorette';
 import { cleanup as cleanupMountedReactTrees, act, renderHook } from '@testing-library/react';
-import { iterateFormatted, useAsyncIterMulti } from '../../src/index.js';
+import { iterateFormatted, useAsyncIterMulti } from '../libEntrypoint.js';
 import { pipe } from '../utils/pipe.js';
 import { asyncIterOf } from '../utils/asyncIterOf.js';
 import { IteratorChannelTestHelper } from '../utils/IteratorChannelTestHelper.js';

spec/tests/useAsyncIterState.spec.tsx

@@ -7,7 +7,7 @@ import {
   cleanup as cleanupMountedReactTrees,
   act,
 } from '@testing-library/react';
-import { useAsyncIterState } from '../../src/index.js';
+import { useAsyncIterState } from '../libEntrypoint.js';
 import { asyncIterToArray } from '../utils/asyncIterToArray.js';
 import { asyncIterTake } from '../utils/asyncIterTake.js';
 import { asyncIterTakeFirst } from '../utils/asyncIterTakeFirst.js';