[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
master 8bdea76: Remove hack from seq-tests.el for broken `with-suppresse
From: |
Lars Ingebrigtsen |
Subject: |
master 8bdea76: Remove hack from seq-tests.el for broken `with-suppressed-warnings' |
Date: |
Sat, 4 Dec 2021 22:21:03 -0500 (EST) |
branch: master
commit 8bdea767208257599f6ab727e51dd94f0c1872e1
Author: Lars Ingebrigtsen <larsi@gnus.org>
Commit: Lars Ingebrigtsen <larsi@gnus.org>
Remove hack from seq-tests.el for broken `with-suppressed-warnings'
* test/lisp/emacs-lisp/seq-tests.el: `with-suppressed-warnings'
now works better in erf-deftests, so remove hack that works around
it (bug#36568).
---
test/lisp/emacs-lisp/seq-tests.el | 24 ++++++++++--------------
1 file changed, 10 insertions(+), 14 deletions(-)
diff --git a/test/lisp/emacs-lisp/seq-tests.el
b/test/lisp/emacs-lisp/seq-tests.el
index 8cfa3bd..4b940af 100644
--- a/test/lisp/emacs-lisp/seq-tests.el
+++ b/test/lisp/emacs-lisp/seq-tests.el
@@ -172,23 +172,19 @@ Evaluate BODY for each created sequence.
(should-not (seq-find #'null '(1 2 3)))
(should (seq-find #'null '(1 2 3) 'sentinel)))
-;; Hack to work around the ERT limitation that we can't reliably use
-;; `with-suppressed-warnings' inside an `ert-deftest'. (Bug#36568)
-(defun seq--contains (&rest args)
- (with-suppressed-warnings ((obsolete seq-contains))
- (apply #'seq-contains args)))
-
(ert-deftest test-seq-contains ()
- (with-test-sequences (seq '(3 4 5 6))
- (should (seq--contains seq 3))
- (should-not (seq--contains seq 7)))
- (with-test-sequences (seq '())
- (should-not (seq--contains seq 3))
- (should-not (seq--contains seq nil))))
+ (with-suppressed-warnings ((obsolete seq-contains))
+ (with-test-sequences (seq '(3 4 5 6))
+ (should (seq-contains seq 3))
+ (should-not (seq-contains seq 7)))
+ (with-test-sequences (seq '())
+ (should-not (seq-contains seq 3))
+ (should-not (seq-contains seq nil)))))
(ert-deftest test-seq-contains-should-return-the-elt ()
- (with-test-sequences (seq '(3 4 5 6))
- (should (= 5 (seq--contains seq 5)))))
+ (with-suppressed-warnings ((obsolete seq-contains))
+ (with-test-sequences (seq '(3 4 5 6))
+ (should (= 5 (seq-contains seq 5))))))
(ert-deftest test-seq-contains-p ()
(with-test-sequences (seq '(3 4 5 6))
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- master 8bdea76: Remove hack from seq-tests.el for broken `with-suppressed-warnings',
Lars Ingebrigtsen <=