Skip to content

docs: Correct PHP unit test setup - #2654

Merged
Su (sushmangupta) merged 2 commits into
mainfrom
docs/correct-php-unit-test-setup
Oct 23, 2026
Merged

Su (sushmangupta) merged 2 commits into
mainfrom
docs/correct-php-unit-test-setup

Conversation

@d-kirkland

Copy link
Copy Markdown
Contributor

Summary

  • The page says plugin:create generates the shown configuration, while the scaffold defines one testsuite pointing to tests (php-unit.md L22; phpunit-xml.stub L18-L21).
  • The generated config has no migration testsuite. The guide now explains that the generated configuration needs adaptation and shows suites pointing to src/Migration/Test and src/Test.
  • The bootstrap example uses BasicExample; the scaffold supplies the configured class name, and addCallingPlugin() activates the final class name from extra.shopware-plugin-class (test-bootstrap.stub L5-L9; TestBootstrapper.php L235-L238). The example now uses SwagBasicExample.
  • The page gives service config paths without src; the scaffold writes the regular service config to src/Resources/config/services.php, and Bundle.php looks for test config under $this->getPath()/Resources/config (CommandGenerator.php L40-L44; Bundle.php L244-L248). The page now includes src in both paths.

Related links

Checklist

  • I reviewed affected links, code samples, and cross-references, including PageRef references where relevant. (Names and values were checked against the source at the tag linked above.)
  • I added or updated redirects in .gitbook.yaml if pages were moved, renamed, or deleted. (N/A, no page moved.)
  • I updated .wordlist.txt (and sorted it) if spellcheck flags new legitimate terms. (No new prose words.)
  • Any required dependent changes in downstream modules have already been merged and published. (N/A)
  • This pull request is ready for review.

Notes

  • This PR was fully AI generated and requires human review. The text was drafted by an OpenAI model (gpt-6-luna) and checked by a second OpenAI model (gpt-6.1-sol), run from a Claude Code session. It has not been merged, approved or reviewed by a person. Please verify the claims and the changed text against the source before merging.
  • Defect type: Breaks (2 of 3 changes). A reader who follows the page as written ends up with code, markup or a command that fails or does not do what the page says.
  • Defect type: Wrong fact (1 of 3 changes). The page states a name, value or behavior that the 6.7 source contradicts. The example may still run, but readers are told something untrue.
  • Codebase files this change was checked against (tag v6.7.15.1): src/Core/Framework/Bundle.php L244-L248; src/Core/Framework/Plugin/Command/Scaffolding/Generator/CommandGenerator.php L40-L44; src/Core/Framework/Plugin/Command/Scaffolding/Generator/TestsGenerator.php L48-L56; src/Core/Framework/Plugin/Command/Scaffolding/stubs/phpunit-xml.stub L18-L21; src/Core/Framework/Plugin/Command/Scaffolding/stubs/test-bootstrap.stub L5-L9; src/Core/TestBootstrapper.php L235-L238.
  • How it was found: a read-through of this page against shopware/shopware at tag v6.7.15.1, then an independent re-check by a second AI model that re-read the page and the cited source files. Please check the cited lines yourself before merging.
  • Scope: main (Shopware 6.7) only. Older docs branches are not touched.

🤖 Generated with Claude Code

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@d-kirkland Daniel Kirkland (d-kirkland) added Improvement PR created to append/modify info in the existing article Info Missed To Document Info Missed To Document labels Oct 22, 2026
@shopware-dev-docs-connector

shopware-dev-docs-connector Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

Developer Docs healthcheck

Status: Completed with success.
Repository: shopware/docs
Commit: ccbcf3e
Preview: https://developer-documentation-qyl25tces-shopware-frontends.vercel.app
Workflow run: #5819

@shopware-dev-docs-connector

Copy link
Copy Markdown

Developer Docs healthcheck

Status: Running.
Repository: shopware/docs
Commit: 7948df5
Preview: Waiting for workflow logs.

1 similar comment
@shopware-dev-docs-connector

Copy link
Copy Markdown

Developer Docs healthcheck

Status: Running.
Repository: shopware/docs
Commit: 7948df5
Preview: Waiting for workflow logs.

@github-actions

Copy link
Copy Markdown
Contributor

📢 Developer Announcement Recommendation

Recommendation: ❌ No Announcement Recommended

No developer announcement signals were detected.

This PR does not appear to introduce developer-facing changes that require a community announcement.

@sushmangupta
Su (sushmangupta) merged commit da4aca3 into main Oct 23, 2026
10 checks passed
@sushmangupta
Su (sushmangupta) deleted the docs/correct-php-unit-test-setup branch October 23, 2026 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Improvement PR created to append/modify info in the existing article Info Missed To Document Info Missed To Document

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants