Fix PHPCS errors in Behat test files

[swissspidy]

Summary by CodeRabbit

• Style
  • Cleaned up indentation and trailing whitespace in plugin-related test scenarios.
  • No user-visible behavior changes.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: f256d4e5-32e7-4f05-ab1a-f4fcb92c5ab6

📥 Commits

Reviewing files that changed from the base of the PR and between 0a41471 and 4f78013.

📒 Files selected for processing (3)

• features/plugin-activate.feature
• features/plugin-deactivate.feature
• features/plugin.feature

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

---

📝 Walkthrough

Walkthrough

The pull request cleans up indentation and trailing whitespace in embedded PHP snippets across three feature fixtures. It does not change behavior or public declarations.

Changes

Feature fixture formatting

Layer / File(s)	Summary
Embedded PHP formatting cleanup features/plugin-activate.feature, features/plugin-deactivate.feature, features/plugin.feature	Embedded PHP callbacks use adjusted indentation and no longer contain specified trailing whitespace. Callback logic remains unchanged.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 4f780

This change only corrects formatting in Behat test files without changing test logic or product behavior; no actionable merge-blocking risk remains.

Suggested reviewers: schlessera

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly identifies the main change: correcting PHPCS issues in Behat test files. The changes remove trailing whitespace and normalize indentation in those files.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (3 skipped: 3 unsupported.)

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/tests-lint

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[Copilot]

Copilot wasn't able to review any files in this pull request.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!