draftsman 0.3.1 → 0.3.2

data/spec/models/draft_spec.rb

@@ -3,28 +3,67 @@ require 'spec_helper'
 describe Draftsman::Draft do
   describe 'class methods' do
     subject { Draftsman::Draft }
-    its(:object_col_is_json?) { should be_false }
-    its(:object_changes_col_is_json?) { should be_false }
+
+    it 'does not have a JSON object column' do
+      expect(subject.object_col_is_json?).to eql false
+    end
+
+    it 'does not have a JSON object_changes column' do
+      expect(subject.object_changes_col_is_json?).to eql false
+    end
   end
 
   describe 'instance methods' do
     let(:trashable) { Trashable.new :name => 'Bob' }
     subject { trashable.draft }
 
-    describe :event, :create?, :update?, :destroy?, :object, :changeset do
+    describe 'event, create?, update?, destroy?, object, changeset' do
       context 'with `create` draft' do
         before { trashable.draft_creation }
-        its(:event) { should eql 'create' }
-        its(:create?) { should be_true }
-        its(:update?) { should be_false }
-        its(:destroy?) { should be_false }
-        its(:object) { should be_present }
-        its(:changeset) { should include :id }
-        its(:changeset) { should include :name }
-        its(:changeset) { should_not include :title }
-        its(:changeset) { should include :created_at }
-        its(:changeset) { should include :updated_at }
-        its(:previous_draft) { should be_nil }
+
+        it 'is a `create` event' do
+          expect(subject.event).to eql 'create'
+        end
+
+        it 'identifies as a `create` event' do
+          expect(subject.create?).to eql true
+        end
+
+        it 'does not identify as an `update` event' do
+          expect(subject.update?).to eql false
+        end
+
+        it 'does not identify as a `destroy` event' do
+          expect(subject.destroy?).to eql false
+        end
+
+        it 'has an object' do
+          expect(subject.object).to be_present
+        end
+
+        it 'has an `id` in the `changeset`' do
+          expect(subject.changeset).to include :id
+        end
+
+        it 'has a `name` in the `changeset`' do
+          expect(subject.changeset).to include :name
+        end
+
+        it 'does not have a `title` in the `changeset`' do
+          expect(subject.changeset).to_not include :title
+        end
+
+        it 'has `created_at` in the `changeset`' do
+          expect(subject.changeset).to include :created_at
+        end
+
+        it 'has `updated_at` in the `changeset`' do
+          expect(subject.changeset).to include :updated_at
+        end
+
+        it 'does not have a `previous_draft`' do
+          expect(subject.previous_draft).to be_nil
+        end
 
         context 'updated create' do
           before do
@@ -32,17 +71,49 @@ describe Draftsman::Draft do
             trashable.draft_update
           end
 
-          it { should be_create }
-          it { should_not be_update }
-          it { should_not be_destroy }
-          its(:event) { should eql 'create' }
-          its(:object) { should be_present }
-          its(:changeset) { should include :id }
-          its(:changeset) { should include :name }
-          its(:changeset) { should_not include :title }
-          its(:changeset) { should include :created_at }
-          its(:changeset) { should include :updated_at }
-          its(:previous_draft) { should be_nil }
+          it 'identifies as a `create` event' do
+            expect(subject.create?).to eql true
+          end
+
+          it 'does not identify as an `update` event' do
+            expect(subject.update?).to eql false
+          end
+
+          it 'does not identify as a `destroy` event' do
+            expect(subject.destroy?).to eql false
+          end
+
+          it 'is a `create` event' do
+            expect(subject.event).to eql 'create'
+          end
+
+          it 'has an `object`' do
+            expect(subject.object).to be_present
+          end
+
+          it 'has an `id` in the `changeset`' do
+            expect(subject.changeset).to include :id
+          end
+
+          it 'has a `name` in the `changeset`' do
+            expect(subject.changeset).to include :name
+          end
+
+          it 'does not have a `title` in the `changeset`' do
+            expect(subject.changeset).to_not include :title
+          end
+
+          it 'has `created_at` in the `changeset`' do
+            expect(subject.changeset).to include :created_at
+          end
+
+          it 'has `updated_at` in the `changeset`' do
+            expect(subject.changeset).to include :updated_at
+          end
+
+          it 'does not have a `previous_draft`' do
+            expect(subject.previous_draft).to be_nil
+          end
         end
       end
 
@@ -54,17 +125,49 @@ describe Draftsman::Draft do
           trashable.draft_update
         end
 
