Skip to content

fix(test): skip version assertions on broken binary after download#620

Merged
Chemaclass merged 1 commit intomainfrom
fix/skip-flaky-network-tests
Apr 9, 2026
Merged

fix(test): skip version assertions on broken binary after download#620
Chemaclass merged 1 commit intomainfrom
fix/skip-flaky-network-tests

Conversation

@Chemaclass
Copy link
Copy Markdown
Member

Summary

  • Skip --version assertions in upgrade/install acceptance tests when the downloaded binary is non-functional (returns empty output)
  • Adds a descriptive skip message so it's clear the test was skipped due to a transient network failure, not silently ignored

Context

The locale CI jobs (Brazilian, Spanish) intermittently fail because GitHub downloads silently return empty/broken binaries (likely rate limiting). This blocks unrelated PRs with failures in bashunit_upgrade_test.sh and install_test.sh.

The download output assertions still run — only the final --version check on the binary is guarded.

Test plan

  • Locale CI jobs no longer fail on transient download issues (shows skip instead)
  • Real upgrade/install bugs still caught by output message assertions

When GitHub downloads fail silently in CI (rate limiting, network
flake), the resulting binary is non-functional and --version returns
empty. Skip the version check with a descriptive message instead of
failing the entire test suite.
@Chemaclass Chemaclass added the bug Something isn't working label Apr 9, 2026
@Chemaclass Chemaclass self-assigned this Apr 9, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 9, 2026

✅ Contributor Report

User: @Chemaclass
Status: Trusted contributor (whitelisted)

This user is on the trusted contributors list and was automatically approved.

@Chemaclass Chemaclass merged commit 36e162c into main Apr 9, 2026
30 checks passed
@Chemaclass Chemaclass deleted the fix/skip-flaky-network-tests branch April 9, 2026 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant