Skip to content

ci: inline LPP integration tests into the workflow - #406

Merged
Teakowa merged 1 commit into
mainfrom
claude/zealous-pascal-gx07b4
Sep 24, 2026
Merged

Teakowa merged 1 commit into
mainfrom
claude/zealous-pascal-gx07b4

Conversation

@Teakowa

@Teakowa Teakowa commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

Summary

Follow-up to #405, continuing the work of making CI steps plain single commands.

scripts/test-lpp-integration.sh only ran three cargo test targets. The LPP integration job now runs them directly as a single command, and the wrapper script is removed:

run: >-
  cargo test --locked -p wright-lpp -p wright-driver
  --test mock_provider --test lpp --test provider_edit

The LPP_MOCK_PROVIDER env and the job's triggers are unchanged.

Validation

  • Locally, the single command runs exactly the three targets: lpp (4 passed), provider_edit (11 passed) and mock_provider (14 passed).
  • Locally the suites self-skip their mock-provider cases because LPP_MOCK_PROVIDER is unset. This PR's CI runs them against the built mock provider.
  • The workflow YAML parses, git diff --check is clean, and nothing else references the removed script.

🤖 Generated with Claude Code

https://claude.ai/code/session_0178vQfKDP4QsRd2Kykm3FZ4


Generated by Claude Code

scripts/test-lpp-integration.sh only ran three `cargo test` targets.
Run them as one `cargo test` command directly in the LPP integration job
and remove the wrapper script.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0178vQfKDP4QsRd2Kykm3FZ4
@Teakowa
Teakowa marked this pull request as ready for review September 24, 2026 07:17
@Teakowa
Teakowa merged commit ac8d762 into main Sep 24, 2026
22 checks passed
@Teakowa
Teakowa deleted the claude/zealous-pascal-gx07b4 branch September 24, 2026 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants