Skip to content

Test stability - #252

Merged
Harshan01 merged 1 commit into
masterfrom
users/harshan01/test-stability
Aug 14, 2026
Merged

Test stability#252
Harshan01 merged 1 commit into
masterfrom
users/harshan01/test-stability

Conversation

@Harshan01

Copy link
Copy Markdown
Contributor

Reset proxy state after the test that modifies it.

@Harshan01
Harshan01 requested a review from a team as a code owner August 13, 2026 19:14
Copilot AI balanced review requested due to automatic review settings August 13, 2026 19:14

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Ensures proxy state does not leak between randomized tests.

Changes:

  • Resets the global proxy after the proxy-setting test.
Show a summary per file
File Description
spec/github_pages_health_check_spec.rb Adds proxy-state cleanup.

Review details

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

  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Balanced

Comment thread spec/github_pages_health_check_spec.rb Outdated
@Harshan01
Harshan01 enabled auto-merge August 13, 2026 19:16
@Harshan01
Harshan01 force-pushed the users/harshan01/test-stability branch from 4597e39 to f2b8955 Compare August 13, 2026 19:23
@Harshan01
Harshan01 merged commit 2fcb1c5 into master Aug 14, 2026
7 checks passed
@Harshan01
Harshan01 deleted the users/harshan01/test-stability branch August 14, 2026 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants