From c37a3713d3b5f2318dc111f190e163960280309c Mon Sep 17 00:00:00 2001
From: "Natalia.Ukhorskaya" <natalia.ukhorskaya@jetbrains.com>
Date: Tue, 28 May 2013 17:27:55 +0400
Subject: [PATCH] Fix tests for j2converter

---
 j2k/src/org/jetbrains/jet/j2k/Converter.java  |  9 ++++++
 j2k/tests/j2k-tests.iml                       |  1 +
 .../StandaloneJavaToKotlinConverterTest.java  | 32 ++++---------------
 3 files changed, 17 insertions(+), 25 deletions(-)

diff --git a/j2k/src/org/jetbrains/jet/j2k/Converter.java b/j2k/src/org/jetbrains/jet/j2k/Converter.java
index 9cefa372f01..f4abee3d06a 100644
--- a/j2k/src/org/jetbrains/jet/j2k/Converter.java
+++ b/j2k/src/org/jetbrains/jet/j2k/Converter.java
@@ -46,6 +46,9 @@ public class Converter {
         "javax.annotation.Nonnull"
     );
 
+    @NotNull
+    private final Project project;
+
     @NotNull
     private Set<String> classIdentifiers = Sets.newHashSet();
 
@@ -59,9 +62,15 @@ public class Converter {
     private final Set<J2KConverterFlags> flags = Sets.newHashSet();
 
     public Converter(@NotNull Project project) {
+        this.project = project;
         KotlinBuiltIns.initialize(project, KotlinBuiltIns.InitializationMode.MULTI_THREADED);
     }
 
+    @NotNull
+    public Project getProject() {
+        return project;
+    }
+
     public boolean addFlag(@NotNull J2KConverterFlags flag) {
         return flags.add(flag);
     }
diff --git a/j2k/tests/j2k-tests.iml b/j2k/tests/j2k-tests.iml
index 0d3d4fd6bce..bd44f2f04ce 100644
--- a/j2k/tests/j2k-tests.iml
+++ b/j2k/tests/j2k-tests.iml
@@ -13,6 +13,7 @@
     <orderEntry type="module" module-name="cli" scope="TEST" />
     <orderEntry type="module" module-name="frontend" scope="TEST" />
     <orderEntry type="module" module-name="util" />
+    <orderEntry type="module" module-name="compiler-tests" scope="TEST" />
   </component>
 </module>
 
diff --git a/j2k/tests/test/org/jetbrains/jet/j2k/StandaloneJavaToKotlinConverterTest.java b/j2k/tests/test/org/jetbrains/jet/j2k/StandaloneJavaToKotlinConverterTest.java
index b5a97417e62..de1f996a202 100644
--- a/j2k/tests/test/org/jetbrains/jet/j2k/StandaloneJavaToKotlinConverterTest.java
+++ b/j2k/tests/test/org/jetbrains/jet/j2k/StandaloneJavaToKotlinConverterTest.java
@@ -16,44 +16,24 @@
 
 package org.jetbrains.jet.j2k;
 
-import com.intellij.openapi.Disposable;
 import com.intellij.openapi.util.io.FileUtil;
 import com.intellij.psi.PsiFile;
 import com.intellij.psi.PsiJavaFile;
+import com.intellij.testFramework.UsefulTestCase;
 import junit.framework.Assert;
 import junit.framework.Test;
-import junit.framework.TestCase;
 import junit.framework.TestSuite;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
-import org.jetbrains.jet.cli.jvm.JVMConfigurationKeys;
+import org.jetbrains.jet.ConfigurationKind;
 import org.jetbrains.jet.cli.jvm.compiler.JetCoreEnvironment;
-import org.jetbrains.jet.config.CompilerConfiguration;
-import org.jetbrains.jet.utils.PathUtil;
 
 import java.io.File;
 import java.io.IOException;
 
-public class StandaloneJavaToKotlinConverterTest extends TestCase {
-    private static final JetCoreEnvironment jetCoreEnvironment;
-
-    static {
-        CompilerConfiguration config = new CompilerConfiguration();
-
-        config.add(JVMConfigurationKeys.CLASSPATH_KEY, PathUtil.findRtJar());
-        File annotations = JavaToKotlinTranslator.findAnnotations();
-        if (annotations != null && annotations.exists()) {
-            config.add(JVMConfigurationKeys.CLASSPATH_KEY, annotations);
-        }
-
-        Disposable disposable = new Disposable() {
-            @Override
-            public void dispose() {
-            }
-        };
-        jetCoreEnvironment = new JetCoreEnvironment(disposable, config);
-    }
+import static org.jetbrains.jet.JetTestUtils.createEnvironmentWithMockJdkAndIdeaAnnotations;
 
+public class StandaloneJavaToKotlinConverterTest extends UsefulTestCase {
     private final String myDataPath;
     private final String myName;
 
@@ -64,7 +44,9 @@ public class StandaloneJavaToKotlinConverterTest extends TestCase {
 
     @Override
     protected void runTest() throws Throwable {
+        JetCoreEnvironment jetCoreEnvironment = createEnvironmentWithMockJdkAndIdeaAnnotations(getTestRootDisposable(), ConfigurationKind.JDK_ONLY);
         Converter converter = new Converter(jetCoreEnvironment.getProject());
+
         String javaPath = "j2k/tests/testData/" + getTestFilePath();
         String kotlinPath = javaPath.replace(".jav", ".kt");
 
@@ -140,7 +122,7 @@ public class StandaloneJavaToKotlinConverterTest extends TestCase {
 
     @NotNull
     private String fileToKotlin(Converter converter, @NotNull String text) {
-        return generateKotlinCode(converter, JavaToKotlinTranslator.createFile(jetCoreEnvironment.getProject(), text));
+        return generateKotlinCode(converter, JavaToKotlinTranslator.createFile(converter.getProject(), text));
     }
 
     @NotNull