Fix PHPCS errors in Behat test files

[swissspidy]

Summary by CodeRabbit

• Style
  • Standardized indentation and removed trailing whitespace in several test fixtures.
  • No functional behavior, test logic, or user-facing functionality changed.

[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: 38359be3-70d8-4a35-b5d2-e8c77782a42d

📥 Commits

Reviewing files that changed from the base of the PR and between ea96e76 and 70b2b7c.

📒 Files selected for processing (6)

• features/icon-collection.feature
• features/icon.feature
• features/post-block.feature
• features/taxonomy.feature
• features/term-migrate.feature
• features/user-term.feature

Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review.

---

📝 Walkthrough

Walkthrough

The pull request reformats embedded PHP fixtures in six Behat feature files from tab indentation to spaces. It also removes one trailing space. Fixture behavior and test logic remain unchanged.

Changes

Fixture formatting

Layer / File(s)	Summary
Normalize embedded PHP fixture indentation features/icon-collection.feature, features/icon.feature, features/post-block.feature, features/taxonomy.feature, features/term-migrate.feature, features/user-term.feature	Embedded PHP callbacks and registration snippets now use space indentation. One trailing space was removed. Logic, arguments, hooks, and assertions are unchanged.

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

Merge Risk: ⚪ Minimal · up to 70b2b

This change only fixes formatting in Behat test files without changing test behavior or product runtime behavior; no actionable merge-blocking risk remains after normal checks and review.

Suggested reviewers: brianhenryie, ernilambar, janw-me

🚥 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 and concisely describes the whitespace-only changes made to fix PHPCS errors in the Behat test 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. (6 skipped: 6 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!