greg/mentat
1
0
Fork 0
Code Issues 294 Pull requests Projects Releases 9 Packages Wiki Activity

Pre: Use SQLite temp files in deftest-db.

Browse source 
These temp files will almost certainly live in memory only, speeding our
test suite evaluation significantly.  Before this patch, in a warmed
REPL environment I get:

Testing datomish.db-test

Ran 19 tests containing 97 assertions.
0 failures, 0 errors.
"Elapsed time: 1408.720681 msecs"
"Elapsed time: 1343.986464 msecs"
"Elapsed time: 1338.660762 msecs"

After this patch, in a warmed REPL environment I get:

Testing datomish.db-test

Ran 19 tests containing 97 assertions.
0 failures, 0 errors.
"Elapsed time: 587.605168 msecs"
"Elapsed time: 569.522333 msecs"
"Elapsed time: 589.080282 msecs"
This commit is contained in:
Nick Alexander 2016-09-02 13:52:18 -07:00
parent cfdce12c1a
commit 94bac9704b
1 changed files with 6 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
src/datomish/test_macros.cljc
View file

@ -5,8 +5,7 @@
(ns datomish.test-macros (ns datomish.test-macros
#?(:cljs #?(:cljs
(:require-macros (:require-macros
[datomish.test-macros] [datomish.test-macros]))
[datomish.node-tempfile-macros]))
(:require (:require
[datomish.pair-chan])) [datomish.pair-chan]))
@ -44,16 +43,8 @@
(defmacro deftest-db (defmacro deftest-db
[n conn-var & body] [n conn-var & body]
`(deftest-async ~n `(deftest-async ~n
(if-cljs (let [~conn-var (datomish.pair-chan/<? (datomish.api/<connect ""))]
(datomish.node-tempfile-macros/with-tempfile [t# (datomish.node-tempfile/tempfile)] (try
(let [~conn-var (datomish.pair-chan/<? (datomish.api/<connect t#))] ~@body
(try (finally
~@body (datomish.pair-chan/<? (datomish.api/<close ~conn-var)))))))
(finally
(datomish.pair-chan/<? (datomish.api/<close ~conn-var))))))
(tempfile.core/with-tempfile [t# (tempfile.core/tempfile)]
(let [~conn-var (datomish.pair-chan/<? (datomish.api/<connect t#))]
(try
~@body
(finally
(datomish.pair-chan/<? (datomish.api/<close ~conn-var)))))))))