Skip to content

Update test environment to fix running tests on GitHub Actions - #65

Merged
clue merged 2 commits into
clue:mainfrom
clue-labs:update-tests
Oct 25, 2026
Merged

clue merged 2 commits into
clue:mainfrom
clue-labs:update-tests

Conversation

@clue

@clue clue commented Oct 25, 2026

Copy link
Copy Markdown
Owner

This changeset updates the test environment to fix running tests on GitHub Actions. CI could not run at all since the ubuntu-20.04 runner images were retired in April 2025, so this updates to ubuntu-24.04 and temporarily skips the functional tests relying on the unavailable SOAP service from #58 as well as legacy PHP 7.2 due to version conflicts.

Note this also temporarily requires the dated ringcentral/psr7 package for the test suite only, so the tests pass again while the library itself is still broken with react/http v1.10+ for now. I've already prepared a follow-up PR that fixes this and drops this requirement again.

Builds on top of #55, #27, clue/reactphp-ssh-proxy#42 and others
Refs #58 and reactphp/http#522

@clue clue added this to the v2.1.0 milestone Oct 25, 2026
@clue
clue requested a balanced review from Copilot October 25, 2026 02:19

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

No blocking issues were identified in the scoped test-environment changes, although CI execution was not verified.

No open issues 🎉

What changed in this PR

This PR updates the test environment to restore GitHub Actions CI, using temporary workarounds for the unavailable SOAP service and dependency conflicts. The underlying react/http compatibility fix remains deferred to a follow-up PR.

Changes:

  • Moves CI to Ubuntu 24.04 and updates the checkout action.
  • Temporarily skips PHP 7.2 and internet-dependent tests.
  • Adds ringcentral/psr7 as a development-only dependency.
File Description
composer.json Adds the temporary PSR-7 dependency for the test suite.
.github/​workflows/​ci.yml Updates the runner and checkout action, and applies temporary test exclusions.

Review effort: Balanced

@clue
clue merged commit 71946d7 into clue:main Oct 25, 2026
3 checks passed
@clue
clue deleted the update-tests branch October 25, 2026 02:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants