-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
tests: Enhance error msgs for static route automation. #12816
Conversation
Continuous Integration Result: SUCCESSFULContinuous Integration Result: SUCCESSFULCongratulations, this patch passed basic tests Tested-by: NetDEF / OpenSourceRouting.org CI System CI System Testrun URL: https://ci1.netdef.org/browse/FRR-PULLREQ2-9712/ This is a comment from an automated CI system. |
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.
Please review again those changes, because they are inconsistent.
tests/topotests/static_routing_with_ebgp/test_static_routes_topo1_ebgp.py
Outdated
Show resolved
Hide resolved
@@ -448,7 +448,7 @@ def test_static_route_2nh_p0_tc_1_ebgp(request): | |||
) | |||
assert ( | |||
result is not True | |||
), "Testcase {} : Failed \nError: Route is" " still present in RIB".format( | |||
), "Testcase {} : Failed \nExpected: Routes should not present in RIB \nError: Route is" " still present in RIB".format( |
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.
Same as above Route is " " ..
looks weird.
tests/topotests/static_routing_with_ebgp/test_static_routes_topo1_ebgp.py
Outdated
Show resolved
Hide resolved
@@ -516,7 +516,7 @@ def test_static_route_2nh_p0_tc_1_ebgp(request): | |||
) | |||
assert ( | |||
result is not True | |||
), "Testcase {} : Failed \nError: Route is" " still present in RIB".format( | |||
), "Testcase {} : Failed \nExpected: Routes should not present in RIB \nError: Route is" " still present in RIB".format( |
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.
Ditto.
tests/topotests/static_routing_with_ebgp/test_static_routes_topo1_ebgp.py
Outdated
Show resolved
Hide resolved
@@ -583,7 +583,7 @@ def test_static_route_2nh_p0_tc_1_ebgp(request): | |||
) | |||
assert ( | |||
result is not True | |||
), "Testcase {} : Failed \nError: Route is" " still present in RIB".format( | |||
), "Testcase {} : Failed \nExpected: Routes should not present in RIB \nError: Route is" " still present in RIB".format( |
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.
Ditto.
tests/topotests/static_routing_with_ebgp/test_static_routes_topo1_ebgp.py
Outdated
Show resolved
Hide resolved
@@ -778,7 +778,7 @@ def test_static_route_2nh_admin_dist_p0_tc_2_ebgp(request): | |||
) | |||
assert ( | |||
result is not True | |||
), "Testcase {} : Failed \nError: Routes is" "missing in RIB".format(tc_name) | |||
), "Testcase {} : Failed \nExpected: Routes should not present in RIB \nError: Routes is" "missing in RIB".format(tc_name) |
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.
Ditto.
tests/topotests/static_routing_with_ebgp/test_static_routes_topo1_ebgp.py
Outdated
Show resolved
Hide resolved
tests: Enhance error messages for static route automation. Signed-off-by: nguggarigoud <nguggarigoud@vmware.com>
a65f6e0
to
1155a66
Compare
Continuous Integration Result: FAILEDContinuous Integration Result: FAILEDSee below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: SuccessfulBasic Tests: FailedTopotests Ubuntu 18.04 i386 part 3: Failed (click for details)Topotests Ubuntu 18.04 i386 part 3: Unknown Log URL: https://ci1.netdef.org/browse/FRR-PULLREQ2-10072/artifact/TOPO3U18I386/TopotestDetails/Topology Test Results are at https://ci1.netdef.org/browse/FRR-PULLREQ2-TOPO3U18I386-10072/test Topology Tests failed for Topotests Ubuntu 18.04 i386 part 3 Successful on other platforms/tests
|
ci: rerun, un related test failed. |
Continuous Integration Result: SUCCESSFULCongratulations, this patch passed basic tests Tested-by: NetDEF / OpenSourceRouting.org CI System CI System Testrun URL: https://ci1.netdef.org/browse/FRR-PULLREQ2-10125/ This is a comment from an automated CI system. |
Signed-off-by: nguggarigoud nguggarigoud@vmware.com