fix: do not report a bucket we've already reported && do not report a bucket that has not completed.

Author: jim80net

lib/slowlog_check.rb

@@ -78,10 +78,10 @@ def slowlog_microseconds(slowlog)
   end
 
   def reporting_interval
-    now_i = Time.now.to_i
+    now_i = minute_precision(Time.now).to_i - 60
     start_time_i = last_time_submitted.to_i + 60
     times = (start_time_i..now_i).step(60).to_a
-    Hash[times.collect {|time| [Time.at(time), nil]}]
+    Hash[times.collect {|time| [Time.at(time), {}]}]
   end
 
   def _95percentile(sorted_values)

spec/slowlog_check_spec.rb

@@ -216,16 +216,13 @@ def redis_slowlog(index, time, microseconds, command='eval')
   end
 
   describe '#reporting_interval' do
-    subject { slowlog_check.reporting_interval.map {|x| x[0]} }
-    it 'generates an array at minute intervals' do
-      minute_precision_time = slowlog_check.minute_precision(Time.now)
-
-      expect(subject).to contain_exactly(
-        minute_precision_time,
-        minute_precision_time - 60,
-        minute_precision_time - 120,
-        minute_precision_time - 180
-      )
+    subject { slowlog_check.reporting_interval }
+    focus 'generates an array at minute intervals' do
+      expect(subject).to eq(
+                             Time.utc(2020,4,20,4,19).localtime => {},
+                             Time.utc(2020,4,20,4,18).localtime => {},
+                             Time.utc(2020,4,20,4,17).localtime => {},
+                         )
     end
   end