Repository navigation
Conversation
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the Core Committers: Use this line as a base for the props when committing in SVN: To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
Test using WordPress PlaygroundThe changes in this pull request can previewed and tested using a WordPress Playground instance. WordPress Playground is an experimental project that creates a full WordPress instance entirely within the browser. Some things to be aware of
For more details about these limitations and more, check out the Limitations page in the WordPress Playground documentation. |
|
Refreshed in 076b40e. Three independent cases cover upload with activation/deactivation, replacement with a newer version, and confirmed deletion. The activation/deactivation scenarios carry forward your work in #8050. Two tiny fixture archives replace Akismet. Each case checks saved state and cleans up only its own fixture. All nine local checks passed without retries (55.8 seconds); scoped E2E and five-steps audits found no blockers. Fresh CI and independent review are next. #8050 remains open pending the consolidation decision, with its contribution recorded in the description. AI review · |
|
Review of 076b40e. One small requested change; the tests themselves are sound. Requested change: put the fixtures in the existing assets folderThis PR adds a new What I checked
CIRun 38281150390 is green. All three new cases passed on the first attempt in both E2E jobs. One existing test was retried in each job: AI review · |
|
Moved the fixture archives and their documentation into the existing |
|
Re-review of a54d884. The requested change is in; no open findings.
One existing test was retried in one job: AI review · |
|
PR #8075 landed in https://core.trac.wordpress.org/changeset/64624 |
Add browser checks for plugin upload, activation and deactivation, replacement with a newer uploaded version, and deletion. Verify saved plugin status, version, and removal through REST. Use small local ZIP fixtures containing only plugin headers. Install prerequisites for each case and clean up only the fixture plugin. Developed in: #8075 Props juhise, lucatume. See #52895. git-svn-id: https://develop.svn.wordpress.org/trunk@64624 602fd350-edb4-49c9-b593-d223f7449a82
Add browser checks for plugin upload, activation and deactivation, replacement with a newer uploaded version, and deletion. Verify saved plugin status, version, and removal through REST. Use small local ZIP fixtures containing only plugin headers. Install prerequisites for each case and clean up only the fixture plugin. Developed in: WordPress/wordpress-develop#8075 Props juhise, lucatume. See #52895. Built from https://develop.svn.wordpress.org/trunk@64624 git-svn-id: http://core.svn.wordpress.org/trunk@63774 1a063a9b-81f0-0310-95a4-ce76da25c4cd
Adds three focused browser checks for plugin management:
Two 251-byte fixture archives replace the bundled Akismet versions. They contain only plugin headers; no third-party plugin code or update availability is needed. The fixture documentation includes their source and regeneration instructions. The network-dependent update-notice assertion is removed.
Each case installs its own prerequisite and cleans up only the fixture plugin. A preexisting fixture makes setup fail without deleting it. The fixed fixture slug assumes the test environment is not shared with another simultaneous run, matching the current CI setup. These cases target single-site plugin management.
This carries the activation/deactivation scenarios forward from #8050, by the same author. That PR remains open pending review of this consolidation.
Trac ticket: https://core.trac.wordpress.org/ticket/52895
Props
juhiseauthored this PR and the activation/deactivation scenarios in #8050.Verification
All three cases passed three local single-site Chromium repetitions without retries (9 checks, 55.8 seconds).
The fixture was removed and the original local multisite configuration restored afterward.
Scoped E2E plugin and five-steps audits found no blockers.
JavaScript syntax and whitespace checks passed.
Fixtures
The archives are committed, not built at test time. Trunk already keeps ZIP fixtures for tests, including a plugin archive for install tests at
tests/phpunit/data/plugins/link-manager.zip. Building them in the spec would need a ZIP library that core does not declare. The note beside each archive lists its full contents and a script that rebuilds it byte for byte.Use of AI Tools
AI assistance: Yes
Tool(s): Codex
Model(s): GPT-6
Used for: Refreshing and consolidating the tests, preparing small fixture archives, and local validation and audits.