From: david-sarah Date: Tue, 4 Jan 2011 11:51:08 +0000 (-0700) Subject: test_cli.py: Test for error message when 'tahoe rm' is invoked without a path. refs... X-Git-Tag: pre-393~86 X-Git-Url: https://git.rkrishnan.org/?a=commitdiff_plain;h=00fefeba4969605d8e7f027b42e47a7d7ed58647;p=tahoe-lafs%2Ftahoe-lafs.git test_cli.py: Test for error message when 'tahoe rm' is invoked without a path. refs #1292 --- diff --git a/src/allmydata/test/test_cli.py b/src/allmydata/test/test_cli.py index 9f08351f..2c2d130c 100644 --- a/src/allmydata/test/test_cli.py +++ b/src/allmydata/test/test_cli.py @@ -2942,6 +2942,12 @@ class Mkdir(GridTestMixin, CLITestMixin, unittest.TestCase): class Rm(GridTestMixin, CLITestMixin, unittest.TestCase): + def _create_test_file(self): + data = "puppies" * 1000 + path = os.path.join(self.basedir, "datafile") + fileutil.write(path, data) + self.datafile = path + def test_rm_without_alias(self): # 'tahoe rm' should behave sensibly when invoked without an explicit # alias before the default 'tahoe' alias has been created. @@ -2975,6 +2981,26 @@ class Rm(GridTestMixin, CLITestMixin, unittest.TestCase): d.addCallback(_check) return d + def test_rm_without_path(self): + # 'tahoe rm' should give a sensible error message when invoked without a path. + self.basedir = "cli/Rm/rm_without_path" + self.set_up_grid() + self._create_test_file() + d = self.do_cli("create-alias", "tahoe") + d.addCallback(lambda ign: self.do_cli("put", self.datafile, "tahoe:test")) + def _do_rm((rc, out, err)): + self.failUnlessReallyEqual(rc, 0) + self.failUnless(out.startswith("URI:"), out) + return self.do_cli("rm", out.strip('\n')) + d.addCallback(_do_rm) + + def _check((rc, out, err)): + self.failUnlessReallyEqual(rc, 1) + self.failUnlessIn("path must be given", err) + self.failUnlessReallyEqual(out, "") + d.addCallback(_check) + return d + class Stats(GridTestMixin, CLITestMixin, unittest.TestCase): def test_empty_directory(self):