Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

windows: more test / testutils fixups #1708

Merged
merged 3 commits into from
Mar 3, 2025
Merged

Conversation

lmb
Copy link
Collaborator

@lmb lmb commented Mar 3, 2025

testutils: make IsVersionLessThan cross platform

We usually rely on feature tests to skip certain portions of a test. This
causes a lot of overhead when the feature tests would only be used by the
unit tests.

For that reason the info tests skip some tests based on the current kernel
version. Extend this concept to other platforms by reusing the selection
logic introduced for feature tests.

Signed-off-by: Lorenz Bauer <[email protected]>

testutils: add DupFD helper

Add a cross-platform DupFD helper and use it from tests.

Signed-off-by: Lorenz Bauer <[email protected]>

ebpf: run TestProgramTestRunInterrupt only on Linux

TestProgramTestRunInterrupt tests Linux specific behaviour around signal 
delivery. It doesn't make sense to run it on a different platform.

The test was copied without making any changes.

Signed-off-by: Lorenz Bauer <[email protected]>

lmb added 3 commits March 3, 2025 04:33
We usually rely on feature tests to skip certain portions of
a test. This causes a lot of overhead when the feature tests
would only be used by the unit tests.

For that reason the info tests skip some tests based on the
current kernel version. Extend this concept to other platforms
by reusing the selection logic introduced for feature tests.

Signed-off-by: Lorenz Bauer <[email protected]>
Add a cross-platform DupFD helper and use it from tests.

Signed-off-by: Lorenz Bauer <[email protected]>
TestProgramTestRunInterrupt tests Linux specific behaviour around signal
delivery. It doesn't make sense to run it on a different platform.

The test was copied without making any changes.

Signed-off-by: Lorenz Bauer <[email protected]>
@lmb lmb marked this pull request as ready for review March 3, 2025 14:58
@lmb lmb requested a review from a team as a code owner March 3, 2025 14:58
@lmb lmb merged commit eb39cd8 into cilium:main Mar 3, 2025
17 checks passed
@lmb lmb deleted the windows-test-fixups branch March 3, 2025 14:58
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