RubyGems - draftsman - Versions diffs - 0.5.1 → 0.6.0 - Mend

draftsman 0.5.1 → 0.6.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (31) hide show

checksums.yaml +4 -4
data/.travis.yml +12 -0
data/CHANGELOG.md +34 -0
data/README.md +244 -181
data/lib/draftsman/config.rb +4 -1
data/lib/draftsman/draft.rb +117 -80
data/lib/draftsman/frameworks/rspec.rb +1 -1
data/lib/draftsman/model.rb +253 -235
data/lib/draftsman/version.rb +1 -1
data/lib/draftsman.rb +26 -5
data/lib/generators/draftsman/templates/config/initializers/draftsman.rb +16 -7
data/spec/draftsman_spec.rb +7 -9
data/spec/dummy/app/controllers/application_controller.rb +2 -2
data/spec/dummy/app/models/overridden_draft.rb +7 -0
data/spec/dummy/app/models/talkative.rb +21 -43
data/spec/dummy/db/schema.rb +1 -0
data/spec/models/child_spec.rb +17 -17
data/spec/models/draft_spec.rb +893 -305
data/spec/models/enumable_spec.rb +3 -2
data/spec/models/overridden_draft_spec.rb +41 -0
data/spec/models/parent_spec.rb +10 -10
data/spec/models/skipper_spec.rb +221 -219
data/spec/models/talkative_spec.rb +107 -108
data/spec/models/trashable_spec.rb +6 -10
data/spec/models/vanilla_spec.rb +570 -229
data/spec/models/whitelister_spec.rb +489 -348
metadata +7 -10
data/spec/dummy/db/migrate/20110208155312_set_up_test_tables.rb +0 -95
data/spec/dummy/db/migrate/20150404203627_add_talkatives_table_to_tests.rb +0 -18
data/spec/dummy/db/migrate/20150408234937_add_only_children.rb +0 -16
data/spec/dummy/db/migrate/20160328184419_create_enumables.rb +0 -9

data/spec/models/whitelister_spec.rb CHANGED Viewed

@@ -1,430 +1,571 @@
 require 'spec_helper'
 describe Whitelister do
-  let(:whitelister) { Whitelister.new :name => 'Bob' }
+  let(:whitelister) { Whitelister.new(name: 'Bob') }
   it { should be_draftable }
-  # Not affected by this customization
-  describe 'draft_creation' do
-  end
-  describe 'draft_update' do
-    subject do
-      whitelister.draft_update
-      whitelister.reload
+  describe '#save_draft' do
+    # Not affected by this customization.
+    context 'on create' do
     end
-    context 'with whitelisted change' do
-      context 'without draft' do
-        before do
-          whitelister.save!
-          whitelister.attributes = { :name => 'Sam', :ignored => 'Meh.' }
-        end
-        it 'is persisted' do
-          expect(subject).to be_persisted
-        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 'has an `update` draft' do
-          expect(subject.draft.update?).to eql true
-        end
-        it 'has its original `name`' do
-          expect(subject.name).to eql 'Bob'
-        end
-        it 'has the ignored value' do
-          expect(subject.ignored).to eql 'Meh.'
-        end
-        it 'creates a new draft' do
-          expect { subject }.to change(Draftsman::Draft, :count).by(1)
-        end
-        it 'has an `update` draft' do
-          expect(subject.draft.update?).to eql true
-        end
-        it "updates the draft's name to `Sam`" do
-          expect(subject.draft.reify.name).to eql 'Sam'
-        end
-        context 'changing back to initial state' do
+    context 'on update' do
+      context 'with whitelisted change' do
+        context 'without draft' do
           before do
-            whitelister.draft_update
-            whitelister.attributes = { :name => 'Bob', :ignored => 'Huzzah!' }
-          end
-          it 'is no longer a draft' do
-            expect(subject.draft?).to eql false
-          end
-          it 'has its original `name`' do
-            expect(subject.name).to eql 'Bob'
-          end
-          it 'updates the ignored attribute' do
-            expect(subject.ignored).to eql 'Huzzah!'
-          end
-          it 'does not have a `draft_id`' do
-            expect(subject.draft_id).to be_nil
+            whitelister.save!
+            whitelister.attributes = { name: 'Sam', ignored: 'Meh.' }
           end
