entity_store_sequel 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8e81c642170575ae02a6faf8f0ec2c4fe71bf4b00eb2fabfd5f9f8099fdb5bd6
-  data.tar.gz: 5b0463a52fc3b0382f6d083ad087c353753214a01b1e17220cd61f769aa4e058
+  metadata.gz: 1fbaafda3599b05b00d50669608de59f140546e61066def1dbb355d2eea59ccf
+  data.tar.gz: 565ac6d7d44085cb366f2250147c20c63f4a1b05143cb8f30286f485172c184b
 SHA512:
-  metadata.gz: 727a2f7db49b844f065e25dda18433916a9a5b8773e936ce7cb5ee6c068c3ff238a242aba7548d03b8d8af8e4de1bd4193a81642fe967c3fbe222ee2ef44ad7f
-  data.tar.gz: a999ecfe3f45f38fc16250f81fb64e73d06e480d753562eca0ce40d900d06c1349f17bd949312c343f4b42cce8d2818ab42c1ec8822c7818ea1e030e7018cb3e
+  metadata.gz: a5ab29eb7d354b40cd35411226e8e96d65ca7df0fda329378a849033fc96a13864558eae931966756fef5ae18733806075dcd69cf723c1f28bcebcca2c606c10
+  data.tar.gz: c8ffb267c8103f4bf4cd03e56099f55e3d6ecec1b93357176de0e1a24018d1c8191763b7ac5578944eeefd9795fce6dfaa024c88c881a6ebbef5f38449dcec04

data/lib/entity_store_sequel/postgres_entity_store.rb

@@ -70,10 +70,13 @@ module EntityStoreSequel
     end
 
     def clear_entity_events(id, excluded_types)
-      events.where(_entity_id: id).each do |event|
-        next if excluded_types.include?(event[:_type])
-
-        events.where(id: event[:id]).delete
+      if excluded_types.empty?
+        events.where(_entity_id: id).delete
+      else
+        events.where(_entity_id: id).select(:id, :_type).each do |event|
+          next if excluded_types.include?(event[:_type])
+          events.where(id: event[:id]).delete
+        end
       end
     end
 

data/lib/entity_store_sequel/version.rb

@@ -1,3 +1,3 @@
 module EntityStoreSequel
-  VERSION = "0.3.0".freeze
+  VERSION = "0.4.0".freeze
 end

data/spec/entity_store_sequel/postgres_entity_store_spec.rb

@@ -47,6 +47,21 @@ describe PostgresEntityStore do
     end
   end
 
+  class DummyEntityExcludedEvent
+    include EntityStore::Event
+
+    attr_accessor :marker
+
+    def apply(entity)
+      # No-op
+    end
+
+    def ==(other)
+      # Crude check relying on inspect, ok for tests
+      self.inspect == other.inspect
+    end
+  end
+
   let(:store) do
     described_class.connection_string = ENV['SQL_CONNECTION'] || 'postgres://localhost/cronofy_test'
     described_class.init
@@ -123,23 +138,42 @@ describe PostgresEntityStore do
     let(:third_event)     { DummyEntityNameSet.new(:entity_id => entity_id, :entity_version => 2, :name => random_string) }
     let(:unrelated_event) { DummyEntityNameSet.new(:entity_id => second_entity_id, :entity_version => 4, :name => random_string) }
     let(:fourth_event)    { DummyEntityNameSet.new(:entity_id => entity_id, :entity_version => 3, :name => random_string) }
+    let(:excluded_event)  { DummyEntityExcludedEvent.new(:entity_id => entity_id, :entity_version => 5, :marker => random_string) }
 
     before do
-      store.add_events([ second_event, unrelated_event, first_event, third_event, fourth_event ])
+      store.add_events([ second_event, unrelated_event, first_event, third_event, fourth_event, excluded_event ])
     end
 
-    subject { store.clear_entity_events(entity_id, []) }
+    context "clearing everything" do
+      subject { store.clear_entity_events(entity_id, []) }
 
-    it "clears the events from the entity" do
-      subject
-      events = store.get_events( [ id: entity_id ])[entity_id]
-      expect(events).to be_empty
+      it "clears the events from the entity" do
+        subject
+        events = store.get_events( [ id: entity_id ])[entity_id]
+        expect(events).to be_empty
+      end
+
+      it "does not delete unrelated the events" do
+        subject
+        events = store.get_events( [ id: second_entity_id ])[second_entity_id]
+        expect(events).to eq [unrelated_event]
+      end
     end
 
-    it "does not delete unrelated the events" do
-      subject
-      events = store.get_events( [ id: second_entity_id ])[second_entity_id]
-      expect(events.count).to eq(1)
+    context "excluding an event type from deletion" do
+      subject { store.clear_entity_events(entity_id, [DummyEntityExcludedEvent.to_s]) }
+
+      it "clears all other events from the entity" do
+        subject
+        events = store.get_events( [ id: entity_id ])[entity_id]
+        expect(events).to eq [excluded_event]
+      end
+
+      it "does not delete unrelated the events" do
+        subject
+        events = store.get_events( [ id: second_entity_id ])[second_entity_id]
+        expect(events).to eq [unrelated_event]
+      end
     end
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: entity_store_sequel
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Stephen Binns
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-04-09 00:00:00.000000000 Z
+date: 2024-02-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sequel
@@ -101,7 +101,7 @@ homepage: http://github.com/cronofy/entity_store_sequel
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -116,11 +116,11 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.4.21
+signing_key:
 specification_version: 4
 summary: Sequel body for Entity Store
 test_files:
-- spec/spec_helper.rb
-- spec/entity_store_spec.rb
 - spec/entity_store_sequel/postgres_entity_store_spec.rb
+- spec/entity_store_spec.rb
+- spec/spec_helper.rb