Skip to content

Conversation

@phillebaba
Copy link
Member

A full refactor of the e2e tests for a while now. Firstly they are not e2e tests but rather integration tests. Second of all depending on shelling out has its limits especially when debugging issues. This change refactors the tests into Go in the same style that the Containerd integration tests use.

@codecov
Copy link

codecov bot commented Dec 4, 2025

Codecov Report

❌ Patch coverage is 70.00000% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
pkg/routing/p2p.go 66.66% 1 Missing and 2 partials ⚠️
Flag Coverage Δ
integration-containerd 19.85% <0.00%> (ø)
unit 59.65% <70.00%> (-0.07%) ⬇️

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

Files with missing lines Coverage Δ
pkg/oci/image.go 88.23% <100.00%> (ø)
pkg/routing/p2p.go 60.46% <66.66%> (-1.02%) ⬇️

... and 1 file with indirect coverage changes

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

@phillebaba phillebaba force-pushed the refactor/e2e-test-integration branch 14 times, most recently from c3f05e9 to 28c98d7 Compare December 4, 2025 17:51
@phillebaba phillebaba force-pushed the refactor/e2e-test-integration branch 11 times, most recently from ca27941 to 57059f1 Compare December 5, 2025 15:14
@phillebaba phillebaba force-pushed the refactor/e2e-test-integration branch from 57059f1 to 241a311 Compare December 5, 2025 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant