-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
Added Unit tests to eos_logging #63248
Conversation
The test
The test
|
The test
|
recheck |
The test
The test
|
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.
Again with commands that aren't asserted. You could probably at least do something with load_config.assert_called_once_with(...)
if you really wanted to, but that's probably overkill
@GomathiselviS this PR contains the following merge commits: Please rebase your branch to remove these commits. |
recheck |
SUMMARY
Added Unit Testcases to eos_logging module
ISSUE TYPE
COMPONENT NAME
test/units/modules/network/eos/test_eos_logging.py