-          it 'does not have a `draft`' do
-            expect(subject.draft).to be_nil
-          end
-          it 'destroys the draft' do
-            expect { subject }.to change(Draftsman::Draft.where(:id => whitelister.draft_id), :count).by(-1)
-          end
-        end
-      end
-      context 'with existing `create` draft' do
-        before { whitelister.draft_creation }
-        context 'with changes' do
-          before { whitelister.attributes = { :name => 'Sam', :ignored => 'Meh.' } }
           it 'is persisted' do
-            expect(subject).to be_persisted
+            whitelister.save_draft
+            whitelister.reload
+            expect(whitelister).to be_persisted
           end
           it 'is a draft' do
-            expect(subject.draft?).to eql true
+            whitelister.save_draft
+            whitelister.reload
+            expect(whitelister.draft?).to eql true
           end
           it 'has a `draft_id`' do
-            expect(subject.draft_id).to be_present
+            whitelister.save_draft
+            whitelister.reload
+            expect(whitelister.draft_id).to be_present
           end
           it 'has a `draft`' do
-            expect(subject.draft).to be_present
-          end
-          it 'has a `create` draft' do
-            expect(subject.draft.create?).to eql true
-          end
-          it 'updates the `name`' do
-            expect(subject.name).to eql 'Sam'
-          end
-          it 'updates the existing draft' do
-            expect { subject }.to_not change(Draftsman::Draft.where(:id => whitelister.draft_id), :count)
-          end
-          it "updates the draft's `name`" do
-            expect(subject.draft.reify.name).to eql 'Sam'
-          end
-          it "updates the draft's ignored attribute" do
-            expect(subject.draft.reify.ignored).to eql 'Meh.'
-          end
-        end
-        context 'with no changes' do
-          it 'is persisted' do
-            expect(subject).to be_persisted
-          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 'has a `create` draft' do
-            expect(subject.draft.create?).to eql true
-          end
-          it 'keeps its original `name`' do
-            expect(subject.name).to eql 'Bob'
-          end
-          it "doesn't change the number of drafts" do
-            expect { subject }.to_not change(Draftsman::Draft.where(:id => whitelister.draft_id), :count)
-          end
-        end
-      end
-      context 'with existing `update` draft' do
-        before do
-          whitelister.save!
-          whitelister.attributes = { :name => 'Sam', :ignored => 'Meh.' }
-        end
-        context 'with changes' do
-          before { whitelister.attributes = { :name => 'Steve', :ignored => 'Huzzah!' } }
-          it 'is persisted' do
-            expect(subject).to be_persisted
-          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
+            whitelister.save_draft
+            whitelister.reload
+            expect(whitelister.draft).to be_present
           end
           it 'has an `update` draft' do
-            expect(subject.draft.update?).to eql true
+            whitelister.save_draft
+            whitelister.reload
+            expect(whitelister.draft.update?).to eql true
           end
           it 'has its original `name`' do
-            expect(subject.name).to eql 'Bob'
+            whitelister.save_draft
+            whitelister.reload
+            expect(whitelister.name).to eql 'Bob'
           end
-          it 'has the updated ignored attribute' do
-            expect(subject.ignored).to eql 'Huzzah!'
+          it 'has the ignored value' do
+            whitelister.save_draft
+            whitelister.reload
+            expect(whitelister.ignored).to eql 'Meh.'
           end
-          it 'updates the existing draft' do
-            expect { subject }.to_not change(Draftsman::Draft.where(:id => whitelister.draft_id), :count)
+          it 'creates a new draft' do
+            expect { whitelister.save_draft }.to change(Draftsman::Draft, :count).by(1)
           end
