diff --git a/MyTracksTest/src/com/google/android/apps/mytracks/endtoendtest/others/SecondaryTestUtils.java b/MyTracksTest/src/com/google/android/apps/mytracks/endtoendtest/others/SecondaryTestUtils.java new file mode 100644 index 000000000..749f87c79 --- /dev/null +++ b/MyTracksTest/src/com/google/android/apps/mytracks/endtoendtest/others/SecondaryTestUtils.java @@ -0,0 +1,14 @@ +package com.google.android.apps.mytracks.endtoendtest.others; + +public class SecondaryTestUtils { + + /** + * Set to false as default. True to run the test. Default to false since this + * test can take a long time. + */ + protected static boolean runTest = false; + + protected static final String DISABLE_MESSAGE = "This test is disabled"; + + +} diff --git a/MyTracksTest/src/com/google/android/apps/mytracks/endtoendtest/others/SensorTest.java b/MyTracksTest/src/com/google/android/apps/mytracks/endtoendtest/others/SensorTest.java index c156d5cf4..7fd6488d9 100644 --- a/MyTracksTest/src/com/google/android/apps/mytracks/endtoendtest/others/SensorTest.java +++ b/MyTracksTest/src/com/google/android/apps/mytracks/endtoendtest/others/SensorTest.java @@ -19,9 +19,7 @@ import com.google.android.apps.mytracks.TrackListActivity; import com.google.android.apps.mytracks.endtoendtest.EndToEndTestUtils; import com.google.android.maps.mytracks.R; -import android.app.AlertDialog; import android.app.Instrumentation; -import android.content.DialogInterface; import android.test.ActivityInstrumentationTestCase2; import android.util.Log; import android.widget.ListView; @@ -36,31 +34,19 @@ import java.util.ArrayList; */ public class SensorTest extends ActivityInstrumentationTestCase2 { - /** - * Set to false as default. True to run the test. Default to false since this - * test can take a long time. - */ - public static boolean runTest = false; - public SensorTest() { super(TrackListActivity.class); } private Instrumentation instrumentation; private TrackListActivity trackListActivity; - public static final String DISABLE_MESSAGE = "This test is disabled"; - public static final String ZEPHYR_NAME = "HXM"; - public static final String POLAR_NAME = "Polar"; - - /** - * This flag is true means the operation of UI thread has done. - */ - private boolean UIThreadDone = false; + private static final String ZEPHYR_NAME = "HXM"; + private static final String POLAR_NAME = "Polar"; @Override protected void setUp() throws Exception { super.setUp(); - if (!runTest) { + if (!SecondaryTestUtils.runTest) { return; } instrumentation = getInstrumentation(); @@ -73,11 +59,11 @@ public class SensorTest extends ActivityInstrumentationTestCase2 allListViews = EndToEndTestUtils.SOLO.getCurrentListViews(); int number = allListViews.get(0).getCount(); - if (number > 0 && EndToEndTestUtils.SOLO.waitForText(nameString, 1, EndToEndTestUtils.SHORT_WAIT_TIME)) { + if (number > 0 + && EndToEndTestUtils.SOLO.waitForText(nameString, 1, EndToEndTestUtils.SHORT_WAIT_TIME)) { return; } - + Log.i(EndToEndTestUtils.LOG_TAG, "No sensor is paried"); - try { - runTestOnUiThread(new Runnable() { - @Override - public void run() { - UIThreadDone = false; - new AlertDialog.Builder(EndToEndTestUtils.SOLO.getCurrentActivity()) - .setTitle("No paried sensor.") - .setMessage("No paried sensor, do you want to pair sensor now?") - .setPositiveButton("Yes", new DialogInterface.OnClickListener() { - public void onClick(DialogInterface dialog, int which) { - UIThreadDone = true; - } - }).setNegativeButton("No", new DialogInterface.OnClickListener() { - public void onClick(DialogInterface dialog, int which) { - fail(); - } - }).show(); - } - }); - // Waiting user click the dialog. - while (!UIThreadDone) { - EndToEndTestUtils.sleep(1000); - Log.i(EndToEndTestUtils.LOG_TAG, "Waiting UI thread done."); - } - - // Open OS setting page of Bluetooth. - EndToEndTestUtils.SOLO.goBack(); - EndToEndTestUtils.SOLO.clickOnText(trackListActivity - .getString(R.string.settings_sensor_bluetooth_pairing)); - - // Now the setting page is on the front. - while (!EndToEndTestUtils.SOLO.waitForText( - trackListActivity.getString(R.string.settings_sensor_bluetooth_pairing), 1, 500)) { - Log.i(EndToEndTestUtils.LOG_TAG, "Waiting back to MyTracks activity."); - EndToEndTestUtils.sleep(200); - } - checkPairedSensorsNumber(nameString); - } catch (Throwable e) { - e.printStackTrace(); - } + fail("No bluetooth sensor is paired. Please pair at Settings->Bluetooth before running the test."); } /** * Tests connecting to a cadence ANT+ sensor. */ public void testConnectANTSensor_Cadence() { - if (!runTest) { - Log.i(EndToEndTestUtils.LOG_TAG, DISABLE_MESSAGE); + if (!SecondaryTestUtils.runTest) { + Log.i(EndToEndTestUtils.LOG_TAG, SecondaryTestUtils.DISABLE_MESSAGE); return; } useANTSeonsor(); @@ -185,8 +133,8 @@ public class SensorTest extends ActivityInstrumentationTestCase2 { - /** - * Set to false as default. True to run the test. Default to false since this - * test can take a long time. - */ - public static boolean runTest = true; - public static final int TEST_DURATION_IN_MILLISECONDS = 30 * 60 * 1000; + private static final int TEST_DURATION_IN_MILLISECONDS = 30 * 60 * 1000; private static final int MAX_TRACK_NUMBER = 30; private long startTime = 0; private int numberOfTracks; @@ -50,7 +45,7 @@ public class StressTest extends ActivityInstrumentationTestCase2