aspector 0.13.1 → 0.14.0

data/spec/units/deferred_logic_spec.rb

@@ -0,0 +1,35 @@
+require 'spec_helper'
+
+RSpec.describe Aspector::DeferredLogic do
+  describe '.new' do
+    pending
+  end
+
+  describe '#apply' do
+    context 'block in deferred logic' do
+      let(:klass) do
+        ClassBuilder.build do
+          def self.exec
+            'class-exec'
+          end
+        end
+      end
+
+      let(:logic) { described_class.new(-> { exec }) }
+
+      it 'should work with block' do
+        expect(logic.apply(klass)).to eq 'class-exec'
+      end
+    end
+
+    context 'block with argument in deferred logic' do
+      let(:argument_value) { rand }
+      let(:klass) { ClassBuilder.build }
+      let(:logic) { described_class.new(->(arg) { arg }) }
+
+      it 'block can take an argument' do
+        expect(logic.apply(klass, argument_value)).to eq argument_value
+      end
+    end
+  end
+end

data/spec/units/logger_spec.rb

@@ -0,0 +1,20 @@
+require 'spec_helper'
+
+RSpec.describe Aspector::Logger do
+  let(:context) { '' }
+  subject { described_class.new(context) }
+
+  describe '.new' do
+    it 'should store a context' do
+      expect(subject.context).to eq context
+    end
+
+    it 'should set a log level' do
+      expect(subject.level).to_not be_nil
+    end
+  end
+
+  describe '#postfix' do
+    pending
+  end
+end

data/spec/units/logging_spec.rb

@@ -0,0 +1,85 @@
+require 'spec_helper'
+
+RSpec.describe Aspector::Logging do
+  subject { described_class }
+  let(:context) { double }
+
+  describe '.get_logger' do
+    before do
+      subject.instance_variable_set('@logger', logger)
+    end
+
+    after do
+      subject.instance_variable_set('@logger', nil)
+    end
+
+    let(:logger) { nil }
+    let(:klass_name) { 'NonExisting' }
+
+    context 'and given logger class doesnt exist' do
+      before do
+        expect(ENV)
+          .to receive(:[])
+          .and_return(klass_name)
+          .at_least(:once)
+
+        expect($stderr)
+          .to receive(:puts)
+          .with("#{klass_name} is not a valid constant name!")
+      end
+
+      it 'should use Aspector::Logger' do
+        expect(subject.get_logger(context)).to be_instance_of(Aspector::Logger)
+      end
+    end
+
+    context 'and given logger class exists' do
+      let(:dummy_logger) { 'DummyLogger' }
+
+      before do
+        expect(ENV)
+          .to receive(:[])
+          .and_return(dummy_logger)
+          .at_least(:once)
+
+        expect(Object)
+          .to receive(:const_get)
+          .with(dummy_logger)
+          .and_return(Aspector::Logger)
+      end
+
+      it 'should use it' do
+        expect(subject.get_logger(context)).to be_instance_of(Aspector::Logger)
+      end
+    end
+  end
+
+  describe '.deconstantize' do
+    context 'and given logger class doesnt exist' do
+      let(:klass_name) { 'NonExisting' }
+
+      before do
+        expect($stderr)
+          .to receive(:puts)
+          .with("#{klass_name} is not a valid constant name!")
+      end
+
+      it 'should return internal logger class' do
+        expect(subject.send(:deconstantize, klass_name)).to eq Aspector::Logger
+      end
+    end
+
+    context 'and given logger class exists' do
+      let(:klass_name) { 'Aspector::Logger' }
+
+      before do
+        expect($stderr)
+          .not_to receive(:puts)
+      end
+
+      it 'should return it deconstantized' do
+        expect(subject.send(:deconstantize, klass_name)).to eq Aspector::Logger
+      end
+    end
+  end
+end

data/spec/units/method_matcher_spec.rb