-          it "updates its draft's `name`" do
-            expect(subject.draft.reify.name).to eql 'Steve'
+          it 'has an `update` draft' do
+            whitelister.save_draft
+            whitelister.reload
+            expect(whitelister.draft.update?).to eql true
+          end
+          it "updates the draft's name to `Sam`" do
+            whitelister.save_draft
+            whitelister.reload
+            expect(whitelister.draft.reify.name).to eql 'Sam'
+          end
+          context 'changing back to initial state' do
+            before do
+              whitelister.save_draft
+              whitelister.attributes = { name: 'Bob', ignored: 'Huzzah!' }
+            end
+            it 'is no longer a draft' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft?).to eql false
+            end
+            it 'has its original `name`' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.name).to eql 'Bob'
+            end
+            it 'updates the ignored attribute' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.ignored).to eql 'Huzzah!'
+            end
+            it 'does not have a `draft_id`' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft_id).to be_nil
+            end
+            it 'does not have a `draft`' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft).to be_nil
+            end
+            it 'destroys the draft' do
+              expect { whitelister.save_draft }.to change(Draftsman::Draft.where(id: whitelister.draft_id), :count).by(-1)
+            end
+          end
+        end
+        context 'with existing `create` draft' do
+          before { whitelister.save_draft }
+          context 'with changes' do
+            before { whitelister.attributes = { name: 'Sam', ignored: 'Meh.' } }
+            it 'is persisted' do
+              whitelister.save_draft
+              expect(whitelister).to be_persisted
+            end
+            it 'is a draft' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft?).to eql true
+            end
+            it 'has a `draft_id`' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft_id).to be_present
+            end
+            it 'has a `draft`' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft).to be_present
+            end
+            it 'has a `create` draft' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft.create?).to eql true
+            end
+            it 'updates the `name`' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.name).to eql 'Sam'
+            end
+            it 'updates the existing draft' do
+              expect { whitelister.save_draft }.to_not change(Draftsman::Draft.where(id: whitelister.draft_id), :count)
+            end
+            it "updates the draft's `name`" do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft.reify.name).to eql 'Sam'
+            end
+            it "updates the draft's ignored attribute" do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft.reify.ignored).to eql 'Meh.'
+            end
+          end
+          context 'with no changes' do
+            it 'is persisted' do
+              whitelister.save_draft
+              expect(whitelister).to be_persisted
+            end
+            it 'is a draft' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft?).to eql true
+            end
+            it 'has a `draft_id`' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft_id).to be_present
+            end
+            it 'has a `draft`' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft).to be_present
+            end
+            it 'has a `create` draft' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft.create?).to eql true
+            end
+            it 'keeps its original `name`' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.name).to eql 'Bob'
+            end
+            it "doesn't change the number of drafts" do
+              expect { whitelister.save_draft }.to_not change(Draftsman::Draft.where(id: whitelister.draft_id), :count)
+            end
+          end
+        end
+        context 'with existing `update` draft' do
+          before do
+            whitelister.save!
+            whitelister.attributes = { name: 'Sam', ignored: 'Meh.' }
+          end
+          context 'with changes' do
+            before { whitelister.attributes = { name: 'Steve', ignored: 'Huzzah!' } }
+            it 'is persisted' do
+              whitelister.save_draft
+              expect(whitelister).to be_persisted
+            end
+            it 'is a draft' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft?).to eql true
+            end
+            it 'has a `draft_id`' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft_id).to be_present
+            end
+            it 'has a `draft`' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft).to be_present
+            end
+            it 'has an `update` draft' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft.update?).to eql true
+            end
+            it 'has its original `name`' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.name).to eql 'Bob'
+            end
+            it 'has the updated ignored attribute' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.ignored).to eql 'Huzzah!'
+            end
+            it 'updates the existing draft' do
+              expect { whitelister.save_draft }.to_not change(Draftsman::Draft.where(id: whitelister.draft_id), :count)
+            end
+            it "updates its draft's `name`" do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft.reify.name).to eql 'Steve'
+            end
+            it "updates its draft's `ignored` attribute" do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft.reify.ignored).to eql 'Huzzah!'
+            end
+          end
+          context 'with no changes' do
+            it 'is persisted' do
+              whitelister.save_draft
+              expect(whitelister).to be_persisted
+            end
+            it 'is a draft' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft?).to eql true
+            end
+            it 'has a `draft_id`' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft_id).to be_present
+            end
+            it 'has a `draft`' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft).to be_present
+            end
+            it 'has its original `name`' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.name).to eql 'Bob'
+            end
+            it "doesn't change the number of drafts" do
+              expect { whitelister.save_draft }.to_not change(Draftsman::Draft.where(id: whitelister.draft_id), :count)
+            end
+            it "does not update its draft's `name`" do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft.reify.name).to eql 'Sam'
+            end
+            it 'still has an `update` draft' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft.update?).to eql true
+            end
           end
+        end
+      end
-          it "updates its draft's `ignored` attribute" do
-            expect(subject.draft.reify.ignored).to eql 'Huzzah!'
+      context 'without whitelisted change' do
+        context 'without existing draft' do
+          before do
+            whitelister.save!
+            whitelister.ignored = 'Huzzah!'
           end
-        end
-        context 'with no changes' do
           it 'is persisted' do
-            expect(subject).to be_persisted
+            whitelister.save_draft
+            expect(whitelister).to be_persisted
           end
-          it 'is a draft' do
-            expect(subject.draft?).to eql true
+          it 'is not a draft' do
+            whitelister.save_draft
+            whitelister.reload
+            expect(whitelister.draft?).to eql false
           end
