Skip to content

refactor(control-plane): decouple pool tests from EC2 - #5307

Open
edersonbrilhante wants to merge 1 commit into
refactor-ec2-runner-operationsfrom
refactor-pool-provider-test-boundary
Open

refactor(control-plane): decouple pool tests from EC2#5307
edersonbrilhante wants to merge 1 commit into
refactor-ec2-runner-operationsfrom
refactor-pool-provider-test-boundary

Conversation

@edersonbrilhante

Copy link
Copy Markdown
Contributor

Description

Depends on #5306.

Refactors the control-plane pool tests to use the PoolComputeProvider capability boundary instead of importing and mocking the EC2 runner implementation.

EC2-specific assertions for running-instance filters, busy and booting runner counts, and the pool-lambda source now live in the EC2 pool provider tests. Generic pool orchestration coverage remains in the control-plane test.

Test Plan

  • Ran the targeted control-plane pool suite: 17 tests passed.
  • Ran the targeted EC2 provider pool suite: 7 tests passed.
  • Ran ESLint and Prettier checks for both changed files.
  • Ran the control-plane TypeScript type-check.
  • Ran git diff --check against refactor-ec2-runner-operations.
  • Verified the generic control-plane pool test contains no EC2 references.

Related Issues

None.

@edersonbrilhante
edersonbrilhante requested a review from a team as a code owner August 27, 2026 11:20
@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

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