-        it { should_not be_create }
-        it { should be_update }
-        it { should_not be_destroy }
-        its(:event) { should eql 'update' }
-        its(:object) { should be_present }
-        its(:changeset) { should_not include :id }
-        its(:changeset) { should include :name }
-        its(:changeset) { should include :title }
-        its(:changeset) { should_not include :created_at }
-        its(:changeset) { should_not include :updated_at }
-        its(:previous_draft) { should be_nil }
+        it 'does not identify as a `create` event' do
+          expect(subject.create?).to eql false
+        end
+
+        it 'identifies as an `update event' do
+          expect(subject.update?).to eql true
+        end
+
+        it 'does not identify as a `destroy` event' do
+          expect(subject.destroy?).to eql false
+        end
+
+        it 'has an `update` event' do
+          expect(subject.event).to eql 'update'
+        end
+
+        it 'has an `object`' do
+          expect(subject.object).to be_present
+        end
+
+        it 'does not have an `id` in the `changeset`' do
+          expect(subject.changeset).to_not include :id
+        end
+
+        it 'has a `name` in the `changeset`' do
+          expect(subject.changeset).to include :name
+        end
+
+        it 'has a `title` in the `changeset`' do
+          expect(subject.changeset).to include :title
+        end
+
+        it 'does not have `created_at` in the `changeset`' do
+          expect(subject.changeset).to_not include :created_at
+        end
+
+        it 'does not have `updated_at` in the `changeset`' do
+          expect(subject.changeset).to_not include :updated_at
+        end
+
+        it 'does not have a `previous_draft`' do
+          expect(subject.previous_draft).to be_nil
+        end
 
         context 'updating the update' do
           before do
@@ -72,17 +175,49 @@ describe Draftsman::Draft do
             trashable.draft_update
           end
 
-          it { should_not be_create }
-          it { should be_update }
-          it { should_not be_destroy }
-          its(:event) { should eql 'update' }
-          its(:object) { should be_present }
-          its(:changeset) { should_not include :id }
-          its(:changeset) { should include :name }
-          its(:changeset) { should_not include :title }
-          its(:changeset) { should_not include :created_at }
-          its(:changeset) { should_not include :updated_at }
-          its(:previous_draft) { should be_nil }
+          it 'does not identify as a `create` event' do
+            expect(subject.create?).to eql false
+          end
+
+          it 'identifies as an `update` event' do
+            expect(subject.update?).to eql true
+          end
+
+          it 'does not identify as a `destroy` event' do
+            expect(subject.destroy?).to eql false
+          end
+
+          it 'has an `update` event' do
+            expect(subject.event).to eql 'update'
+          end
+
+          it 'has an `object`' do
+            expect(subject.object).to be_present
+          end
+
+          it 'does not have an `id` in the `changeset`' do
+            expect(subject.changeset).to_not include :id
+          end
+
+          it 'has a `name` in the `changeset`' do
+            expect(subject.changeset).to include :name
+          end
+
+          it 'does not have a `title` in the `changeset`' do
+            expect(subject.changeset).to_not include :title
+          end
+
+          it 'does not have `created_at` in the `changeset`' do
+            expect(subject.changeset).to_not include :created_at
+          end
+
+          it 'does not have `updated_at` in the `changeset`' do
+            expect(subject.changeset).to_not include :updated_at
+          end
+
+          it 'does not have a `previous_draft`' do
+            expect(subject.previous_draft).to be_nil
+          end
         end
       end
 
@@ -93,14 +228,37 @@ describe Draftsman::Draft do
             trashable.draft_destroy
           end
 
-          it { should_not be_create }
-          it { should_not be_update }
-          it { should be_destroy }
-          it { should_not be_destroyed }
-          its(:event) { should eql 'destroy' }
-          its(:object) { should be_present }
-          its(:changeset) { should eql Hash.new }
-          its(:previous_draft) { should be_nil }
+          it 'does not identify as a `create` event' do
+            expect(subject.create?).to eql false
+          end
+
+          it 'does not identify as an `update` event' do
+            expect(subject.update?).to eql false
+          end
+
+          it 'identifies as a `destroy` event' do
+            expect(subject.destroy?).to eql true
+          end
+
+          it 'is not destroyed' do
+            expect(subject.destroyed?).to eql false
+          end
+
+          it 'is a `destroy` event' do
+            expect(subject.event).to eql 'destroy'
+          end
+
+          it 'has an `object`' do
+            expect(subject.object).to be_present
+          end
+
+          it 'has an empty `changeset`' do
+            expect(subject.changeset).to eql Hash.new
+          end
+
+          it 'does not have a `previous_draft`' do
+            expect(subject.previous_draft).to be_nil
+          end
         end
 
         context 'with previous `create` draft' do
