From df90dd8e73d44c2ef3e4194ec0ed9d2c752d4861 Mon Sep 17 00:00:00 2001 From: Brian Warner <warner@lothar.com> Date: Sun, 15 Feb 2009 21:04:51 -0700 Subject: [PATCH] test_cli: improve test coverage slightly --- src/allmydata/test/test_cli.py | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/src/allmydata/test/test_cli.py b/src/allmydata/test/test_cli.py index da940bd7..1ee4b9f2 100644 --- 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()) -- 2.45.2