Skip to content

Restore 'Install a package with a dependency' test#224

Draft
Copilot wants to merge 3 commits intomainfrom
copilot/restore-install-package-test
Draft

Restore 'Install a package with a dependency' test#224
Copilot wants to merge 3 commits intomainfrom
copilot/restore-install-package-test

Conversation

Copy link
Contributor

Copilot AI commented Mar 2, 2026

The Install a package with a dependency test was marked @broken due to rate limiting. Additionally, yoast/wp-cli-faker (the original test package) has since been removed from Packagist and the WP-CLI package index.

Changes

  • Replace test package: Swap yoast/wp-cli-fakerwp-cli-test/updateable-package
    • Available on Packagist, maintained in the WP-CLI test organization
    • Has wp-cli/wp-cli as a dependency, which is installed into the packages vendor dir but does not appear in wp package list
    • Avoids rate limiting (no GitHub REST API calls)
  • Remove tags: Dropped @broken (restoring the test) and @require-php-5.6 (no PHP version restriction needed)
  • Update assertions: vendor directory checks, is-installed, list, and uninstall steps updated to reflect the new package and its wp-cli/wp-cli dependency
Original prompt

This section details on the original issue you should resolve

<issue_title>Restore 'Install a package with a dependency' test</issue_title>
<issue_description>We marked it broken with #170

I think we're getting hit by some rate limiting. My best bet would be to only run it on one CI build.

Related #164 wp-cli/.github#51

Done is:

  • The 'Install a package with a dependency' test is restored.</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

- Remove @broken tag from the test scenario
- Update @require-php-5.6 to @require-php-7.2 (matches halaxa/json-machine requirement)
- Replace yoast/wp-cli-faker (unavailable) with 10up/wpcli-vulnerability-scanner
- Update assertions to use new package and dependency names (halaxa/json-machine)

Co-authored-by: swissspidy <841956+swissspidy@users.noreply.github.com>
Copilot AI changed the title [WIP] Restore 'Install a package with a dependency' test Restore 'Install a package with a dependency' test Mar 2, 2026
@swissspidy

This comment was marked as resolved.

Replace 10up/wpcli-vulnerability-scanner (deprecated) with
wp-cli-test/updateable-package as suggested in code review.
The package requires wp-cli/wp-cli as its dependency, which
appears in vendor but not in wp package list.

Remove @require-php-7.2 tag since updateable-package has no PHP version restriction.

Co-authored-by: swissspidy <841956+swissspidy@users.noreply.github.com>

This comment was marked as resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope:testing Related to testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Restore 'Install a package with a dependency' test

2 participants