Skip to content
Browse files

Separate tests into their own system.

This removes the dependency on fiveam from the primary system and means we don't load the tests if we're not using them.

This does NOT change how tests are run. `(asdf:test-system :cl-oauth)` still works.
  • Loading branch information...
1 parent d5d1774 commit 4f676451242aac41c4f08b10889660b3ad84d575 @sellout committed Apr 2, 2012
Showing with 19 additions and 16 deletions.
  1. +19 −16 cl-oauth.asd
View
35 cl-oauth.asd
@@ -41,26 +41,29 @@
(:file "service-provider"
:depends-on ("tokens" "parameters"
"error-handling")))
- :depends-on ("package" "util"))))
- (:module "test"
- :components ((:file "package")
- (:module "core"
- :components ((:file "request-adapter")
- (:file "parameters"
- :depends-on ("request-adapter"))
- (:file "signature"
- :depends-on ("request-adapter"))
- (:file "tokens")
- (:file "service-provider"
- :depends-on ("request-adapter")))
- :depends-on ("package")))
- :depends-on ("src")))
+ :depends-on ("package" "util")))))
:depends-on (:ironclad :cl-base64 :babel
:closer-mop
:alexandria :anaphora :f-underscore :split-sequence
:trivial-garbage
- :fiveam
:drakma
:puri :hunchentoot)
- :in-order-to ((asdf:test-op (load-op "cl-oauth"))))
+ :in-order-to ((test-op (load-op cl-oauth.tests))))
+
+(defmethod operation-done-p ((op test-op) (c (eql (find-system :cl-oauth))))
+ (values nil))
+(defsystem :cl-oauth.tests
+ :depends-on (:fiveam :cl-oauth)
+ :pathname "test/"
+ :components ((:file "package")
+ (:module "core"
+ :components ((:file "request-adapter")
+ (:file "parameters"
+ :depends-on ("request-adapter"))
+ (:file "signature"
+ :depends-on ("request-adapter"))
+ (:file "tokens")
+ (:file "service-provider"
+ :depends-on ("request-adapter")))
+ :depends-on ("package"))))

1 comment on commit 4f67645

@skypher
skypher commented on 4f67645 Apr 3, 2012

Looks good, merged. Thank you!

Please sign in to comment.
Something went wrong with that request. Please try again.