sinclair 1.6.6 → 1.8.0

data/lib/sinclair/matchers/change_class_method.rb

@@ -0,0 +1,42 @@
+# frozen_string_literal: true
+
+class Sinclair
+  module Matchers
+    # @api private
+    # @author darthjee
+    #
+    # AddInstanceMethod is able to build an instance of
+    # {Sinclair::Matchers::ChangeClassMethodOn}
+    class ChangeClassMethod < Base
+      include AddMethod
+
+      # @api public
+      #
+      # Builds final matcher
+      #
+      # @return [Sinclair::Matchers::ChangeClassMethodOn]
+      alias on to
+
+      private
+
+      # @private
+      #
+      # Error description on wrong usage
+      #
+      # @return String
+      def matcher_error
+        'You should specify which class the method is being changed on' \
+          "change_class_method(:#{method_name}).on(klass)"
+      end
+
+      # @private
+      #
+      # Class of the real matcher
+      #
+      # @return [Class<ChangeClassMethodOn>]
+      def add_method_to_class
+        ChangeClassMethodOn
+      end
+    end
+  end
+end

data/lib/sinclair/matchers/change_class_method_on.rb

@@ -0,0 +1,64 @@
+# frozen_string_literal: true
+
+class Sinclair
+  module Matchers
+    # @api private
+    # @author darthjee
+    #
+    # Checks if a class method was changed
+    # by the call of a block
+    #
+    #  This is used with a RSpec DSL method
+    #  change_class_method(method_name).on(class_object)
+    class ChangeClassMethodOn < ChangeMethodOn
+      # @param target [Class]
+      #   Class where the class method should be added to
+      #
+      # @param method_name [Symbol,String] method name
+      def initialize(target, method_name)
+        @klass = target
+        super(method_name)
+      end
+
+      # Return expectaton description
+      #
+      # @return [String]
+      def description
+        "change class method '#{method_name}' on #{klass}"
+      end
+
+      # Returns message on expectation failure
+      #
+      # @return [String]
+      def failure_message_for_should
+        "expected class method '#{method_name}' to be changed on #{klass} but " \
+          "#{initial_state ? "it didn't" : "it didn't exist"}"
+      end
+
+      # Returns message on expectation failure for negative expectation
+      #
+      # @return [String]
+      def failure_message_for_should_not
+        "expected class method '#{method_name}' not to be changed on #{klass} but it was"
+      end
+
+      private
+
+      # Checks if class has instance method defined
+      #
+      # @return [Boolean]
+      def state
+        klass.methods(false).include?(method_name.to_sym) \
+          && klass.method(method_name)
+      end
+
+      # Raises when block was not given
+      #
+      # @raise SyntaxError
+      def raise_block_syntax_error
+        raise SyntaxError, 'Block not received by the `change_class_method_on` matcher. ' \
+          'Perhaps you want to use `{ ... }` instead of do/end?'
+      end
+    end
+  end
+end

data/lib/sinclair/matchers/change_instance_method.rb

@@ -0,0 +1,42 @@
+# frozen_string_literal: true
+
+class Sinclair
+  module Matchers
+    # @api private
+    # @author darthjee
+    #
+    # AddInstanceMethod is able to build an instance of
+    # {Sinclair::Matchers::ChangeInstanceMethodOn}
+    class ChangeInstanceMethod < Base
+      include AddMethod
+
+      # @api public
+      #
+      # Builds final matcher
+      #
+      # @return [Sinclair::Matchers::ChangeInstanceMethodOn]
+      alias on to
+
+      private
+
+      # @private
+      #
+      # Error description on wrong usage
+      #
+      # @return String
+      def matcher_error
+        'You should specify which instance the method is being changed on' \
+          "change_method(:#{method_name}).on(instance)"
+      end
+
+      # @private
+      #
+      # Class of the real matcher
+      #
+      # @return [Class<Sinclair::Matchers::Base>]
+      def add_method_to_class
+        ChangeInstanceMethodOn
+      end
+    end
+  end
+end

data/lib/sinclair/matchers/change_instance_method_on.rb

