mongoid 9.0.11 → 9.1.0

data/spec/mongoid/cacheable_spec.rb

@@ -1,104 +1,106 @@
 # frozen_string_literal: true
-# rubocop:todo all
 
-require "spec_helper"
+require 'spec_helper'
 
 describe Mongoid::Cacheable do
-
-  describe ".included" do
-
+  describe '.included' do
     let(:klass) do
       Class.new do
         include Mongoid::Cacheable
       end
     end
 
-    it "adds an cache_timestamp_format accessor" do
+    it 'adds an cache_timestamp_format accessor' do
       expect(klass).to respond_to(:cache_timestamp_format)
     end
 
-    it "defaults cache_timestamp_format to :nsec" do
+    it 'defaults cache_timestamp_format to :nsec' do
       expect(klass.cache_timestamp_format).to be(:nsec)
     end
   end
 
-  describe "#cache_key" do
-
+  describe '#cache_key' do
     let(:document) do
       Dokument.new
     end
 
-    context "when the document is new" do
-
-      it "has a new key name" do
-        expect(document.cache_key).to eq("dokuments/new")
+    context 'when the document is new' do
+      it 'has a new key name' do
+        expect(document.cache_key).to eq('dokuments/new')
       end
     end
 
-    context "when persisted" do
-
+    context 'when persisted' do
       before do
         document.save!
       end
 
-      context "with updated_at" do
-
-        context "with the default cache_timestamp_format" do
-
-          let!(:updated_at) do
-            document.updated_at.utc.to_formatted_s(:nsec)
+      context 'with cache_version' do
+        context 'with updated_at' do
+          it 'does not include the timestamp in the key' do
+            expect(document.cache_key).to eq("dokuments/#{document.id}")
           end
+        end
+      end
 
-          it "has the id and updated_at key name" do
-            expect(document.cache_key).to eq("dokuments/#{document.id}-#{updated_at}")
-          end
+      context 'without cache_version' do
+        before do
+          document.define_singleton_method(:cache_version) { nil }
         end
 
-        context "with a different cache_timestamp_format" do
+        context 'with updated_at' do
+          context 'with the default cache_timestamp_format' do
+            let!(:updated_at) do
+              document.updated_at.utc.to_formatted_s(:nsec)
+            end
 
-          before do
-            Dokument.cache_timestamp_format = :number
+            it 'has the id and updated_at key name' do
+              expect(document.cache_key).to eq("dokuments/#{document.id}-#{updated_at}")
+            end
           end
 
-          after do
-            Dokument.cache_timestamp_format = :nsec
-          end
+          context 'with a different cache_timestamp_format' do
+            before do
+              Dokument.cache_timestamp_format = :number
+            end
 
-          let!(:updated_at) do
-            document.updated_at.utc.to_formatted_s(:number)
-          end
+            after do
+              Dokument.cache_timestamp_format = :nsec
+            end
+
+            let!(:updated_at) do
+              document.updated_at.utc.to_formatted_s(:number)
+            end
 
-          it "has the id and updated_at key name" do
-            expect(document.cache_key).to eq("dokuments/#{document.id}-#{updated_at}")
+            it 'has the id and updated_at key name' do
+              expect(document.cache_key).to eq("dokuments/#{document.id}-#{updated_at}")
+            end
           end
         end
       end
 
-      context "without updated_at, with Timestamps" do
-
+      context 'without updated_at, with Timestamps' do
         before do
           document.updated_at = nil
         end
 
-        it "has the id key name" do
+        it 'has the id key name' do
           expect(document.cache_key).to eq("dokuments/#{document.id}")
         end
       end
     end
 
-    context "when model dont have Timestamps" do
-
+    context 'when model dont have Timestamps' do
       let(:artist) do
         Artist.create!
       end
 
-      it "should have the id key name" do
+      it 'has the id key name' do
         expect(artist.cache_key).to eq("artists/#{artist.id}")
       end
     end
 
-    context "when model has Short Timestamps" do
-
+    context 'when model has Short Timestamps' do
       let(:agent) do
         ShortAgent.create!
       end
@@ -107,9 +109,37 @@ describe Mongoid::Cacheable do
         agent.updated_at.utc.to_formatted_s(:nsec)
       end
 
-      it "has the id and updated_at key name" do
+      it 'has the id and updated_at key name' do
         expect(agent.cache_key).to eq("short_agents/#{agent.id}-#{updated_at}")
       end
     end
   end
+
+  describe 'cache_version' do
+    context 'when model has no updated_at field' do
+      let(:model) { Artist.create! }
+
+      it 'has a nil cache_version' do
+        expect(model.cache_version).to be_nil
+      end
+    end
+
+    context 'when model has updated_at field' do
+      context 'when updated_at is nil' do
+        let(:model) { Dokument.create!.tap { |v| v.updated_at = nil } }
+
+        it 'has a nil cache_version' do
+          expect(model.cache_version).to be_nil
+        end
+      end
+
+      context 'when updated_at is present' do
+        let(:model) { Dokument.create! }
+
+        it 'has a non-nil cache_version' do
+          expect(model.cache_version).not_to be_nil
+        end
+      end
+    end
+  end
 end