19146: Remove unneeded special case checks, explain the needed one.

[arvados.git] / lib / deduplicationreport / report_test.go

diff --git a/lib/deduplicationreport/report_test.go b/lib/deduplicationreport/report_test.go
index dc760f78ad77cd1b4856c6c01ccb1bbd0ae7b188..e172008c7214720a48df5411f803c5e3f8b8aa48 100644 (file)
--- a/lib/deduplicationreport/report_test.go
+++ b/lib/deduplicationreport/report_test.go
@@ -28,8 +28,8 @@ func (s *Suite) TearDownSuite(c *check.C) {
 
 func (*Suite) TestUsage(c *check.C) {
        var stdout, stderr bytes.Buffer
-       exitcode := Command.RunCommand("deduplicationreport.test", []string{"-log-level=debug"}, &bytes.Buffer{}, &stdout, &stderr)
-       c.Check(exitcode, check.Equals, 2)
+       exitcode := Command.RunCommand("deduplicationreport.test", []string{"-h", "-log-level=debug"}, &bytes.Buffer{}, &stdout, &stderr)
+       c.Check(exitcode, check.Equals, 0)
        c.Check(stdout.String(), check.Equals, "")
        c.Log(stderr.String())
        c.Check(stderr.String(), check.Matches, `(?ms).*Usage:.*`)
@@ -40,7 +40,6 @@ func (*Suite) TestTwoIdenticalUUIDs(c *check.C) {
        // Run dedupreport with 2 identical uuids
        exitcode := Command.RunCommand("deduplicationreport.test", []string{arvadostest.FooCollection, arvadostest.FooCollection}, &bytes.Buffer{}, &stdout, &stderr)
        c.Check(exitcode, check.Equals, 0)
-       //c.Check(stdout.String(), check.Equals, "")
        c.Check(stdout.String(), check.Matches, "(?ms).*Collections:[[:space:]]+1.*")
        c.Check(stdout.String(), check.Matches, "(?ms).*Nominal size of stored data:[[:space:]]+3 bytes \\(3 B\\).*")
        c.Check(stdout.String(), check.Matches, "(?ms).*Actual size of stored data:[[:space:]]+3 bytes \\(3 B\\).*")