@@ -0,0 +1,98 @@
+# frozen_string_literal: true
+
+class Sinclair
+  module Matchers
+    # @api private
+    # @author darthjee
+    #
+    # Checks if a method was changed
+    # by the call of a block
+    #
+    #  This is used with a RSpec DSL method
+    #  change_method(method_name).on(class_object)
+    class ChangeInstanceMethodOn < ChangeMethodOn
+      # @overload initialize(klass, method_name)
+      #   @param [Class] klass
+      #     class where the method should be added to
+      #
+      # @overload initialize(instance, method_name)
+      #   @param [Object] instance
+      #     instance of the class where the method should be added to
+      #
+      # @param method_name [Symbol,String] method name
+      def initialize(target, method_name)
+        if target.is_a?(Class)
+          @klass = target
+        else
+          @instance = target
+        end
+
+        super(method_name)
+      end
+
+      # Returnst expectaton description
+      #
+      # @return [String]
+      def description
+        "change method '#{method_name}' on #{klass} instances"
+      end
+
+      # Returns message on expectation failure
+      #
+      # @return [String]
+      def failure_message_for_should
+        "expected '#{method_name}' to be changed on #{klass} but " \
+          "#{initial_state ? "it didn't" : "it didn't exist"}"
+      end
+
+      # Returns message on expectation failure for negative expectation
+      #
+      # @return [String]
+      def failure_message_for_should_not
+        "expected '#{method_name}' not to be changed on #{klass} but it was"
+      end
+
+      protected
+
+      # @method instance
+      # @api private
+      # @private
+      #
+      # Instance of the class where the method should be added
+      #
+      # @return [Object]
+      attr_reader :instance
+
+      # @private
+      #
+      # Class to be analised
+      #
+      # @return [Class]
+      def klass
+        @klass ||= instance.class
+      end
+
+      private
+
+      # @private
+      #
+      # Checks if class has instance method defined
+      #
+      # @return [Boolean]
+      def state
+        klass.method_defined?(method_name) &&
+          klass.instance_method(method_name)
+      end
+
+      # @private
+      #
+      # Raises when block was not given
+      #
+      # @raise SyntaxError
+      def raise_block_syntax_error
+        raise SyntaxError, 'Block not received by the `change_instance_method_on` matcher. ' \
+          'Perhaps you want to use `{ ... }` instead of do/end?'
+      end
+    end
+  end
+end

data/lib/sinclair/matchers/change_method_on.rb

@@ -0,0 +1,25 @@
+# frozen_string_literal: true
+
+class Sinclair
+  module Matchers
+    # @api private
+    # @author darthjee
+    # @abstract
+    #
+    # Base class for change_method_on matcher
+    class ChangeMethodOn < Base
+      include MethodTo
+
+      private
+
+      # @private
+      #
+      # Checks if a method was changed
+      #
+      # @return Boolean
+      def check
+        initial_state && initial_state != final_state
+      end
+    end
+  end
+end

data/lib/sinclair/matchers/method_to.rb

@@ -0,0 +1,82 @@
+# frozen_string_literal: true
+
+class Sinclair
+  module Matchers
+    # @api private
+    #
+    # Common methods on final matchers
+    module MethodTo
+      # Used for other versions of rspec
+      #
+      # Some versions call failure_message, others
+      # call failure_message_for_should
+      #
+      # @return [String]
+      def failure_message
+        failure_message_for_should
+      end
+
+      # Used for other versions of rspec
+      #
+      # Some versions call failure_message_when_negated, others
+      # call failure_message_for_should_not
+      #
+      # @return [String]
+      def failure_message_when_negated
+        failure_message_for_should_not
+      end
+
+      # Checks if expectation is true or not
+      #
+      # @return [Boolean] expectation check
+      def matches?(event_proc)
+        return false unless event_proc.is_a?(Proc)
+
+        raise_block_syntax_error if block_given?
+        perform_change(event_proc)
+        check
+      end
+
+      protected
+
+      # @method klass
+      # @api private
+      # @private
+      #
+      # Class where class method should be added to
+      #
+      # @return [Class]
+      attr_reader :klass
+
+      private
+
+      # @method initial_state
+      # @api private
+      # @private
+      #
+      # State before running the block
+      #
+      # @return [Object]
+
+      # @method final_state
+      # @api private
+      # @private
+      #
+      # State after running the block
+      #
+      # @return [Object]
+      attr_reader :initial_state, :final_state
+
+      # @private
+      #
+      # Call block to check if it aded a method or not
+      #
+      # @return [Boolan]
+      def perform_change(event_proc)
+        @initial_state = state
+        event_proc.call
+        @final_state = state
+      end
+    end
+  end
+end

data/lib/sinclair/matchers.rb

@@ -7,33 +7,21 @@ class Sinclair
   # Matchers module will have the DSL to be included in RSpec in order to have
   # access to the matchers
   #
-  # @example
-  #  RSpec.configure do |config|
-  #    config.include Sinclair::Matchers
-  #  end
-  #
-  #  class MyModel
-  #  end
-  #
-  #  RSpec.describe 'my test' do
-  #    let(:klass)   { Class.new(MyModel) }
-  #    let(:builder) { Sinclair.new(klass) }
-  #
-  #    before do
-  #      builder.add_method(:class_name, 'self.class.name')
-  #    end
-  #
-  #    it do
-  #      expect { builder.build }.to add_method(:class_name).to(klass)
-  #    end
-  #  end
+  # @example (see Sinclair::Matchers::AddMethod#to)
   module Matchers
