Added OutOfOrder and Empty tests in At functions.

Author: VitorCioletti

FluentScheduler.UnitTests/Fluent/3.Duration/EverydayUnitTests.cs

@@ -90,7 +90,7 @@ public void AtMultipleTimeSpan()
 
         // Assert
         Equal(expected, calculated.Value);
-        
+
         // Arrange
         now = now.AddHours(5.1);
         expected = expected.Date.Add(timeSpans[1]);
@@ -121,7 +121,7 @@ public void AtMultipleTimeSpan()
         // Assert
         Equal(expected, calculated.Value);
     }
-    
+
     [Fact]
     public void AtMultipleTimeSpan2()
     {
@@ -142,7 +142,7 @@ public void AtMultipleTimeSpan2()
 
         // Assert
         Equal(expected, calculated.Value);
-        
+
         // Arrange
         now = now.AddHours(1.1);
         expected = expected.Date.Add(timeSpans[2]);
@@ -227,7 +227,7 @@ public void EverydayAt15()
         // Assert
         Equal(expectedDate, calculated.Value);
     }
-    
+
     [Fact]
     public void EverydayAt16()
     {
@@ -400,4 +400,26 @@ public void Between19and22()
         // Assert
         Equal(expectedDate, calculated.Value);
     }
+
+    [Fact]
+    public void ThrowIfAtOutOfOrder()
+    {
+        // Arrange
+        var fluentCalculator = new FluentTimeCalculator();
+        var run = new EverydayUnit(fluentCalculator);
+
+        // Act
+        Throws<ArgumentException>(() => run.At(new TimeSpan(10, 0, 0), new TimeSpan(9, 0, 0)));
+    }
+    
+    [Fact]
+    public void ThrowIfAtEmpty()
+    {
+        // Arrange
+        var fluentCalculator = new FluentTimeCalculator();
+        var run = new EverydayUnit(fluentCalculator);
+
+        // Act
+        Throws<ArgumentException>(() => run.At());
+    }
 }

FluentScheduler.UnitTests/Fluent/3.Duration/PeriodOnceSetTests.cs

@@ -21,7 +21,7 @@ public void At()
 
         // Act
         run.At(8, 40);
-        var calculated = calculator.Calculate(now);
+        var calculated = calculator.Calculate(now)!;
 
         // Assert
         Equal(expected, calculated.Value);
@@ -31,7 +31,7 @@ public void At()
     public void AtTimeSpan()
     {
         // Arrange
-        var now = new DateTime(2018, 3, 3, 10, 0 ,0);
+        var now = new DateTime(2018, 3, 3, 10, 0, 0);
         var expected = new DateTime(2018, 3, 3, 12, 30, 0);
 
         var timeSpan = new TimeSpan(12, 30, 0);
@@ -44,12 +44,34 @@ public void AtTimeSpan()
 
         // Act
         run.At(timeSpan);
-        var calculated = calculator.Calculate(now);
+        var calculated = calculator.Calculate(now)!;
 
         // Assert
         Equal(expected, calculated.Value);
     }
 
+    [Fact]
+    public void ThrowIfAtEmpty()
+    {
+        // Arrange
+        var fluentCalculator = new FluentTimeCalculator();
+        var run = new PeriodOnceSet(fluentCalculator);
+
+        // Act
+        Throws<ArgumentException>(() => run.At());
+    }
+
+    [Fact]
+    public void ThrowIfAtOutOfOrder()
+    {
+        // Arrange
+        var fluentCalculator = new FluentTimeCalculator();
+        var run = new PeriodOnceSet(fluentCalculator);
+
+        // Act
+        Throws<ArgumentException>(() => run.At(new TimeSpan(10, 0, 0), new TimeSpan(9, 0, 0)));
+    }
+
     [Fact]
     public void InTheNextMonth()
     {
@@ -65,9 +87,9 @@ public void InTheNextMonth()
 
         // Act
         run.Every(1).Months().On(10).At(11, 30);
-        var calculated = calculator.Calculate(now);
+        var calculated = calculator.Calculate(now)!;
 
         // Assert
         Equal(expectedDate, calculated.Value);
     }
-}
+}

FluentScheduler/Fluent/3.Duration/PeriodOnceSet.cs

@@ -32,6 +32,7 @@ public void At(int hour, int minute)
     public void At(params TimeSpan[] timeCollection)
     {
         ThrowHelper.ThrowIfEmpty(timeCollection);
+        ThrowHelper.ThrowIfOutOfOrder(timeCollection);
         ThrowHelper.ThrowIfOutOfMilitaryTimeRange(timeCollection);
 
         foreach (var time in timeCollection)