promiscuous 0.7.2 → 0.7.3

data/lib/promiscuous/common/lint/base.rb

@@ -0,0 +1,22 @@
+module Promiscuous::Common::Lint::Base
+  extend ActiveSupport::Concern
+
+  included do
+    attr_accessor :options
+  end
+
+  def initialize(options)
+    self.options = options
+  end
+
+  def lint
+  end
+
+  module ClassMethods
+    def use_option(attr)
+      define_method(attr) do
+        self.options[attr]
+      end
+    end
+  end
+end

data/lib/promiscuous/common/lint.rb

@@ -0,0 +1,3 @@
+module Promiscuous::Common::Lint
+  autoload :Base, 'promiscuous/common/lint/base'
+end

data/lib/promiscuous/common.rb

@@ -1,7 +1,4 @@
 module Promiscuous::Common
   autoload :Options, 'promiscuous/common/options'
-
-  def self.lint(*args)
-    Lint.lint(*args)
-  end
+  autoload :Lint,    'promiscuous/common/lint'
 end

data/lib/promiscuous/publisher/lint/amqp.rb

@@ -4,7 +4,7 @@ module Promiscuous::Publisher::Lint::AMQP
   def lint
     super
 
-    pub_to = publisher_instance.to
+    pub_to = publisher.to
     if pub_to != to
       raise "#{publisher} publishes #{klass} to #{pub_to} instead of #{to}"
     end

data/lib/promiscuous/publisher/lint/base.rb

@@ -1,22 +1,5 @@
 class Promiscuous::Publisher::Lint::Base
-  attr_accessor :options
-
-  def initialize(options)
-    self.options = options
-  end
-
-  def publisher_instance
-    @publisher_instance ||= publisher.new({})
-  end
-
-  def lint
-  end
-
-  def self.use_option(attr)
-    define_method(attr) do
-      self.options[attr]
-    end
-  end
+  include Promiscuous::Common::Lint::Base
 
   use_option(:publisher)
 end

data/lib/promiscuous/publisher/lint/{class_bind.rb → class.rb}

@@ -1,4 +1,4 @@
-module Promiscuous::Publisher::Lint::ClassBind
+module Promiscuous::Publisher::Lint::Class
   extend ActiveSupport::Concern
 
   def lint
@@ -11,4 +11,3 @@ module Promiscuous::Publisher::Lint::ClassBind
 
   included { use_option :klass }
 end
-

data/lib/promiscuous/publisher/lint.rb

@@ -1,6 +1,6 @@
 module Promiscuous::Publisher::Lint
   autoload :Base,        'promiscuous/publisher/lint/base'
-  autoload :ClassBind,   'promiscuous/publisher/lint/class_bind'
+  autoload :Class,       'promiscuous/publisher/lint/class'
   autoload :Attributes,  'promiscuous/publisher/lint/attributes'
   autoload :Polymorphic, 'promiscuous/publisher/lint/polymorphic'
   autoload :AMQP,        'promiscuous/publisher/lint/amqp'
@@ -17,8 +17,8 @@ module Promiscuous::Publisher::Lint
     classes.each do |klass, to|
       pub = get_publisher(klass)
 
-      lint = Class.new(Base)
-      lint.__send__(:include, ClassBind)   if pub.include?(Promiscuous::Publisher::ClassBind)
+      lint = ::Class.new(Base)
+      lint.__send__(:include, Class)       if pub.include?(Promiscuous::Publisher::Class)
       lint.__send__(:include, Attributes)  if pub.include?(Promiscuous::Publisher::Attributes)
       lint.__send__(:include, Polymorphic) if pub.include?(Promiscuous::Publisher::Polymorphic)
       lint.__send__(:include, AMQP)        if pub.include?(Promiscuous::Publisher::AMQP)

data/lib/promiscuous/publisher/mock.rb

@@ -6,9 +6,10 @@ class Promiscuous::Publisher::Mock
         self.attributes = self.attributes + options[:attributes]
       end
     else
-      class_attribute :attributes, :to
+      class_attribute :to, :attributes, :klass
 
       self.to = options[:to]
+      self.klass = options[:class]
       self.attributes = options[:attributes].to_a
 
       attr_accessor :id, :new_record