-          it 'has a `draft_id`' do
-            expect(subject.draft_id).to be_present
+          it 'does not have a `draft_id`' do
+            whitelister.save_draft
+            whitelister.reload
+            expect(whitelister.draft_id).to be_nil
           end
-          it 'has a `draft`' do
-            expect(subject.draft).to be_present
+          it 'does not create a `draft`' do
+            whitelister.save_draft
+            whitelister.reload
+            expect(whitelister.draft).to be_nil
           end
-          it 'has its original `name`' do
-            expect(subject.name).to eql 'Bob'
+          it 'has the same `name`' do
+            whitelister.save_draft
+            whitelister.reload
+            expect(whitelister.name).to eql 'Bob'
           end
-          it "doesn't change the number of drafts" do
-            expect { subject }.to_not change(Draftsman::Draft.where(:id => whitelister.draft_id), :count)
+          it 'has an updated ignored attribute' do
+            whitelister.save_draft
+            whitelister.reload
+            expect(whitelister.ignored).to eql 'Huzzah!'
           end
-          it "does not update its draft's `name`" do
-            expect(subject.draft.reify.name).to eql 'Sam'
+          it 'does not create a draft' do
+            expect { whitelister.save_draft }.to_not change(Draftsman::Draft, :count)
           end
-          it 'still has an `update` draft' do
-            expect(subject.draft.update?).to eql true
+          # Not affected by this customization
+          context 'changing back to initial state' do
           end
         end
-      end
-    end
-    context 'without whitelisted change' do
-      context 'without existing draft' do
-        before do
-          whitelister.save!
-          whitelister.ignored = 'Huzzah!'
-        end
-        it 'is persisted' do
-          expect(subject).to be_persisted
-        end
-        it 'is not 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 create a `draft`' do
-          expect(subject.draft).to be_nil
-        end
-        it 'has the same `name`' do
-          expect(subject.name).to eql 'Bob'
-        end
-        it 'has an updated ignored attribute' do
-          expect(subject.ignored).to eql 'Huzzah!'
-        end
-        it 'does not create a draft' do
-          expect { subject }.to_not change(Draftsman::Draft, :count)
-        end
-        # Not affected by this customization
-        context 'changing back to initial state' do
-        end
-      end
-      context 'with existing `create` draft' do
-        before do
-          whitelister.draft_creation
-          whitelister.ignored = 'Huzzah!'
-        end
-        it 'is persisted' do
-          expect(subject).to be_persisted
-        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 'has the same `name`' do
-          expect(subject.name).to eql 'Bob'
-        end
-        it 'has an updated ignored attribute' do
-          expect(subject.ignored).to eql 'Huzzah!'
-        end
-        it 'updates the existing draft' do
-          expect { subject }.to_not change(Draftsman::Draft.where(:id => whitelister.draft_id), :count)
-        end
-        it "updates its draft's `ignored` attribute" do
-          expect(subject.draft.reify.ignored).to eql 'Huzzah!'
-        end
-        it 'has a `create` draft' do
-          expect(subject.draft.create?).to eql true
-        end
-      end
-      context 'with existing `update` draft' do
-        before do
-          whitelister.save!
-          whitelister.attributes = { :name => 'Sam', :ignored => 'Meh.' }
-        end
+        context 'with existing `create` draft' do
+          before do
+            whitelister.save_draft
+            whitelister.ignored = 'Huzzah!'
+          end
-        context 'with changes' do
-          before { whitelister.ignored = 'Huzzah!' }
           it 'is persisted' do
-            expect(subject).to be_persisted
+            whitelister.save_draft
+            expect(whitelister).to be_persisted
           end
           it 'is a draft' do
-            expect(subject.draft?).to eql true
+            whitelister.save_draft
+            whitelister.reload
+            expect(whitelister.draft?).to eql true
           end
           it 'has a `draft_id`' do
-            expect(subject.draft_id).to be_present
+            whitelister.save_draft
+            whitelister.reload
+            expect(whitelister.draft_id).to be_present
           end
           it 'has a `draft`' do
-            expect(subject.draft).to be_present
-          end
-          it 'has an `update` draft' do
-            expect(subject.draft.update?).to eql true
+            whitelister.save_draft
+            whitelister.reload
+            expect(whitelister.draft).to be_present
           end
-          it 'has its original `name`' do
-            expect(subject.name).to eql 'Bob'
+          it 'has the same `name`' do
+            whitelister.save_draft
+            whitelister.reload
+            expect(whitelister.name).to eql 'Bob'
           end
           it 'has an updated ignored attribute' do
