Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Side by Side Diff: adblock-android-tests/src/org/adblockplus/libadblockplus/tests/FilterEngineUpdaterTest.java

Issue 29819555: Fix libadblockplus-android tests
Patch Set: Fixed indentation issues. Created Aug. 27, 2018, 8:03 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * This file is part of Adblock Plus <https://adblockplus.org/>, 2 * This file is part of Adblock Plus <https://adblockplus.org/>,
3 * Copyright (C) 2006-present eyeo GmbH 3 * Copyright (C) 2006-present eyeo GmbH
4 * 4 *
5 * Adblock Plus is free software: you can redistribute it and/or modify 5 * Adblock Plus is free software: you can redistribute it and/or modify
6 * it under the terms of the GNU General Public License version 3 as 6 * it under the terms of the GNU General Public License version 3 as
7 * published by the Free Software Foundation. 7 * published by the Free Software Foundation.
8 * 8 *
9 * Adblock Plus is distributed in the hope that it will be useful, 9 * Adblock Plus is distributed in the hope that it will be useful,
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of
11 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the 11 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12 * GNU General Public License for more details. 12 * GNU General Public License for more details.
13 * 13 *
14 * You should have received a copy of the GNU General Public License 14 * You should have received a copy of the GNU General Public License
15 * along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>. 15 * along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>.
16 */ 16 */
17 17
18 package org.adblockplus.libadblockplus.tests; 18 package org.adblockplus.libadblockplus.tests;
19 19
20 import android.os.SystemClock;
21
20 import org.adblockplus.libadblockplus.MockUpdateAvailableCallback; 22 import org.adblockplus.libadblockplus.MockUpdateAvailableCallback;
21 import org.adblockplus.libadblockplus.NoOpUpdaterCallback; 23 import org.adblockplus.libadblockplus.NoOpUpdaterCallback;
22 import org.adblockplus.libadblockplus.ServerResponse; 24 import org.adblockplus.libadblockplus.ServerResponse;
23 25
26 import org.adblockplus.libadblockplus.UpdaterTest;
24 import org.junit.Test; 27 import org.junit.Test;
25 28
26 public class FilterEngineUpdaterTest extends UpdaterTest 29 public class FilterEngineUpdaterTest extends UpdaterTest
27 { 30 {
28 @Test 31 @Test
29 public void testSetRemoveUpdateAvailableCallback() throws InterruptedException 32 public void testSetRemoveUpdateAvailableCallback()
30 { 33 {
31 mockWebRequest.response.setStatus(ServerResponse.NsStatus.OK); 34 mockWebRequest.response.setStatus(ServerResponse.NsStatus.OK);
32 mockWebRequest.response.setResponseStatus(200); 35 mockWebRequest.response.setResponseStatus(200);
33 mockWebRequest.response.setResponse( 36 mockWebRequest.response.setResponse(
34 "{\n" + 37 "{\n" +
35 " \"test\": {\n" + 38 " \"test\": {\n" +
36 " \"version\": \"1.0.2\",\n" + 39 " \"version\": \"1.0.2\",\n" +
37 " \"url\": \"https://downloads.adblockplus.org/test-1.0.2.tar.gz?upd ate\"\n" + 40 " \"url\": \"https://downloads.adblockplus.org/test-1.0.2.tar.gz?upd ate\"\n" +
38 " }\n" + 41 " }\n" +
39 "}"); 42 "}");
40 43
41 MockUpdateAvailableCallback mockUpdateAvailableCallback = 44 MockUpdateAvailableCallback mockUpdateAvailableCallback =
42 new MockUpdateAvailableCallback(0); 45 new MockUpdateAvailableCallback(0);
46 NoOpUpdaterCallback noOpUpdater = new NoOpUpdaterCallback();
43 filterEngine.setUpdateAvailableCallback(mockUpdateAvailableCallback); 47 filterEngine.setUpdateAvailableCallback(mockUpdateAvailableCallback);
44 filterEngine.forceUpdateCheck(new NoOpUpdaterCallback()); 48 filterEngine.forceUpdateCheck(noOpUpdater);
45 Thread.sleep(1000); 49 while (noOpUpdater.getUpdateCount() < 1)
50 {
51 SystemClock.sleep(100);
52 }
53 assertEquals(1, noOpUpdater.getUpdateCount());
46 assertEquals(1, mockUpdateAvailableCallback.getTimesCalled()); 54 assertEquals(1, mockUpdateAvailableCallback.getTimesCalled());
47 55
48 filterEngine.removeUpdateAvailableCallback(); 56 filterEngine.removeUpdateAvailableCallback();
49 filterEngine.forceUpdateCheck(new NoOpUpdaterCallback()); 57 filterEngine.forceUpdateCheck(noOpUpdater);
50 Thread.sleep(1000); 58 while (noOpUpdater.getUpdateCount() < 2)
59 {
60 SystemClock.sleep(100);
61 }
62 assertEquals(2, noOpUpdater.getUpdateCount());
51 assertEquals(1, mockUpdateAvailableCallback.getTimesCalled()); 63 assertEquals(1, mockUpdateAvailableCallback.getTimesCalled());
52 } 64 }
53 } 65 }
OLDNEW

Powered by Google App Engine
This is Rietveld