@@ -109,36 +267,95 @@ describe Draftsman::Draft do
             trashable.draft_destroy
           end
 
-          it { should_not be_create }
-          it { should_not be_update }
-          it { should be_destroy }
-          it { should_not be_destroyed }
-          its(:event) { should eql 'destroy' }
-          its(:object) { should be_present }
-          its(:changeset) { should include :id }
-          its(:changeset) { should include :name }
-          its(:changeset) { should_not include :title }
-          its(:changeset) { should include :created_at }
-          its(:changeset) { should include :updated_at }
-          its(:previous_draft) { should be_present }
+          it 'does not identify as a `create` event' do
+            expect(subject.create?).to eql false
+          end
+
+          it 'does not identify as an `update` event' do
+            expect(subject.update?).to eql false
+          end
+
+          it 'identifies as a `destroy` event' do
+            expect(subject.destroy?).to eql true
+          end
+
+          it 'is not destroyed' do
+            expect(subject.destroyed?).to eql false
+          end
+
+          it 'is a `destroy` event' do
+            expect(subject.event).to eql 'destroy'
+          end
+
+          it 'has an `object`' do
+            expect(subject.object).to be_present
+          end
+
+          it 'has an `id` in the `changeset`' do
+            expect(subject.changeset).to include :id
+          end
+
+          it 'has a `name` in the `changeset`' do
+            expect(subject.changeset).to include :name
+          end
+
+          it 'does not have a `title` in the `changeset`' do
+            expect(subject.changeset).to_not include :title
+          end
+
+          it 'has `created_at` in the `changeset`' do
+            expect(subject.changeset).to include :created_at
+          end
+
+          it 'has `updated_at` in the `changeset`' do
+            expect(subject.changeset).to include :updated_at
+          end
+
+          it 'has a `previous_draft`' do
+            expect(subject.previous_draft).to be_present
+          end
         end
       end
     end
 
-    describe :publish! do
+    describe 'publish!' do
       context 'with `create` draft' do
         before { trashable.draft_creation }
         subject { trashable.draft.publish!; return trashable.reload }
-        it { expect { subject }.to_not raise_exception }
-        it { should be_published }
-        it { should_not be_trashed }
-        it { should_not be_draft }
-        its(:published_at) { should be_present }
-        its(:draft_id) { should be_nil }
-        its(:draft) { should be_nil }
-        its(:trashed_at) { should be_nil }
-
-        it 'deletes the draft' do
+
+        it 'does not raise an exception' do
+          expect { subject }.to_not raise_exception
+        end
+
+        it 'publishes the item' do
+          expect(subject.published?).to eql true
+        end
+
+        it 'is not trashed' do
+          expect(subject.trashed?).to eql false
+        end
+
+        it 'is no longer a draft' do
+          expect(subject.draft?).to eql false
+        end
+
+        it 'should have a `published_at` timestamp' do
+          expect(subject.published_at).to be_present
+        end
+
+        it 'does not have a `draft_id`' do
+          expect(subject.draft_id).to be_nil
+        end
+
+        it 'does not have a draft' do
+          expect(subject.draft).to be_nil
+        end
+
+        it 'does not have a `trashed_at` timestamp' do
+          expect(subject.trashed_at).to be_nil
+        end
+
+        it 'deletes the draft record' do
           expect { subject }.to change(Draftsman::Draft, :count).by(-1)
         end
       end
@@ -151,17 +368,44 @@ describe Draftsman::Draft do
         end
 
         subject { trashable.draft.publish!; return trashable.reload }
