proclaimer 0.2.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f828bb32881de602413bf79b13d3d7cfe93e75e1
-  data.tar.gz: 22716b13b309b4fe2d049eefb1f8d28bf93e699d
+  metadata.gz: 52c69df73835fe00be8d56f7f7deeee774ad6a5e
+  data.tar.gz: 0c6b1b232a4c32c4886576acde714c3bc29e917d
 SHA512:
-  metadata.gz: 8721b30d047b3f7ff45becdee6933ad63b083cbbcb010e3dba79b3cbee9b1374842355a104b3aacfcceb7ddea9e71ecd1c71e38532de26f8138a1ecf338c1e57
-  data.tar.gz: 47e10b63f25a17f165dd0a54050824c5f5d5c43254ae4342ee0db493be48fd01d3912c1b0e70bfc6a7a2badfc7500721bd990d281231a112febc80184b511637
+  metadata.gz: dce7f586aabe933a013d6c905588839a739886a6181c0e6c61db22e8fc7d6c8835de4d3a97e3589c8693690aa98865cbfa7626f374d0c4d77a1e5c411da71308
+  data.tar.gz: fcfde9c493ee65e6b3dd47864b6a667539cbf8eb7d14fd8594050ff4c181a67b81951f4d87ea814888f830652ed0ac081bddf1eaafd3a4dd1c09235d50c32969

data/Gemfile.lock

@@ -74,7 +74,7 @@ GIT
 PATH
   remote: .
   specs:
-    proclaimer (0.2.0)
+    proclaimer (0.3.0)
       spree_backend (~> 3.0.0)
 
 GEM

data/app/models/spree/product_decorator.rb

@@ -0,0 +1,14 @@
+module Spree
+  Product.class_eval do
+    after_save :broadcast_changes, if: :anything_changed?
+
+    private
+
+    def broadcast_changes
+      ActiveSupport::Notifications.instrument(
+        "spree.product.updated",
+        product: self
+      )
+    end
+  end
+end

data/lib/proclaimer/version.rb

@@ -1,3 +1,3 @@
 module Proclaimer
-  VERSION = "0.2.0".freeze
+  VERSION = "0.3.0".freeze
 end

data/spec/models/spree/order_spec.rb

@@ -1,36 +1,38 @@
 require "spec_helper"
 
-RSpec.describe Spree::Order do
-  before { stub_const("EMPTY_PAYLOAD", Object.new) }
+module Spree
+  RSpec.describe Order, type: :model do
+    before { stub_const("EMPTY_PAYLOAD", Object.new) }
 
-  context "when order is first created" do
-    it "does not instrument any order event" do
-      payload = EMPTY_PAYLOAD
+    context "when order is first created" do
+      it "does not instrument any order event" do
+        payload = EMPTY_PAYLOAD
 
-      ActiveSupport::Notifications.subscribed(
-        -> (*args) { payload = args.last },
-        /^spree\.order/
-      ) do
-        create(:order)
+        ActiveSupport::Notifications.subscribed(
+          -> (*args) { payload = args.last },
+          /^spree\.order/
+        ) do
+          create(:order)
 
-        expect(payload).to eq EMPTY_PAYLOAD
+          expect(payload).to eq EMPTY_PAYLOAD
+        end
       end
     end
-  end
 
-  context "when order is complete" do
-    it "instruments order.complete event" do
-      payload = EMPTY_PAYLOAD
+    context "when order is complete" do
+      it "instruments order.complete event" do
+        payload = EMPTY_PAYLOAD
 
-      ActiveSupport::Notifications.subscribed(
-        -> (*args) { payload = args.last },
-        "spree.order.complete"
-      ) do
-        order = create(:order_with_line_items, state: "confirm")
-        create(:payment, amount: order.total, order: order)
-        order.next!
+        ActiveSupport::Notifications.subscribed(
+          -> (*args) { payload = args.last },
+          "spree.order.complete"
+        ) do
+          order = create(:order_with_line_items, state: "confirm")
+          create(:payment, amount: order.total, order: order)
+          order.next!
 
-        expect(payload[:order]).to eq order
+          expect(payload[:order]).to eq order
+        end
       end
     end
   end

data/spec/models/spree/order_updater_spec.rb

