Chromium Code Reviews Chromium Code Reviews
Issue 1324853004:
Add browsertests for VibrationManager java impl on android. (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: content/public/android/javatests/src/org/chromium/content/browser/VibrationManagerImplTest.java |
| diff --git a/content/public/android/javatests/src/org/chromium/content/browser/VibrationManagerImplTest.java b/content/public/android/javatests/src/org/chromium/content/browser/VibrationManagerImplTest.java |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..b29c3c7a1beeb930baf40053a36fbdbe61574883 |
| --- /dev/null |
| +++ b/content/public/android/javatests/src/org/chromium/content/browser/VibrationManagerImplTest.java |
| @@ -0,0 +1,117 @@ |
| +// Copyright 2015 The Chromium Authors. All rights reserved. |
| +// Use of this source code is governed by a BSD-style license that can be |
| +// found in the LICENSE file. |
| + |
| +package org.chromium.content.browser; |
| + |
| +import android.test.suitebuilder.annotation.MediumTest; |
| + |
| +import org.chromium.base.test.util.Feature; |
| +import org.chromium.base.test.util.UrlUtils; |
| +import org.chromium.content.browser.test.util.Criteria; |
| +import org.chromium.content.browser.test.util.CriteriaHelper; |
| +import org.chromium.content_shell_apk.ContentShellTestBase; |
| +import org.chromium.device.vibration.VibrationManagerImpl; |
| + |
| +/** |
| + * Tests java implementation of VibrationManager mojo service on android. |
| + */ |
| +public class VibrationManagerImplTest extends ContentShellTestBase { |
| + private static final String URL_VIBRATOR_VIBRATE = UrlUtils.encodeHtmlDataUri("<html><body>" |
| + + " <script type=\"text/javascript\">" |
| + + " navigator.vibrate(3000);" |
| + + " </script>" |
| + + "</body></html>"); |
| + private static final String URL_VIBRATOR_CANCEL = UrlUtils.encodeHtmlDataUri("<html><body>" |
| + + " <script type=\"text/javascript\">" |
| + + " navigator.vibrate(10000);" |
| + + " navigator.vibrate(0);" |
| + + " </script>" |
| + + "</body></html>"); |
| + |
| + // Override AndroidVibratorWrapper API to record the calling. |
| + private static class FakeAndroidVibratorWrapper |
| + extends VibrationManagerImpl.AndroidVibratorWrapper { |
| + // Record the parameters of vibrate() and cancel(). |
| + public long mMilliSeconds; |
| + public boolean mCancelled; |
| + |
| + protected FakeAndroidVibratorWrapper() { |
| + // No need to provide real system Vibrator for testing, just pass null. |
| + super(null); |
| + mMilliSeconds = -1; |
| + mCancelled = false; |
| + } |
| + |
| + @Override |
| + public void vibrate(long milliseconds) { |
| + mMilliSeconds = milliseconds; |
| + } |
| + |
| + @Override |
| + public void cancel() { |
| + mCancelled = true; |
| + } |
| + } |
| + |
| + @Override |
| + protected void setUp() throws Exception { |
| + super.setUp(); |
| + launchContentShellWithUrl("about:blank"); |
| + assertTrue("Page failed to load", waitForActiveShellToBeDoneLoading()); |
| + } |
| + |
| + /** |
| + * Inject our fake wrapper into VibrationManagerImpl, |
| + * load the webpage which will request vibrate for 3000 milliseconds, |
| + * the fake wrapper vibrate() should be called and 3000 milliseconds should be recorded |
| + * correctly. |
| + */ |
| + @MediumTest |
| + @Feature({"Vibration"}) |
| + public void testVibrate() throws Throwable { |
| + final FakeAndroidVibratorWrapper fakeWrapper = new FakeAndroidVibratorWrapper(); |
| + VibrationManagerImpl.setVibratorWrapperForTesting(fakeWrapper); |
|
timvolodine
2015/10/20 13:16:22
put these 2 lines in setUp()? applies to the secon
leonhsl(Using Gerrit)
2015/10/22 10:47:58
Done.
|
| + |
| + assertEquals(-1, fakeWrapper.mMilliSeconds); |
| + assertFalse(fakeWrapper.mCancelled); |
|
timvolodine
2015/10/20 13:16:22
maybe also these two lines as well
leonhsl(Using Gerrit)
2015/10/22 10:47:58
Done.
|
| + |
| + loadNewShell(URL_VIBRATOR_VIBRATE); |
|
timvolodine
2015/10/20 13:16:22
nit: empty line after for readability
leonhsl(Using Gerrit)
2015/10/22 10:47:58
Done.
|
| + // Waits until VibrationManagerImpl.Vibrate() got called. |
| + assertTrue(CriteriaHelper.pollForUIThreadCriteria(new Criteria() { |
| + @Override |
| + public boolean isSatisfied() { |
| + return fakeWrapper.mMilliSeconds != -1; |
| + } |
| + })); |
| + |
| + assertEquals( |
| + "Did not get vibrate mMilliSeconds correctly", 3000, fakeWrapper.mMilliSeconds); |
| + } |
| + |
| + /** |
| + * Inject our fake wrapper into VibrationManagerImpl, |
| + * load the webpage which will request vibrate and then request cancel, |
| + * the fake wrapper cancel() should be called. |
| + */ |
| + @MediumTest |
| + @Feature({"Vibration"}) |
| + public void testCancel() throws Throwable { |
| + final FakeAndroidVibratorWrapper fakeWrapper = new FakeAndroidVibratorWrapper(); |
| + VibrationManagerImpl.setVibratorWrapperForTesting(fakeWrapper); |
|
timvolodine
2015/10/20 13:16:22
same here
leonhsl(Using Gerrit)
2015/10/22 10:47:57
Done.
|
| + |
| + assertEquals(-1, fakeWrapper.mMilliSeconds); |
| + assertFalse(fakeWrapper.mCancelled); |
|
timvolodine
2015/10/20 13:16:22
and here
leonhsl(Using Gerrit)
2015/10/22 10:47:58
Done.
|
| + |
| + loadNewShell(URL_VIBRATOR_CANCEL); |
| + // Waits until VibrationManagerImpl.Cancel() got called. |
| + assertTrue(CriteriaHelper.pollForUIThreadCriteria(new Criteria() { |
| + @Override |
| + public boolean isSatisfied() { |
| + return fakeWrapper.mCancelled; |
| + } |
| + })); |
| + |
| + assertTrue("Did not get cancelled", fakeWrapper.mCancelled); |
| + } |
| +} |