-        it { expect { subject }.to_not raise_exception }
-        it { should be_published }
-        it { should_not be_draft }
-        it { should_not be_trashed }
-        its(:name) { should eql 'Sam' }
-        its(:published_at) { should be_present }
-        its(:draft_id) { should be_nil }
-        its(:draft) { should be_nil }
-        its(:trashed_at) { should be_nil }
-
-        it 'deletes the draft' do
+
+        it 'does not raise an exception' do
+          expect { subject }.to_not raise_exception
+        end
+
+        it 'publishes the item' do
+          expect(subject.published?).to eql true
+        end
+
+        it 'is no longer a draft' do
+          expect(subject.draft?).to eql false
+        end
+
+        it 'is not trashed' do
+          expect(subject.trashed?).to eql false
+        end
+
+        it 'has an updated `name`' do
+          expect(subject.name).to eql 'Sam'
+        end
+
+        it 'has a `published_at` timestamp' do
+          expect(subject.published_at).to be_present
+        end
+
+        it 'does not have a `draft_id`' do
+          expect(subject.draft_id).to be_nil
+        end
+
+        it 'does not have a `draft`' do
+          expect(subject.draft).to be_nil
+        end
+
+        it 'does not have a `trashed_at` timestamp' do
+          expect(subject.trashed_at).to be_nil
+        end
+
+        it 'destroys the draft' do
           expect { subject }.to change(Draftsman::Draft, :count).by(-1)
         end
 
@@ -207,17 +451,20 @@ describe Draftsman::Draft do
       end
     end
 
-    describe :revert! do
+    describe 'revert!' do
       context 'with `create` draft' do
         before { trashable.draft_creation }
         subject { trashable.draft.revert! }
-        it { expect { subject }.to_not raise_exception }
 
-        it 'deletes the draft' do
+        it 'does not raise an exception' do
+          expect { subject }.to_not raise_exception
+        end
+
+        it 'destroys the draft' do
           expect { subject }.to change(Draftsman::Draft, :count).by(-1)
         end
 
-        it 'deletes associated item' do
+        it 'destroys associated item' do
           expect { subject }.to change(Trashable, :count).by(-1)
         end
       end
@@ -230,17 +477,32 @@ describe Draftsman::Draft do
         end
 
         subject { trashable.draft.revert!; return trashable.reload }
-        it { expect { subject }.to_not raise_exception }
-        it { should_not be_draft }
-        its(:name) { should eql 'Bob' }
-        its(:draft_id) { should be_nil }
-        its(:draft) { should be_nil }
 
-        it 'deletes the draft' do
+        it 'does not raise an exception' do
+          expect { subject }.to_not raise_exception
+        end
+
+        it 'is no longer a draft' do
+          expect(subject.draft?).to eql false
+        end
+
+        it 'reverts its `name`' do
+          expect(subject.name).to eql 'Bob'
+        end
+
+        it 'does not have a `draft_id`' do
+          expect(subject.draft_id).to be_nil
+        end
+
+        it 'does not have a `draft`' do
+          expect(subject.draft).to be_nil
+        end
+
+        it 'destroys the draft record' do
           expect { subject }.to change(Draftsman::Draft, :count).by(-1)
         end
 
-        it 'does not delete the associated item' do
+        it 'does not destroy the associated item' do
           expect { subject }.to_not change(Trashable, :count)
         end
       end
@@ -253,18 +515,36 @@ describe Draftsman::Draft do
           end
 
           subject { trashable.draft.revert!; return trashable.reload }
-          it { expect { subject }.to_not raise_exception }
-          it { should_not be_trashed }
-          it { should_not be_draft }
-          its(:draft_id) { should be_nil }
-          its(:draft) { should be_nil }
-          its(:trashed_at) { should be_nil }
-
-          it 'deletes the draft' do
+
+          it 'does not raise an exception' do
+            expect { subject }.to_not raise_exception
+          end
+
+          it 'is not trashed' do
+            expect(subject.trashed?).to eql false
+          end
+
+          it 'is no longer a draft' do
+            expect(subject.draft?).to eql false
+          end
+
+          it 'does not have a `draft_id`' do
+            expect(subject.draft_id).to be_nil
+          end
+
+          it 'does not have a `draft`' do
+            expect(subject.draft).to be_nil
+          end
+
+          it 'does not have a `trashed_at` timestamp' do
+            expect(subject.trashed_at).to be_nil
+          end
+
+          it 'destroys the draft record' do
             expect { subject }.to change(Draftsman::Draft, :count).by(-1)
           end
 
-          it 'does not delete the associated item' do
+          it 'does not destroy the associated item' do
             expect { subject }.to_not change(Trashable, :count)
           end
         end
@@ -276,38 +556,59 @@ describe Draftsman::Draft do
           end
 
           subject { trashable.draft.revert!; return trashable.reload }