-            expect(subject.ignored).to eql 'Huzzah!'
+            whitelister.save_draft
+            whitelister.reload
+            expect(whitelister.ignored).to eql 'Huzzah!'
           end
           it 'updates the existing draft' do
-            expect { subject }.to_not change(Draftsman::Draft.where(:id => whitelister.draft_id), :count)
+            expect { whitelister.save_draft }.to_not change(Draftsman::Draft.where(id: whitelister.draft_id), :count)
           end
-          it "updates its draft's `name`" do
-            expect(subject.draft.reify.name).to eql 'Sam'
+          it "updates its draft's `ignored` attribute" do
+            whitelister.save_draft
+            whitelister.reload
+            expect(whitelister.draft.reify.ignored).to eql 'Huzzah!'
           end
-          it "updated its draft's `ignored` attribute" do
-            expect(subject.draft.reify.ignored).to eql 'Huzzah!'
+          it 'has a `create` draft' do
+            whitelister.save_draft
+            whitelister.reload
+            expect(whitelister.draft.create?).to eql true
           end
         end
-        context 'with no changes' do
-          it 'is persisted' do
-            expect(subject).to be_persisted
-          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 'has an `update` draft' do
-            expect(subject.draft.update?).to eql true
-          end
-          it 'has its original `name`' do
-            expect(subject.name).to eql 'Bob'
-          end
-          it 'has its original `ignored` attribute' do
-            expect(subject.ignored).to eql 'Meh.'
-          end
-          it "doesn't change the number of drafts" do
-            expect { subject }.to_not change(Draftsman::Draft.where(:id => whitelister.draft_id), :count)
-          end
-          it "does not update its draft's `name`" do
-            expect(subject.draft.reify.name).to eql 'Sam'
-          end
-          it "does not update its draft's `ignored` attribute" do
-            expect(subject.draft.reify.ignored).to eql 'Meh.'
+        context 'with existing `update` draft' do
+          before do
+            whitelister.save!
+            whitelister.attributes = { name: 'Sam', ignored: 'Meh.' }
+          end
+          context 'with changes' do
+            before { whitelister.ignored = 'Huzzah!' }
+            it 'is persisted' do
+              whitelister.save_draft
+              expect(whitelister).to be_persisted
+            end
+            it 'is a draft' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft?).to eql true
+            end
+            it 'has a `draft_id`' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft_id).to be_present
+            end
+            it 'has a `draft`' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft).to be_present
+            end
+            it 'has an `update` draft' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft.update?).to eql true
+            end
+            it 'has its original `name`' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.name).to eql 'Bob'
+            end
+            it 'has an updated ignored attribute' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.ignored).to eql 'Huzzah!'
+            end
+            it 'updates the existing draft' do
+              expect { whitelister.save_draft }.to_not change(Draftsman::Draft.where(id: whitelister.draft_id), :count)
+            end
+            it "updates its draft's `name`" do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft.reify.name).to eql 'Sam'
+            end
+            it "updated its draft's `ignored` attribute" do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft.reify.ignored).to eql 'Huzzah!'
+            end
+          end
+          context 'with no changes' do
+            it 'is persisted' do
+              whitelister.save_draft
+              expect(whitelister).to be_persisted
+            end
+            it 'is a draft' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft?).to eql true
+            end
+            it 'has a `draft_id`' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft_id).to be_present
+            end
+            it 'has a `draft`' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft).to be_present
+            end
+            it 'has an `update` draft' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft.update?).to eql true
+            end
+            it 'has its original `name`' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.name).to eql 'Bob'
+            end
+            it 'has its original `ignored` attribute' do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.ignored).to eql 'Meh.'
+            end
+            it "doesn't change the number of drafts" do
+              expect { whitelister.save_draft }.to_not change(Draftsman::Draft.where(id: whitelister.draft_id), :count)
+            end
+            it "does not update its draft's `name`" do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft.reify.name).to eql 'Sam'
+            end
+            it "does not update its draft's `ignored` attribute" do
+              whitelister.save_draft
+              whitelister.reload
+              expect(whitelister.draft.reify.ignored).to eql 'Meh.'
+            end
           end
         end
       end
@@ -432,7 +573,7 @@ describe Whitelister do
   end
   # Not affected by this customization
-  describe 'draft_destruction' do
+  describe '#draft_destruction' do
   end
   # Not affected by this customization