@@ -0,0 +1,95 @@
+require 'spec_helper'
+
+RSpec.describe Aspector::MethodMatcher do
+  describe '#initialize' do
+    pending
+  end
+
+  describe '#match?' do
+    subject { described_class.new(match_data) }
+
+    context 'String matching - when we want to match method by string name' do
+      let(:method) { rand.to_s }
+      let(:match_data) { method }
+
+      it 'should match it' do
+        expect(subject.match?(method)).not_to be_nil
+      end
+    end
+
+    context 'Regular expression matching - when we want to match method by regexp' do
+      let(:method) {  rand.to_s + 'constant-part' }
+      let(:match_data) { /constant/ }
+
+      it 'should match it' do
+        expect(subject.match?(method)).not_to be_nil
+      end
+    end
+
+    context 'when we want to add more match data (multiple elements' do
+      context 'and one of them matches' do
+        let(:match_data) { %w( test1 test2 ) }
+
+        it 'should not be nil' do
+          expect(subject.match?(match_data.last)).not_to be_nil
+        end
+      end
+
+      context 'and none of them match' do
+        let(:match_data) { %w( test1 test2 ) }
+
+        it 'should be nil' do
+          expect(subject.match?('test3')).to be_nil
+        end
+      end
+
+      context 'and all of them match' do
+        let(:match_data) { [/test/, /te/]  }
+
+        it 'should not be nil' do
+          expect(subject.match?('test2')).not_to be_nil
+        end
+      end
+    end
+
+    context 'deferred logic matching' do
+      let(:match_data) { Aspector::DeferredLogic.new('') }
+      let(:method) { double }
+
+      before do
+        expect(method)
+          .to receive(:deferred_logic_results)
+          .with(match_data)
+          .and_return(/test/)
+      end
+
+      it 'should get aspect deferred logic results and match them' do
+        expect(subject.match?('test', method)).not_to be_nil
+      end
+    end
+
+    context 'deferred option matching' do
+      let(:match_data) { Aspector::DeferredOption.new[:methods] }
+      let(:method) { double }
+
+      before do
+        expect(method)
+          .to receive(:options)
+          .and_return(methods: /test/)
+      end
+
+      it 'should get aspect options value for a proper key and match it' do
+        expect(subject.match?('test', method)).not_to be_nil
+      end
+    end
+  end
+
+  describe '#to_s' do
+    let(:match_data) { [rand, rand] }
+    subject { described_class.new(match_data) }
+
+    it 'should use inspected match_data as a string representation of MethodMatcher' do
+      expect(subject.to_s).to eq match_data.map(&:inspect).join(', ')
+    end
+  end
+end

data/spec/units/object_extension_spec.rb

@@ -0,0 +1,11 @@
+require 'spec_helper'
+
+RSpec.describe Aspector::ObjectExtension do
+  describe '.aspector' do
+    pending
+  end
+
+  describe '.Aspector' do
+    pending
+  end
+end

data/spec/units/special_chars_spec.rb