@@ -6,8 +6,8 @@ module Spree
     describe "#update_shipments_with_status_tracking" do
       it "calls shipment#broadcast on shipments that have changed state" do
         fake_shipments = [
-          fake_shipment_with_state('pending', 'ready'),
-          fake_shipment_with_state('pending')
+          fake_shipment_with_state("pending", "ready"),
+          fake_shipment_with_state("pending")
         ]
         fake_order = double(Order, shipments: fake_shipments)
         updater = described_class.new(fake_order)

data/spec/models/spree/payment_spec.rb

@@ -1,18 +1,20 @@
 require "spec_helper"
 
-RSpec.describe Spree::Payment do
-  context "when payment is complete" do
-    it "instruments payment.complete event" do
-      payload = nil
+module Spree
+  RSpec.describe Payment, type: :model do
+    context "when payment is complete" do
+      it "instruments payment.complete event" do
+        payload = nil
 
-      ActiveSupport::Notifications.subscribed(
-        -> (*args) { payload = args.last },
-        "spree.payment.complete"
-      ) do
-        payment = create(:payment)
-        payment.complete
+        ActiveSupport::Notifications.subscribed(
+          -> (*args) { payload = args.last },
+          "spree.payment.complete"
+        ) do
+          payment = create(:payment)
+          payment.complete
 
-        expect(payload[:payment]).to eq payment
+          expect(payload[:payment]).to eq payment
+        end
       end
     end
   end

data/spec/models/spree/product_spec.rb

@@ -0,0 +1,36 @@
+require "spec_helper"
+
+module Spree
+  RSpec.describe Product, type: :model do
+    context "when changes are present" do
+      it "instruments spree.product.updated event" do
+        product = build_product
+
+        product.run_callbacks(:save)
+
+        expect(ActiveSupport::Notifications).
+          to have_received(:instrument).
+          with("spree.product.updated", product: product)
+      end
+    end
+
+    context "when no changes are present" do
+      it "instruments spree.product.updated event" do
+        product = build_product
+        allow(product).to receive(:anything_changed?) { false }
+
+        product.run_callbacks(:save)
+
+        expect(ActiveSupport::Notifications).
+          not_to have_received(:instrument).
+          with("spree.product.updated", product: product)
+      end
+    end
+
+    def build_product
+      build(:product).tap do
+        allow(ActiveSupport::Notifications).to receive(:instrument)
+      end
+    end
+  end
+end

data/spec/models/spree/refund_spec.rb

@@ -1,34 +1,36 @@
 require "spec_helper"
 
-describe Spree::Refund do
-  before { stub_const("EMPTY_PAYLOAD", Object.new) }
+module Spree
+  RSpec.describe Refund, type: :model do
+    before { stub_const("EMPTY_PAYLOAD", Object.new) }
 
-  context "when refund have not been processed" do
-    it "does not instrument any refund event" do
-      payload = EMPTY_PAYLOAD
+    context "when refund have not been processed" do
+      it "does not instrument any refund event" do
+        payload = EMPTY_PAYLOAD
 
-      ActiveSupport::Notifications.subscribed(
-        -> (*args) { payload = args.last },
-        /^spree\.refund/
-      ) do
-        build(:refund)
+        ActiveSupport::Notifications.subscribed(
+          -> (*args) { payload = args.last },
+          /^spree\.refund/
+        ) do
+          build(:refund)
 
-        expect(payload).to eq EMPTY_PAYLOAD
+          expect(payload).to eq EMPTY_PAYLOAD
+        end
       end
     end
-  end
 
-  context "when the refund has been processed" do
-    it "instruments refund.complete event" do
-      payload = EMPTY_PAYLOAD
+    context "when the refund has been processed" do
+      it "instruments refund.complete event" do
+        payload = EMPTY_PAYLOAD
 
-      ActiveSupport::Notifications.subscribed(
-        -> (*args) { payload = args.last },
-        "spree.refund.complete"
-      ) do
-        refund = create(:refund, payment: create(:payment, amount: 100))
+        ActiveSupport::Notifications.subscribed(
+          -> (*args) { payload = args.last },
+          "spree.refund.complete"
+        ) do
+          refund = create(:refund, payment: create(:payment, amount: 100))
 
-        expect(payload[:refund]).to eq refund
+          expect(payload[:refund]).to eq refund
+        end
       end
     end
   end

data/spec/models/spree/shipment_spec.rb

@@ -1,58 +1,61 @@
 require "spec_helper"
 
