The worst-case for testing a time-period, and one used in the

parser test, is matching against a period that covers the full
24 hours.

Correctly terminate this case, and add a new test-case to ensure
that this is always correct.

1 files changed, 13 insertions, 0 deletions

diff --git a/t/test-custodian-util-timespan.rb b/t/test-custodian-util-timespan.rb
index 307a171..048f582 100755
--- a/t/test-custodian-util-timespan.rb
+++ b/t/test-custodian-util-timespan.rb
@@ -150,4 +150,17 @@ class TestTimeSpanUtil < Test::Unit::TestCase
 
   end
 
+
+  #
+  #  Since the implementation of our test involves
+  # iterating over the specified period then any 24-hour
+  # period is the worst case.
+  #
+  #
+  def test_worst()
+    for i in 0..23
+      assert( Custodian::Util::TimeSpan.inside?( 0, 23, i ) )
+    end
+
+  end
 end