From 9d326b1016a8ac7ad83c3f6d3550cfc7e18215dd Mon Sep 17 00:00:00 2001
From: "Jason N." <33561705+JasonN3@users.noreply.github.com>
Date: Fri, 15 Mar 2024 18:08:29 -0400
Subject: [PATCH] check return codes

---
 Makefile                                  | 1 +
 tests/vm/flatpak_fedora_repo_disabled.yml | 3 ++-
 tests/vm/flatpak_installed.yml            | 2 +-
 3 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/Makefile b/Makefile
index 7c18e8f..64001f2 100644
--- a/Makefile
+++ b/Makefile
@@ -42,6 +42,7 @@ run_tests = tests="$(shell ls tests/$(1)/$(2)_*)"; \
 	  		  for test in $$tests; \
 	  		  do \
 				$(foreach var,$(_VARS),$(var)=$($(var))) ./$${test}; \
+				RC=$$?; if [ $$RC != 0 ]; then exit $$RC; fi; \
 	  		  done; \
 			fi
 
diff --git a/tests/vm/flatpak_fedora_repo_disabled.yml b/tests/vm/flatpak_fedora_repo_disabled.yml
index 2fc77a4..f87ff80 100644
--- a/tests/vm/flatpak_fedora_repo_disabled.yml
+++ b/tests/vm/flatpak_fedora_repo_disabled.yml
@@ -12,4 +12,5 @@
   - name: Check that flatpak-add-fedora-repos is disabled
     ansible.builtin.assert:
       that:
-        - services_state['ansible_facts']['services']['flatpak-add-fedora-repos.service']['status'] == 'disabled'
\ No newline at end of file
+        - services_state['ansible_facts']['services']['flatpak-add-fedora-repos.service']['status'] == 'disabled'
+      fail_msg: 'flatpak-add-fedora-repos.service is not disabled'
\ No newline at end of file
diff --git a/tests/vm/flatpak_installed.yml b/tests/vm/flatpak_installed.yml
index d1630e4..da4f331 100644
--- a/tests/vm/flatpak_installed.yml
+++ b/tests/vm/flatpak_installed.yml
@@ -18,7 +18,7 @@
         - "'VLC' in flatpaks.stdout"
       fail_msg: 'VLC is not installed'
 
-  - name: Check that VLC is installed
+  - name: Check that Firefox is installed
     ansible.builtin.assert:
       that:
         - "'Firefox' in flatpaks.stdout"