Merge pull request #704 from jrafanie/delete_nil_timestamp_event_streams

Delete invalid nil timestamp event streams

db/migrate/20230830134742_delete_nil_timestamp_event_streams.rb

@@ -0,0 +1,11 @@
+class DeleteNilTimestampEventStreams < ActiveRecord::Migration[6.1]
+  class EventStream < ActiveRecord::Base
+    self.inheritance_column = :_type_disabled
+  end
+
+  def up
+    say_with_time("Deleting event_streams with nil timestamp values") do
+      EventStream.where(:timestamp => nil).delete_all
+    end
+  end
+end

spec/migrations/20230830134742_delete_nil_timestamp_event_streams_spec.rb

@@ -0,0 +1,18 @@
+require_migration
+
+describe DeleteNilTimestampEventStreams do
+  let(:event_stream_stub) { migration_stub(:EventStream) }
+
+  migration_context :up do
+    it "removes nil timestamp events, leaving others" do
+      good = event_stream_stub.create!(:timestamp => Time.now.utc)
+      event_stream_stub.create!(:timestamp => nil)
+
+      migrate
+
+      expect(event_stream_stub.count).to eq(1)
+      expect(event_stream_stub.first.id).to eq(good.id)
+      expect(event_stream_stub.first.timestamp).to be_present
+    end
+  end
+end