fix(wpcarro/emacs): Use should macro in al.el tests

:)

Change-Id: Idcd0e51737c4331b1c95fe2589f0758f443228ac
Reviewed-on: https://cl.tvl.fyi/c/depot/+/6009
Reviewed-by: wpcarro <wpcarro@gmail.com>
Autosubmit: wpcarro <wpcarro@gmail.com>
Tested-by: BuildkiteCI
This commit is contained in:
William Carroll 2022-07-30 07:13:38 -07:00 committed by clbot
parent 3935848a5d
commit 4193f24e5d

View file

@ -10,38 +10,44 @@
;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
(ert-deftest al-has-key? ()
(and
(al-has-key? 'fname '((fname . "William")))
(not (al-has-key? 'lname '((fname . "William"))))))
(should (al-has-key? 'fname '((fname . "William"))))
(should (not (al-has-key? 'lname '((fname . "William"))))))
(ert-deftest al-get ()
(let ((xs (->> (al-new)
(al-set 'fname "John")
(al-set 'employed? nil))))
(and
(string= "John" (al-get 'fname xs))
(string= "Cleese" (al-get 'lname xs "Cleese"))
;; Test that the value of nil is returned even when a default is defined,
;; which could be a subtle bug in the typical Elisp pattern of supporting
;; defaults with: (or foo default).
(eq nil (al-get 'employed? xs))
(eq nil (al-get 'employed? xs "default")))))
(should (string= "John" (al-get 'fname xs)))
(should (string= "Cleese" (al-get 'lname xs "Cleese")))
;; Test that the value of nil is returned even when a default is defined,
;; which could be a subtle bug in the typical Elisp pattern of supporting
;; defaults with: (or foo default).
(should (eq nil (al-get 'employed? xs)))
(should (eq nil (al-get 'employed? xs "default")))))
(ert-deftest al-has-value? ()
(and
(al-has-value? "William" '((fname . "William")))
(not (al-has-key? "John" '((fname . "William"))))))
(should (al-has-value? "William" '((fname . "William"))))
(should (not (al-has-key? "John" '((fname . "William"))))))
(ert-deftest al-map-keys ()
(equal '((2 . one)
(3 . two))
(al-map-keys #'1+
'((1 . one)
(2 . two)))))
(should
(equal '((2 . one)
(3 . two))
(al-map-keys #'1+
'((1 . one)
(2 . two))))))
(ert-deftest al-map-values ()
(equal '((one . 2)
(two . 3))
(al-map-values #'1+
'((one . 1)
(two . 2)))))
(should (equal '((one . 2)
(two . 3))
(al-map-values #'1+
'((one . 1)
(two . 2))))))
(ert-deftest al-delete ()
(let ((person (->> (al-new)
(al-set "fname" "John")
(al-set "lname" "Cleese")
(al-set "age" 82))))
(should (al-has-key? "age" person))
(should (not (al-has-key? "age" (al-delete "age" person))))))