-          it { expect { subject }.to_not raise_exception }
-          it { should_not be_trashed }
-          it { should be_draft }
-          its(:draft_id) { should be_present }
-          its(:draft) { should be_present }
-          its(:trashed_at) { should be_nil }
-
-          it 'deletes the `destroy` draft' do
+
+          it 'does not raise an exception' do
+            expect { subject }.to_not raise_exception
+          end
+
+          it 'is not trashed' do
+            expect(subject.trashed?).to eql false
+          end
+
+          it 'is a draft' do
+            expect(subject.draft?).to eql true
+          end
+
+          it 'has a `draft_id`' do
+            expect(subject.draft_id).to be_present
+          end
+
+          it 'has a `draft`' do
+            expect(subject.draft).to be_present
+          end
+
+          it 'does not have a `trashed_at` timestamp' do
+            expect(subject.trashed_at).to be_nil
+          end
+
+          it 'destroys the `destroy` draft record' do
             expect { subject }.to change(Draftsman::Draft.where(:event => 'destroy'), :count).by(-1)
           end
 
-          it 'reifies the previous `create` draft' do
+          it 'reifies the previous `create` draft record' do
             expect { subject }.to change(Draftsman::Draft.where(:event => 'create'), :count).by(1)
           end
 
-          it 'does not delete the associated item' do
+          it 'does not destroy the associated item' do
             expect { subject }.to_not change(Trashable, :count)
           end
 
-          its "draft's previous draft should be nil" do
-            subject.draft.previous_draft.should be_nil
+          it "no longer has a `previous_draft`" do
+            expect(subject.draft.previous_draft).to be_nil
           end
         end
       end
     end
 
-    describe :reify do
+    describe 'reify' do
       subject { trashable.draft.reify }
 
       context 'with `create` draft' do
         before { trashable.draft_creation }
-        its(:title) { should eql trashable.title }
+
+        it "has a `title` that matches the item's" do
+          expect(subject.title).to eql trashable.title
+        end
 
         context 'updated create' do
           before do
@@ -315,8 +616,13 @@ describe Draftsman::Draft do
             trashable.draft_update
           end
 
-          its(:name) { should eql 'Sam' }
-          its(:title) { should be_nil }
+          it 'has an updated `name`' do
+            expect(subject.name).to eql 'Sam'
+          end
+
+          it 'has no `title`' do
+            expect(subject.title).to be_nil
+          end
         end
       end
 
@@ -328,8 +634,13 @@ describe Draftsman::Draft do
           trashable.draft_update
         end
 
-        its(:name) { should eql 'Sam' }
-        its(:title) { should eql 'My Title' }
+        it 'has the updated `name`' do
+          expect(subject.name).to eql 'Sam'
+        end
+
+        it 'has the updated `title`' do
+          expect(subject.title).to eql 'My Title'
+        end
 
         context 'updating the update' do
           before do
@@ -337,8 +648,13 @@ describe Draftsman::Draft do
             trashable.draft_update
           end
 
-          its(:name) { should eql 'Sam' }
-          its(:title) { should be_nil }
+          it 'has the same `name`' do
+            expect(subject.name).to eql 'Sam'
+          end
+
+          it 'has the updated `title`' do
+            expect(subject.title).to be_nil
+          end
         end
       end
 
@@ -349,8 +665,13 @@ describe Draftsman::Draft do
             trashable.draft_destroy
           end
 
-          its(:name) { should eql 'Bob' }
-          its(:title) { should be_nil }
+          it 'records the `name`' do
+            expect(subject.name).to eql 'Bob'
+          end
+
+          it 'records the `title`' do
+            expect(subject.title).to be_nil
+          end
         end
 
         context 'with previous `create` draft' do
@@ -359,8 +680,13 @@ describe Draftsman::Draft do
             trashable.draft_destroy
           end
 
-          its(:name) { should eql 'Bob' }
-          its(:title) { should be_nil }
+          it 'records the `name`' do
+            expect(subject.name).to eql 'Bob'
+          end
+
+          it 'records the `title`' do
+            expect(subject.title).to be_nil
+          end
         end
 
         context 'with previous `update` draft' do
@@ -373,8 +699,13 @@ describe Draftsman::Draft do
             trashable.reload.draft_destroy
           end
 
-          its(:name) { should eql 'Sam' }
-          its(:title) { should eql 'My Title' }
+          it 'records the updated `name`' do
+            expect(subject.name).to eql 'Sam'
+          end
+
+          it 'records the updated `title`' do
+            expect(subject.title).to eql 'My Title'
+          end
         end
       end
     end