-    autoload :AddMethod,           'sinclair/matchers/add_method'
-    autoload :AddInstanceMethod,   'sinclair/matchers/add_instance_method'
-    autoload :AddClassMethod,      'sinclair/matchers/add_class_method'
-    autoload :AddMethodTo,         'sinclair/matchers/add_method_to'
-    autoload :AddInstanceMethodTo, 'sinclair/matchers/add_instance_method_to'
-    autoload :AddClassMethodTo,    'sinclair/matchers/add_class_method_to'
+    autoload :Base,                   'sinclair/matchers/base'
+    autoload :AddInstanceMethod,      'sinclair/matchers/add_instance_method'
+    autoload :AddClassMethod,         'sinclair/matchers/add_class_method'
+    autoload :AddMethod,              'sinclair/matchers/add_method'
+    autoload :AddMethodTo,            'sinclair/matchers/add_method_to'
+    autoload :AddInstanceMethodTo,    'sinclair/matchers/add_instance_method_to'
+    autoload :AddClassMethodTo,       'sinclair/matchers/add_class_method_to'
+    autoload :ChangeClassMethod,      'sinclair/matchers/change_class_method'
+    autoload :ChangeInstanceMethod,   'sinclair/matchers/change_instance_method'
+    autoload :ChangeMethodOn,         'sinclair/matchers/change_method_on'
+    autoload :ChangeClassMethodOn,    'sinclair/matchers/change_class_method_on'
+    autoload :ChangeInstanceMethodOn, 'sinclair/matchers/change_instance_method_on'
+    autoload :MethodTo,               'sinclair/matchers/method_to'
 
     # DSL to AddInstanceMethod
     #
@@ -41,8 +29,8 @@ class Sinclair
     # @example (see Sinclair::Matchers::AddInstanceMethod#to)
     #
     # @return [AddInstanceMethod] RSpec Matcher
-    def add_method(method)
-      Sinclair::Matchers::AddInstanceMethod.new(method)
+    def add_method(method_name)
+      Sinclair::Matchers::AddInstanceMethod.new(method_name)
     end
 
     # DSL to AddClassMethod
@@ -51,8 +39,28 @@ class Sinclair
     # @example (see Sinclair::Matchers::AddClassMethod#to)
     #
     # @return [AddClassMethod] RSpec Matcher
-    def add_class_method(method)
-      Sinclair::Matchers::AddClassMethod.new(method)
+    def add_class_method(method_name)
+      Sinclair::Matchers::AddClassMethod.new(method_name)
+    end
+
+    # DSL to ChangeInstanceMethod
+    #
+    # @example (see Sinclair::Matchers)
+    # @example (see Sinclair::Matchers::ChangeInstanceMethod#to)
+    #
+    # @return [ChangeInstanceMethod] RSpec Matcher
+    def change_method(method_name)
+      Sinclair::Matchers::ChangeInstanceMethod.new(method_name)
+    end
+
+    # DSL to ChangeClassMethod
+    #
+    # @example (see Sinclair::Matchers)
+    # @example (see Sinclair::Matchers::ChangeClassMethod#to)
+    #
+    # @return [ChangeClassMethod] RSpec Matcher
+    def change_class_method(method_name)
+      Sinclair::Matchers::ChangeClassMethod.new(method_name)
     end
   end
 end

data/lib/sinclair/options/class_methods.rb

@@ -4,7 +4,7 @@ require 'set'
 
 class Sinclair
   class Options
-    # Class Methods for {Sinclai::Options}
+    # Class Methods for {Sinclair::Options}
     module ClassMethods
       # @api private
       #
@@ -47,10 +47,7 @@ class Sinclair
         @skip_validation ||= superclass.try(:skip_validation?) || false
       end
 
-      private
-
       # @api public
-      # @!visibility public
       #
       # Add available options
       #
@@ -71,7 +68,6 @@ class Sinclair
       end
 
       # @api public
-      # @!visibility public
       #
       # Changes class to skip attributes validation
       #

data/lib/sinclair/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 class Sinclair
-  VERSION = '1.6.6'
+  VERSION = '1.8.0'
 end

data/sinclair.gemspec

@@ -21,20 +21,20 @@ Gem::Specification.new do |gem|
 
   gem.add_runtime_dependency 'activesupport', '~> 5.2.0'
 
-  gem.add_development_dependency 'bundler',            '1.16.1'
-  gem.add_development_dependency 'pry',                '0.12.2'
-  gem.add_development_dependency 'pry-nav',            '0.3.0'
+  gem.add_development_dependency 'bundler',            '2.3.20'
+  gem.add_development_dependency 'pry',                '0.14.1'
+  gem.add_development_dependency 'pry-nav',            '1.0.0'
   gem.add_development_dependency 'rake',               '13.0.1'
