From cc29ff70e5cf92f718e0b647ed5f934077dece40 Mon Sep 17 00:00:00 2001
From: Roman Shevchenko <roman.shevchenko@jetbrains.com>
Date: Thu, 5 Jan 2017 19:25:52 +0100
Subject: [PATCH] Cleanup (further simplifies VM options bean test)

---
 .../src/com/intellij/diagnostic/VMOptions.java  | 17 -----------------
 .../com/intellij/diagnostic/VMOptionsTest.java  | 10 ++++------
 2 files changed, 4 insertions(+), 23 deletions(-)

diff --git a/platform/platform-impl/src/com/intellij/diagnostic/VMOptions.java b/platform/platform-impl/src/com/intellij/diagnostic/VMOptions.java
index cadf2260b988..9e795d442703 100644
--- a/platform/platform-impl/src/com/intellij/diagnostic/VMOptions.java
+++ b/platform/platform-impl/src/com/intellij/diagnostic/VMOptions.java
@@ -24,7 +24,6 @@ import com.intellij.openapi.util.text.StringUtil;
 import com.intellij.util.SystemProperties;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
-import org.jetbrains.annotations.TestOnly;
 
 import java.io.File;
 import java.io.IOException;
@@ -161,10 +160,6 @@ public class VMOptions {
 
   @Nullable
   public static File getWriteFile() {
-    if (ourTestPath != null) {
-      return new File(ourTestPath);
-    }
-
     String vmOptionsFile = System.getProperty("jb.vmOptionsFile");
     if (vmOptionsFile == null) {
       // launchers should specify a path to an options file used to configure a JVM
@@ -188,18 +183,6 @@ public class VMOptions {
     return new File(location, fileName);
   }
 
-  private static String ourTestPath;
-
-  @TestOnly
-  static void setTestFile(String path) {
-    ourTestPath = path;
-  }
-
-  @TestOnly
-  static void clearTestFile() {
-    ourTestPath = null;
-  }
-
   //<editor-fold desc="Deprecated stuff.">
   /** @deprecated use {@link #readOption(MemoryKind, boolean)} (to be removed in IDEA 2018) */
   public static int readXmx() {
diff --git a/platform/platform-tests/testSrc/com/intellij/diagnostic/VMOptionsTest.java b/platform/platform-tests/testSrc/com/intellij/diagnostic/VMOptionsTest.java
index 529ba651c3e3..2bf78290f525 100644
--- a/platform/platform-tests/testSrc/com/intellij/diagnostic/VMOptionsTest.java
+++ b/platform/platform-tests/testSrc/com/intellij/diagnostic/VMOptionsTest.java
@@ -37,12 +37,12 @@ public class VMOptionsTest {
   public void setUp() throws IOException {
     myFile = myTempDir.newFile("vmoptions.txt");
     FileUtil.writeToFile(myFile, "-Xmx512m\n-XX:MaxPermSize=128m");
-    VMOptions.setTestFile(myFile.getPath());
+    System.setProperty("jb.vmOptionsFile", myFile.getPath());
   }
 
   @After
   public void tearDown() {
-    VMOptions.clearTestFile();
+    System.clearProperty("jb.vmOptionsFile");
   }
 
   @Test
@@ -125,13 +125,11 @@ public class VMOptionsTest {
 
   @Test
   public void testWritingNonExistingFile() throws IOException {
-    File testFile = myTempDir.newFile("vmoptions.non.existing.txt");
-    FileUtil.delete(testFile);
-    VMOptions.setTestFile(testFile.getPath());
+    FileUtil.delete(myFile);
 
     VMOptions.writeOption(VMOptions.MemoryKind.HEAP, 1024);
     VMOptions.writeOption(VMOptions.MemoryKind.PERM_GEN, 256);
 
-    assertThat(FileUtil.loadFile(testFile)).isEqualToIgnoringWhitespace("-Xmx1024m -XX:MaxPermSize=256m");
+    assertThat(FileUtil.loadFile(myFile)).isEqualToIgnoringWhitespace("-Xmx1024m -XX:MaxPermSize=256m");
   }
 }
\ No newline at end of file