[tracker/sam/test-fixes: 3/5] functional-tests: Don't ignore failures in ipc/ tests



commit 1e66800ff94d58629db02406187b8230ca2afd72
Author: Sam Thursfield <sam afuera me uk>
Date:   Sat Sep 14 13:51:17 2019 +0200

    functional-tests: Don't ignore failures in ipc/ tests
    
    This fixes a regression introduced in 14e557aad2675c8a07f680a2.
    
    Tracker Sandbox now passes on the returncode from the subprocess
    so that failing tests are noticed by `meson test` as expected.

 utils/trackertestutils/__main__.py | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)
---
diff --git a/utils/trackertestutils/__main__.py b/utils/trackertestutils/__main__.py
index 859a82545..b67b23d12 100644
--- a/utils/trackertestutils/__main__.py
+++ b/utils/trackertestutils/__main__.py
@@ -432,11 +432,15 @@ def main():
     try:
         if args.command:
             command = [shell, '-c', ' '.join(shlex.quote(c) for c in args.command)]
+
             log.debug("Running: %s", command)
-            subprocess.run(command)
+            result = subprocess.run(command)
 
             if len(miner_watches) > 0:
                 wait_for_miners(miner_watches)
+
+            log.debug("Process finished with returncode %i", result.returncode)
+            sys.exit(result.returncode)
         else:
             if args.dbus_config:
                 print(f"Using Tracker daemons from build tree with D-Bus config {args.dbus_config}")


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]