test: Make it easier to resolve conflicts when adding new tests
authorMichal Sojka <sojkam1@fel.cvut.cz>
Sun, 23 Jan 2011 11:12:24 +0000 (12:12 +0100)
committerCarl Worth <cworth@cworth.org>
Wed, 26 Jan 2011 12:37:21 +0000 (22:37 +1000)
Currently, there are two places in the test framework that contain very
long list on a single line. Whenever a test is added (or changed) in
several branches and these branches are merged, it results in conflict
which is hard to resolve because one has to go through the whole long
line to find where the conflict is.

This patch splits these long lists to several lines so that the
conflicts are easier to resolve.

test/basic
test/notmuch-test

index e1269e28b2d9a8af4b067f9cba55b78bb5d49e5b..3b43ad9cda4e5c1c447f4547c05f5562432628f3 100755 (executable)
@@ -51,8 +51,13 @@ test_expect_code 2 'failure to clean up causes the test to fail' '
 
 # Ensure that all tests are being run
 test_begin_subtest 'Ensure that all available tests will be run by notmuch-test'
-tests_in_suite=$(grep TESTS= ../notmuch-test | sed -e "s/TESTS=\"\(.*\)\"/\1/" | tr " " "\n" | sort)
-available=$(ls -1 ../ | grep -v -E "^(aggregate-results.sh|Makefile|Makefile.local|notmuch-test|README|test-lib.sh|test-results|tmp.*|valgrind|corpus*|emacs.expected-output|smtp-dummy|smtp-dummy.c|test-verbose|test.expected-output|.*~)" | sort)
+eval $(sed -n -e '/^TESTS="$/,/^"$/p' notmuch-test ../notmuch-test)
+tests_in_suite=$(for i in $TESTS; do echo $i; done | sort)
+available=$(ls -1 ../ | \
+    sed -r -e "/^(aggregate-results.sh|Makefile|Makefile.local|notmuch-test)/d" \
+          -e "/^(README|test-lib.sh|test-results|tmp.*|valgrind|corpus*)/d" \
+          -e "/^(emacs.expected-output|smtp-dummy|smtp-dummy.c|test-verbose)/d" \
+          -e "/^(test.expected-output|.*~)/d" | sort)
 test_expect_equal "$tests_in_suite" "$available"
 
 EXPECTED=../test.expected-output
index b2bfdff008ec81c17b5ffa4860b4a957a1e00211..9d77c0fa172559e187f4216afceee0e41823a2e8 100755 (executable)
@@ -16,7 +16,27 @@ fi
 
 cd $(dirname "$0")
 
-TESTS="basic new search search-output search-by-folder search-position-overlap-bug json thread-naming raw reply dump-restore uuencode thread-order author-order from-guessing long-id encoding emacs maildir-sync"
+TESTS="
+  basic
+  new
+  search
+  search-output
+  search-by-folder
+  search-position-overlap-bug
+  json
+  thread-naming
+  raw
+  reply
+  dump-restore
+  uuencode
+  thread-order
+  author-order
+  from-guessing
+  long-id
+  encoding
+  emacs
+  maildir-sync
+"
 
 # Clean up any results from a previous run
 rm -r test-results >/dev/null 2>/dev/null