service: --bus-uri no longer has a default

Closes projectatomic#28
ashcrow · Mar 8, 2017 · d7a64ca · d7a64ca
1 parent 2dcd844
commit d7a64ca
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/src/commissaire_service/service/__init__.py b/src/commissaire_service/service/__init__.py
@@ -38,7 +38,7 @@ def add_service_arguments(parser):
 
         --config-file   (string, no default)
         --bus-exchange  (string, with default)
-        --bus-uri       (string, with default)
+        --bus-uri       (string, no default)
 
     :param parser: An argument parser instance
     :type parser: argparse.ArgumentParser
@@ -51,7 +51,6 @@ def add_service_arguments(parser):
         help='Message bus exchange name.')
     parser.add_argument(
         '--bus-uri', type=str, metavar='BUS_URI',
-        default='redis://127.0.0.1:6379/',  # FIXME Remove before release
         help=(
             'Message bus connection URI. See: '
             'http://kombu.readthedocs.io/en/latest/userguide/connections.html'))  # noqa

diff --git a/test/test_service_add_service_arguments.py b/test/test_service_add_service_arguments.py
@@ -43,5 +43,5 @@ def test_add_service_arguments(self):
 
         # Check for presence of defaults.
         self.assertIsNone(args.config_file)
+        self.assertIsNone(args.bus_uri)
         self.assertIsNotNone(args.bus_exchange)
-        self.assertIsNotNone(args.bus_uri)