luciano/specter
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

finish test for inline caching

Browse source
This commit is contained in:
Nathan Marz 2016-05-23 15:17:47 -04:00
parent 4f3990c239
commit 8dd0f7e168
1 changed files with 20 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

34
test/com/rpl/specter/core_test.cljx
View file

@ -908,18 +908,24 @@
[{:a 1 :b 2} {:a 10 :b 6} {:c 7 :b 8} {:c 1 :d 9} {:c 3 :d -1}]
[[:a :b] [:b :a] [:c :d] [:b :c]]
)
;; TODO: test exception thrown when can't cache and must-cache-paths!
(s/must-cache-paths!)
(is (thrown? #+clj Exception #+cljs js/Error
(select (if true :a :b) nil)
))
(is (thrown? #+clj Exception #+cljs js/Error
(select (*APATH* :a) nil)
))
(is (thrown? #+clj Exception #+cljs js/Error
(select [:a (identity even?)] {:a 2})
))
(let [p :a]
(is (thrown? #+clj Exception #+cljs js/Error
(select [p even?] {:a 2})
)))
(let [p :a]
(is (thrown? #+clj Exception #+cljs js/Error
(select [s/ALL (s/selected? p even?)] [{:a 2}])
)))
(s/must-cache-paths! false)
)
;;TODO:
;; make a macro to help with testing inline caching
;; - put into test-helpers
;; - do must-cache-paths!
;; - take in params vec, path expression, data, transform fn, and sequence of param vecs
;; - verify that functional version gets same result as inline
;; caching version run on the data multiple times (make sure same callsite)
;; - have another test that turns must-cache-paths on and verifies certain paths
;; don't cache (but still run with must-cache-paths off)