js.frontend new module

js/js.frontend/js.frontend.iml

@@ -0,0 +1,16 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<module type="JAVA_MODULE" version="4">
+  <component name="NewModuleRootManager" inherit-compiler-output="true">
+    <exclude-output />
+    <content url="file://$MODULE_DIR$">
+      <sourceFolder url="file://$MODULE_DIR$/src" isTestSource="false" />
+    </content>
+    <orderEntry type="inheritedJdk" />
+    <orderEntry type="sourceFolder" forTests="false" />
+    <orderEntry type="module" module-name="frontend" />
+    <orderEntry type="library" scope="PROVIDED" name="intellij-core" level="project" />
+    <orderEntry type="module" module-name="util" />
+    <orderEntry type="module" module-name="js.dart-ast" exported="" />
+  </component>
+</module>
+

js/js.frontend/src/org/jetbrains/k2js/config/Config.java

@@ -23,8 +23,6 @@ import org.jetbrains.annotations.Nullable;
 import org.jetbrains.jet.lang.descriptors.ModuleDescriptor;
 import org.jetbrains.jet.lang.psi.JetFile;
 import org.jetbrains.jet.lang.resolve.BindingContext;
-import org.jetbrains.k2js.translate.test.JSTester;
-import org.jetbrains.k2js.translate.test.QUnitTester;
 
 import java.util.Arrays;
 import java.util.Collection;
@@ -222,9 +220,7 @@ public abstract class Config {
         return allFiles;
     }
 
-    //TODO: should be null by default I suppose but we can't communicate it to K2JSCompiler atm
-    @Nullable
-    public JSTester getTester() {
-        return new QUnitTester();
+    public boolean isTestConfig() {
+        return false;
     }
 }

js/js.tests/test/org/jetbrains/k2js/test/config/TestConfigWithUnitTests.java

@@ -22,8 +22,6 @@ import org.jetbrains.jet.lang.descriptors.ModuleDescriptor;
 import org.jetbrains.jet.lang.psi.JetFile;
 import org.jetbrains.jet.lang.resolve.BindingContext;
 import org.jetbrains.k2js.config.EcmaVersion;
-import org.jetbrains.k2js.translate.test.JSRhinoUnitTester;
-import org.jetbrains.k2js.translate.test.JSTester;
 
 import java.util.List;
 
@@ -41,10 +39,6 @@ public class TestConfigWithUnitTests extends TestConfig {
         }
     };
 
-    @Override
-    public JSTester getTester() {
-        return new JSRhinoUnitTester();
-    }
 
     public TestConfigWithUnitTests(@NotNull Project project,
             @NotNull EcmaVersion version,
@@ -53,4 +47,9 @@ public class TestConfigWithUnitTests extends TestConfig {
             @NotNull ModuleDescriptor module) {
         super(project, version, files, libraryContext, module, false);
     }
+
+    @Override
+    public boolean isTestConfig() {
+        return true;
+    }
 }

js/js.translator/js.translator.iml

@@ -12,6 +12,7 @@
     <orderEntry type="module" module-name="util" />
     <orderEntry type="module" module-name="backend-common" />
     <orderEntry type="module" module-name="js.dart-ast" exported="" />
+    <orderEntry type="module" module-name="js.frontend" exported="" />
   </component>
 </module>
 

js/js.translator/src/org/jetbrains/k2js/translate/general/Translation.java

@@ -42,8 +42,10 @@ import org.jetbrains.k2js.translate.declaration.PackageDeclarationTranslator;
 import org.jetbrains.k2js.translate.expression.ExpressionVisitor;
 import org.jetbrains.k2js.translate.expression.FunctionTranslator;
 import org.jetbrains.k2js.translate.expression.PatternTranslator;
+import org.jetbrains.k2js.translate.test.JSRhinoUnitTester;
 import org.jetbrains.k2js.translate.test.JSTestGenerator;
 import org.jetbrains.k2js.translate.test.JSTester;
+import org.jetbrains.k2js.translate.test.QUnitTester;
 import org.jetbrains.k2js.translate.utils.JsAstUtils;
 import org.jetbrains.k2js.translate.utils.dangerous.DangerousData;
 import org.jetbrains.k2js.translate.utils.dangerous.DangerousTranslator;
@@ -162,12 +164,10 @@ public final class Translation {
 
     private static void mayBeGenerateTests(@NotNull Collection<JetFile> files, @NotNull Config config,
             @NotNull JsBlock rootBlock, @NotNull TranslationContext context) {
-        JSTester tester = config.getTester();
-        if (tester != null) {
-            tester.initialize(context, rootBlock);
-            JSTestGenerator.generateTestCalls(context, files, tester);
-            tester.deinitialize();
-        }
+        JSTester tester = config.isTestConfig() ? new JSRhinoUnitTester() : new QUnitTester();
+        tester.initialize(context, rootBlock);
+        JSTestGenerator.generateTestCalls(context, files, tester);
+        tester.deinitialize();
     }
 
     //TODO: determine whether should throw exception