test_cli: improve test coverage slightly

diff --git a/src/allmydata/test/test_cli.py b/src/allmydata/test/test_cli.py
index da940bd7ba0398fa83880ef02e5ea7c03313dd20..1ee4b9f293fb726fc9f806395063fcdb310d389c 100644 (file)
--- a/src/allmydata/test/test_cli.py
+++ b/src/allmydata/test/test_cli.py
@@ -20,6 +20,7 @@ from allmydata.scripts import cli, debug, runner, backupdb
 from allmydata.test.common import SystemTestMixin
 from allmydata.test.common_util import StallMixin
 from twisted.internet import threads # CLI tests use deferToThread
+from twisted.python import usage
 
 class CLI(unittest.TestCase):
     # this test case only looks at argument-processing and simple stuff.
@@ -69,6 +70,17 @@ class CLI(unittest.TestCase):
         self.failUnlessEqual(o.aliases[DEFAULT_ALIAS], other_uri)
         self.failUnlessEqual(o.where, "subdir")
 
+        o = cli.ListOptions()
+        self.failUnlessRaises(usage.UsageError,
+                              o.parseOptions,
+                              ["--node-directory", "cli/test_options",
+                               "--node-url", "NOT-A-URL"])
+
+        o = cli.ListOptions()
+        o.parseOptions(["--node-directory", "cli/test_options",
+                        "--node-url", "http://localhost:8080"])
+        self.failUnlessEqual(o["node-url"], "http://localhost:8080/")
+
     def _dump_cap(self, *args):
         config = debug.DumpCapOptions()
         config.stdout,config.stderr = StringIO(), StringIO()
@@ -575,6 +587,11 @@ class Put(SystemTestMixin, CLITestMixin, unittest.TestCase):
         return d
 
 class Cp(SystemTestMixin, CLITestMixin, unittest.TestCase):
+    def test_not_enough_args(self):
+        o = cli.CpOptions()
+        self.failUnlessRaises(usage.UsageError,
+                              o.parseOptions, ["onearg"])
+
     def test_unicode_filename(self):
         self.basedir = os.path.dirname(self.mktemp())