@olasd is expecting this to exit with an error code.

fedora-infra · Feb 28, 2014 · 07ade12 · 07ade12
1 parent 5589ad1
commit 07ade12
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/fedmsg/commands/config.py b/fedmsg/commands/config.py
@@ -92,6 +92,10 @@ def config():
 
     if args.query:
         cur = fedmsg.utils.dict_query(cur, args.query)
+        if cur[args.query] is None:
+            print >>sys.stderr, (
+                "Key `%s` does not exist in config" % args.query)
+            sys.exit(1)
 
     if isinstance(cur, list):
         for i in cur:

diff --git a/fedmsg/tests/test_commands.py b/fedmsg/tests/test_commands.py
@@ -230,7 +230,9 @@ def test_config_query_broken(self, stderr, stdout, argv):
         except SystemExit as exc:
             eq_(exc.code, 1)
         else:
-            assert False
+            output = "output: %r, error: %r" % (
+                stdout.getvalue(), stderr.getvalue())
+            assert False, output
 
         output = stdout.getvalue()
         error = stderr.getvalue()