Skip to content

test: refactor e2e test suite#1187

Merged
lukasmetzner merged 7 commits intomainfrom
test-refactor-e2e-tests
Apr 15, 2026
Merged

test: refactor e2e test suite#1187
lukasmetzner merged 7 commits intomainfrom
test-refactor-e2e-tests

Conversation

@lukasmetzner
Copy link
Copy Markdown
Contributor

@lukasmetzner lukasmetzner commented Mar 16, 2026

  • Use t.Cleanup for teardown
  • context.Background()t.Context() in tests/helpers
  • %w error wrapping
  • Standardize on testify
  • Exponential backoff (1s → 30s cap) for CreateService + WaitForHTTPAvailable

@lukasmetzner lukasmetzner requested a review from a team as a code owner March 16, 2026 09:08
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.62%. Comparing base (a9d4851) to head (8270299).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1187      +/-   ##
==========================================
+ Coverage   66.05%   69.62%   +3.57%     
==========================================
  Files          24       24              
  Lines        2657     2657              
==========================================
+ Hits         1755     1850      +95     
+ Misses        735      634     -101     
- Partials      167      173       +6     
Flag Coverage Δ
e2e 43.79% <ø> (?)
unit 66.05% <ø> (ø)

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

☔ View full report in Codecov by Sentry.
📢 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.

@lukasmetzner lukasmetzner merged commit bccaddb into main Apr 15, 2026
15 of 18 checks passed
@lukasmetzner lukasmetzner deleted the test-refactor-e2e-tests branch April 15, 2026 06:57
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