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

[Platform API][watchdog] Inherit from PlatformApiTestBase #1881

Merged
merged 11 commits into from
Jul 18, 2020
Merged

[Platform API][watchdog] Inherit from PlatformApiTestBase #1881

merged 11 commits into from
Jul 18, 2020

Conversation

sujinmkang
Copy link
Contributor

Updated test_watchdog to use PlatformApiTestBase to report the test failures

@lgtm-com
Copy link

lgtm-com bot commented Jul 11, 2020

This pull request introduces 1 alert when merging 16dba83 into c42201c - view on LGTM.com

new alerts:

  • 1 for Unused import

@jleveque jleveque changed the title Changed to use PlatformApiTestBase to report the test failure. [Platform API][watchdog] Inherit from PlatformApiTestBase Jul 11, 2020
tests/platform_tests/api/test_watchdog.py Outdated Show resolved Hide resolved
tests/platform_tests/api/test_watchdog.py Show resolved Hide resolved
tests/platform_tests/api/test_watchdog.py Outdated Show resolved Hide resolved
tests/platform_tests/api/test_watchdog.py Outdated Show resolved Hide resolved
tests/platform_tests/api/test_watchdog.py Outdated Show resolved Hide resolved
tests/platform_tests/api/test_watchdog.py Outdated Show resolved Hide resolved
tests/platform_tests/api/test_watchdog.py Outdated Show resolved Hide resolved
tests/platform_tests/api/test_watchdog.py Outdated Show resolved Hide resolved
tests/platform_tests/api/test_watchdog.py Outdated Show resolved Hide resolved
tests/platform_tests/api/test_watchdog.py Outdated Show resolved Hide resolved
@lgtm-com
Copy link

lgtm-com bot commented Jul 13, 2020

This pull request introduces 1 alert when merging 2c27b13 into 5afa492 - view on LGTM.com

new alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Jul 15, 2020

This pull request introduces 2 alerts when merging a40ccd9 into 15ed97d - view on LGTM.com

new alerts:

  • 2 for Except block handles 'BaseException'

tests/platform_tests/api/test_watchdog.py Outdated Show resolved Hide resolved
tests/platform_tests/api/test_watchdog.py Outdated Show resolved Hide resolved
@lgtm-com
Copy link

lgtm-com bot commented Jul 15, 2020

This pull request introduces 1 alert when merging 7eea20d into e048d8f - view on LGTM.com

new alerts:

  • 1 for Syntax error

@sujinmkang
Copy link
Contributor Author

retest vsimage please

tests/platform_tests/api/test_watchdog.py Outdated Show resolved Hide resolved
tests/platform_tests/api/test_watchdog.py Outdated Show resolved Hide resolved
tests/platform_tests/api/test_watchdog.py Show resolved Hide resolved
tests/platform_tests/api/test_watchdog.py Outdated Show resolved Hide resolved
tests/platform_tests/api/test_watchdog.py Outdated Show resolved Hide resolved
tests/platform_tests/api/test_watchdog.py Outdated Show resolved Hide resolved
tests/platform_tests/api/test_watchdog.py Outdated Show resolved Hide resolved
tests/platform_tests/api/test_watchdog.py Outdated Show resolved Hide resolved
tests/platform_tests/api/test_watchdog.py Outdated Show resolved Hide resolved
tests/platform_tests/api/test_watchdog.py Outdated Show resolved Hide resolved
@lgtm-com
Copy link

lgtm-com bot commented Jul 16, 2020

This pull request introduces 1 alert when merging b3b05fa into b22f3cf - view on LGTM.com

new alerts:

  • 1 for Syntax error

@sujinmkang sujinmkang merged commit e98c12a into sonic-net:master Jul 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants