noticent 0.0.1.pre.pre → 0.0.1

data/lib/noticent/definitions/channel.rb

@@ -9,11 +9,17 @@ module Noticent
       attr_reader :options
 
       def initialize(config, name, group: :default, klass: nil)
+        raise BadConfiguration, 'name should be a symbol' unless name.is_a? Symbol
+        raise BadConfiguration, '\'_any_\' is a reserved channel name' if name == :_any_
+        raise BadConfiguration, '\'_none_\' is a reserved channel name' if name == :_none_
+
         @name = name
         @group = group
         @config = config
 
-        suggested_class_name = @config.base_module_name + '::' + name.to_s.camelize
+        sub_module = @config.use_sub_modules ? '::Channels::' : '::'
+        suggested_class_name = @config.base_module_name + sub_module + name.to_s.camelize
+
         @klass = klass.nil? ? suggested_class_name.camelize.constantize : klass
       rescue NameError
         raise Noticent::BadConfiguration, "no class found for #{suggested_class_name}"
@@ -37,7 +43,6 @@ module Noticent
       rescue ArgumentError
         raise Noticent::BadConfiguration, "channel #{@klass} initializer arguments are mismatching."
       end
-
     end
   end
 end

data/lib/noticent/definitions/scope.rb

@@ -5,22 +5,26 @@ module Noticent
     class Scope
       attr_reader :name
       attr_reader :payload_class
+      attr_reader :check_constructor
 
-      def initialize(config, name, payload_class: nil)
+      def initialize(config, name, payload_class: nil, check_constructor: true)
         @config = config
         @name = name
-        suggeste_name = config.base_module_name + "::#{name.capitalize}Payload"
-        @payload_class = payload_class.nil? ? suggeste_name.constantize : payload_class
+        @check_constructor = check_constructor
+
+        sub_module = @config.use_sub_modules ? '::Payloads::' : '::'
+        suggested_name = config.base_module_name + sub_module + "#{name.capitalize}Payload"
+        @payload_class = payload_class.nil? ? suggested_name.constantize : payload_class
       rescue NameError
-        raise BadConfiguration, "scope #{suggeste_name} class not found"
+        raise BadConfiguration, "scope #{suggested_name} class not found"
       end
 
-      def alert(name, &block)
+      def alert(name, constructor_name: nil, &block)
         alerts = @config.instance_variable_get(:@alerts) || {}
 
         raise BadConfiguration, "alert '#{name}' already defined" if alerts.include? name
 
-        alert = Noticent::Definitions::Alert.new(@config, name: name, scope: self)
+        alert = Noticent::Definitions::Alert.new(@config, name: name, scope: self, constructor_name: constructor_name.nil? ? name : constructor_name)
         @config.hooks&.run(:pre_alert_registration, alert)
         alert.instance_eval(&block) if block_given?
         @config.hooks&.run(:post_alert_registration, alert)

data/lib/noticent/dispatcher.rb

@@ -2,11 +2,11 @@
 
 module Noticent
   class Dispatcher
-    def initialize(config, alert_name, payload, context = {})
+    def initialize(config, alert_name, payload, configuration = {})
       @config = config
       @alert_name = alert_name
       @payload = payload
-      @context = context
+      @configuration = configuration
 
       validate!
 
@@ -34,8 +34,8 @@ module Noticent
 
     # only returns recipients that have opted-in for this channel
     def filter_recipients(recipients, channel)
-      raise ArgumentError, 'channel should be a string or symbol' unless channel.is_a?(String) || channel.is_a?(Symbol)
-      raise ArgumentError, 'recipients is nil' if recipients.nil?
+      raise ArgumentError, "channel should be a string or symbol" unless channel.is_a?(String) || channel.is_a?(Symbol)
+      raise ArgumentError, "recipients is nil" if recipients.nil?
 
       recipients.select { |recipient| @config.opt_in_provider.opted_in?(recipient_id: recipient.id, scope: scope.name, entity_id: @entity_id, alert_name: alert.name, channel_name: channel) }
     end
@@ -43,9 +43,15 @@ module Noticent
     def dispatch
       notifiers.values.each do |notifier|
         recs = recipients(notifier.recipient)
