From 8d78c16c8acbd544d65b1df9aef1bba5497bd527 Mon Sep 17 00:00:00 2001
From: Rodrigo Damazio <Rodrigo Damazio@localhost>
Date: Sun, 28 Nov 2010 03:37:27 -0800
Subject: [PATCH] Adding more tests and other minor fixes.

---
 .../proguard.cfg                              |   0
 SignalStrengthSampler/res/values/strings.xml  |   4 +
 SignalStrengthSampler/res/xml/preferences.xml |   5 +
 .../RecordingStateReceiver.java               |  20 ++-
 .../signalstrength/SignalStrengthService.java |   7 +-
 .../test/RecordingStateReceiverTest.java      | 121 ++++++++++++++++++
 6 files changed, 147 insertions(+), 10 deletions(-)
 rename {SignalStrengthSamplerTest => MyTracks}/proguard.cfg (100%)
 create mode 100644 SignalStrengthSamplerTest/src/com/google/android/apps/mytracks/signalstrength/test/RecordingStateReceiverTest.java

diff --git a/SignalStrengthSamplerTest/proguard.cfg b/MyTracks/proguard.cfg
similarity index 100%
rename from SignalStrengthSamplerTest/proguard.cfg
rename to MyTracks/proguard.cfg
diff --git a/SignalStrengthSampler/res/values/strings.xml b/SignalStrengthSampler/res/values/strings.xml
index 187f169ed..1549968fc 100644
--- a/SignalStrengthSampler/res/values/strings.xml
+++ b/SignalStrengthSampler/res/values/strings.xml
@@ -21,6 +21,9 @@
   <string name="settings_auto_start">Automatic start</string>
   <string name="settings_auto_start_summary_on">Automatically start inserting signal strength waypoints whenever My Tracks starts recording a track.</string>
   <string name="settings_auto_start_summary_off">You will need to manually start signal sampling using the controls below.</string>
+  <string name="settings_auto_stop">Automatic stop</string>
+  <string name="settings_auto_stop_summary_on">Automatically stop inserting signal strength waypoints whenever My Tracks stops recording a track.</string>
+  <string name="settings_auto_stop_summary_off">You will need to manually stop signal sampling using the controls below.</string>
   <string name="settings_min_signal_sampling_period">Min. sampling period</string>
   <string name="settings_min_signal_sampling_period_summary">Minimum time between two recordings of the phone\'s signal strength</string>
 
@@ -44,6 +47,7 @@
 
   <!-- Non-translatable strings -->
   <string name="settings_auto_start_key">autoStartSampling</string>
+  <string name="settings_auto_stop_key">autoStopSampling</string>
   <string name="settings_min_signal_sampling_period_key">signalSamplingFrequency</string>
   <string name="settings_control_start_key">startSamplingControl</string>
   <string name="settings_control_stop_key">stopSamplingControl</string>
diff --git a/SignalStrengthSampler/res/xml/preferences.xml b/SignalStrengthSampler/res/xml/preferences.xml
index 7277d0ba0..ba4de39c2 100644
--- a/SignalStrengthSampler/res/xml/preferences.xml
+++ b/SignalStrengthSampler/res/xml/preferences.xml
@@ -21,6 +21,11 @@
                         android:summaryOn="@string/settings_auto_start_summary_on"
                         android:summaryOff="@string/settings_auto_start_summary_off"
                         android:defaultValue="false" />
+    <CheckBoxPreference android:key="@string/settings_auto_stop_key"
+                        android:title="@string/settings_auto_stop"
+                        android:summaryOn="@string/settings_auto_stop_summary_on"
+                        android:summaryOff="@string/settings_auto_stop_summary_off"
+                        android:defaultValue="true" />
 
     <ListPreference
         android:key="@string/settings_min_signal_sampling_period_key"
