commit c4975fabe94417d8aa793ae2728c29750d42dde2
author Florian Weikert <fwe@google.com> Thu Dec 03 10:27:10 2015 +0000
committer Kristina Chodorow <kchodorow@google.com> Thu Dec 03 18:37:44 2015 +0000
tree 04b5b168b3d79b3993a15df2390f3cb20211aaf5
parent 92b22366a6a7bace048b48e56931c8c1ea111045

Migrated remaining tests in devtools/build/lib to JUnit 4.

--
MOS_MIGRATED_REVID=109287267

src/test/java/com/google/devtools/build/lib/standalone/StandaloneSpawnStrategyTest.java

diff --git a/src/test/java/com/google/devtools/build/lib/standalone/StandaloneSpawnStrategyTest.java b/src/test/java/com/google/devtools/build/lib/standalone/StandaloneSpawnStrategyTest.java
index 715b792..6e4bfd2 100644
--- a/src/test/java/com/google/devtools/build/lib/standalone/StandaloneSpawnStrategyTest.java
+++ b/src/test/java/com/google/devtools/build/lib/standalone/StandaloneSpawnStrategyTest.java
@@ -14,6 +14,9 @@
 package com.google.devtools.build.lib.standalone;
 
 import static com.google.common.truth.Truth.assertThat;
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertTrue;
+import static org.junit.Assert.fail;
 
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.ImmutableMap;
@@ -46,7 +49,10 @@
 import com.google.devtools.build.lib.vfs.util.FileSystems;
 import com.google.devtools.common.options.OptionsParser;
 
-import junit.framework.TestCase;
+import org.junit.Before;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.junit.runners.JUnit4;
 
 import java.io.IOException;
 import java.util.Arrays;
@@ -54,7 +60,8 @@
 /**
  * Test StandaloneSpawnStrategy.
  */
-public class StandaloneSpawnStrategyTest extends TestCase {
+@RunWith(JUnit4.class)
+public class StandaloneSpawnStrategyTest {
 
   private Reporter reporter = new Reporter(PrintingEventHandler.ERRORS_AND_WARNINGS_TO_STDERR);
   private BlazeExecutor executor;
@@ -72,9 +79,8 @@
     return testRoot;
   }
 
-  @Override
-  protected void setUp() throws Exception {
-    super.setUp();
+  @Before
+  public final void setUp() throws Exception {
     Path testRoot = createTestRoot();
     Path workspaceDir = testRoot.getRelative(TestConstants.WORKSPACE_NAME);
     workspaceDir.createDirectory();
@@ -120,6 +126,7 @@
     return outErr.errAsLatin1();
   }
 
+  @Test
   public void testBinTrueExecutesFine() throws Exception {
     Spawn spawn = createSpawn(getTrueCommand());
     executor.getSpawnActionContext(spawn.getMnemonic()).exec(spawn, createContext());
@@ -141,6 +148,7 @@
         null);
   }
 
+  @Test
   public void testBinFalseYieldsException() throws Exception {
     try {
       run(createSpawn(getFalseCommand()));
@@ -159,7 +167,7 @@
     return OS.getCurrent() == OS.DARWIN ? "/usr/bin/true" : "/bin/true";
   }
 
-
+  @Test
   public void testBinEchoPrintsArguments() throws Exception {
     Spawn spawn = createSpawn("/bin/echo", "Hello,", "world.");
     run(spawn);
@@ -167,12 +175,14 @@
     assertThat(err()).isEmpty();
   }
 
+  @Test
   public void testCommandRunsInWorkingDir() throws Exception {
     Spawn spawn = createSpawn("/bin/pwd");
     run(spawn);
     assertEquals(executor.getExecRoot() + "\n", out());
   }
 
+  @Test
   public void testCommandHonorsEnvironment() throws Exception {
     Spawn spawn = new BaseSpawn.Local(Arrays.asList("/usr/bin/env"),
         ImmutableMap.of("foo", "bar", "baz", "boo"),
@@ -181,6 +191,7 @@
     assertEquals(Sets.newHashSet("foo=bar", "baz=boo"), Sets.newHashSet(out().split("\n")));
   }
 
+  @Test
   public void testStandardError() throws Exception {
     Spawn spawn = createSpawn("/bin/sh", "-c", "echo Oops! >&2");
     run(spawn);
@@ -191,6 +202,7 @@
   // Test an action with environment variables set indicating an action running on a darwin host 
   // system. Such actions should fail given the fact that these tests run on a non darwin
   // architecture.
+  @Test
   public void testIOSEnvironmentOnNonDarwin() throws Exception {
     if (OS.getCurrent() == OS.DARWIN) {
       return;