]> code.delx.au - gnu-emacs/commitdiff
* test/automated/ert-tests.el (ert-test-deftest): Add FIXME.
authorStefan Monnier <monnier@iro.umontreal.ca>
Sat, 8 Aug 2015 23:43:14 +0000 (19:43 -0400)
committerStefan Monnier <monnier@iro.umontreal.ca>
Sat, 8 Aug 2015 23:43:14 +0000 (19:43 -0400)
test/automated/ert-tests.el

index fcfc7ee4823ed56046119c7ac24c6cb68b1f3daa..5382c400962fc806ea178cf7b5c72873a5c726d5 100644 (file)
@@ -345,6 +345,10 @@ This macro is used to test if macroexpansion in `should' works."
         (should (equal actual-condition expected-condition)))))))
 
 (ert-deftest ert-test-deftest ()
+  ;; FIXME: These tests don't look very good.  What is their intent, i.e. what
+  ;; are they really testing?  The precise generated code shouldn't matter, so
+  ;; we should either test the behavior of the code, or else try to express the
+  ;; kind of efficiency guarantees we're looking for.
   (should (equal (macroexpand '(ert-deftest abc () "foo" :tags '(bar)))
                 '(progn
                    (ert-set-test 'abc