@@ -17,7 +18,13 @@ class Promiscuous::Publisher::Mock
   end
 
   def self.class_name
-    name.split('::').last
+    if self.klass
+      self.klass
+    elsif name
+      class_name = "#{name.split('::').last}"
+      class_name = $1 if class_name =~ /^(.+)Publisher$/
+      class_name
+    end
   end
 
   def initialize

data/lib/promiscuous/subscriber/lint/amqp.rb

@@ -0,0 +1,21 @@
+module Promiscuous::Subscriber::Lint::AMQP
+  extend ActiveSupport::Concern
+
+  def publisher
+    Promiscuous::Publisher::Mock.descendants.
+      select { |pub| pub.superclass == Promiscuous::Publisher::Mock }.
+      select { |pub| pub.to == from }.
+      tap { |pubs| raise "#{from} has multiple publishers: #{pubs}" if pubs.size > 1 }.
+      first
+  end
+
+  def lint
+    super
+
+    if check_publisher
+      raise "No publisher found for #{publisher}" if publisher.nil?
+    end
+  end
+
+  included { use_option :from }
+end

data/lib/promiscuous/subscriber/lint/attributes.rb

@@ -0,0 +1,19 @@
+module Promiscuous::Subscriber::Lint::Attributes
+  extend ActiveSupport::Concern
+
+  def lint
+    super
+
+    instance = subscriber.klass.new
+    subscriber.attributes.each do |attr|
+      instance.respond_to?("#{attr}=") or instance.__send__("#{attr}=")
+    end
+
+    if check_publisher
+      missing_attributes = subscriber.attributes - publisher.attributes
+      if missing_attributes.present?
+        raise "#{publisher} subscribes to non published attributes: #{missing_attributes.join(", ")}"
+      end
+    end
+  end
+end

data/lib/promiscuous/subscriber/lint/base.rb

@@ -0,0 +1,8 @@
+class Promiscuous::Subscriber::Lint::Base
+  include Promiscuous::Common::Lint::Base
+
+  class_attribute :check_publisher
+  self.check_publisher = Promiscuous::Publisher::Mock.descendants.present?
+
+  use_option(:subscriber)
+end

data/lib/promiscuous/subscriber/lint/class.rb

@@ -0,0 +1,13 @@
+module Promiscuous::Subscriber::Lint::Class
+  extend ActiveSupport::Concern
+
+  def lint
+    super
+
+    if klass && subscriber.klass != klass
+      raise "Subscriber #{subscriber} does not replicate #{klass}"
+    end
+  end
+
+  included { use_option :klass }
+end

data/lib/promiscuous/subscriber/lint/polymorphic.rb

@@ -0,0 +1,41 @@
+module Promiscuous::Subscriber::Lint::Polymorphic
+  extend ActiveSupport::Concern
+
+  def parent_publisher
+    publisher
+    @parent_publisher
+  end
+
+  def publisher
+    @parent_publisher = super
+    if @parent_publisher
+      ([parent_publisher] + parent_publisher.descendants).
+        select { |pub| pub.class_name == subscriber.from_type }.
+        first
+    end
+  end
+
+  def lint
+    super
+
+    if check_publisher
+      subscriber_types = subscriber.descendants.map &:from_type
+      publisher_types = publisher.descendants.map &:class_name
+      missing_types = publisher_types - subscriber_types
+      if missing_types.present?
+        raise "#{publisher} misses some child types: #{missing_types.join(", ")}"
+      end
+    end
+
+    unless skip_polymorphic
+      subscriber.descendants.each do |pub|
+        self.class.new(options.merge(:publisher => pub,
+                                     :skip_polymorphic => true)).lint
+      end
+    end
+  end
+
+  included do
+    use_option :skip_polymorphic
+  end
+end

data/lib/promiscuous/subscriber/lint.rb

