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

-- MOS_MIGRATED_REVID=108983674
nitinreddy3 · Nov 30, 2015 · 18565b0 · 18565b0
1 parent 4359ca2
commit 18565b0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/test/java/com/google/devtools/build/lib/concurrent/KeyedLockerTest.java b/src/test/java/com/google/devtools/build/lib/concurrent/KeyedLockerTest.java
@@ -49,14 +49,14 @@ public abstract class KeyedLockerTest {
   protected abstract KeyedLocker<String> makeFreshLocker();
 
   @Before
-  public void setUp_KeyedLockerTest() {
+  public final void setUp_KeyedLockerTest() {
     locker = makeFreshLocker();
     executorService = Executors.newFixedThreadPool(NUM_EXECUTOR_THREADS);
     wrapper = new ThrowableRecordingRunnableWrapper("KeyedLockerTest");
   }
 
   @After
-  public void tearDown() {
+  public final void shutdownExecutor() throws Exception  {
     locker = null;
     MoreExecutors.shutdownAndAwaitTermination(executorService, TestUtils.WAIT_TIMEOUT_SECONDS,
         TimeUnit.SECONDS);

diff --git a/src/test/java/com/google/devtools/build/lib/concurrent/MoreFuturesTest.java b/src/test/java/com/google/devtools/build/lib/concurrent/MoreFuturesTest.java
@@ -47,12 +47,12 @@ public class MoreFuturesTest {
   private ExecutorService executorService;
 
   @Before
-  public void setUp() throws Exception {
+  public final void createExecutor() throws Exception  {
     executorService = Executors.newFixedThreadPool(5);
   }
 
   @After
-  public void tearDown() throws Exception {
+  public final void shutdownExecutor() throws Exception  {
     MoreExecutors.shutdownAndAwaitTermination(executorService, TestUtils.WAIT_TIMEOUT_SECONDS,
         TimeUnit.SECONDS);