-RSpec.describe Spree::Shipment do
-  context "when shipment is pending" do
-    it "instruments shipment.pending event" do
-      payload = nil
-
-      ActiveSupport::Notifications.subscribed(
-        -> (*args) { payload = args.last },
-        "spree.shipment.pending"
-      ) do
-        shipment = create(:shipment)
-
-        expect(payload[:shipment]).to eq shipment
+module Spree
+  RSpec.describe Shipment, type: :model do
+    context "when shipment is pending" do
+      it "instruments shipment.pending event" do
+        payload = nil
+
+        ActiveSupport::Notifications.subscribed(
+          -> (*args) { payload = args.last },
+          "spree.shipment.pending"
+        ) do
+          shipment = create(:shipment)
+
+          expect(payload[:shipment]).to eq shipment
+        end
       end
     end
-  end
 
-  context "when shipment is ready" do
-    it "instruments shipment.ready event" do
-      shipment = create(:shipment)
-      allow(shipment).to receive(:determine_state) { 'ready' }
-      allow(ActiveSupport::Notifications).to receive(:instrument)
+    context "when shipment is ready" do
+      it "instruments shipment.ready event" do
+        shipment = create(:shipment)
+        allow(shipment).to receive(:determine_state) { 'ready' }
+        allow(ActiveSupport::Notifications).to receive(:instrument)
 
-      shipment.ready
+        shipment.ready
 
-      expect(ActiveSupport::Notifications).
-        to have_received(:instrument).
-        with("spree.shipment.ready", shipment: shipment)
+        expect(ActiveSupport::Notifications).
+          to have_received(:instrument).
+          with("spree.shipment.ready", shipment: shipment)
+      end
     end
-  end
 
-  context "when shipment is canceled" do
-    it "instruments shipment.canceled event" do
-      shipment = create(:shipment)
-      allow(ActiveSupport::Notifications).to receive(:instrument)
+    context "when shipment is canceled" do
+      it "instruments shipment.canceled event" do
+        shipment = create(:shipment)
+        allow(ActiveSupport::Notifications).to receive(:instrument)
 
-      shipment.cancel
+        shipment.cancel
 
-      expect(ActiveSupport::Notifications).
-        to have_received(:instrument).
-        with("spree.shipment.canceled", shipment: shipment)
+        expect(ActiveSupport::Notifications).
+          to have_received(:instrument).
+          with("spree.shipment.canceled", shipment: shipment)
+      end
     end
-  end
 
-  describe "#broadcast_state" do
-    it "instruments shipment.STATE event" do
-      shipment = build(:shipment, state: 'pending')
-      allow(ActiveSupport::Notifications).to receive(:instrument)
+    describe "#broadcast_state" do
+      it "instruments shipment.STATE event" do
+        shipment = build(:shipment, state: 'pending')
+        allow(ActiveSupport::Notifications).to receive(:instrument)
 
-      shipment.broadcast_state
+        shipment.broadcast_state
 
-      expect(ActiveSupport::Notifications).
-        to have_received(:instrument).
-        with("spree.shipment.pending", shipment: shipment)
+        expect(ActiveSupport::Notifications).
+          to have_received(:instrument).
+          with("spree.shipment.pending", shipment: shipment)
+      end
     end
   end
 end
+

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: proclaimer
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Prem Sichanugrist
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-10-06 00:00:00.000000000 Z
+date: 2015-10-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: spree_backend
@@ -168,6 +168,7 @@ files:
 - app/models/spree/order_decorator.rb
 - app/models/spree/order_updater_decorator.rb
 - app/models/spree/payment_decorator.rb
+- app/models/spree/product_decorator.rb
 - app/models/spree/refund_decorator.rb
 - app/models/spree/shipment_decorator.rb
 - bin/rails
@@ -181,6 +182,7 @@ files:
 - spec/models/spree/order_spec.rb
 - spec/models/spree/order_updater_spec.rb
 - spec/models/spree/payment_spec.rb
+- spec/models/spree/product_spec.rb
 - spec/models/spree/refund_spec.rb
 - spec/models/spree/shipment_spec.rb
 - spec/proclaimer_spec.rb
@@ -214,6 +216,7 @@ test_files:
 - spec/models/spree/order_spec.rb
 - spec/models/spree/order_updater_spec.rb
 - spec/models/spree/payment_spec.rb
+- spec/models/spree/product_spec.rb
 - spec/models/spree/refund_spec.rb
 - spec/models/spree/shipment_spec.rb
 - spec/proclaimer_spec.rb