Skip to content

ci: swap httpbin.org for a locally-hosted version for reliability (doesn't affect local testing)#15

Merged
talagrand merged 1 commit into
mainfrom
fixes
May 30, 2026
Merged

ci: swap httpbin.org for a locally-hosted version for reliability (doesn't affect local testing)#15
talagrand merged 1 commit into
mainfrom
fixes

Conversation

@talagrand

Copy link
Copy Markdown
Owner

No description provided.

@codecov

codecov Bot commented May 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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

This PR replaces direct calls to https://httpbin.org in tests/real_world.rs with a helper that reads HTTPBIN_URL, and wires CI to run a local go-httpbin instance instead, reducing flakiness from the public service. Local developer runs remain unchanged (falling back to httpbin.org).

Changes:

  • Add httpbin(path) helper in tests/real_world.rs and update all httpbin-backed tests/comments to use it.
  • New composite action .github/actions/start-httpbin that installs Go, runs go-httpbin on 127.0.0.1:8080, waits for readiness, and exports HTTPBIN_URL; wired into the CI test (stable/full-checks) and coverage jobs with failure-log dumping. Also factors should-run-full-checks into a reusable step output and enables set -euo pipefail / RUST_BACKTRACE=1.
  • Add Go module under .github/ci-tools/ pinning go-httpbin v2.23.0 (with Dependabot gomod tracking) and a CHANGELOG entry.

Reviewed changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tests/real_world.rs Introduce httpbin() helper; route httpbin tests through HTTPBIN_URL; update doc/log strings.
.github/workflows/ci.yml Add should-run-full-checks output, start go-httpbin and dump log on failure in test & coverage jobs, set pipefail and RUST_BACKTRACE.
.github/actions/start-httpbin/action.yml New composite action: install Go, build/start go-httpbin, wait for readiness, export HTTPBIN_URL.
.github/ci-tools/go.mod New Go module pinning go-httpbin/v2 v2.23.0 via tool directive.
.github/ci-tools/go.sum Checksums for the pinned go-httpbin module.
.github/dependabot.yml Track the new gomod directory monthly.
CHANGELOG.md Add Unreleased "Changed" entry describing the CI swap.

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

@talagrand talagrand merged commit a0765ae into main May 30, 2026
20 checks passed
@talagrand talagrand deleted the fixes branch May 30, 2026 18:20
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.

2 participants