rspectre 0.0.4 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b0c337fb9cef9cb104d3ca686aa194cbb81062c0407e52ffdecec1e0c393d0fa
-  data.tar.gz: 144eee36cdd52ba9f99d38b3bc7bb57bcbd5d283accc4ba0eef11e474fe99ed6
+  metadata.gz: 77d1fd05a8a9608c30ed29f93c408bf519507fe5e8560cb9a80542f13f2783fa
+  data.tar.gz: a2b683a3a2289c8f8b1e39fa09a0d3b55ca6e5d9337d922c8a0124b821d59242
 SHA512:
-  metadata.gz: 1a5cd59d5dff5c9becdfc36c28e1605f7a3e222fe7ffa7543974ad6e80bb80238adb654140d3b976c910d5fda36e946456539720801dfdc72844c3c59040c8cf
-  data.tar.gz: ea374c050f039f1c027d12586f091c2fd15731b3a62648a95c15153bca1c03f145357ad3162eed228e3cedec293af7ba2009fc4178bf23d506bae33aa485f3c4
+  metadata.gz: d8042b9d677bc5fbbd8ef98bac37f271a244622782391dda4926b0c63b5c5b21e36cdfa08e24e94b98a29079d5525180f5129e0534114fbe946538d1160ab679
+  data.tar.gz: 9e5bd54c5c3faedddaebb97a3265163ece4fece63d884ac22ba412dfe8c78de1e31cb097e83d3d0047ca10043908f74865e6c0c20b780c9e3fd0aa8f76cf546c

data/bin/rspectre

@@ -20,8 +20,8 @@ OptionParser.new do |opts|
   opts.on(
     '--auto-correct',
     'Enables auto-correct.',
-    'When auto-correct is enabled, rspectre will modify your source files in place and delete any'\
-    ' unused test setup.'
+    'When auto-correct is enabled, rspectre will modify your source files in place and delete ' \
+    'any unused test setup.'
   ) do |value|
     options[:'auto-correct'] = value
   end
@@ -29,4 +29,4 @@ end.parse(ARGV)
 
 rspec, auto_correct = options.fetch_values(:rspec, :'auto-correct')
 
-RSpectre::Runner.new(rspec, auto_correct).lint
+RSpectre::Runner.new(rspec_arguments: rspec, auto_correct: auto_correct).lint

data/lib/rspectre/auto_corrector.rb

@@ -2,19 +2,19 @@
 
 module RSpectre
   class AutoCorrector < Parser::TreeRewriter
-    include Concord.new(:filename, :nodes, :buffer)
+    include KeywordStruct.new(:filename, :nodes, :buffer)
 
     def initialize(filename, nodes)
       buffer = Parser::Source::Buffer.new("(#{filename})")
       buffer.source = File.read(filename)
 
-      super(filename, nodes, buffer)
+      super(filename: filename, nodes: nodes, buffer: buffer)
     end
 
     def correct
       File.write(
         filename,
-        rewrite(buffer, Parser::CurrentRuby.new.parse(buffer))
+        rewrite(buffer, SourceMap::Parser.parser_class.new.parse(buffer))
       )
     end
 

data/lib/rspectre/keyword_struct.rb

