Skip to content

fix(test): align QwenPaw test with AgentTeams contracts - #1033

Open
RerankerGuo wants to merge 2 commits into
agentscope-ai:mainfrom
RerankerGuo:fix/qwenpaw-agentteams-provider-assertion
Open

fix(test): align QwenPaw test with AgentTeams contracts#1033
RerankerGuo wants to merge 2 commits into
agentscope-ai:mainfrom
RerankerGuo:fix/qwenpaw-agentteams-provider-assertion

Conversation

@RerankerGuo

@RerankerGuo RerankerGuo commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Summary

Test 26 still targeted several retired HiClaw contracts after the hard rename: the QwenPaw provider ID, embedded controller container and data directory, Kubernetes API path, and Worker/Team apiVersion values. Align the integration test with the canonical AgentTeams runtime and API contracts so it can create resources and validate the generated workspace once CI reaches the shard.

This changes only the integration test; production behavior already uses these AgentTeams values.

Verification

  • extracted workspace projection check against an agentteams-gateway fixture: ok
  • exact static contract check for controller, token path, API group, resource declarations, and provider ID
  • bash -n tests/test-26-qwenpaw-teamharness-plugin-mode.sh
  • go test ./api/v1beta1 ./internal/config ./internal/service with Go 1.25.0
  • git diff --check

The previous integration run stopped during install-embedded with the base-workflow LLM secret issue tracked in #1026, before any test shard ran.

@github-actions

github-actions Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

❌ Integration Tests Failed (llm-interaction-2 / mgr=openclaw / wk=openclaw)

Commit: c5596da
Workflow run: #1662

Test Results
No test output captured.
Debug Log (tail)
No debug logs available.

📦 Download full debug logs & test artifacts

@github-actions

github-actions Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

❌ Integration Tests Failed (llm-interaction / mgr=openclaw / wk=openclaw)

Commit: c5596da
Workflow run: #1662

Test Results
No test output captured.
Debug Log (tail)
No debug logs available.

📦 Download full debug logs & test artifacts

@github-actions

github-actions Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

❌ Integration Tests Failed (llm-interaction-2 / mgr=copaw / wk=copaw)

Commit: c5596da
Workflow run: #1662

Test Results
No test output captured.
Debug Log (tail)
No debug logs available.

📦 Download full debug logs & test artifacts

@github-actions

github-actions Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

❌ Integration Tests Failed (controller-cr / mgr=copaw / wk=copaw)

Commit: c5596da
Workflow run: #1662

Test Results
No test output captured.
Debug Log (tail)
No debug logs available.

📦 Download full debug logs & test artifacts

@github-actions

github-actions Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

❌ Integration Tests Failed (llm-interaction / mgr=copaw / wk=copaw)

Commit: c5596da
Workflow run: #1662

Test Results
No test output captured.
Debug Log (tail)
No debug logs available.

📦 Download full debug logs & test artifacts

@github-actions

github-actions Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

❌ Integration Tests Failed (llm-interaction / mgr=copaw / wk=hermes)

Commit: c5596da
Workflow run: #1662

Test Results
No test output captured.
Debug Log (tail)
No debug logs available.

📦 Download full debug logs & test artifacts

@github-actions

github-actions Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

❌ Integration Tests Failed (llm-interaction-2 / mgr=copaw / wk=hermes)

Commit: c5596da
Workflow run: #1662

Test Results
No test output captured.
Debug Log (tail)
No debug logs available.

📦 Download full debug logs & test artifacts

@github-actions

github-actions Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

❌ Integration Tests Failed (controller-cr / mgr=copaw / wk=hermes)

Commit: c5596da
Workflow run: #1662

Test Results
No test output captured.
Debug Log (tail)
No debug logs available.

📦 Download full debug logs & test artifacts

@github-actions

github-actions Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

❌ Integration Tests Failed (controller-cr / mgr=openclaw / wk=openclaw)

Commit: c5596da
Workflow run: #1662

Test Results
No test output captured.
Debug Log (tail)
No debug logs available.

📦 Download full debug logs & test artifacts

@github-actions

github-actions Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

❌ Integration Tests Failed (controller-cr-2 / mgr=copaw / wk=copaw)

Commit: c5596da
Workflow run: #1662

Test Results
No test output captured.
Debug Log (tail)
No debug logs available.

📦 Download full debug logs & test artifacts

@RerankerGuo

Copy link
Copy Markdown
Contributor Author

CI attribution: all 9 translation/base/image build checks passed. I inspected the raw logs for all 10 failed integration jobs in run #1660; every matrix entry exits during install-embedded, before its test shard starts, with LLM API Key is required. This is the same base pull_request_target secret-name bootstrap regression addressed by #1026, not a failure of this focused test assertion. I am keeping this branch unchanged.

@RerankerGuo RerankerGuo changed the title fix(test): expect AgentTeams provider in QwenPaw projection fix(test): align QwenPaw test with AgentTeams contracts Jul 15, 2026
@RerankerGuo

Copy link
Copy Markdown
Contributor Author

Latest-head CI attribution: all 9 translation/base/image checks passed for c5596da. I inspected the raw logs for all 10 failed matrix jobs in run #1662; each log has an empty AGENTTEAMS_LLM_API_KEY, exits from install-embedded with code 2 and LLM API Key is required, and produces no shard test output. This is the same pull_request_target base-workflow secret blocker addressed by #1026, so the locally passing QwenPaw contract patch is unchanged.

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.

1 participant