draftsman 0.3.7 → 0.4.0

data/spec/dummy/db/schema.rb

@@ -11,10 +11,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20150408234937) do
-
-  # These are extensions that must be enabled in order to support this database
-  enable_extension "plpgsql"
+ActiveRecord::Schema.define(version: 20160328184419) do
 
   create_table "bastards", force: :cascade do |t|
     t.string   "name"
@@ -56,6 +53,21 @@ ActiveRecord::Schema.define(version: 20150408234937) do
     t.string   "user_agent"
   end
 
+  create_table "enumables", force: :cascade do |t|
+    t.integer  "status",       null: false
+    t.integer  "draft_id"
+    t.datetime "published_at"
+  end
+
+  create_table "moneybags", force: :cascade do |t|
+    t.integer  "price_cents",    default: 0,     null: false
+    t.string   "price_currency", default: "USD", null: false
+    t.integer  "draft_id"
+    t.datetime "published_at"
+    t.datetime "created_at",                     null: false
+    t.datetime "updated_at",                     null: false
+  end
+
   create_table "only_children", force: :cascade do |t|
     t.string   "name"
     t.integer  "parent_id"
@@ -85,6 +97,18 @@ ActiveRecord::Schema.define(version: 20150408234937) do
     t.datetime "updated_at"
   end
 
+  create_table "talkatives", force: :cascade do |t|
+    t.string   "before_comment"
+    t.string   "around_early_comment"
+    t.string   "around_late_comment"
+    t.string   "after_comment"
+    t.integer  "draft_id"
+    t.datetime "trashed_at"
+    t.datetime "published_at"
+    t.datetime "created_at"
+    t.datetime "updated_at"
+  end
+
   create_table "trashables", force: :cascade do |t|
     t.string   "name"
     t.string   "title"
@@ -112,11 +136,4 @@ ActiveRecord::Schema.define(version: 20150408234937) do
     t.datetime "updated_at"
   end
 
-  add_foreign_key "children", "drafts"
-  add_foreign_key "only_children", "drafts"
-  add_foreign_key "parents", "drafts"
-  add_foreign_key "skippers", "drafts"
-  add_foreign_key "trashables", "drafts"
-  add_foreign_key "vanillas", "drafts"
-  add_foreign_key "whitelisters", "drafts"
 end

data/spec/models/child_spec.rb

@@ -50,8 +50,8 @@ describe Child, :type => :model do
       before do
         parent.save!
         child.save!
-        child.draft_destroy
-        parent.draft_destroy
+        child.draft_destruction
+        parent.draft_destruction
       end
 
       subject { child.draft.publish! }
@@ -104,8 +104,8 @@ describe Child, :type => :model do
       before do
         parent.save!
         child.save!
-        child.draft_destroy
-        parent.draft_destroy
+        child.draft_destruction
+        parent.draft_destruction
       end
 
       subject do
@@ -216,7 +216,7 @@ describe Child, :type => :model do
     context 'parent `destroy` draft with child `destroy` draft' do
       before do
         child.save!
-        parent.draft_destroy
+        parent.draft_destruction
         child.reload
       end
 
@@ -245,7 +245,7 @@ describe Child, :type => :model do
     context 'parent `destroy` draft with child `destroy` draft' do
       before do
         child.save!
-        parent.draft_destroy
+        parent.draft_destruction
         child.reload
       end
 

data/spec/models/draft_spec.rb

@@ -229,7 +229,7 @@ describe Draftsman::Draft do
         context 'without previous draft' do
           before do
             trashable.save!
-            trashable.draft_destroy
+            trashable.draft_destruction
           end
 
           it 'does not identify as a `create` event' do
@@ -268,7 +268,7 @@ describe Draftsman::Draft do
         context 'with previous `create` draft' do
           before do
             trashable.draft_creation
-            trashable.draft_destroy
+            trashable.draft_destruction
           end
 
           it 'does not identify as a `create` event' do
@@ -422,7 +422,7 @@ describe Draftsman::Draft do
         context 'without previous draft' do
           before do
             trashable.save!
-            trashable.draft_destroy
+            trashable.draft_destruction
           end
 
           subject { trashable.draft.publish! }
@@ -439,7 +439,7 @@ describe Draftsman::Draft do
         context 'with previous `create` draft' do
           before do
             trashable.draft_creation
-            trashable.draft_destroy
+            trashable.draft_destruction
           end
 
           subject { trashable.draft.publish! }
@@ -515,7 +515,7 @@ describe Draftsman::Draft do
         context 'without previous draft' do
           before do
             trashable.save!
