Remove unnecessary additional target thanks to cmdline option

docs/dev-practices.md

@@ -22,4 +22,8 @@ We have unit and end to end tests as described in https://github.com/logseq/logs
 
 ### Unit Testing
 
-When writing unit tests it is helpful to have tests automatically run on file save with `clojure -M:test watch test-auto`. For more about the shadow-cljs test runner, see [this documentation](https://shadow-cljs.github.io/docs/UsersGuide.html#_testing).
+When writing unit tests it is helpful to have tests automatically run on file
+save with `yarn shadow-cljs watch test --config-merge '{:autorun true}'`. The
+test output may appear where shadow-cljs was first invoked e.g. where `yarn
+watch` is running. For more about the shadow-cljs test runner, see [this
+documentation](https://shadow-cljs.github.io/docs/UsersGuide.html#_testing).

shadow-cljs.edn

@@ -71,12 +71,6 @@
          :closure-defines {frontend.util/NODETEST true}
          :devtools        {:enabled false}}
 
-  :test-auto {:target          :node-test
-              :output-to       "static/tests.js"
-              :closure-defines {frontend.util/NODETEST true}
-              :devtools        {:enabled false}
-              :autorun true}
-
   :publishing {:target        :browser
                :module-loader true
                :js-options    {:ignore-asset-requires true}