Skip to content

Move API-key-gated tests from test:local to test:live by file/suite split#2165

Merged
curtisman merged 2 commits intomainfrom
copilot/review-tests-to-test-live
Apr 8, 2026
Merged

Move API-key-gated tests from test:local to test:live by file/suite split#2165
curtisman merged 2 commits intomainfrom
copilot/review-tests-to-test-live

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 8, 2026

test:local was still picking up tests that depend on external services and API keys via runtime gating (testIf/describeIf). This PR reclassifies those suites so service-dependent tests run under test:live instead of local runs.

  • Reclassify service-dependent suites to live

    • Renamed fully API-key-gated test files from *.spec.ts to *.test.ts in:
      • packages/aiclient
      • packages/knowPro
      • packages/knowledgeProcessor
      • packages/memory/conversation
      • packages/memory/storage
      • packages/memory/website
    • Result: these suites are no longer selected by local *.spec patterns.
  • Add explicit live test scripts in affected packages

    • Added test:live scripts with --testPathPattern=".*[.]test[.]js" to each affected package so workspace-level pnpm run test:live includes them.
  • Align website online alias with live routing

    • Updated packages/memory/website/package.json:
      • test:online now delegates to test:live (instead of test:local) to match .online suite intent.
{
  "scripts": {
    "test:local": "pnpm run jest-esm --testPathPattern=\".*[.]spec[.]js\"",
    "test:live": "pnpm run jest-esm --testPathPattern=\".*[.]test[.]js\""
  }
}

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • ip-api.com
    • Triggering command: /opt/hostedtoolcache/node/24.14.1/x64/bin/node /opt/hostedtoolcache/node/24.14.1/x64/bin/node --no-warnings --experimental-vm-modules /home/REDACTED/work/TypeAgent/TypeAgent/ts/node_modules/.pnpm/jest-worker@29.7.0/node_modules/jest-worker/build/workers/processChild.js Agen�� ignore src/preload dules/.bin/sh electron.vite.prsh --logLevel error node k/Ty�� /bin/jest.js '--testPathPattern=.*[.]spec[.]js' 1 dules/.bin/sh gnore dist r uname (dns block)
    • Triggering command: /opt/hostedtoolcache/node/24.14.1/x64/bin/node /opt/hostedtoolcache/node/24.14.1/x64/bin/node --no-warnings --experimental-vm-modules /home/REDACTED/work/TypeAgent/TypeAgent/ts/node_modules/.pnpm/jest-worker@29.7.0/node_modules/jest-worker/build/workers/processChild.js /hom�� build src/preload k/_temp/ghcca-node/node/bin/node electron.vite.prsh --logLevel .bin/node node k/Ty�� /bin/jest.js '--testPathPattern=.*[.]spec[.]js' 1 dules/.bin/sh gnore dist ct/node_modules/.bin/node node (dns block)
    • Triggering command: /opt/hostedtoolcache/node/24.14.1/x64/bin/node /opt/hostedtoolcache/node/24.14.1/x64/bin/node --no-warnings --experimental-vm-modules /home/REDACTED/work/TypeAgent/TypeAgent/ts/node_modules/.pnpm/jest-worker@29.7.0/node_modules/jest-worker/build/workers/processChild.js Agen�� build src/preload uid-build electron.vite.prsh --logLevel error node Agen�� /bin/jest.js '--testPathPattern=.*[.]spec[.]js' 1 odules/pnpm/dist/node-gyp-bin/sh src/**/*.txt dist in/copyfiles node (dns block)
  • www.electronjs.org
    • Triggering command: /opt/hostedtoolcache/node/24.14.1/x64/bin/node /opt/hostedtoolcache/node/24.14.1/x64/bin/node /home/REDACTED/work/TypeAgent/TypeAgent/ts/node_modules/.pnpm/@electron+rebuild@4.0.3/node_modules/@electron/rebuild/lib/module-type/node-gyp/worker.js (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Move tests requiring services from test:local to test:live Move API-key-gated tests from test:local to test:live by file/suite split Apr 8, 2026
Copilot AI requested a review from curtisman April 8, 2026 19:21
Copilot AI requested a deployment to development-fork April 8, 2026 19:45 Waiting
Copilot AI requested a deployment to development-fork April 8, 2026 19:45 Waiting
@curtisman curtisman marked this pull request as ready for review April 8, 2026 19:54
@curtisman curtisman enabled auto-merge April 8, 2026 19:55
@curtisman curtisman added this pull request to the merge queue Apr 8, 2026
Merged via the queue into main with commit 65675ca Apr 8, 2026
19 of 21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Review tests triggered with the pnpm script test:local and move those that require services to test:live

2 participants