git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH]: cogito testsuite fix - expect "m foo", not "M foo"

From
PRPavel Roskin <proski@gnu.org>
Date
Sep 27, 2005, 01:06 UTC
Message-ID
<1127783161.11578.8.camel@dv>
Signed-off-by: Pavel Roskin <proski@gnu.org>
testsuite fix - expect "m foo", not "M foo"

commit bb42b67ac543416f974d502e269f134ed80bdd4b introduced status 'm' for modified files that won't be committed. This change also broke test 19 in t9202-merge-on-dirty.sh, which expects status 'M'. I believe the 'm' status is correct in this test, so it's the testsuite that need to be adjusted.

diff --git a/t/t9202-merge-on-dirty.sh b/t/t9202-merge-on-dirty.sh
--- a/t/t9202-merge-on-dirty.sh
+++ b/t/t9202-merge-on-dirty.sh
@@ -89,7 +89,7 @@ cp branch1/foo branch1/foo-
 test_expect_failure 'merging branch2 to branch1 (clean)' \
 		"(cd branch1 && cg-merge </dev/null)"
 test_expect_success 'checking if we still have our local change' \
-		'(cd branch1 && cg-status -w | grep -q "^M foo" && cmp foo foo-)'
+		'(cd branch1 && cg-status -w | grep -q "^m foo" && cmp foo foo-)'
 # This test is useful if the previous one failed - did it get lost or
 # accidentally committed?
 test_expect_success 'checking that we didn'\''t commit the local change' \
-- 
Regards,
Pavel Roskin
Message 1 of 1 in “: cogito testsuite fix - expect "m foo", not "M foo"”
  1. : cogito testsuite fix - expect "m foo", not "M foo"Pavel Roskin, Sep 27, 2005

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.