@@ -0,0 +1,128 @@
+require 'spec_helper'
+
+RSpec.describe 'Special chars in method names' do
+  subject { klass.new }
+
+  methods_postfixes = %w( ? ! = )
+  full_methods = %w( + - * / ~ | % & ^ < > [] []= )
+
+  context 'special characters in instance method names' do
+    methods_postfixes.each do |char|
+      context "when method name contains '#{char}'" do
+        let(:klass) do
+          ClassBuilder.build do
+            aspector do
+              before "exec#{char}", :exec
+            end
+
+            define_method "exec#{char}" do |*_args|
+              values << char
+            end
+          end
+        end
+
+        it 'should handle it without any issues' do
+          subject.send "exec#{char}"
+          expect(subject.values).to eq ['exec-result', char]
+        end
+      end
+    end
+
+    full_methods.each do |char|
+      context "when method name is '#{char}'" do
+        let(:klass) do
+          ClassBuilder.build do
+            aspector do
+              before char, :exec
+            end
+
+            define_method char do |*_args|
+              values << char
+            end
+          end
+        end
+
+        it 'should handle it without any issues' do
+          subject.send(char, 1)
+          expect(subject.values).to eq ['exec-result', char]
+        end
+      end
+    end
+  end
+
+  context 'special characters in class method names' do
+    methods_postfixes.each do |char|
+      context "when method name contains '#{char}'" do
+        self.class.send :define_method, :methods_postfixes do
+          methods_postfixes
+        end
+
+        let(:klass) do
+          ClassBuilder.build do
+            aspector class_methods: true do
+              before "exec#{char}", :exec
+            end
+
+            class << self
+              def values
+                @values ||= []
+              end
+
+              def exec(*_args)
+                values << 'exec-result'
+              end
+
+              methods_postfixes.each do |char|
+                define_method "exec#{char}" do |*args|
+                  values << args.first
+                end
+              end
+            end
+          end
+        end
+
+        it 'should handle it without any issues' do
+          klass.send("exec#{char}", char)
+          expect(klass.values).to eq ['exec-result', char]
+        end
+      end
+    end
+
+    full_methods.each do |char|
+      context "when method name is '#{char}'" do
+        self.class.send :define_method, :full_methods_names do
+          full_methods
+        end
+
+        let(:klass) do
+          ClassBuilder.build do
+            aspector class_methods: true do
+              before char, :exec
+            end
+
+            class << self
+              def values
+                @values ||= []
+              end
+
+              def exec(*_args)
+                values << 'exec-result'
+              end
+
+              full_methods_names.each do |char|
+                define_method char do |*args|
+                  values << args.first
+                end
+              end
+            end
+          end
+        end
+
+        it 'should handle it without any issues' do
+          klass.send(char, char)
+          expect(klass.values).to eq ['exec-result', char]
+        end
+      end
+    end
+  end
+end

metadata

@@ -1,212 +1,47 @@
---- !ruby/object:Gem::Specification 
+--- !ruby/object:Gem::Specification
 name: aspector
-version: !ruby/object:Gem::Version 
-  hash: 41
-  prerelease: 
-  segments: 
-  - 0
-  - 13
-  - 1
-  version: 0.13.1
+version: !ruby/object:Gem::Version
+  version: 0.14.0
 platform: ruby
-authors: 
+authors:
 - Guoliang Cao
+- Maciej Mensfeld
 autorequire: 
 bindir: bin
 cert_chain: []
-
-date: 2013-01-23 00:00:00 -05:00
-default_executable: 
-dependencies: 
-- !ruby/object:Gem::Dependency 
-  requirement: &id001 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
-  version_requirements: *id001
-  type: :development
-  name: rspec
-  prerelease: false
-- !ruby/object:Gem::Dependency 
-  requirement: &id002 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ~>
-      - !ruby/object:Gem::Version 
-        hash: 7
-        segments: 
-        - 1
-        - 6
-        - 4
-        version: 1.6.4
-  version_requirements: *id002
-  type: :development
-  name: jeweler
-  prerelease: false
-- !ruby/object:Gem::Dependency 
-  requirement: &id003 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
-  version_requirements: *id003
-  type: :development
-  name: ruby-prof
-  prerelease: false
-- !ruby/object:Gem::Dependency 
-  requirement: &id004 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ~>
-      - !ruby/object:Gem::Version 
-        hash: 55
-        segments: 
-        - 0
-        - 8
-        - 4
-        version: 0.8.4
-  version_requirements: *id004
-  type: :development
-  name: guard
-  prerelease: false
-- !ruby/object:Gem::Dependency 
-  requirement: &id005 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ~>
-      - !ruby/object:Gem::Version 
-        hash: 29
-        segments: 
-        - 0
-        - 1
-        - 3
-        version: 0.1.3
-  version_requirements: *id005
-  type: :development
-  name: guard-bundler
-  prerelease: false
-- !ruby/object:Gem::Dependency 
-  requirement: &id006 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ~>
-      - !ruby/object:Gem::Version 
-        hash: 15
-        segments: 
-        - 0
-        - 5
-        - 2
-        version: 0.5.2
-  version_requirements: *id006
-  type: :development
-  name: guard-rspec
-  prerelease: false
-- !ruby/object:Gem::Dependency 
-  requirement: &id007 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ~>
-      - !ruby/object:Gem::Version 
-        hash: 25
-        segments: 
-        - 0
-        - 1
-        - 1
-        version: 0.1.1
-  version_requirements: *id007
-  type: :development
-  name: guard-shell
-  prerelease: false
-- !ruby/object:Gem::Dependency 
-  requirement: &id008 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
-  version_requirements: *id008
-  type: :development
-  name: rb-fsevent
-  prerelease: false
-- !ruby/object:Gem::Dependency 
-  requirement: &id009 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ~>
-      - !ruby/object:Gem::Version 
-        hash: 17
-        segments: 
-        - 1
-        - 0
-        - 3
-        version: 1.0.3
-  version_requirements: *id009
-  type: :development
-  name: growl
-  prerelease: false
-- !ruby/object:Gem::Dependency 
-  requirement: &id010 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
-  version_requirements: *id010
-  type: :development
-  name: awesome_print
-  prerelease: false
-- !ruby/object:Gem::Dependency 
-  requirement: &id011 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
-  version_requirements: *id011
-  type: :development
-  name: pry
-  prerelease: false
-description: ""
-email: gcao99@gmail.com
+date: 2015-07-10 00:00:00.000000000 Z
+dependencies: []
+description: "[]"
+email:
+- gcao99@gmail.com
+- maciej@mensfeld.pl
 executables: []