-        @config.channels_by_group(notifier.channel_group).each do |channel|
+        notifier.applicable_channels.each do |channel|
           to_send = filter_recipients(recs, channel.name)
-          channel_instance = channel.instance(@config, to_send, @payload, @context)
+
+          if to_send.count == 0 && @config.skip_alert_with_no_subscribers
+            Noticent.configuration.logger.info "Skipping alert #{alert.name} as they are no subscribers"
+            return
+          end
+
+          channel_instance = channel.instance(@config, to_send, @payload, @configuration)
           begin
             raise Noticent::BadConfiguration, "channel #{channel.name} (#{channel.klass}) doesn't have a method called #{alert.name}" unless channel_instance.respond_to? alert.name
 
@@ -54,7 +60,7 @@ module Noticent
             # log and move on
             raise if @config.halt_on_error
 
-            Noticent.logger.error e
+            Noticent.configuration.logger.error e
           end
         end
       end
@@ -63,12 +69,12 @@ module Noticent
     private
 
     def validate!
-      raise Noticent::BadConfiguration, 'no base_dir defined' if @config.base_dir.nil?
+      raise Noticent::BadConfiguration, "no base_dir defined" if @config.base_dir.nil?
       raise Noticent::MissingConfiguration if @config.nil?
       raise Noticent::BadConfiguration if @config.alerts.nil?
       raise Noticent::InvalidAlert, "no alert #{@alert_name} found" if @config.alerts[@alert_name].nil?
-      raise ::ArgumentError, 'payload is nil' if @payload.nil?
-      raise ::ArgumentError, 'alert is not a symbol' unless @alert_name.is_a?(Symbol)
+      raise ::ArgumentError, "payload is nil" if @payload.nil?
+      raise ::ArgumentError, "alert is not a symbol" unless @alert_name.is_a?(Symbol)
       raise Noticent::BadConfiguration, "payload (#{@payload.class}) doesn't belong to this scope (#{scope.name}) as it requires #{scope.payload_class}" unless !scope.payload_class.nil? && @payload.is_a?(scope.payload_class)
       raise Noticent::BadConfiguration, "payload doesn't have a #{scope.name}_id method" unless @payload.respond_to?("#{scope.name}_id")
     end

data/lib/noticent/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Noticent
-  VERSION ||= '0.0.1-pre'
+  VERSION ||= '0.0.1'
 end

data/lib/noticent/view.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'yaml'
+require "yaml"
 
 module Noticent
   class View
@@ -16,38 +16,40 @@ module Noticent
     attr_reader :raw_data         # frontmatter in their raw (pre render) format
     attr_reader :rendered_data    # frontmatter rendered in string format
 
-    def initialize(filename, template_filename: '', channel:)
+    def initialize(filename, template_filename: "", channel:)
       raise ViewNotFound, "view #{filename} not found" unless File.exist?(filename)
-      raise ViewNotFound, "template #{template_filename} not found" if template_filename != '' && !File.exist?(template_filename)
-      raise ArgumentError, 'channel is nil' if channel.nil?
+      raise ViewNotFound, "template #{template_filename} not found" if template_filename != "" && !File.exist?(template_filename)
+      raise ArgumentError, "channel is nil" if channel.nil?
 
       @filename = filename
       @view_content = File.read(filename)
-      @template_content = template_filename != '' ? File.read(template_filename) : '<%= yield %>'
-      @template_filename = template_filename != '' ? template_filename : ''
+      @template_content = template_filename != "" ? File.read(template_filename) : "<%= @content %>"
+      @template_filename = template_filename != "" ? template_filename : ""
       @channel = channel
     end
 
-    def process
+    def process(context)
       parse
-      render_content
-      render_data
+      render_data(context)
       read_data
+      # TODO this is nasty. we need to refactor to have an independent render context which somehow merges the binding with the channel.
+      @channel.data = @data
+      render_content(context)
     end
 
     private
 
-    def render_content
-      @content = @channel.render_within_context(@template_content, @view_content)
+    def render_content(context)
+      @content = @channel.render_within_context(template: @template_content, content: @raw_content, context: context)
     end
 
-    def render_data
+    def render_data(context)
       if @raw_data.nil?
         @rendered_data = nil
         return
       end
 