@@ -0,0 +1,48 @@
+# frozen_string_literal: true
+
+module RSpectre
+  # Heavily influenced by dkubb/equalizer, mbj/concord, and mbj/anima
+  class KeywordStruct < Module
+    def initialize(*names) # rubocop:disable Lint/MissingSuper
+      raise 'Specify at least one keyword name!' if names.empty?
+
+      names.each do |name|
+        next if /\A\w+\z/.match?(name)
+
+        raise "Invalid keyword name #{name.inspect}!"
+      end
+
+      @module =
+        Module.new do
+          attr_reader(*names)
+
+          define_method(:eql?) do |other|
+            other.instance_of?(self.class) && names.all? do |name|
+              __send__(name).eql?(other.__send__(name))
+            end
+          end
+          alias_method :==, :eql?
+
+          define_method(:hash) do
+            [self.class, *names.map { |name| __send__(name) }].hash
+          end
+
+          define_method(:inspect) do
+            class_name = self.class.name || self.class.inspect
+            attributes = names.map { |name| "#{name}=#{__send__(name).inspect}" }.join(' ')
+            "#<#{class_name} #{attributes}>"
+          end
+        end.tap do |generated_module|
+          generated_module.class_eval(<<~RUBY, __FILE__, __LINE__ + 1) # rubocop:disable Style/DocumentDynamicEvalDefinition
+            def initialize(#{names.map { |name| "#{name}:" }.join(', ')})
+              #{names.map { |name| "@#{name} = #{name}" }.join("\n  ")}
+            end
+          RUBY
+        end
+    end
+
+    def included(descendant)
+      descendant.include(@module)
+    end
+  end
+end

data/lib/rspectre/linter/unused_let.rb

@@ -5,8 +5,8 @@ module RSpectre
     class UnusedLet < self
       TAG = 'UnusedLet'
 
-      def example_group.let(name, &block)
-        super(name, &block)
+      def example_group.let(name, &)
+        super
 
         UnusedLet.register(:let, caller_locations) do |node|
           UnusedLet.prepend_behavior(self, name) { UnusedLet.record(node) }

data/lib/rspectre/linter/unused_shared_setup.rb

@@ -13,14 +13,14 @@ module RSpectre
         receiver.__send__(:define_singleton_method, method) do |name, *args, &block|
           # When we can locate the source of the node, tag it
           if (node = UnusedSharedSetup.register(method, caller_locations))
-            # And call the orignal
+            # And call the original
             original_method.(name, *args) do |*shared_args|
               # But record that it was used in a `before`
               before { UnusedSharedSetup.record(node) }
 
               # And then perform the original block in a `class_exec` like the original block was
               # supposed to be
-              class_exec(*shared_args, &block)
+              RSpec::Support::WithKeywordsWhenNeeded.class_exec(self, *shared_args, &block)
             end
           else
             # If we couldn't locate the source, just delegate to the original method.
@@ -48,37 +48,37 @@ module RSpectre
       # for now.
       def example_group.shared_examples(name, *args, &block)
         if (node = UnusedSharedSetup.register(:shared_examples, caller_locations))
-          super(name, *args) do |*shared_args|
+          super do |*shared_args|
             before { UnusedSharedSetup.record(node) }
 
-            class_exec(*shared_args, &block)
+            RSpec::Support::WithKeywordsWhenNeeded.class_exec(self, *shared_args, &block)
           end
         else
-          super(name, *args, &block)
+          super
         end
       end
 
       def example_group.shared_examples_for(name, *args, &block)
         if (node = UnusedSharedSetup.register(:shared_examples_for, caller_locations))
-          super(name, *args) do |*shared_args|
+          super do |*shared_args|
             before { UnusedSharedSetup.record(node) }
 
-            class_exec(*shared_args, &block)
+            RSpec::Support::WithKeywordsWhenNeeded.class_exec(self, *shared_args, &block)
           end
         else
-          super(name, *args, &block)
+          super
         end
       end
 
       def example_group.shared_context(name, *args, &block)
         if (node = UnusedSharedSetup.register(:shared_context, caller_locations))
-          super(name, *args) do |*shared_args|
+          super do |*shared_args|
             before { UnusedSharedSetup.record(node) }
 
-            class_exec(*shared_args, &block)
+            RSpec::Support::WithKeywordsWhenNeeded.class_exec(self, *shared_args, &block)
           end
         else
-          super(name, *args, &block)
+          super
         end
       end
     end

data/lib/rspectre/linter/unused_subject.rb

@@ -5,8 +5,8 @@ module RSpectre
     class UnusedSubject < self
       TAG = 'UnusedSubject'
 
-      def example_group.subject(name = nil, &block)
-        super(*name, &block)
+      def example_group.subject(name = nil, &)
+        super(*name, &)
 
         UnusedSubject.register(:subject, caller_locations) do |node|
           UnusedSubject.prepend_behavior(self, :subject) { UnusedSubject.record(node) }

data/lib/rspectre/linter.rb

@@ -19,7 +19,7 @@ module RSpectre
       raw_node = node_map(file).find_method(selector, line)
 
       if raw_node
-        node = RSpectre::Node.new(file, line, raw_node)
+        node = RSpectre::Node.new(file: file, line: line, node: raw_node)
         TRACKER.register(self::TAG, node)
         if block_given?
           yield node
@@ -40,10 +40,12 @@ module RSpectre
     def self.prepend_behavior(scope, method_name)
       original_method = scope.instance_method(method_name)
 
-      scope.__send__(:define_method, method_name) do |*args, &block|
+      # Removing the method first prevents method redefined warnings when $VERBOSE is true
+      scope.remove_method(method_name)
+      scope.__send__(:define_method, method_name) do |*args, **kwargs, &block|
         yield
 
-        original_method.bind(self).(*args, &block)
+        original_method.bind_call(self, *args, **kwargs, &block)
       end
     end
 

data/lib/rspectre/node.rb

@@ -2,7 +2,7 @@
 
 module RSpectre
   class Node
-    include Concord::Public.new(:file, :line, :node)
+    include KeywordStruct.new(:file, :line, :node)
 
     def start_column
       location.column + 1
@@ -27,7 +27,11 @@ module RSpectre
     end
 
     def location
-      node.children.first.location
+      if node.type.equal?(:block)
+        node.children.first.location
+      else
+        node.location
+      end
     end
   end
 end

data/lib/rspectre/offense.rb

@@ -2,13 +2,13 @@
 
 module RSpectre
   class Offense
-    include Anima.new(:file, :line, :source_line, :start_column, :end_column, :type)
+    include KeywordStruct.new(:file, :line, :source_line, :start_column, :end_column, :type)
 
     DESCRIPTIONS = {
       'UnusedLet'         => 'Unused `let` definition.',
       'UnusedSubject'     => 'Unused `subject` definition.',
-      'UnusedSharedSetup' => 'Unused `shared_examples`, `shared_examples_for`, or'\
-                             ' `shared_context` definition.'
+      'UnusedSharedSetup' => 'Unused `shared_examples`, `shared_examples_for`, or ' \
+                             '`shared_context` definition.'
     }.freeze
 
     def self.parse(type, node)
@@ -23,7 +23,7 @@ module RSpectre
     end
 
     def warn
-      puts to_s
+      puts self
     end
 
     def to_s

data/lib/rspectre/runner.rb

@@ -2,11 +2,11 @@
 
 module RSpectre
   class Runner
-    include Concord.new(:rspec_arguments, :auto_correct)
+    include KeywordStruct.new(:rspec_arguments, :auto_correct)
 
     EXIT_SUCCESS = 0
 
-    def initialize(*)
+    def initialize(**)
       super
       @rspec_output = StringIO.new
     end
@@ -39,7 +39,7 @@ module RSpectre
 
       abort(
         Color.red(
-          'Running the specs failed. Either your tests do not pass '\
+          'Running the specs failed. Either your tests do not pass ' \
           'normally or this is a bug in RSpectre.'
         ) + <<~TEXT
 

data/lib/rspectre/source_map/parser.rb

@@ -3,7 +3,7 @@
 module RSpectre
   class SourceMap
     class Parser
-      include Concord.new(:file)
+      include KeywordStruct.new(:file)
 
       def populate(map)
         walk(parsed_source) { |node| map.add(node) }
@@ -15,6 +15,14 @@ module RSpectre
         Null.new
       end
 
+      def self.parser_class
+        if RSpectre.ruby_version_supports_prism?
+          Prism::Translation::Parser
+        else
+          ::Parser::CurrentRuby
+        end
+      end
+
       private
 
       def walk(node, &block)
@@ -28,9 +36,8 @@ module RSpectre
       end
 
       def parsed_source
-        parser = ::Parser::CurrentRuby.new(PermissiveASTBuilder.new)
+        parser = self.class.parser_class.new(PermissiveASTBuilder.new)
         buffer = ::Parser::Source::Buffer.new(file, source: raw_source)
-
         parser.parse(buffer)
       end
 

data/lib/rspectre/source_map.rb

@@ -2,15 +2,15 @@
 
 module RSpectre
   class SourceMap
-    include Concord.new(:map)
+    include KeywordStruct.new(:map)
 
     def initialize
-      super(Hash.new { [] })
+      super(map: Hash.new { [] })
     end
     private_class_method :new
 
     def self.parse(file)
-      self::Parser.new(file).populate(new)
+      self::Parser.new(file: file).populate(new)
     end
 
     def add(node)
@@ -32,18 +32,35 @@ module RSpectre
     private
 
     def find_methods(target_selector, line)
-      block_nodes(line).select do |node|
-        send, = *node
-        _receiver, selector = *send
+      block_candidates =
+        block_nodes(line).select do |node|
+          send, = *node
+          matching_send?(send, target_selector)
+        end
 
-        selector.equal?(target_selector)
+      if block_candidates.any?
+        block_candidates
+      else
+        send_nodes(line).select do |node|
+          matching_send?(node, target_selector)
+        end
       end
     end
 
+    def matching_send?(node, method_name)
+      _receiver, selector = *node
+
+      selector.equal?(method_name)
+    end
+
     def block_nodes(line)
       map.fetch(line, []).select { |node| node.type.equal?(:block) }
     end
 
+    def send_nodes(line)
+      map.fetch(line, []).select { |node| node.type.equal?(:send) }
+    end
+
     class Null < self
       public_class_method :new
 

data/lib/rspectre/tracker.rb

@@ -2,10 +2,10 @@
 
 module RSpectre
   class Tracker
-    include Concord.new(:registry, :tracker)
+    include KeywordStruct.new(:registry, :tracker)
 
     def initialize
-      super(Hash.new { Set.new }, Hash.new { Set.new })
+      super(registry: Hash.new { Set.new }, tracker: Hash.new { Set.new })
     end
 
     def register(type, node)

data/lib/rspectre/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module RSpectre
-  VERSION = '0.0.4'
+  VERSION = '0.2.0'
 end

data/lib/rspectre.rb

@@ -6,10 +6,26 @@ require 'pathname'
 require 'set'
 require 'stringio'
 
-require 'anima'
-require 'concord'
-require 'parser/current'
 require 'rspec'
+module RSpectre
+  LIB_PATH = Pathname.new(File.expand_path('../lib', __dir__))
+
+  MINIMUM_RUBY_FOR_PRISM = Gem::Version.new('3.3')
+  private_constant(:MINIMUM_RUBY_FOR_PRISM)
+
+  def self.ruby_version_supports_prism?
+    Gem::Version.new(RUBY_VERSION) >= MINIMUM_RUBY_FOR_PRISM
+  end
+end
+
+if RSpectre.ruby_version_supports_prism?
+  require 'parser'
+  require 'prism'
+else
+  require 'parser/current'
+end
+
+require 'rspectre/keyword_struct'
 
 require 'rspectre/auto_corrector'
 require 'rspectre/color'
@@ -28,5 +44,4 @@ require 'rspectre/linter/unused_shared_setup'
 
 module RSpectre
   TRACKER = Tracker.new
-  LIB_PATH = Pathname.new(File.expand_path('../lib', __dir__))
 end

metadata

@@ -1,115 +1,84 @@
 --- !ruby/object:Gem::Specification
 name: rspectre
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.2.0
 platform: ruby
 authors:
 - Daniel Gollahon
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-01-10 00:00:00.000000000 Z
+date: 2025-03-17 00:00:00.000000000 Z
 dependencies:
-- !ruby/object:Gem::Dependency
-  name: anima
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.3'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.3'
-- !ruby/object:Gem::Dependency
-  name: concord
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.1'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.1'
 - !ruby/object:Gem::Dependency
   name: parser
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '2.6'
+        version: 3.3.7.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '2.6'
+        version: 3.3.7.1
 - !ruby/object:Gem::Dependency
-  name: rspec
+  name: prism
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '1.3'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '1.3'
 - !ruby/object:Gem::Dependency
-  name: pry
+  name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.14'
-  type: :development
+        version: '3.10'
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.14'
+        version: '3.10'
 - !ruby/object:Gem::Dependency
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.24.1
+        version: 1.71.2
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.24.1
+        version: 1.71.2
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.7.0
+        version: 3.4.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.7.0
-description:
-email:
+        version: 3.4.0
 executables:
 - rspectre
 extensions: []
@@ -119,6 +88,7 @@ files:
 - lib/rspectre.rb
 - lib/rspectre/auto_corrector.rb
 - lib/rspectre/color.rb
+- lib/rspectre/keyword_struct.rb
 - lib/rspectre/linter.rb
 - lib/rspectre/linter/unused_let.rb
 - lib/rspectre/linter/unused_shared_setup.rb
@@ -136,7 +106,6 @@ licenses:
 - MIT
 metadata:
   rubygems_mfa_required: 'true'
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -144,15 +113,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.6'
+      version: '3.1'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
-signing_key:
+rubygems_version: 3.6.2
 specification_version: 4
 summary: A tool for linting RSpec test suites.
 test_files: []