test_buildpackage_rpm: correctly mock False state

This way we don't need to rely on python comparing our tristate to
True / False correctly.

1 files changed, 5 insertions, 2 deletions

diff --git a/tests/component/rpm/test_buildpackage_rpm.py b/tests/component/rpm/test_buildpackage_rpm.py
index 4193e1d7..e236cf45 100644
--- a/tests/component/rpm/test_buildpackage_rpm.py
+++ b/tests/component/rpm/test_buildpackage_rpm.py
@@ -54,12 +54,15 @@ def mock_gbp(args):
 
 def mock_notify(summary, message, notify_opt):
     """Mock notification system"""
+    if notify_opt.is_off():
+        return True
     # Auto will succeed
-    if notify_opt.is_auto():
+    elif notify_opt.is_auto():
         MOCK_NOTIFICATIONS.append((summary, message))
         return True
     # Otherwise fail
-    return False
+    else:
+        return False
 
 
 class TestGbpRpm(RpmRepoTestBase):