Modify the check units flow so that it asks the user to pick either

metric or imperial. Metric is the default. Thus no longer need to
send the user to the 'Display' settings page.

SignalStrengthSamplerTest/src/com/google/android/apps/mytracks/signalstrength/test/RecordingStateReceiverTest.java

@@ -0,0 +1,109 @@
+/*
+ * Copyright 2010 Google Inc.
+ * 
+ * Licensed under the Apache License, Version 2.0 (the "License"); you may not
+ * use this file except in compliance with the License. You may obtain a copy of
+ * the License at
+ * 
+ * http://www.apache.org/licenses/LICENSE-2.0
+ * 
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+ * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
+ * License for the specific language governing permissions and limitations under
+ * the License.
+ */
+package com.google.android.apps.mytracks.signalstrength.test;
+
+import com.google.android.apps.mytracks.signalstrength.R;
+import com.google.android.apps.mytracks.signalstrength.RecordingStateReceiver;
+
+import android.content.Context;
+import android.content.Intent;
+import android.content.SharedPreferences;
+import android.preference.PreferenceManager;
+import android.test.AndroidTestCase;
+import android.test.RenamingDelegatingContext;
+
+/**
+ * Tests for {@link RecordingStateReceiver}.
+ *
+ * @author Rodrigo Damazio
+ */
+public class RecordingStateReceiverTest extends AndroidTestCase {
+
+  private RecordingStateReceiver receiver;
+  private RenamingDelegatingContext context;
+  private SharedPreferences preferences;
+  private int serviceStopCalls;
+  private int serviceStartCalls;
+
+  @Override
+  protected void setUp() throws Exception {
+    super.setUp();
+    
+    context = new RenamingDelegatingContext(getContext(), "test.");
+    setContext(context);
+
+    receiver = new RecordingStateReceiver() {
+      @Override
+      protected void stopService(Context ctx) {
+        serviceStopCalls++;
+      }
+
+      @Override
+      protected void startService(Context ctx) {
+        serviceStartCalls++;
+      }
+    };
+
+    preferences = PreferenceManager.getDefaultSharedPreferences(context);
+    preferences
+        .edit()
+        .putBoolean(context.getString(R.string.settings_auto_start_key), true)
+        .putBoolean(context.getString(R.string.settings_auto_stop_key), true)
+        .commit();
+  }
+
+  public void testOnReceive_start() {
+    receiver.onReceive(context,
+        new Intent(context.getString(R.string.track_started_broadcast_action)));
+    assertServiceCalls(1, 0);
+  }
+
+  public void testOnReceive_stop() {
+    receiver.onReceive(context,
+        new Intent(context.getString(R.string.track_stopped_broadcast_action)));
+    assertServiceCalls(0, 1);
+  }
+
+  public void testOnReceive_noAutoStart() {
+    preferences
+        .edit()
+        .putBoolean(context.getString(R.string.settings_auto_start_key), false)
+        .commit();
+    receiver.onReceive(context,
+        new Intent(context.getString(R.string.track_started_broadcast_action)));
+    assertServiceCalls(0, 0);
+  }
+
+  public void testOnReceive_noAutoStop() {
+    preferences
+        .edit()
+        .putBoolean(context.getString(R.string.settings_auto_stop_key), false)
+        .commit();
+    receiver.onReceive(context,
+        new Intent(context.getString(R.string.track_stopped_broadcast_action)));
+    assertServiceCalls(0, 0);
+  }
+
+  public void testOnReceive_other() {
+    receiver.onReceive(context, new Intent("bla"));
+    assertServiceCalls(0, 0);
+  }
+
+  private void assertServiceCalls(int startCalls, int stopCalls) {
+    assertEquals(startCalls, serviceStartCalls);
+    assertEquals(stopCalls, serviceStopCalls);
+  }
+}

SignalStrengthSamplerTest/src/com/google/android/apps/mytracks/signalstrength/test/SignalStrengthListenerFactoryTest.java

@@ -0,0 +1,68 @@
+/*
+ * Copyright 2010 Google Inc.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License"); you may not
+ * use this file except in compliance with the License. You may obtain a copy of
+ * the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+ * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
+ * License for the specific language governing permissions and limitations under
+ * the License.
+ */
+package com.google.android.apps.mytracks.signalstrength.test;
+
+import com.google.android.apps.mytracks.signalstrength.SignalStrengthListener;
+import com.google.android.apps.mytracks.signalstrength.SignalStrengthListener.SignalStrengthCallback;
+import com.google.android.apps.mytracks.signalstrength.SignalStrengthListenerCupcake;
+import com.google.android.apps.mytracks.signalstrength.SignalStrengthListenerEclair;
+import com.google.android.apps.mytracks.signalstrength.SignalStrengthListenerFactory;
+import com.google.android.testing.mocking.AndroidMock;
+
+import android.test.AndroidTestCase;
+
+/**
+ * Tests for {@link SignalStrengthListenerFactory}.
+ * These tests require Eclair+ (API level 7) to run.
+ *
+ * @author Rodrigo Damazio
+ */
+public class SignalStrengthListenerFactoryTest extends AndroidTestCase {
+  private boolean hasModernSignalStrength;
+  private TestableSignalStrengthListenerFactory factory;
+  private SignalStrengthCallback callback;
+
+  private class TestableSignalStrengthListenerFactory extends SignalStrengthListenerFactory {
+    @Override
+    protected boolean hasModernSignalStrength() {
+      return hasModernSignalStrength;
+    }
+  }
+  
+  @Override
+  protected void setUp() throws Exception {
+    super.setUp();
+
+    factory = new TestableSignalStrengthListenerFactory();
+    callback = AndroidMock.createMock(SignalStrengthCallback.class);
+  }
+
+  public void testCreate_eclair() {
+    hasModernSignalStrength = true;
+
+    SignalStrengthListener listener = factory.create(getContext(), callback);
+    assertTrue(listener.getClass().getName(),
+        listener instanceof SignalStrengthListenerEclair);
+  }
+
+  public void testCreate_legacy() {
+    hasModernSignalStrength = false;
+
+    SignalStrengthListener listener = factory.create(getContext(), callback);
+    assertTrue(listener.getClass().getName(),
+        listener instanceof SignalStrengthListenerCupcake);
+  }
+}