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

Add tests for bpf_prog_test_run (i.e context_create/delete) #142

Open
matthewige opened this issue Oct 18, 2024 · 0 comments
Open

Add tests for bpf_prog_test_run (i.e context_create/delete) #142

matthewige opened this issue Oct 18, 2024 · 0 comments
Assignees
Labels
P1 Priority 1 test triaged The issue is triaged by the maintainers.
Milestone

Comments

@matthewige
Copy link
Contributor

Describe the bug

Both the ntosext and neteventext both support context_create/context_delete functions. However, there are no unit tests in this repo that invoke them.

We should add tests for both of these ensuring that the context_create/delete functions are properly implemented and there are no bugs in them, so that other codebases consuming the extension can invoke bpf_prog_test_run_opts with confidence that the extension code is correct and valid.

OS information

n/a

Steps taken to reproduce bug

n/a

Expected behavior

n/a

Actual outcome

n/a

Additional details

No response

@matthewige matthewige added the bug Something isn't working label Oct 18, 2024
@shankarseal shankarseal added test P1 Priority 1 and removed bug Something isn't working labels Oct 21, 2024
@shankarseal shankarseal added this to the 2410 milestone Oct 21, 2024
@shankarseal shankarseal added the triaged The issue is triaged by the maintainers. label Oct 21, 2024
@shankarseal shankarseal modified the milestones: 2410, 2501 Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 Priority 1 test triaged The issue is triaged by the maintainers.
Projects
None yet
Development

No branches or pull requests

3 participants