Merge pull request #37 from RADAR-CNS/fix_battery_alert

Update BatteryLevelMonitor.java

Author: yatharthranjan

src/main/java/org/radarcns/monitor/BatteryLevelMonitor.java

@@ -97,7 +97,10 @@ protected void evaluateRecord(ConsumerRecord<GenericRecord, GenericRecord> recor
                             key.getSourceId(), key.getUserId(), record.value());
                 }
             } else if (previousLevel <= Status.LOW.getLevel()) {
-                updateStatus(key, Status.NORMAL);
+                // Remove the email alert for battery monitor for normal level because 
+                // it is not crucial and to prevent spamming a user's email account. 
+                // Uncomment the line below if needed.
+                // updateStatus(key, Status.NORMAL);
                 logger.info("Battery of sensor {} of user {} is has returned to normal: {}",
                         key.getSourceId(), key.getUserId(), record.value());
             }

src/test/java/org/radarcns/monitor/BatteryLevelMonitorTest.java

@@ -89,14 +89,14 @@ public void evaluateRecord() throws Exception {
         sendMessage(monitor, 1.0f, false);
         sendMessage(monitor, 0.1f, true);
         sendMessage(monitor, 0.1f, false);
-        sendMessage(monitor, 0.3f, true);
+        sendMessage(monitor, 0.3f, false);
         sendMessage(monitor, 0.4f, false);
         sendMessage(monitor, 0.01f, true);
         sendMessage(monitor, 0.01f, false);
         sendMessage(monitor, 0.1f, false);
         sendMessage(monitor, 0.1f, false);
         sendMessage(monitor, 0.01f, true);
-        sendMessage(monitor, 1f, true);
+        sendMessage(monitor, 1f, false);
     }
 
     private void sendMessage(BatteryLevelMonitor monitor, float batteryLevel, boolean sentMessage)
@@ -130,4 +130,4 @@ public void retrieveState() throws Exception {
         Map<String, Float> values = state2.getLevels();
         assertThat(values, hasEntry(measurementKeyToString(key1), 0.1f));
     }
-}
+}