-
 extensions: []
-
-extra_rdoc_files: 
-- LICENSE.txt
-- README.rdoc
-files: 
-- .document
-- .irbrc
-- .rspec
-- .rvmrc
-- .travis.yml
+extra_rdoc_files: []
+files:
+- ".gitignore"
+- ".irbrc"
+- ".rspec"
+- ".rubocop.yml"
+- ".ruby-gemset"
+- ".ruby-version"
+- ".travis.yml"
+- Changelog.md
 - Gemfile
 - Gemfile.lock
 - Guardfile
 - LICENSE.txt
-- README.rdoc
+- README.md
 - Rakefile
-- VERSION
 - aspector.gemspec
+- benchmarks/after_benchmark.rb
+- benchmarks/around_advice_benchmark.rb
+- benchmarks/around_benchmark.rb
+- benchmarks/before_benchmark.rb
+- benchmarks/benchmark_helper.rb
+- benchmarks/combined_benchmark.rb
+- benchmarks/method_invocation_benchmark.rb
+- benchmarks/raw_benchmark.rb
 - examples/activerecord_hooks.rb
 - examples/around_example.rb
 - examples/aspector_apply_example.rb
@@ -216,7 +51,9 @@ files:
 - examples/exception_handler.rb
 - examples/exception_handler2.rb
 - examples/implicit_method_option_test.rb
+- examples/interception_options_example.rb
 - examples/logging_aspect.rb
+- examples/process_aspector.rb
 - examples/retry_aspect.rb
 - lib/aspector.rb
 - lib/aspector/advice.rb
@@ -226,70 +63,85 @@ files:
 - lib/aspector/base_class_methods.rb
 - lib/aspector/deferred_logic.rb
 - lib/aspector/deferred_option.rb
+- lib/aspector/interception.rb
 - lib/aspector/logger.rb
 - lib/aspector/logging.rb
 - lib/aspector/method_matcher.rb
 - lib/aspector/module_extension.rb
 - lib/aspector/object_extension.rb
