Skip to content

test: move provider auth parity coverage out of core source#104974

Merged
steipete merged 1 commit into
mainfrom
codex/fix-provider-auth-parity-test-boundary
Jul 12, 2026
Merged

test: move provider auth parity coverage out of core source#104974
steipete merged 1 commit into
mainfrom
codex/fix-provider-auth-parity-test-boundary

Conversation

@steipete

Copy link
Copy Markdown
Contributor

What Problem This Solves

Fixes an issue where current-main build-artifacts fails the core extension boundary because the bundled provider-auth parity integration test was placed under src/ while intentionally loading bundled plugin public surfaces.

Why This Change Was Made

The test moves to the root integration-test owner and updates only its relative core imports. The core boundary remains strict; no allowlist or production behavior changes are added.

User Impact

Maintainers regain a green core-support boundary while preserving the full 72-case provider-auth parity coverage. No shipped runtime or release behavior changes.

Evidence

  • node scripts/run-vitest.mjs test/extension-test-boundary.test.ts test/bundled-provider-auth-literal-parity.test.ts — 11 boundary tests and 72 parity tests passed.
  • Targeted oxfmt and git diff --check — clean.
  • Reproduced on PR fix: recover live Gateway snapshot updates without a control build #104964 exact-head CI run 29179984173: build-artifacts reported only src/plugins/bundled-provider-auth-literal-parity.test.ts as the boundary offender.

@openclaw-barnacle openclaw-barnacle Bot added size: XS maintainer Maintainer-authored PR labels Jul 12, 2026
@steipete

Copy link
Copy Markdown
Contributor Author

Land-ready proof for exact head ff8f3c48126788a0fc04a85cceeff496f27de4f1:

  • Full hosted CI run 29180206960: 48 successful checks, no failures or pending checks.
  • Focused regression proof: node scripts/run-vitest.mjs test/extension-test-boundary.test.ts test/bundled-provider-auth-literal-parity.test.ts — 11 boundary tests and 72 provider-auth parity tests passed.
  • pnpm exec oxfmt --check test/bundled-provider-auth-literal-parity.test.ts passed.
  • git diff --check passed.
  • Review artifacts validated with no findings. OPENCLAW_TESTBOX=1 scripts/pr prepare-run 104974 reused exact-head hosted proof successfully.

This is a pure test-ownership move: it restores the enforced src/ production boundary without weakening the guard or changing runtime behavior.

@steipete
steipete merged commit 4b82e1a into main Jul 12, 2026
119 of 121 checks passed
@steipete
steipete deleted the codex/fix-provider-auth-parity-test-boundary branch July 12, 2026 04:55
@steipete

Copy link
Copy Markdown
Contributor Author

Merged via squash.

vincentkoc added a commit that referenced this pull request Jul 12, 2026
* origin/main:
  test(plugins): move auth parity integration coverage (#104974)
  fix(qa): run mixed channel suites
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request Jul 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintainer Maintainer-authored PR size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant