unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
* [PATCH] test: cleanup gdb external dependency in atomicity tests
@ 2011-12-01 22:02 Dmitry Kurochkin
  2011-12-11 14:49 ` David Bremner
  0 siblings, 1 reply; 2+ messages in thread
From: Dmitry Kurochkin @ 2011-12-01 22:02 UTC (permalink / raw)
  To: notmuch

Change atomicity tests to use the new external binary dependencies.
This simplifies the code and makes output consistent.
---
 test/atomicity |   12 ++++--------
 1 files changed, 4 insertions(+), 8 deletions(-)

diff --git a/test/atomicity b/test/atomicity
index ad7d4a3..6df0a00 100755
--- a/test/atomicity
+++ b/test/atomicity
@@ -7,8 +7,7 @@ test_description='atomicity'
 # final database contents should be the same regardless of when (or
 # if) it is killed and restarted.
 
-if which gdb 1>/dev/null 2>&1; then
-    test_set_prereq GDB
+if test_require_external_prereq gdb; then
 
 # Create a maildir structure to also stress flag synchronization
     mkdir $MAIL_DIR/cur
@@ -91,14 +90,11 @@ if which gdb 1>/dev/null 2>&1; then
 	    i=$(expr $end - 1)
 	fi
     done
-else
-    say_color info "%-6s" "WARNING"
-    echo " Missing test prerequisite GDB"
-fi  
+fi
 
 test_begin_subtest '"notmuch new" is idempotent under arbitrary aborts'
-test_expect_equal_file GDB searchall expectall
+test_expect_equal_file searchall expectall
 
-test_expect_success GDB "detected $outcount>10 abort points" "test $outcount -gt 10"
+test_expect_success "detected $outcount>10 abort points" "test $outcount -gt 10"
 
 test_done
-- 
1.7.7.3

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] test: cleanup gdb external dependency in atomicity tests
  2011-12-01 22:02 [PATCH] test: cleanup gdb external dependency in atomicity tests Dmitry Kurochkin
@ 2011-12-11 14:49 ` David Bremner
  0 siblings, 0 replies; 2+ messages in thread
From: David Bremner @ 2011-12-11 14:49 UTC (permalink / raw)
  To: Dmitry Kurochkin, notmuch

On Fri,  2 Dec 2011 02:02:46 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmail.com> wrote:
> Change atomicity tests to use the new external binary dependencies.
> This simplifies the code and makes output consistent.

Pushed.

d

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-12-11 14:49 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-12-01 22:02 [PATCH] test: cleanup gdb external dependency in atomicity tests Dmitry Kurochkin
2011-12-11 14:49 ` David Bremner

Code repositories for project(s) associated with this public inbox

	https://yhetil.org/notmuch.git/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).