From cd7c09e258bf96a6ae314f7c7b3467982cd92d0c Mon Sep 17 00:00:00 2001 From: Ben Atkins Date: Wed, 14 Jan 2015 16:07:28 -0500 Subject: [PATCH] Adjust specs for PaperTrail::Version#where_object_changes to ensure consistent sort order of return values --- spec/models/version_spec.rb | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/spec/models/version_spec.rb b/spec/models/version_spec.rb index 9d3d7b32..85d9b643 100644 --- a/spec/models/version_spec.rb +++ b/spec/models/version_spec.rb @@ -134,13 +134,13 @@ describe PaperTrail::Version, :type => :model do specify { expect(PaperTrail.serializer).to be PaperTrail::Serializers::YAML } it "should be able to locate versions according to their `object_changes` contents" do - expect(PaperTrail::Version.where_object_changes(:name => name)).to eq(widget.versions[0..1]) - expect(PaperTrail::Version.where_object_changes(:an_integer => 100)).to eq(widget.versions[1..2]) - expect(PaperTrail::Version.where_object_changes(:an_integer => int)).to eq([widget.versions.last]) + expect(widget.versions.where_object_changes(:name => name)).to eq(widget.versions[0..1]) + expect(widget.versions.where_object_changes(:an_integer => 100)).to eq(widget.versions[1..2]) + expect(widget.versions.where_object_changes(:an_integer => int)).to eq([widget.versions.last]) end it "should be able to handle queries for multiple attributes" do - expect(PaperTrail::Version.where_object_changes(:an_integer => 100, :name => 'foobar')).to eq(widget.versions[1..2]) + expect(widget.versions.where_object_changes(:an_integer => 100, :name => 'foobar')).to eq(widget.versions[1..2]) end end @@ -149,13 +149,13 @@ describe PaperTrail::Version, :type => :model do specify { expect(PaperTrail.serializer).to be PaperTrail::Serializers::JSON } it "should be able to locate versions according to their `object_changes` contents" do - expect(PaperTrail::Version.where_object_changes(:name => name)).to eq(widget.versions[0..1]) - expect(PaperTrail::Version.where_object_changes(:an_integer => 100)).to eq(widget.versions[1..2]) - expect(PaperTrail::Version.where_object_changes(:an_integer => int)).to eq([widget.versions.last]) + expect(widget.versions.where_object_changes(:name => name)).to eq(widget.versions[0..1]) + expect(widget.versions.where_object_changes(:an_integer => 100)).to eq(widget.versions[1..2]) + expect(widget.versions.where_object_changes(:an_integer => int)).to eq([widget.versions.last]) end it "should be able to handle queries for multiple attributes" do - expect(PaperTrail::Version.where_object_changes(:an_integer => 100, :name => 'foobar')).to eq(widget.versions[1..2]) + expect(widget.versions.where_object_changes(:an_integer => 100, :name => 'foobar')).to eq(widget.versions[1..2]) end after(:all) { PaperTrail.serializer = PaperTrail::Serializers::YAML }