Improve flaky HTTP logging test

[swissspidy]

Summary by CodeRabbit

• Bug Fixes
  • Improved HTTP logging test reliability by providing consistent responses for GET and POST requests.
  • Ensured HTTP method debugging works correctly when external request behavior is mocked.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 8587da13-a1f5-4b45-ad79-02a3d4b58df0

📥 Commits

Reviewing files that changed from the base of the PR and between 90946a6 and 0a970eb.

📒 Files selected for processing (1)

• features/http-logging.feature

---

📝 Walkthrough

Walkthrough

The HTTP logging feature now mocks deterministic GET and POST responses from httpbin.org and loads the mock request handlers through the scenario’s WP-CLI configuration.

Changes

HTTP logging test behavior

Layer / File(s)	Summary
Mock HTTP method responses features/http-logging.feature	The HTTP methods scenario defines fixed GET and POST JSON responses and adds mock-requests.php to the wp-cli.yml requirements.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: copilot

🚥 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 accurately summarizes the main change: improving the stability of the flaky HTTP logging test.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

✨ Finishing Touches

🧪 Generate unit tests (beta)

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

---

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

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[Copilot]

Pull request overview

This PR aims to reduce flakiness in the Behat coverage for HTTP request logging by stubbing external HTTP responses (instead of depending on live httpbin.org behavior) in the “Different HTTP methods are logged correctly” scenario.

Changes:

• Adds mocked responses for https://httpbin.org/get and https://httpbin.org/post within features/http-logging.feature.
• Updates the scenario’s wp-cli.yml require: list (currently adding mock-requests.php, which appears to be missing).

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.