-
Notifications
You must be signed in to change notification settings - Fork 0
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 #4
Conversation
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.
See comments - need to clean up unused code and fix the asserts. Great start, I like the approach.
Another thought - since these are all "functional tests," should they go under a new directory |
* Run all SAIC tests, not just vnet. * Change name of test script. Co-authored-by: Chris Sommers <chrispsommers@gmail.com>
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.
See comments
* Adding SAI api tests for CRUD operation (#4) * 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> * Incorporated changes from feedback from community 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 * remove unneeded files * Incorporated requested changes Logging and dependencies in tests will be implemented in another PR. Implemented the remaining changes requested --------- Co-authored-by: Chris Sommers <31145757+chrispsommers@users.noreply.github.com> Co-authored-by: Chris Sommers <chrispsommers@gmail.com>
requesting review of SAI api tests