Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(361)

Unified Diff: webrtc/examples/androidjunit/src/org/appspot/apprtc/util/LooperExecutorTest.java

Issue 2206473003: Remove LooperExecutor and replace it with built-in HandlerThread. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Changes according to magjed's comments. Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webrtc/examples/androidapp/src/org/appspot/apprtc/util/LooperExecutor.java ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/examples/androidjunit/src/org/appspot/apprtc/util/LooperExecutorTest.java
diff --git a/webrtc/examples/androidjunit/src/org/appspot/apprtc/util/LooperExecutorTest.java b/webrtc/examples/androidjunit/src/org/appspot/apprtc/util/LooperExecutorTest.java
deleted file mode 100644
index aecf6b168ac7d362b2cc1c196fa36e680e0527b8..0000000000000000000000000000000000000000
--- a/webrtc/examples/androidjunit/src/org/appspot/apprtc/util/LooperExecutorTest.java
+++ /dev/null
@@ -1,122 +0,0 @@
-/*
- * Copyright 2016 The WebRTC Project Authors. All rights reserved.
- *
- * Use of this source code is governed by a BSD-style license
- * that can be found in the LICENSE file in the root of the source
- * tree. An additional intellectual property rights grant can be found
- * in the file PATENTS. All contributing project authors may
- * be found in the AUTHORS file in the root of the source tree.
- */
-
-package org.appspot.apprtc.util;
-
-import static org.mockito.Mockito.times;
-import static org.mockito.Mockito.verify;
-import static org.mockito.Mockito.verifyNoMoreInteractions;
-import static org.robolectric.Robolectric.shadowOf;
-
-import org.junit.After;
-import org.junit.Before;
-import org.junit.Test;
-import org.junit.runner.RunWith;
-import org.mockito.Mock;
-import org.mockito.MockitoAnnotations;
-import org.robolectric.RobolectricTestRunner;
-import org.robolectric.annotation.Config;
-import org.robolectric.shadows.ShadowLooper;
-
-@RunWith(RobolectricTestRunner.class)
-@Config(manifest = Config.NONE)
-public class LooperExecutorTest {
- private final static int RUN_TIMES = 10;
-
- @Mock private Runnable mockRunnable;
- private LooperExecutor executor;
-
- @Before
- public void setUp() {
- MockitoAnnotations.initMocks(this);
- executor = new LooperExecutor();
- }
-
- @After
- public void tearDown() {
- executor.requestStop();
- executePendingRunnables();
- }
-
- @Test
- public void testExecute() {
- executor.requestStart();
-
- for (int i = 0; i < RUN_TIMES; i++) {
- executor.execute(mockRunnable);
- }
-
- verifyNoMoreInteractions(mockRunnable);
- executePendingRunnables();
- verify(mockRunnable, times(RUN_TIMES)).run();
- }
-
- /**
- * Test that runnables executed before requestStart are ignored.
- */
- @Test
- public void testExecuteBeforeStart() {
- executor.execute(mockRunnable);
-
- executor.requestStart();
- executePendingRunnables();
-
- verifyNoMoreInteractions(mockRunnable);
- }
-
- /**
- * Test that runnables executed after requestStop are not executed.
- */
- @Test
- public void testExecuteAfterStop() {
- executor.requestStart();
- executor.requestStop();
-
- executor.execute(mockRunnable);
- executePendingRunnables();
-
- verifyNoMoreInteractions(mockRunnable);
- }
-
- /**
- * Test multiple requestStart calls are just ignored.
- */
- @Test
- public void testMultipleStarts() {
- executor.requestStart();
- testExecute();
- }
-
- /**
- * Test multiple requestStop calls are just ignored.
- */
- @Test
- public void testMultipleStops() {
- executor.requestStart();
- executor.requestStop();
- executor.requestStop();
- executePendingRunnables();
- }
-
- /**
- * Calls ShadowLooper's idle method in order to execute pending runnables.
- */
- private void executePendingRunnables() {
- ShadowLooper shadowLooper = getShadowLooper();
- shadowLooper.idle();
- }
-
- /**
- * Get ShadowLooper of the executor thread.
- */
- private ShadowLooper getShadowLooper() {
- return shadowOf(executor.getHandler().getLooper());
- }
-}
« no previous file with comments | « webrtc/examples/androidapp/src/org/appspot/apprtc/util/LooperExecutor.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698