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
Adding SAI api tests for CRUD operation #301
Adding SAI api tests for CRUD operation #301
Conversation
murthyvijay
commented
Dec 14, 2022
•
edited
edited
- Created a sai-api folder for all functional sai-api testing
- Test cases to test, create, and remove of:
- VNET, ENI, ENI mac address
- Inbound/Outbound Routes
- ACL groups
- PA validation entry
- VNET direction lookup entry
- Test cases to test get and set are marked to be skipped in CI/CD as they have not been implemented. These tests will be executed when get and set are implemented
- Minor fixes to execute all newly included test cases
* Adding SAI api tests for CRUD operation * Cleanup import and fix assert statements * Undo gitignore changes * Update .gitignore * Update .gitignore * Skip get and set tests, not implemented yet * Add dependency tests for VNET In Route test * Fixed some dependency tests * Run all SAIC tests, not just vnet. (#5) * Run all SAIC tests, not just vnet. * Change name of test script. Co-authored-by: Chris Sommers <chrispsommers@gmail.com> Co-authored-by: Chris Sommers <31145757+chrispsommers@users.noreply.github.com> Co-authored-by: Chris Sommers <chrispsommers@gmail.com>
1. Updated assert statements to remove false pass 2. Added setup and cleanup to ensure tests pass independently 3. Skip get/set tests because feature not available yet in BMv2 4. Inbound route for the ACL test is skipped due to issue sonic-net#233
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
test/test-cases/functional/saic/sai-api/test_sai_api_vnet_acl_in.py
Outdated
Show resolved
Hide resolved
test/test-cases/functional/saic/sai-api/test_sai_api_vnet_acl_in.py
Outdated
Show resolved
Hide resolved
test/test-cases/functional/saic/sai-api/test_sai_api_vnet_direction_lookup.py
Outdated
Show resolved
Hide resolved
test/test-cases/functional/saic/sai-api/test_sai_api_vnet_eni.py
Outdated
Show resolved
Hide resolved
test/test-cases/functional/saic/sai-api/test_sai_api_vnet_pa_entry.py
Outdated
Show resolved
Hide resolved
test/test-cases/functional/saic/sai-api/test_sai_api_vnet_eni_addr.py
Outdated
Show resolved
Hide resolved
Logging and dependencies in tests will be implemented in another PR. Implemented the remaining changes requested
print("\n======= SAI commands RETURN values create =======") | ||
pprint(results) | ||
|
||
assert all(results), "SAI_IP_ADDR_FAMILY_IPV4 Create error" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@murthyvijay Could you please check all assert messages? I see that the next 3 messages do not correspond to the SAI calls.
Per Community Call 1/25/2023 merging |