Skip to content

Commit

Permalink
Adjust specs for PaperTrail::Version#where_object_changes to ensure c…
Browse files Browse the repository at this point in the history
…onsistent sort order of return values
  • Loading branch information
batter committed Jan 14, 2015
1 parent 642774e commit cd7c09e
Showing 1 changed file with 8 additions and 8 deletions.
16 changes: 8 additions & 8 deletions spec/models/version_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -134,13 +134,13 @@
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

Expand All @@ -149,13 +149,13 @@
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 }
Expand Down

0 comments on commit cd7c09e

Please sign in to comment.