]> git.puffer.fish Git - mirror/frr.git/commitdiff
all-protocol-startup: Fix bad assert in case of failed vtysh output test
authorMartin Winter <mwinter@opensourcerouting.org>
Mon, 24 Apr 2017 23:12:55 +0000 (16:12 -0700)
committerDonald Sharp <sharpd@cumulusnetworks.com>
Wed, 28 Nov 2018 01:22:11 +0000 (20:22 -0500)
Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>
tests/topotests/all-protocol-startup/test_all_protocol_startup.py

index ad0e6d20320fd0d9e96b8e99ef50c63548e08f40..ca6cac4b61bc1335f92a8566212a6109bd82f2d6 100755 (executable)
@@ -181,10 +181,11 @@ def test_error_messages_vtysh():
 
         if (vtystdout != ''):
             sys.stderr.write('\nr%s created some spurious VTYSH start StdOut messages:\n%s\n' % (i, vtystdout))
-            failures += 1
         else:
             print("r%s StdOut ok" % i)
 
+        assert vtystdout == '', "Vtysh StdOut Output check failed for router r%s:\n%s" % (i, vtystdout)
+
         #
         # Second checking Standard Error
         #
@@ -199,11 +200,10 @@ def test_error_messages_vtysh():
 
         if (vtystderr != ''):
             sys.stderr.write('\nr%s created some spurious VTYSH start StdErr messages:\n<%s>\n' % (i, vtystderr))
-            failures += 1
         else:
             print("r%s StdErr ok" % i)
 
-        assert failures == 0, "IP RIP status failed for router r%s:\n%s" % (i, diff)
+        assert vtystderr == '', "Vtysh StdErr Output check failed for router r%s:\n%s" % (i, vtystderr)
 
     # For debugging after starting FRR/Quagga daemons, uncomment the next line
     # CLI(net)