Tests fail if hypen in directory name

Description

Check out compiler into a foo-bar directory and run lein test :only cljs.module-processing-tests.

Seems very similar to CLJS-2703.

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 $ lein test :only cljs.module-processing-tests lein test cljs.module-processing-tests lein test :only cljs.module-processing-tests/commonjs-module-processing-preprocess-symbol FAIL in (commonjs-module-processing-preprocess-symbol) (module_processing_tests.clj:191) Processed modules are added to :js-module-index expected: (= {"React" {:name (absolute-module-path "src/test/cljs/reactJS.js"), :module-type :commonjs}, "Circle" {:name (absolute-module-path "src/test/cljs/Circle.js"), :module-type :commonjs}} (:js-module-index (clojure.core/deref cenv))) actual: (not (= {"React" {:name "module$Users$mfikes$foo_bar$clojurescript$src$test$cljs$reactJS", :module-type :commonjs}, "Circle" {:name "module$Users$mfikes$foo_bar$clojurescript$src$test$cljs$Circle", :module-type :commonjs}} {"React" {:name "module$Users$mfikes$foo-bar$clojurescript$src$test$cljs$reactJS", :module-type :commonjs}, "Circle" {:name "module$Users$mfikes$foo-bar$clojurescript$src$test$cljs$Circle", :module-type :commonjs}})) lein test :only cljs.module-processing-tests/es6-module-processing FAIL in (es6-module-processing) (module_processing_tests.clj:97) Processed modules are added to :js-module-index expected: (= {"es6-hello" {:name (absolute-module-path "src/test/cljs/es6_hello.js"), :module-type :es6}} (:js-module-index (clojure.core/deref cenv))) actual: (not (= {"es6-hello" {:name "module$Users$mfikes$foo_bar$clojurescript$src$test$cljs$es6-hello", :module-type :es6}} {"es6-hello" {:name "module$Users$mfikes$foo-bar$clojurescript$src$test$cljs$es6-hello", :module-type :es6}})) lein test :only cljs.module-processing-tests/test-cljs-1822 FAIL in (test-cljs-1822) (module_processing_tests.clj:162) Processed modules are added to :js-module-index expected: (= {"React" {:name (absolute-module-path "src/test/cljs/react-min.js"), :module-type :commonjs}, "Circle" {:name (absolute-module-path "src/test/cljs/Circle-min.js"), :module-type :commonjs}} (:js-module-index (clojure.core/deref cenv))) actual: (not (= {"React" {:name "module$Users$mfikes$foo_bar$clojurescript$src$test$cljs$react-min", :module-type :commonjs}, "Circle" {:name "module$Users$mfikes$foo_bar$clojurescript$src$test$cljs$Circle-min", :module-type :commonjs}} {"React" {:name "module$Users$mfikes$foo-bar$clojurescript$src$test$cljs$react-min", :module-type :commonjs}, "Circle" {:name "module$Users$mfikes$foo-bar$clojurescript$src$test$cljs$Circle-min", :module-type :commonjs}})) lein test :only cljs.module-processing-tests/commonjs-module-processing FAIL in (commonjs-module-processing) (module_processing_tests.clj:71) Processed modules are added to :js-module-index expected: (= {"React" {:name (absolute-module-path "src/test/cljs/reactJS.js"), :module-type :commonjs}, "Circle" {:name (absolute-module-path "src/test/cljs/Circle.js"), :module-type :commonjs}} (:js-module-index (clojure.core/deref cenv))) actual: (not (= {"React" {:name "module$Users$mfikes$foo_bar$clojurescript$src$test$cljs$reactJS", :module-type :commonjs}, "Circle" {:name "module$Users$mfikes$foo_bar$clojurescript$src$test$cljs$Circle", :module-type :commonjs}} {"React" {:name "module$Users$mfikes$foo-bar$clojurescript$src$test$cljs$reactJS", :module-type :commonjs}, "Circle" {:name "module$Users$mfikes$foo-bar$clojurescript$src$test$cljs$Circle", :module-type :commonjs}})) Ran 5 tests containing 14 assertions. 4 failures, 0 errors. Tests failed.

Environment

None

Status

Assignee

Ray Mcdermott

Reporter

Mike Fikes

Labels

None

Approval

None

Patch

None

Priority

Major
Configure