Could watch also reload local deps?

[msolli]

Hi, and thanks for Lazytest, really enjoying it!

I'm running with watch mode, and noticed that my tests wouldn't re-run when the implementation ns changed. I have a Polylith project, and it requires that "components" are included using :extra-deps (in the :dev alias), but test are included using :extra-paths in the :test alias.

I went and looked at the code, and it seems every path on the classpath that is an absolute path is removed from the list of dirs to be watched: https://github.com/NoahTheDuke/lazytest/blob/main/src/clojure/lazytest/watch.clj#L19

I don't have the whole picture, but would it be possible to not remove these absolute paths? It would solve the issue for me, at least, because code in my :extra-deps would be eligible for reloading.

[msolli]

As a local experiment I've monkey-patched get-local-classpath such that only files (that is, JAR files) on the classpath are excluded from reloading. It seems to work fine for us:

(ns dev.test
  (:require [clojure.java.io :as io]
            [clojure.string :as str]
            [lazytest.main :as l.main]
            [lazytest.watch :as l.watch])
  (:import (java.io File)))

(defn get-local-classpath []
  (-> (System/getProperty "java.class.path")
      (str/split (re-pattern File/pathSeparator))
      (->> (into []
                 (comp (map io/file)
                       (remove #(File/.isFile ^File %)))))))

;; Monkey-patch lazytest.watch/get-local-classpath to use our implementation.
;; Instead of filtering out all paths that are absolute, this one filters out files (JAR files).
;; This means that :local/root deps are included in the watched paths.
(alter-var-root #'l.watch/get-local-classpath (constantly get-local-classpath))

(defn -main
  "Wrap lazytest.main/-main so that we can monkey-patch the function that determines the paths to watch."
  [& args]
  (apply l.main/-main args))

In deps.edn we use ["-m" "dev.test"] rather than ["-m" "lazytest.main"] for :main-opts.

It works as workaround for us. Hope this is useful!

[NoahTheDuke]

Oh hell yeah. If you open a PR, I'll review!