Skip to content

tests: require opt-in for remote fixtures#609

Open
HNO3Miracle wants to merge 1 commit into
pb33f:mainfrom
HNO3Miracle:hno3/gate-network-fixtures
Open

tests: require opt-in for remote fixtures#609
HNO3Miracle wants to merge 1 commit into
pb33f:mainfrom
HNO3Miracle:hno3/gate-network-fixtures

Conversation

@HNO3Miracle

Copy link
Copy Markdown

Gate tests that fetch GitHub, DigitalOcean, or other remote specifications behind LIBOPENAPI_RUN_NETWORK_TESTS. The default unit suite remains fully exercised against local fixtures, while network-dependent coverage can be enabled explicitly with LIBOPENAPI_RUN_NETWORK_TESTS=1 go test ./....

This makes the test suite deterministic in offline and sandboxed environments instead of relying on network availability and external repository state.

Tested with go test ./....

Signed-off-by: HNO3Miracle <xiangao.or@isrc.iscas.ac.cn>
@codecov

codecov Bot commented Jul 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.66%. Comparing base (0837c9b) to head (02a4bd6).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #609      +/-   ##
==========================================
- Coverage   99.78%   99.66%   -0.12%     
==========================================
  Files         283      283              
  Lines       34456    34456              
==========================================
- Hits        34382    34341      -41     
- Misses         46       81      +35     
- Partials       28       34       +6     
Flag Coverage Δ
unittests 99.66% <ø> (-0.12%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

1 participant