[PATCH] test: final named query test works regardless of Xapian FieldProcessor

classic Classic list List threaded Threaded
2 messages Options
Daniel Kahn Gillmor Daniel Kahn Gillmor
Reply | Threaded
Open this post in threaded view
|

[PATCH] test: final named query test works regardless of Xapian FieldProcessor

This test passes with older versions of Xapian as well, because
neither query returns any results.

This should resolve the travis build failure at
https://travis-ci.org/notmuch/notmuch/builds/318571658

Signed-off-by: Daniel Kahn Gillmor <[hidden email]>
---
 test/T600-named-queries.sh | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/test/T600-named-queries.sh b/test/T600-named-queries.sh
index e38dc5bb..abaee3b7 100755
--- a/test/T600-named-queries.sh
+++ b/test/T600-named-queries.sh
@@ -69,9 +69,6 @@ test_expect_equal_file EXPECTED OUTPUT
 test_begin_subtest "search nested named query"
 notmuch search query:test2 > OUTPUT
 notmuch search $QUERYSTR2 > EXPECTED
-if [ $NOTMUCH_HAVE_XAPIAN_FIELD_PROCESSOR -ne 1 ]; then
-    test_subtest_known_broken
-fi
 test_expect_equal_file EXPECTED OUTPUT
 
 test_done
--
2.15.1

_______________________________________________
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: final named query test works regardless of Xapian FieldProcessor

Daniel Kahn Gillmor <[hidden email]> writes:

> This test passes with older versions of Xapian as well, because
> neither query returns any results.
>
> This should resolve the travis build failure at
> https://travis-ci.org/notmuch/notmuch/builds/318571658
>
> Signed-off-by: Daniel Kahn Gillmor <[hidden email]>

pushed to master.

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