@@ -0,0 +1,22 @@
+module Promiscuous::Subscriber::Lint
+  autoload :Base,        'promiscuous/subscriber/lint/base'
+  autoload :Class,       'promiscuous/subscriber/lint/class'
+  autoload :Attributes,  'promiscuous/subscriber/lint/attributes'
+  autoload :Polymorphic, 'promiscuous/subscriber/lint/polymorphic'
+  autoload :AMQP,        'promiscuous/subscriber/lint/amqp'
+
+  def self.lint(binding_classes)
+    binding_classes.each do |from, klass|
+      sub = Promiscuous::Subscriber::AMQP.subscribers[from]
+      raise "#{from} has no subscriber" if sub.nil?
+
+      lint = ::Class.new(Base)
+      lint.__send__(:include, Class)       if sub.include?(Promiscuous::Subscriber::Class)
+      lint.__send__(:include, Attributes)  if sub.include?(Promiscuous::Subscriber::Attributes)
+      lint.__send__(:include, Polymorphic) if sub.include?(Promiscuous::Subscriber::Polymorphic)
+      lint.__send__(:include, AMQP)        if sub.include?(Promiscuous::Subscriber::AMQP)
+      lint.new(:klass => klass, :subscriber => sub, :from => from).lint
+    end
+    true
+  end
+end

data/lib/promiscuous/subscriber.rb

@@ -6,11 +6,16 @@ module Promiscuous::Subscriber
   autoload :Class,        'promiscuous/subscriber/class'
   autoload :Envelope,     'promiscuous/subscriber/envelope'
   autoload :Error,        'promiscuous/subscriber/error'
+  autoload :Lint,         'promiscuous/subscriber/lint'
   autoload :Model,        'promiscuous/subscriber/model'
   autoload :Mongoid,      'promiscuous/subscriber/mongoid'
   autoload :Polymorphic,  'promiscuous/subscriber/polymorphic'
   autoload :Upsert,       'promiscuous/subscriber/upsert'
 
+  def self.lint(*args)
+    Lint.lint(*args)
+  end
+
   def self.get_subscriber_from(payload)
     sub = AMQP.subscriber_from(payload)
     if sub && defined?(Polymorphic) && sub.include?(Polymorphic)

data/lib/promiscuous/version.rb

@@ -1,3 +1,3 @@
 module Promiscuous
-  VERSION = '0.7.2'
+  VERSION = '0.7.3'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: promiscuous
 version: !ruby/object:Gem::Version
-  version: 0.7.2
+  version: 0.7.3
   prerelease: 
 platform: ruby
 authors:
@@ -122,8 +122,8 @@ files:
 - lib/promiscuous/publisher/attributes.rb
 - lib/promiscuous/publisher/class.rb
 - lib/promiscuous/publisher/mongoid.rb
-- lib/promiscuous/publisher/lint/class_bind.rb
 - lib/promiscuous/publisher/lint/attributes.rb
+- lib/promiscuous/publisher/lint/class.rb
 - lib/promiscuous/publisher/lint/amqp.rb
 - lib/promiscuous/publisher/lint/base.rb
 - lib/promiscuous/publisher/lint/polymorphic.rb
@@ -139,10 +139,16 @@ files:
 - lib/promiscuous/amqp/rubyamqp.rb
 - lib/promiscuous/amqp/bunny.rb
 - lib/promiscuous/publisher.rb
+- lib/promiscuous/subscriber/lint.rb
 - lib/promiscuous/subscriber/attributes.rb
 - lib/promiscuous/subscriber/class.rb
 - lib/promiscuous/subscriber/mongoid.rb
 - lib/promiscuous/subscriber/error.rb
+- lib/promiscuous/subscriber/lint/attributes.rb
+- lib/promiscuous/subscriber/lint/class.rb
+- lib/promiscuous/subscriber/lint/amqp.rb
+- lib/promiscuous/subscriber/lint/base.rb
+- lib/promiscuous/subscriber/lint/polymorphic.rb
 - lib/promiscuous/subscriber/mongoid/embedded.rb
 - lib/promiscuous/subscriber/active_record.rb
 - lib/promiscuous/subscriber/upsert.rb
@@ -151,7 +157,9 @@ files:
 - lib/promiscuous/subscriber/amqp.rb
 - lib/promiscuous/subscriber/base.rb
 - lib/promiscuous/subscriber/polymorphic.rb
+- lib/promiscuous/common/lint.rb
 - lib/promiscuous/common/options.rb
+- lib/promiscuous/common/lint/base.rb
 - lib/promiscuous/worker.rb
 - lib/promiscuous/railtie/replicate.rake
 - lib/promiscuous/railtie.rb