CmdStart: Allow tracking time in the future

Signed-off-by: Shaun Ruffell <sruffell@sruffell.net>

Author: sruffell

src/commands/CmdStart.cpp

@@ -40,11 +40,7 @@ int CmdStart (
 
   auto interval = cli.getFilter ({now, 0});
 
-  if (interval.start > now)
-  {
-    throw std::string ("Time tracking cannot be set in the future.");
-  }
-  else if (!interval.is_started () || interval.is_ended ())
+  if (!interval.is_started () || interval.is_ended ())
   {
     throw std::string ("The start command does not accept ranges but only a single datetime. "
                        "Perhaps you want the track command?.");

test/start.t

@@ -72,9 +72,9 @@ class TestStart(TestCase):
 
         one_hour_after_utc = now_utc + timedelta(hours=1)
 
-        code, out, err = self.t.runError("start {:%Y-%m-%dT%H:%M:%S}Z FOO".format(one_hour_after_utc))
+        code, out, err = self.t("start {:%Y-%m-%dT%H:%M:%S}Z FOO".format(one_hour_after_utc))
 
-        self.assertIn("Time tracking cannot be set in the future.", err)
+        self.assertIn("Will start tracking", out)
 
     def test_start_with_open_interval(self):
         """Test start with already open interval, which should be auto-stopped"""