-- performance-tests/after_test.rb
-- performance-tests/around_advice_benchmark.rb
-- performance-tests/around_test.rb
-- performance-tests/before_test.rb
-- performance-tests/combined_test.rb
-- performance-tests/method_invocation_test.rb
-- performance-tests/raw_test.rb
-- performance-tests/test_helper.rb
-- run_all_examples.sh
-- spec/functional/advices_on_private_methods_spec.rb
-- spec/functional/aspect_on_eigen_class_spec.rb
-- spec/functional/aspect_on_object_spec.rb
-- spec/functional/aspector_spec.rb
-- spec/functional/aspects_combined_spec.rb
-- spec/functional/execution_order_spec.rb
+- lib/aspector/version.rb
+- spec/examples_spec.rb
+- spec/functionals/aspect_for_multiple_targets_spec.rb
+- spec/functionals/aspect_interception_options_accessing_spec.rb
+- spec/functionals/aspect_on_a_class_spec.rb
+- spec/functionals/aspect_on_an_instance_spec.rb
+- spec/functionals/aspector_spec.rb
+- spec/functionals/aspects_combined_spec.rb
+- spec/functionals/aspects_execution_order_spec.rb
+- spec/functionals/aspects_on_private_methods_spec.rb
 - spec/spec_helper.rb
-- spec/unit/advice_spec.rb
-- spec/unit/after_spec.rb
-- spec/unit/around_spec.rb
-- spec/unit/base_class_methods_spec.rb
-- spec/unit/base_spec.rb
-- spec/unit/before_spec.rb
-- spec/unit/deferred_logic_spec.rb
-- spec/unit/method_matcher_spec.rb
-- spec/unit/raw_spec.rb
-- spec/unit/special_chars_spec.rb
-has_rdoc: true
+- spec/support/class_builder.rb
+- spec/units/advice_spec.rb
+- spec/units/advices/after_spec.rb
+- spec/units/advices/around_spec.rb
+- spec/units/advices/before_filter_spec.rb
+- spec/units/advices/before_spec.rb
+- spec/units/advices/raw_spec.rb
+- spec/units/base_class_methods_spec.rb
+- spec/units/base_spec.rb
+- spec/units/deferred_logic_spec.rb
+- spec/units/logger_spec.rb
+- spec/units/logging_spec.rb
+- spec/units/method_matcher_spec.rb
+- spec/units/object_extension_spec.rb
+- spec/units/special_chars_spec.rb
 homepage: http://github.com/gcao/aspector
-licenses: 
+licenses:
 - MIT
+metadata: {}
 post_install_message: 
 rdoc_options: []
-
-require_paths: 
+require_paths:
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement 
-  none: false
-  requirements: 
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
   - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
-required_rubygems_version: !ruby/object:Gem::Requirement 
-  none: false
-  requirements: 
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
   - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
+    - !ruby/object:Gem::Version
+      version: '0'
 requirements: []
-
 rubyforge_project: 
-rubygems_version: 1.6.2
+rubygems_version: 2.4.5
 signing_key: 
-specification_version: 3
-summary: Aspect Oriented Ruby Programming
-test_files: []
-
+specification_version: 4
+summary: '["Aspect", "Oriented", "Ruby", "Programming", "library"]'
+test_files:
+- spec/examples_spec.rb
+- spec/functionals/aspect_for_multiple_targets_spec.rb
+- spec/functionals/aspect_interception_options_accessing_spec.rb
+- spec/functionals/aspect_on_a_class_spec.rb
+- spec/functionals/aspect_on_an_instance_spec.rb
+- spec/functionals/aspector_spec.rb
+- spec/functionals/aspects_combined_spec.rb
+- spec/functionals/aspects_execution_order_spec.rb
+- spec/functionals/aspects_on_private_methods_spec.rb
+- spec/spec_helper.rb
+- spec/support/class_builder.rb
+- spec/units/advice_spec.rb
+- spec/units/advices/after_spec.rb
+- spec/units/advices/around_spec.rb
+- spec/units/advices/before_filter_spec.rb
+- spec/units/advices/before_spec.rb
+- spec/units/advices/raw_spec.rb
+- spec/units/base_class_methods_spec.rb
+- spec/units/base_spec.rb
+- spec/units/deferred_logic_spec.rb
+- spec/units/logger_spec.rb
+- spec/units/logging_spec.rb
+- spec/units/method_matcher_spec.rb
+- spec/units/object_extension_spec.rb
+- spec/units/special_chars_spec.rb