-            trashable.draft_destroy
+            trashable.draft_destruction
           end
 
           subject { trashable.draft.revert!; return trashable.reload }
@@ -556,7 +556,7 @@ describe Draftsman::Draft do
         context 'with previous `create` draft' do
           before do
             trashable.draft_creation
-            trashable.draft_destroy
+            trashable.draft_destruction
           end
 
           subject { trashable.draft.revert!; return trashable.reload }
@@ -666,7 +666,7 @@ describe Draftsman::Draft do
         context 'without previous draft' do
           before do
             trashable.save!
-            trashable.draft_destroy
+            trashable.draft_destruction
           end
 
           it 'records the `name`' do
@@ -681,7 +681,7 @@ describe Draftsman::Draft do
         context 'with previous `create` draft' do
           before do
             trashable.draft_creation
-            trashable.draft_destroy
+            trashable.draft_destruction
           end
 
           it 'records the `name`' do
@@ -700,7 +700,7 @@ describe Draftsman::Draft do
             trashable.title = 'My Title'
             trashable.draft_update
             # Typically, 2 draft operations won't happen in the same request, so reload before draft-destroying.
-            trashable.reload.draft_destroy
+            trashable.reload.draft_destruction
           end
 
           it 'records the updated `name`' do

data/spec/models/enumable_spec.rb

@@ -0,0 +1,14 @@
+require 'spec_helper'
+
+RSpec.describe Enumable, :type => :model do
+  let(:enumable) { Enumable.new(status: :active) }
+  describe '#draft' do
+    describe '#reify' do
+      before { enumable.draft_creation }
+
+      it 'does not raise an exception' do
+        expect { enumable.draft.reify }.to_not raise_error
+      end
+    end
+  end
+end

data/spec/models/parent_spec.rb

@@ -45,8 +45,8 @@ describe Parent do
     context 'parent `destroy` draft with child `destroy` draft' do
       before do
         child.save!
-        child.draft_destroy
-        parent.draft_destroy
+        child.draft_destruction
+        parent.draft_destruction
       end
 
       subject { parent.draft.publish! }
@@ -90,8 +90,8 @@ describe Parent do
     context 'parent `destroy` draft with child `destroy` draft' do
       before do
         child.save!
-        child.draft_destroy
-        parent.draft_destroy
+        child.draft_destruction
+        parent.draft_destruction
       end
 
       subject do
@@ -169,7 +169,7 @@ describe Parent do
     context 'parent `destroy` draft with child `destroy` draft' do
       before do
         child.save!
-        parent.draft_destroy
+        parent.draft_destruction
         child.reload
       end
 
@@ -206,7 +206,7 @@ describe Parent do
     context 'parent `destroy` draft with child `destroy` draft' do
       before do
         child.save!
-        parent.draft_destroy
+        parent.draft_destruction
         child.reload
       end
 

data/spec/models/skipper_spec.rb

@@ -345,7 +345,7 @@ describe Skipper do
   end
 
   # Not applicable to this customization
-  describe 'draft_destroy' do
+  describe 'draft_destruction' do
   end
 
   # Not applicable to this customization

data/spec/models/talkative_spec.rb

