Skip to content

Make the tests run locally again under Windows #312

Closed
@expez

Description

@expez

This snippet:

(ns global-test-setup
  "This namespace's contents will be automatically loaded by `lein test` (or any test runner)."
  (:require
   [refactor-nrepl.inlined-deps.toolsnamespace.v1v1v0.clojure.tools.namespace.repl :refer [set-refresh-dirs]]))

references an ns that doesn't exist until mranderson has run.

My current side project is Windows only and mranderson doesn't run on Windows. The presence of his ns therefore
makes it impossible to run the tests locally. Using the CI server to test that everything is OK makes the feedback loop quite a bit slower.

Why is this ns referencing a namespace that doesn't exist?

I also don't get this part:

(when (System/getenv "CI") ; don't alter local environments (which may also use clojure.tools.namespace.repl)
  ;; Intentionally exclude the `lein-plugin/` source path:
  (set-refresh-dirs "src" "test"))

Why would we need a refresh dir to be set on the CI server instead of locally? It looks like this NS is a noop as written. Can we just remove this file?

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions