From 0ef164bc04dcfda49e9746a7e7a4abc819d3eee0 Mon Sep 17 00:00:00 2001 From: Roman Tokarev <rtokarev@corp.mail.ru> Date: Thu, 3 Feb 2011 15:20:26 +0300 Subject: [PATCH] [test-runner] Always check for valgrind errors. --- test/lib/test_suite.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/test/lib/test_suite.py b/test/lib/test_suite.py index 3a2c479ef9..e1fa405760 100644 --- a/test/lib/test_suite.py +++ b/test/lib/test_suite.py @@ -131,12 +131,12 @@ class Test: if not self.is_executed_ok: self.print_diagnostics() where = ": test execution aborted, reason '{0}'".format(diagnostics) - elif not self.is_valgrind_clean: - print "Test failed! Valgrind reports errors" \ - " (see {0}/valgrind.log)".format(self.args.vardir) - else: + elif not self.is_equal_result: self.print_unidiff() where = ": wrong test output" + if not self.is_valgrind_clean: + print "Test failed! Valgrind reports errors" \ + " (see {0}/valgrind.log)".format(self.args.vardir) if not self.suite_ini["is_force"]: raise TestRunException("Failed to run test " + self.name + where) -- GitLab