-  gem.add_development_dependency 'reek',               '5.6.0'
-  gem.add_development_dependency 'rspec',              '3.9.0'
-  gem.add_development_dependency 'rspec-core',         '3.9.1'
-  gem.add_development_dependency 'rspec-expectations', '3.9.1'
-  gem.add_development_dependency 'rspec-mocks',        '3.9.1'
-  gem.add_development_dependency 'rspec-support',      '3.9.2'
+  gem.add_development_dependency 'reek',               '6.0.3'
+  gem.add_development_dependency 'rspec',              '3.11.0'
+  gem.add_development_dependency 'rspec-core',         '3.11.0'
+  gem.add_development_dependency 'rspec-expectations', '3.11.0'
+  gem.add_development_dependency 'rspec-mocks',        '3.11.1'
+  gem.add_development_dependency 'rspec-support',      '3.11.0'
   gem.add_development_dependency 'rubocop',            '0.80.1'
   gem.add_development_dependency 'rubocop-rspec',      '1.38.1'
-  gem.add_development_dependency 'rubycritic',         '4.4.1'
-  gem.add_development_dependency 'simplecov',          '0.17.1'
-  gem.add_development_dependency 'yard',               '0.9.24'
+  gem.add_development_dependency 'rubycritic',         '4.7.0'
+  gem.add_development_dependency 'simplecov',          '0.21.2'
+  gem.add_development_dependency 'yard',               '0.9.27'
   gem.add_development_dependency 'yardstick',          '0.9.9'
 end

data/spec/integration/readme/my_class_spec.rb

@@ -3,7 +3,7 @@
 require 'spec_helper'
 
 describe MyClass do
-  subject(:model) { klass.new(attributes) }
+  subject(:model) { klass.new(**attributes) }
 
   let(:klass) { described_class }
   let(:name)  { 'name' }

data/spec/integration/readme/sinclair/configurable_spec.rb

@@ -25,7 +25,17 @@ describe Sinclair::Configurable do
           .to eq(5555)
       end
 
-      context 'when #rest_config is called' do
+      it 'enables options to be returned' do
+        expect(MyConfigurable.as_options.host)
+          .to eq('interstella.art')
+      end
+
+      it 'enables custom options to be returned' do
+        expect(MyConfigurable.as_options(host: 'other').host)
+          .to eq('other')
+      end
+
+      context 'when #reset_config is called' do
         before do
           MyConfigurable.reset_config
         end

data/spec/integration/yard/sinclair/config_spec.rb

@@ -24,5 +24,32 @@ describe Sinclair::Config do
           .to eq('{"password":null,"username":"bob"}')
       end
     end
+
+    describe '#options' do
+      subject(:config) { configurable.config }
+
+      let(:configurable) { LoginConfigurable }
+
+      before do
+        LoginConfigurable.configure do |conf|
+          conf.username :some_username
+          conf.password :some_password
+        end
+      end
+
+      it 'returns options with correct values' do
+        expect(config.as_options.username).to eq(:some_username)
+        expect(config.as_options.password).to eq(:some_password)
+      end
+
+      context 'when merging with given attributes' do
+        subject(:options) { config.as_options(password: :correct_password) }
+
+        it 'returns options with custom values' do
+          expect(options.username).to eq(:some_username)
+          expect(options.password).to eq(:correct_password)
+        end
+      end
+    end
   end
 end

data/spec/lib/sinclair/config_class_spec.rb

@@ -16,38 +16,6 @@ describe Sinclair::ConfigClass do
   end
 
   describe '.add_configs' do
-    let(:setter_block) do
-      proc { |value| config.instance_variable_set(:@name, value) }
-    end
-
-    it_behaves_like 'a config methods builder adding config' do
-      let(:code_block) { proc { klass.add_configs(:name) } }
-
-      it 'sets nil value by default' do
-        code_block.call
-        expect(config.name).to be_nil
-      end
-
-      it 'adds attributes to class' do
-        expect(&code_block).to change(klass, :config_attributes)
-          .from([]).to(%i[name])
-      end
-    end
-
-    context 'when giving defaults' do
-      it_behaves_like 'a config methods builder adding config' do
-        let(:code_block) { proc { klass.add_configs(name: 'Bob') } }
-
-        it 'sets default value' do
-          code_block.call
-          expect(config.name).to eq('Bob')
-        end
-
-        it 'adds attributes to class' do
-          expect(&code_block).to change(klass, :config_attributes)
-            .from([]).to(%i[name])
-        end
-      end
-    end
+    it_behaves_like 'a config class with .add_configs method'
   end
 end