@@ -0,0 +1,224 @@
+require 'spec_helper'
+
+# A Talkative has a call to `has_drafts` and all nine callbacks defined:
+#
+# -  `before_draft_creation`
+# -  `around_draft_creation`
+# -  `after_draft_creation`
+# -  `before_draft_update`
+# -  `around_draft_update`
+# -  `after_draft_update`
+# -  `before_draft_destruction`
+# -  `around_draft_destruction`
+# -  `after_draft_destruction`
+RSpec.describe Talkative do
+  let(:talkative) { subject }
+
+  describe '.draftable?' do
+    it 'is draftable' do
+      expect(talkative.class.draftable?).to eql true
+    end
+  end
+
+  describe '#draft_creation' do
+    before { talkative.draft_creation }
+
+    describe '`before_draft_creation` callback' do
+      it 'changes `before_comment` attribute' do
+        expect(talkative.before_comment).to eql "I changed before creation"
+      end
+
+      it 'persists updated `before_comment` attribute to draft' do
+        talkative.reload
+        expect(talkative.draft.reify.before_comment).to eql "I changed before creation"
+      end
+    end
+
+    describe 'around_draft_creation callback' do
+      it 'changes `around_early_comment` attribute (before yield)' do
+        expect(talkative.around_early_comment).to eql "I changed around creation (before yield)"
+      end
+
+      it 'persists updated `around_early_comment` attribute to draft' do
+        talkative.reload
+        expect(talkative.draft.reify.around_late_comment).to be_nil
+      end
+
+      it 'changes `around_late_comment` attribute (after yield)' do
+        expect(talkative.around_late_comment).to eql "I changed around creation (after yield)"
+      end
+
+      it 'does not persist updated `around_late_comment` attribute' do
+        talkative.reload
+        expect(talkative.around_late_comment).to be_nil
+      end
+
+      it 'does not persist updated `around_late_comment` attribute to draft' do
+        talkative.reload
+        expect(talkative.draft.reify.around_late_comment).to be_nil
+      end
+    end
+
+    describe '`after_draft_creation` callback' do
+      it 'changes `after_comment` attribute' do
+        expect(talkative.after_comment).to eql "I changed after creation"
+      end
+
+      it 'does not persist updated `after_comment` attribute' do
+        talkative.reload
+        expect(talkative.after_comment).to be_nil
+      end
+
+      it 'does not persist updated `after_comment` attribute to draft' do
+        talkative.reload
+        expect(talkative.draft.reify.after_comment).to be_nil
+      end
+    end
+  end
+
+  describe '#draft_update' do
+    before do
+      talkative.draft_creation
+      talkative.draft_update
+    end
+
+    describe '`before_draft_update` callback' do
+      it 'changes `before_comment` attribute' do
+        expect(talkative.before_comment).to eql "I changed before update"
+      end
+
+      it 'persists updated `before_comment` attribute to draft' do
+        talkative.reload
+        expect(talkative.draft.reify.before_comment).to eql "I changed before update"
+      end
+    end
+
+    describe '`around_draft_update` callback' do
+      it 'changes `around_early_comment` attribute (before yield)' do
+        expect(talkative.around_early_comment).to eql "I changed around update (before yield)"
+      end
+
+      it 'persists updated `around_early_comment` attribute' do
+        talkative.reload
+        expect(talkative.around_early_comment).to eql "I changed around update (before yield)"
+      end
+
+      it 'persists updated `around_early_comment` attribute to draft' do
+        talkative.reload
+        expect(talkative.draft.reify.around_early_comment).to eql "I changed around update (before yield)"
+      end
+
+      it 'changes `around_late_comment` attribute (after yield)' do
+        expect(talkative.around_late_comment).to eql "I changed around update (after yield)"
+      end
+
+      it 'does not persist updated `around_late_comment` attribute' do
+        talkative.reload
+        expect(talkative.around_late_comment).to eql 'I changed around creation (after yield)'
+      end
+
+      it 'does not persist updated `around_late_comment` attribute to draft' do
+        talkative.reload
+        expect(talkative.draft.reify.around_late_comment).to eql 'I changed around creation (after yield)'
+      end
+    end
+
+    describe 'after callback' do
+      it 'changes `after_comment` attribute' do
+        expect(talkative.after_comment).to eql "I changed after update"
+      end
+
+      it 'does not persist updated `after_comment` attribute' do
+        talkative.reload
+        expect(talkative.after_comment).to eql 'I changed after creation'
+      end
+
+      it 'does not persist updated `after_comment` attribute to draft' do
+        talkative.reload
+        expect(talkative.draft.reify.after_comment).to eql 'I changed after creation'
+      end
+    end
+  end
+
+  describe '#draft_destruction' do
+    before do
+      talkative.draft_creation
+      talkative.draft_destruction
+    end
+
+    describe '`before_draft_destruction` callback' do
+      it 'changes `before_comment` attribute' do
+        expect(talkative.before_comment).to eql "I changed before destroy"
+      end
+
+      it 'does not persist updated `before_comment` attribute' do
+        talkative.reload
+        expect(talkative.before_comment).to eql "I changed before creation"
+      end
+
+      it 'does not persist updated `before_comment` attribute to draft' do
+        talkative.reload
+        expect(talkative.draft.reify.before_comment).to eql "I changed before creation"
+      end
+    end
+
+    describe '`around_draft_destruction` callback' do
+      it 'changes `around_early_comment` attribute (before yield)' do
+        expect(talkative.around_early_comment).to eql "I changed around destroy (before yield)"
+      end
+
+      it 'does not persist `around_early_comment` attribute (before yield)' do
+        expect(talkative.around_early_comment).to eql "I changed around destroy (before yield)"
+      end
+
+      it 'does not persist updated `around_early_comment` attribute to draft' do
+        talkative.reload
+        expect(talkative.draft.reify.around_early_comment).to eql "I changed around creation (before yield)"
+      end
+
+      it 'changes `around_late_comment` attribute (after yield)' do
+        expect(talkative.around_late_comment).to eql "I changed around destroy (after yield)"
+      end
+
+      it 'does not persist updated `around_late_comment` attribute' do
+        talkative.reload
+        expect(talkative.around_late_comment).to be_nil
+      end
+
+      it 'does not persist updated `around_late_comment` attribute to draft' do
+        talkative.reload
+        expect(talkative.draft.reify.around_late_comment).to be_nil
+      end
+    end
+
+    describe '`after_draft_destruction` callback' do
+      it 'changes `before_comment` attribute' do
+        expect(talkative.before_comment).to eql "I changed before destroy"
+      end
+
+      it 'does not persist updated `before_comment` attribute' do
+        talkative.reload
+        expect(talkative.before_comment).to eql 'I changed before creation'
+      end
+
+      it 'does not persist updated `before_comment` attribute to draft' do
+        talkative.reload
+        expect(talkative.draft.reify.before_comment).to eql 'I changed before creation'
+      end
+
+      it 'changes `after_comment` attribute' do
+        expect(talkative.after_comment).to eql "I changed after destroy"
+      end
+
+      it 'does not persist updated `after_comment` attribute' do
+        talkative.reload
+        expect(talkative.after_comment).to be_nil
+      end
+
+      it 'does not persist updated `after_comment` attribute to draft' do
+        talkative.reload
+        expect(talkative.draft.reify.after_comment).to be_nil
+      end
+    end
+  end
+end