-      @rendered_data = @channel.render_within_context(nil, @raw_data)
+      @rendered_data = @channel.render_within_context(template: nil, content: @raw_data, context: context)
     end
 
     def parse
@@ -70,7 +72,7 @@ module Noticent
       if @raw_data.nil?
         @data = nil
       else
-        raise ArgumentError, 'read_data was called before rendering' if @rendered_data.nil?
+        raise ArgumentError, "read_data was called before rendering" if @rendered_data.nil?
 
         data = ::YAML.safe_load(@rendered_data)
         @data = data.deep_symbolize_keys

data/noticent.gemspec

@@ -1,36 +1,40 @@
 # frozen_string_literal: true
 
-lib = File.expand_path('lib', __dir__)
+lib = File.expand_path("lib", __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require 'noticent/version'
+require "noticent/version"
 
 Gem::Specification.new do |spec|
-  spec.name          = 'noticent'
-  spec.version       = Noticent::VERSION
-  spec.authors       = ['Khash Sajadi']
-  spec.email         = ['khash@cloud66.com']
+  spec.name = "noticent"
+  spec.version = Noticent::VERSION
+  spec.authors = ["Khash Sajadi"]
+  spec.email = ["khash@cloud66.com"]
 
-  spec.summary       = 'Act as Notified is a flexible framework to add notifications to a Rails application'
+  spec.summary = "Act as Notified is a flexible framework to add notifications to a Rails application"
 
   # Specify which files should be added to the gem when it is released.
   # The `git ls-files -z` loads the files in the RubyGem that have been added into git.
   spec.files = Dir.chdir(File.expand_path(__dir__)) do
     `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
   end
-  spec.bindir        = 'exe'
-  spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
-  spec.require_paths = ['lib']
+  spec.bindir = "exe"
+  spec.executables = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
+  spec.require_paths = ["lib"]
 
-  spec.add_dependency 'activerecord', '~> 5.2'
-  spec.add_dependency 'activesupport', '~> 5.2'
+  spec.add_dependency "activerecord", "~> 5.2"
+  spec.add_dependency "activesupport", "~> 5.2"
+  spec.add_dependency "actionpack", "~> 5.2"
 
-  spec.add_development_dependency 'bundler', '~> 2.0'
-  spec.add_development_dependency 'factory_bot', '~> 5.0'
-  spec.add_development_dependency 'generator_spec', '~> 0.9'
-  spec.add_development_dependency 'rake', '~> 10.0'
-  spec.add_development_dependency 'rspec', '~> 3.8'
-  spec.add_development_dependency 'rubocop', '~> 0.69'
-  spec.add_development_dependency 'rubocop-performance', '~> 1.3'
-  spec.add_development_dependency 'rufo', '~> 0.7'
-  spec.add_development_dependency 'sqlite3', '~> 1.4'
+  spec.add_development_dependency "combustion", "~> 1.1"
+  spec.add_development_dependency "bundler", "~> 2.0"
+  spec.add_development_dependency "factory_bot", "~> 5.0"
+  spec.add_development_dependency "generator_spec", "~> 0.9"
+  spec.add_development_dependency "rails", "~> 5.2"
+  spec.add_development_dependency "rake", "~> 10.0"
+  spec.add_development_dependency "rspec", "~> 3.8"
+  spec.add_development_dependency "rubocop", "~> 0.69"
+  spec.add_development_dependency "rubocop-performance", "~> 1.3"
+  spec.add_development_dependency "rufo", "~> 0.7"
+  spec.add_development_dependency "sqlite3", "~> 1.4"
+  spec.add_development_dependency "byebug", "~> 11.0"
 end

data/testing/channels/email.rb

@@ -8,6 +8,7 @@ module Noticent
       end
 
       def some_event
+        @some_value = 1
         render
       end
 

data/testing/channels/exclusive.rb

@@ -0,0 +1,9 @@
+module Noticent
+  module Testing
+    class Exclusive < ::Noticent::Channel
+      def only_here
+        render
+      end
+    end
+  end
+end

data/testing/channels/simple.rb

@@ -0,0 +1,9 @@
+module Noticent
+  module Testing
+    class Simple < ::Noticent::Channel
+      def some_event
+        render
+      end
+    end
+  end
+end

data/testing/payloads/comment_payload.rb

@@ -3,6 +3,16 @@ module Noticent
     # this is a example payload for comment as scope
     class CommentPayload < Noticent::Testing::Payload
       attr_accessor :comment_id
+      attr_reader :users
+
+      def self.three
+        # nop
+      end
+
+      def self.new_signup
+        # nop
+      end
+
     end
   end
 end

data/testing/payloads/post_payload.rb

@@ -16,6 +16,50 @@ module Noticent
         []
       end
 
+      def self.some_event(some_value)
+        # nop
+      end
+
+      def self.foo(options)
+        # nop
+      end
+
+      def self.boo
+        # nop
+      end
+
+      def self.one
+        # nop
+	  end
+	  
+	  def self.only_here
+		# nop
+	  end
+
+      def self.two
+        # nop
+      end
+
+      def self.tfa_enabled
+        # nop
+      end
+
+      def self.sign_up
+        # nop
+      end
+
+      def self.new_signup
+        # nop
+      end
+
+      def self.fuzz
+        # nop
+      end
+
+      def self.buzz
+        # nop
+      end
+
     end
   end
 end

data/testing/views/email/some_event.html.erb

@@ -4,3 +4,6 @@ fuzz: <%= @payload.some_attribute %>
 This is normal test
 Some non-context stuff <%= 1 + 1 %>
 This comes from <%= @payload.some_attribute %>
+This comes from an instance variable <%= @some_value %>
+This is from Rails <%= @routes.hello_path %>
+This is foo <%= @data[:foo] %> and this is bar <%= @data[:fuzz] %>

data/testing/views/email/some_event.txt.erb

@@ -4,3 +4,5 @@ fuzz: <%= @payload.some_attribute %>
 This is normal test
 Some non-context stuff <%= 1 + 1 %>
 This comes from <%= @payload.some_attribute %>
+This comes from an instance variable <%= @some_value %>
+

data/testing/views/layouts/layout.html.erb

@@ -1,5 +1,5 @@
 Header
 
-<%= yield %>
+<%= @content %>
 
 Footer

data/testing/views/simple/default.html.erb

@@ -0,0 +1 @@
+this is from the default view

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: noticent
 version: !ruby/object:Gem::Version
-  version: 0.0.1.pre.pre
+  version: 0.0.1
 platform: ruby
 authors:
 - Khash Sajadi
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-05-29 00:00:00.000000000 Z
+date: 2019-08-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -38,6 +38,34 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '5.2'
+- !ruby/object:Gem::Dependency
+  name: actionpack
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '5.2'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '5.2'
+- !ruby/object:Gem::Dependency
+  name: combustion
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.1'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.1'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -80,6 +108,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.9'
+- !ruby/object:Gem::Dependency
+  name: rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '5.2'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '5.2'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -164,6 +206,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.4'
+- !ruby/object:Gem::Dependency
+  name: byebug
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '11.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '11.0'
 description: 
 email:
 - khash@cloud66.com
@@ -178,10 +234,12 @@ files:
 - ".vscode/settings.json"
 - Gemfile
 - Gemfile.lock
+- LICENSE
 - README.md
 - Rakefile
 - bin/console
 - bin/setup
+- config.ru
 - lib/generators/noticent/noticent.rb
 - lib/generators/noticent/templates/create_opt_ins.rb
 - lib/generators/noticent/templates/noticent_initializer.rb
@@ -205,7 +263,9 @@ files:
 - noticent.gemspec
 - testing/channels/boo.rb
 - testing/channels/email.rb
+- testing/channels/exclusive.rb
 - testing/channels/foo.rb
+- testing/channels/simple.rb
 - testing/channels/slack.rb
 - testing/channels/webhook.rb
 - testing/models/receipient.rb
@@ -215,6 +275,7 @@ files:
 - testing/views/email/some_event.html.erb
 - testing/views/email/some_event.txt.erb
 - testing/views/layouts/layout.html.erb
+- testing/views/simple/default.html.erb
 homepage: 
 licenses: []
 metadata: {}
@@ -229,12 +290,11 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6.2
+rubygems_version: 3.0.4
 signing_key: 
 specification_version: 4
 summary: Act as Notified is a flexible framework to add notifications to a Rails application