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 unit tests for show commands on multi ASIC platform for per namespace output. #1093

Closed
smaheshm opened this issue Sep 3, 2020 · 1 comment
Assignees

Comments

@smaheshm
Copy link
Contributor

smaheshm commented Sep 3, 2020

The "display" option will not work. Requires a bit of explanation but it has to do with 'multi-asic-click-options' which populates the parameters before the environment variables are set un 'setup()' method. Due to this the multi asic namespace options are not available when the pytest framework loads all the .py files. Tried several options including 'reload', but it didn't work. We are dicussing internally what other options are available including changing implementation of 'multi-asic-click-option'.

Can you include cases to cover all the options for 'show stats' and 'show config'. display for a namespace, display when a set of ports/queues are included?

Originally posted by @neethajohn in #1080 (comment)

@smaheshm
Copy link
Contributor Author

smaheshm commented Dec 3, 2020

added unit tests.

@smaheshm smaheshm closed this as completed Dec 3, 2020
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

No branches or pull requests

1 participant