Skip to content

Commit a0ed92a

Browse files
oaldersclaude
andcommitted
Move live httpbin.org test to xt/ so outages can't fail installs
t/example.t makes live HTTPS requests to httpbin.org during the default test suite. Test::RequiresInternet only checks TCP reachability, so when the service is up-but-unhealthy (e.g. a 503) the guard passes and the assertions fail -- failing an otherwise-healthy end-user install. Move it to xt/author/example.t so it runs under author/release testing (and in CI) but never during a normal make test. Relocate the Test::RequiresInternet prereq from the test phase to develop, since example.t was its only consumer. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 7c1023a commit a0ed92a

3 files changed

Lines changed: 4 additions & 1 deletion

File tree

Changes

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,9 @@
11
Release history for LWP-Protocol-https
22

33
{{$NEXT}}
4+
- Move the live httpbin.org test from t/example.t to xt/author/example.t so
5+
that a transient outage of the external service (e.g. a 503) can no longer
6+
fail an end-user install. The test still runs in CI. (Claude Opus 4.8)
47

58
6.16 2026-07-23 03:48:07Z
69
- Remove undeclared Try::Tiny dependency from t/diag.t, which could cause

cpanfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,12 +24,12 @@ on 'test' => sub {
2424
requires "Socket" => "0";
2525
requires "Test::More" => "0.96";
2626
requires "Test::Needs" => "0.002010";
27-
requires "Test::RequiresInternet" => "0";
2827
requires "warnings" => "0";
2928
};
3029

3130
on 'develop' => sub {
3231
requires 'Capture::Tiny' => '0.48';
32+
requires "Test::RequiresInternet" => "0";
3333
requires 'Test::CheckManifest' => '1.29';
3434
requires 'Test::CleanNamespaces';
3535
requires "Test::CPAN::Changes" => "0.19";
File renamed without changes.

0 commit comments

Comments
 (0)