Modify New-ScheduledCommand so that it always includes StartAfter

Author: JasonNoonan

PoshBot/Plugins/Builtin/Public/New-ScheduledCommand.ps1

@@ -81,7 +81,8 @@ function New-ScheduledCommand {
         if ($PSBoundParameters.ContainsKey('StartAfter')) {
             $schedMsg = [ScheduledMessage]::new($Interval, $value, $botMsg, [datetime]$StartAfter)
         } else {
-            $schedMsg = [ScheduledMessage]::new($Interval, $value, $botMsg)
+            $StartAfter = Get-Date
+            $schedMsg = [ScheduledMessage]::new($Interval, $value, $botMsg, [datetime]$StartAfter)
         }
     } elseIf ($PSCmdlet.ParameterSetName -eq 'once') {
         # This command will be executed once then removed from the scheduler

Tests/Unit/Classes/ScheduledMessage.tests.ps1

@@ -0,0 +1,45 @@
+InModuleScope PoshBot {
+
+    Describe ScheduledMessage {
+        # Mock a message
+        $msg = [Message]::new()
+        $msg.Id = 1
+        $msg.Text = "This is a test message"
+
+        # Call constructor
+        $Interval = 'Days'
+        $TimeValue = 1
+        $Message = $msg
+        $StartAfter = (Get-Date).AddMinutes(1)
+
+        Context "Constructor: String, Int, Message, DateTime" {
+
+            $ScheduledMessage = [ScheduledMessage]::new($Interval, $TimeValue, $Message, $StartAfter)
+
+            It 'TimeInterval should match argument' {
+                $ScheduledMessage.TimeInterval | Should be $Interval
+            }
+
+            It 'TimeValue should match argument' {
+                $ScheduledMessage.TimeValue | Should be $TimeValue
+            }
+
+            It 'Message Id should be 1' {
+                $ScheduledMessage.Message.Id | Should be 1
+            }
+            It 'Message Text should match argument' {
+                $ScheduledMessage.Message.Text | Should match $msg.Text
+            }
+
+            It 'StartAfter value should match argument converted to UTC' {
+                $ScheduledMessage.StartAfter | Should be $StartAfter.ToUniversalTime()
+            }
+        }
+
+        Context "Methods" {
+
+        }
+
+    }
+
+}