Unit Tests

- Improved args.5.t, which was attempting to stop tasks that were
  not started.

src/Task.cpp

@@ -150,6 +150,16 @@ void Task::setEnd ()
   recalc_urgency = true;
 }
 
+////////////////////////////////////////////////////////////////////////////////
+void Task::setStart ()
+{
+  char startTime[16];
+  sprintf (startTime, "%u", (unsigned int) time (NULL));
+  set ("start", startTime);
+
+  recalc_urgency = true;
+}
+
 ////////////////////////////////////////////////////////////////////////////////
 Task::status Task::getStatus () const
 {

src/Task.h

@@ -60,6 +60,7 @@ public:
 
   void setEntry ();
   void setEnd ();
+  void setStart ();
 
   status getStatus () const;
   void setStatus (status);

src/commands/CmdStart.cpp

@@ -87,9 +87,7 @@ int CmdStart::execute (std::string& output)
       apply_defaults (*task);
 
       // Add a start time.
-      char startTime[16];
+      task->setStart ();
-      sprintf (startTime, "%u", (unsigned int) time (NULL));
-      task->set ("start", startTime);
 
       if (context.config.getBoolean ("journal.time"))
         task->addAnnotation (context.config.get ("journal.time.start.annotation"));

test/args.5.t

@@ -46,6 +46,12 @@ qx{../src/task rc:args.rc add three};
 qx{../src/task rc:args.rc add four};
 qx{../src/task rc:args.rc add five};
 
+qx{../src/task rc:args.rc 1 start};
+qx{../src/task rc:args.rc 2 start};
+qx{../src/task rc:args.rc 3 start};
+qx{../src/task rc:args.rc 4 start};
+qx{../src/task rc:args.rc 5 start};
+
 qx{../src/task rc:args.rc 1 stop oneanno};
 my $output = qx{../src/task rc:args.rc 1 info};
 like ($output, qr/oneanno/, 'stop enpassant anno');