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

mef_eline new test case to cover set_queue 1 (number diff than 0) with data plane ping too #329

Open
viniarck opened this issue Oct 18, 2024 · 0 comments
Labels
epic_e2e_tests_coverage Increase end-to-end test coverage good first issue Good for newcomers

Comments

@viniarck
Copy link
Member

We do have proper coverage for set_queue action when updating an EVC on this test case test_120_patch_queue_id, but it'd be good to also either augment this one to include data plane ping or add a new one. It's expected that the queue id shouldn't impact, and the existing test case is already good since it's checking for the flow on the switch so it's guaranteed that the switch didn't reject it. But, it'd be nice to know that ping also succeeds in this case.

@viniarck viniarck added good first issue Good for newcomers epic_e2e_tests_coverage Increase end-to-end test coverage labels Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
epic_e2e_tests_coverage Increase end-to-end test coverage good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant