From 9895c56984518fe3701c42f231d175ad7f0efa7a Mon Sep 17 00:00:00 2001
From: Georg Pfuetzenreuter <mail@georg-pfuetzenreuter.net>
Date: Sun, 21 May 2023 11:23:10 +0200
Subject: [PATCH] Improve test result logging

Handle Enum correctly.

Signed-off-by: Georg Pfuetzenreuter <mail@georg-pfuetzenreuter.net>
---
 scullery.py | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/scullery.py b/scullery.py
index 2ccaef3..b014ede 100755
--- a/scullery.py
+++ b/scullery.py
@@ -17,6 +17,7 @@ from configparser import ConfigParser
 import logging
 import os
 import sys
+from pytest import ExitCode
 
 argparser = ArgumentParser()
 config = ConfigParser()
@@ -163,8 +164,10 @@ def runtests(payload, hosts):
     if not os.path.isfile(sshfile):
         vagrant_sshconfig(sshfile)
     testresult = pytest.main(['--verbose', '--hosts={}'.format(','.join(hosts)), '--ssh-config={}'.format(sshfile), payload])
-    log.debug('Test result is {}'.format(str(testresult)))
-    if not testresult:
+    log.debug('Test result is {}'.format(str(testresult.value)))
+    if testresult == ExitCode.OK:
+        log.debug('Test succeeded')
+    else:
         log.warning('Tests failed')
         return False
     return True