patronus_fati 1.1.0 → 1.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1d187f9773be2d8c8fec8c15ab55a3dcc620db38
-  data.tar.gz: 8eb9d743c20db9adeca447df32a35b361682a368
+  metadata.gz: 10eb60457868f46c81ae5387c9df1c2762274426
+  data.tar.gz: 386a39bbc8dfee4a70096b765afc710c6472db4a
 SHA512:
-  metadata.gz: bb30a888c6b1df1eb3869d35204dea70a3ad69d6fcade84d09ded204d92f666e464ddb7ab89fecd9deb141d0f63d722cc3b5c36c3b4f47a555164ada35372435
-  data.tar.gz: 4f3ddb8616a7a0120bad8f5358af118c2ee9b90698b4c7b9130bb72f4313e465504a96ebd524e053c2ed84d9ca3427fcbdf430e6063e5ae0795891d2f0e5c167
+  metadata.gz: eda0ce738c8de86a5f9364e7471d48be1323434e7d97f8503b174c6268d6daf785a1f4a57ce211c476d4e1c57270339fd7586e335a7f39ef7a73c06317f5c871
+  data.tar.gz: a21660802b3f6cf4657df4718cde39171739aea24fbafb3711d066f00ccb8ef277c5263f7ccbe7a6dc31078f40934322e83fe8e68f2985b850a98bce12c9e1eb

data/lib/patronus_fati/presence.rb

@@ -71,7 +71,7 @@ module PatronusFati
     def mark_visible
       rotate_presence
 
-      self.first_seen ||= Time.now.to_i
+      set_first_seen unless first_seen
       self.current_presence.set_bit(current_bit_offset)
     end
 
@@ -87,6 +87,14 @@ module PatronusFati
       self.current_presence = BitField.new(WINDOW_INTERVALS)
     end
 
+    # Set the time we first saw whatever we're tracking to be the beginning of
+    # the current interval. This prevents negative durations in the event we
+    # only see it once.
+    def set_first_seen
+      cur_time = Time.now.to_i
+      self.first_seen = cur_time - (cur_time % INTERVAL_DURATION)
+    end
+
     # Translate a bit into an absolute unix time relative to the reference
     # window
     def time_for_bit(reference_window, bit)

data/lib/patronus_fati/version.rb

@@ -1,3 +1,3 @@
 module PatronusFati
-  VERSION = '1.1.0'
+  VERSION = '1.1.1'
 end

data/spec/patronus_fati/presence_spec.rb

@@ -97,7 +97,7 @@ RSpec.describe(PatronusFati::Presence) do
     end
 
     it 'should set the first_seen timestamp if it hasn\'t been seen yet' do
-      expect { subject.mark_visible }.to change { subject.first_seen }.from(nil).to(Time.now.to_i)
+      expect { subject.mark_visible }.to change { subject.first_seen }.from(nil)
     end
 
     it 'should not modify the first_seen timestamp if it has already been set' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: patronus_fati
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.1.1
 platform: ruby
 authors:
 - Sam Stelfox