prevent expiration in pause

Author: tfeldmann

src/Timeout.cpp

@@ -49,6 +49,9 @@ void Timeout::expire(void)
 
 bool Timeout::time_over(void)
 {
+    if (paused_) {
+        return false;
+    }
     bool result = (time_over_forced_ || (uint32_t)(millis() - reset_time_ms_) >= time_ms_);
     if (result) {
         // make sure to stay expired and not roll over again
@@ -64,12 +67,12 @@ bool Timeout::is_paused(void)
 
 uint32_t Timeout::time_left_ms(void)
 {
-    if (time_over_forced_) {
-        return 0;
-    }
     if (paused_) {
         return time_ms_;
     }
+    if (time_over()) {
+        return 0;
+    }
     int32_t ms_left = time_ms_ - (millis() - reset_time_ms_);
     return ms_left > 0 ? ms_left : 0;
 }

test/test_timeout.cpp

@@ -144,6 +144,77 @@ void pause_resume()
     TEST_ASSERT_EQUAL(t.is_paused(), false);
 }
 
+void no_expiration_in_pause()
+{
+    Timeout t;
+    TEST_ASSERT_EQUAL(t.time_left_ms(), 0);
+    TEST_ASSERT_EQUAL(t.time_over(), true);
+    TEST_ASSERT_EQUAL(t.is_paused(), false);
+
+    set_millis(500);
+    t.start(500);
+    TEST_ASSERT_EQUAL(t.time_left_ms(), 500);
+    TEST_ASSERT_EQUAL(t.time_over(), false);
+    TEST_ASSERT_EQUAL(t.is_paused(), false);
+
+    set_millis(600);
+    TEST_ASSERT_EQUAL(t.time_left_ms(), 400);
+    TEST_ASSERT_EQUAL(t.time_over(), false);
+    TEST_ASSERT_EQUAL(t.is_paused(), false);
+
+    set_millis(700);
+    t.pause();
+    TEST_ASSERT_EQUAL(t.time_left_ms(), 300);
+    TEST_ASSERT_EQUAL(t.time_over(), false);
+    TEST_ASSERT_EQUAL(t.is_paused(), true);
+
+    set_millis(800);
+    t.pause();
+    TEST_ASSERT_EQUAL(t.time_left_ms(), 300);
+    TEST_ASSERT_EQUAL(t.time_over(), false);
+    TEST_ASSERT_EQUAL(t.is_paused(), true);
+
+    set_millis(900);
+    t.pause();
+    TEST_ASSERT_EQUAL(t.time_left_ms(), 300);
+    TEST_ASSERT_EQUAL(t.time_over(), false);
+    TEST_ASSERT_EQUAL(t.is_paused(), true);
+
+    set_millis(1000);
+    TEST_ASSERT_EQUAL(t.time_left_ms(), 300);
+    TEST_ASSERT_EQUAL(t.time_over(), false);
+    TEST_ASSERT_EQUAL(t.is_paused(), true);
+
+    set_millis(1100);
+    TEST_ASSERT_EQUAL(t.time_left_ms(), 300);
+    TEST_ASSERT_EQUAL(t.time_over(), false);
+    TEST_ASSERT_EQUAL(t.is_paused(), true);
+
+    set_millis(1200);
+    t.resume();
+    TEST_ASSERT_EQUAL(t.time_left_ms(), 300);
+    TEST_ASSERT_EQUAL(t.time_over(), false);
+    TEST_ASSERT_EQUAL(t.is_paused(), false);
+
+    set_millis(1300);
+    t.resume();
+    TEST_ASSERT_EQUAL(t.time_left_ms(), 200);
+    TEST_ASSERT_EQUAL(t.time_over(), false);
+    TEST_ASSERT_EQUAL(t.is_paused(), false);
+
+    set_millis(1400);
+    t.resume();
+    TEST_ASSERT_EQUAL(t.time_left_ms(), 100);
+    TEST_ASSERT_EQUAL(t.time_over(), false);
+    TEST_ASSERT_EQUAL(t.is_paused(), false);
+
+    set_millis(1500);
+    t.resume();
+    TEST_ASSERT_EQUAL(t.time_left_ms(), 0);
+    TEST_ASSERT_EQUAL(t.time_over(), true);
+    TEST_ASSERT_EQUAL(t.is_paused(), false);
+}
+
 void periodic()
 {
     Timeout t;
@@ -199,6 +270,7 @@ int main(int argc, char** argv)
     RUN_TEST(heartbeat);
     RUN_TEST(expiration);
     RUN_TEST(pause_resume);
+    RUN_TEST(no_expiration_in_pause);
     RUN_TEST(periodic);
     RUN_TEST(rollover);
     return UNITY_END();