From ccd32cb7d6950c58e37109bffae5dfb0bc2c165c Mon Sep 17 00:00:00 2001 From: Paul Beckingham Date: Sun, 31 Jul 2011 13:34:10 -0400 Subject: [PATCH] Unit Tests - add.t contained a broken test that did not take into account a (working) substitution. --- src/A3.cpp | 2 +- test/add.t | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/A3.cpp b/src/A3.cpp index 4b33fe5c1..bdfa4b7b8 100644 --- a/src/A3.cpp +++ b/src/A3.cpp @@ -1457,7 +1457,7 @@ bool A3::is_pattern (Nibbler& n, std::string& result) // two-directory absolute path, (or three-level, if the third directory is // named 'g') can be misinterpreted. To help (but not solve) this, if a // substition exists on the local disk, it is not considered a subst. -// This needs to be changed to a better solution. +// This needs to be changed to a better solution. When I think of one. bool A3::is_subst (Nibbler& n, std::string& result) { n.save (); diff --git a/test/add.t b/test/add.t index 197bd6963..8f6f29625 100755 --- a/test/add.t +++ b/test/add.t @@ -48,12 +48,12 @@ like ($output, qr/Status\s+Pending\n/, 'add Pending'); like ($output, qr/UUID\s+[0-9A-Fa-f]{8}-[0-9A-Fa-f]{4}-[0-9A-Fa-f]{4}-[0-9A-Fa-f]{4}-[0-9A-Fa-f]{12}\n/, 'add UUID'); # Test the /// modifier. -qx{../src/task rc:add.rc 1 /test/TEST/}; -qx{../src/task rc:add.rc 1 "/is //"}; +qx{../src/task rc:add.rc 1 modify /test/TEST/}; +qx{../src/task rc:add.rc 1 modify "/is //"}; $output = qx{../src/task rc:add.rc info 1}; like ($output, qr/ID\s+1\n/, 'add ID'); like ($output, qr/Status\s+Pending\n/, 'add Pending'); -like ($output, qr/Description\s+This is a TEST\n/, 'add Description'); +like ($output, qr/Description\s+This a TEST\n/, 'add Description'); # Cleanup. unlink 'pending.data';