test: reduce and improve tests in mcp_http_tests.rs

[github-actions]

Test Suite Reduction: tests/mcp_http_tests.rs

What was wrong

• test_health_endpoint_no_auth_required: duplicate of test_health_endpoint_returns_ok. Both tests make an unauthenticated GET /health and assert 200. The "no auth required" test is a strict subset — it omits the body check (assert_eq!(resp.text(), "ok")), so its coverage is entirely contained by the first test.

• test_auth_accepts_correct_token: weak assertion. assert_ne!(resp.status(), 401) would pass silently for 500, 404, or any non-401 status code, including error responses. The test name claims the token is accepted, which requires asserting a successful response, not merely the absence of a 401.

Changes

Test	Action	Reason
test_health_endpoint_no_auth_required	Removed	Duplicate — strict subset of test_health_endpoint_returns_ok
test_auth_accepts_correct_token	Rewritten	Weak assertion: assert_ne!(status, 401) → assert!(status.is_success())

Verification

• cargo test --test mcp_http_tests --test audit_it: all 10 tests pass ✅
• cargo test: all 1840+ tests pass, 0 failures ✅
• cargo clippy --all-targets --all-features: no warnings ✅

Warning

Firewall blocked 2 domains

The following domains were blocked by the firewall during workflow execution:

• spsprodeus21.vssps.visualstudio.com
• spsprodweu4.vssps.visualstudio.com

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "spsprodeus21.vssps.visualstudio.com"
    - "spsprodweu4.vssps.visualstudio.com"

See Network Configuration for more information.

Generated by Test Reducer · sonnet46 1.6M · ◷