CLJS-1469 :modules regression

The sortInputsByDeps method now depends on the Google CompilerOptions
which need to be set before calling the method.

A test to check for similar regressions was added.

Author: bensu

src/main/clojure/cljs/closure.clj

@@ -965,6 +965,7 @@
   (let [closure-compiler (make-closure-compiler)
         ^List externs (load-externs opts)
         compiler-options (make-options opts)
+        _ (.initOptions closure-compiler compiler-options)
         sources (if (= :whitespace (:optimizations opts))
                   (cons "var CLOSURE_NO_DEPS = true;" sources)
                   sources)

src/test/clojure/cljs/build_api_tests.clj

@@ -2,7 +2,9 @@
   (:refer-clojure :exclude [compile])
   (:use cljs.build.api)
   (:use clojure.test)
-  (:require [cljs.env :as env]
+  (:import java.io.File)
+  (:require [clojure.java.io :as io]
+            [cljs.env :as env]
             [cljs.analyzer :as ana]))
 
 (deftest test-target-file-for-cljs-ns
@@ -114,3 +116,20 @@
         '(graph.bar.core graph.baz.core)))
   (is (= (cljs-ns-dependents test-cenv 'graph.foo.core)
         '(graph.bar.core graph.baz.core))))
+
+(deftest cljs-1469
+  (let [srcs "samples/hello/src"
+        [common-tmp app-tmp] (mapv #(File/createTempFile  % ".js")
+                                   ["common" "app"])
+        opts {:optimizations :simple
+              :modules {:common {:entries #{"hello.foo.bar"}
+                                 :output-to (.getAbsolutePath common-tmp)}
+                        :app {:entries #{"hello.core"}
+                              :output-to (.getAbsolutePath app-tmp)}}}]
+    (.deleteOnExit common-tmp)
+    (.deleteOnExit app-tmp)
+    (is (every? #(zero? (.length %)) [common-tmp app-tmp])
+      "The initial files are empty")
+    (build srcs opts)
+    (is (not (every? #(zero? (.length %)) [common-tmp app-tmp]))
+      "The files are not empty after compilation")))