[PATCH] test: remove remainder of previously killed basic test

classic Classic list List threaded Threaded
2 messages Options
Yuri Volchkov Yuri Volchkov
Reply | Threaded
Open this post in threaded view
|

[PATCH] test: remove remainder of previously killed basic test

In the commit 51cd69feb1d131db7a468e33e0fa2e043caad41e the part of the
test "test runs if prerequisite is satisfied" has been
removed. However, there was a remainder of that test - variable
'haveit'.

Kill it, to not to confuse people.

Signed-off-by: Yuri Volchkov <[hidden email]>
---
 test/T000-basic.sh | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/test/T000-basic.sh b/test/T000-basic.sh
index 36a7ca4..a4efa3c 100755
--- a/test/T000-basic.sh
+++ b/test/T000-basic.sh
@@ -23,8 +23,7 @@ test_expect_success ':'
 
 test_begin_subtest 'test runs if prerequisite is satisfied'
 test_set_prereq HAVEIT
-haveit=no
-test_expect_success 'test_have_prereq HAVEIT && haveit=yes'
+test_expect_success 'test_have_prereq HAVEIT'
 
 test_begin_subtest 'tests clean up after themselves'
 clean=no
--
2.7.4

_______________________________________________
notmuch mailing list
[hidden email]
https://notmuchmail.org/mailman/listinfo/notmuch
David Bremner-2 David Bremner-2
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] test: remove remainder of previously killed basic test

Yuri Volchkov <[hidden email]> writes:

> In the commit 51cd69feb1d131db7a468e33e0fa2e043caad41e the part of the
> test "test runs if prerequisite is satisfied" has been
> removed. However, there was a remainder of that test - variable
> 'haveit'.
>
> Kill it, to not to confuse people.
>

pushed to master

d
_______________________________________________
notmuch mailing list
[hidden email]
https://notmuchmail.org/mailman/listinfo/notmuch