Don't put a literal suite object into a test form's macro expnsion.
authorMarco Baringer <mb@bese.it>
Tue, 26 Jun 2007 11:49:00 +0000 (13:49 +0200)
committerMarco Baringer <mb@bese.it>
Tue, 26 Jun 2007 11:49:00 +0000 (13:49 +0200)
Use either *SUITE* or (GET-TEST NAME)

src/test.lisp

index 9190003..29e8361 100644 (file)
@@ -50,9 +50,9 @@ FIXTURE specifies a fixtrue to wrap the body in.
 If PROFILE is T profiling information will be collected as well."
   (let* ((tmp (gensym))
          (suite-arg (getf (cdr (ensure-list name)) :suite tmp))
-         (suite (cond
-                  ((eq tmp suite-arg) *suite*)
-                  (t (get-test suite-arg)))))
+         (suite-form (cond
+                       ((eq tmp suite-arg) '*suite*)
+                       (t                  `(get-test ',suite-arg)))))
     (when (consp name)
       (remf (cdr name) :suite))
     (destructuring-bind (name &key depends-on (compile-at :run-time) fixture profile)
@@ -81,7 +81,7 @@ If PROFILE is T profiling information will be collected as well."
                                                   :description ,description
                                                   :depends-on ',depends-on
                                                   :collect-profiling-info ,profile))
-           (setf (gethash ',name (tests (get-test ',(name suite)))) ',name)
+           (setf (gethash ',name (tests ,suite-form)) ',name)
            (when *run-test-when-defined*
              (run! ',name))
            ',name)))))