diff --git a/SignalStrengthSampler/src/com/google/android/apps/mytracks/signalstrength/RecordingStateReceiver.java b/SignalStrengthSampler/src/com/google/android/apps/mytracks/signalstrength/RecordingStateReceiver.java
index b114c8ba7..1b57700ac 100644
--- a/SignalStrengthSampler/src/com/google/android/apps/mytracks/signalstrength/RecordingStateReceiver.java
+++ b/SignalStrengthSampler/src/com/google/android/apps/mytracks/signalstrength/RecordingStateReceiver.java
@@ -32,19 +32,27 @@ public class RecordingStateReceiver extends BroadcastReceiver {
   @Override
   public void onReceive(Context ctx, Intent intent) {
     SharedPreferences preferences = PreferenceManager.getDefaultSharedPreferences(ctx);
-    boolean autoStart = preferences.getBoolean(
-        ctx.getString(R.string.settings_auto_start_key), false);
-    if (!autoStart) {
-      Log.d(TAG, "Not auto-starting signal sampling");
-      return;
-    }
 
     String action = intent.getAction();
     if (ctx.getString(R.string.track_started_broadcast_action).equals(action) ||
         ctx.getString(R.string.track_resumed_broadcast_action).equals(action)) {
+      boolean autoStart = preferences.getBoolean(
+          ctx.getString(R.string.settings_auto_start_key), false);
+      if (!autoStart) {
+        Log.d(TAG, "Not auto-starting signal sampling");
+        return;
+      }
+
       startService(ctx);
     } else if (ctx.getString(R.string.track_stopped_broadcast_action).equals(action) ||
                ctx.getString(R.string.track_paused_broadcast_action).equals(action)) {
+      boolean autoStop = preferences.getBoolean(
+          ctx.getString(R.string.settings_auto_stop_key), true);
+      if (!autoStop) {
+        Log.d(TAG, "Not auto-stopping signal sampling");
+        return;
+      }
+
       stopService(ctx);
     } else {
       Log.e(TAG, "Unknown action received: " + action);
diff --git a/SignalStrengthSampler/src/com/google/android/apps/mytracks/signalstrength/SignalStrengthService.java b/SignalStrengthSampler/src/com/google/android/apps/mytracks/signalstrength/SignalStrengthService.java
index 96658b935..7f80c7433 100644
--- a/SignalStrengthSampler/src/com/google/android/apps/mytracks/signalstrength/SignalStrengthService.java
+++ b/SignalStrengthSampler/src/com/google/android/apps/mytracks/signalstrength/SignalStrengthService.java
@@ -58,10 +58,9 @@ public class SignalStrengthService extends Service
   public void onCreate() {
     super.onCreate();
 
-    mytracksServiceComponent =
-      new ComponentName(
-          getString(R.string.mytracks_service_package),
-          getString(R.string.mytracks_service_class));
+    mytracksServiceComponent = new ComponentName(
+        getString(R.string.mytracks_service_package),
+        getString(R.string.mytracks_service_class));
     preferences = PreferenceManager.getDefaultSharedPreferences(this);
     signalListenerFactory = new SignalStrengthListenerFactory();
   }
diff --git a/SignalStrengthSamplerTest/src/com/google/android/apps/mytracks/signalstrength/test/RecordingStateReceiverTest.java b/SignalStrengthSamplerTest/src/com/google/android/apps/mytracks/signalstrength/test/RecordingStateReceiverTest.java
new file mode 100644
index 000000000..8f5b6c9cc
--- /dev/null
+++ b/SignalStrengthSamplerTest/src/com/google/android/apps/mytracks/signalstrength/test/RecordingStateReceiverTest.java
@@ -0,0 +1,121 @@
+/*
+ * 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_resume() {
+    receiver.onReceive(context,
+        new Intent(context.getString(R.string.track_resumed_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_pause() {
+    receiver.onReceive(context,
+        new Intent(context.getString(R.string.track_paused_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);
+  }
+}