data/spec/models/trashable_spec.rb

@@ -5,22 +5,24 @@ require 'spec_helper'
 describe Trashable do
   let(:trashable) { Trashable.new :name => 'Bob' }
 
-  it 'is draftable' do
-    expect(subject.class.draftable?).to eql true
+  describe '.draftable?' do
+    it 'is draftable' do
+      expect(subject.class.draftable?).to eql true
+    end
   end
 
   # Not affected by this customization
-  describe 'draft_creation' do
+  describe '#draft_creation' do
   end
 
   # Not affected by this customization
-  describe 'draft_update' do
+  describe '#draft_update' do
   end
 
-  describe 'draft_destroy' do
+  describe '#draft_destruction' do
     context 'with `:create` draft' do
       before { trashable.draft_creation }
-      subject { trashable.draft_destroy; return trashable }
+      subject { trashable.draft_destruction; return trashable }
 
       it 'is persisted' do
         expect(subject).to be_persisted
@@ -84,7 +86,7 @@ describe Trashable do
         trashable.draft_update
       end
 
-      subject { trashable.draft_destroy; return trashable.reload }
+      subject { trashable.draft_destruction; return trashable.reload }
 
       it 'is persisted' do
         expect(subject).to be_persisted
@@ -145,8 +147,8 @@ describe Trashable do
         trashable.update_attributes! :published_at => Time.now
       end
 
-      subject { trashable.draft_destroy; return trashable.reload }
-      
+      subject { trashable.draft_destruction; return trashable.reload }
+
       it 'is persisted' do
         expect(subject).to be_persisted
       end
@@ -203,11 +205,11 @@ describe Trashable do
     let!(:trashed_trashable)   { Trashable.create :name => 'Ralph' }
 
     # Not affected by this customization
-    describe 'drafted' do
+    describe '.drafted' do
     end
 
-    describe 'live' do
-      before { trashed_trashable.draft_destroy }
+    describe '.live' do
+      before { trashed_trashable.draft_destruction }
       subject { Trashable.live }
 
       it 'returns 2 records' do
@@ -232,11 +234,11 @@ describe Trashable do
     end
 
     # Not affected by this customization
-    describe 'published' do
+    describe '.published' do
     end
 
-    describe 'trashed' do
-      before { trashed_trashable.draft_destroy }
+    describe '.trashed' do
+      before { trashed_trashable.draft_destruction }
       subject { Trashable.trashed }
 
       it 'returns 1 record' do

data/spec/models/vanilla_spec.rb

@@ -260,7 +260,7 @@ describe Vanilla do
   end
 
   # Not applicable to this customization
-  describe 'draft_destroy' do
+  describe 'draft_destruction' do
   end
 
   describe 'scopes' do

data/spec/models/whitelister_spec.rb

@@ -432,7 +432,7 @@ describe Whitelister do
   end
 
   # Not affected by this customization
-  describe 'draft_destroy' do
+  describe 'draft_destruction' do
   end
 
   # Not affected by this customization