sevencop 0.17.2 → 0.18.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bb03ae9347c8419a82327535021c4567f50ee667fa6b49d01de0766c9db17b19
-  data.tar.gz: 219b887de52c24ffd4c6baf30b6443311e297b4ef1e3b03895d60f874b145b1f
+  metadata.gz: ddb1ab8850e168dd10c98cf0954d8e9df26cf1b0d43c1019780c2ce76d84d8d6
+  data.tar.gz: d159188fae4618097806c22ef26825993b821f2277671011f9c89997056fb364
 SHA512:
-  metadata.gz: 4a8c1fba03d1a7e5bacae6aa878f9d6284c8b1c5becd745285b5e5f3910784ecc5b815191d0a1f4c7f0e6ee06e70109a58af8bd5dd78797bacaf053a7857e42b
-  data.tar.gz: b0d4562158515e27a4bbc8c3e90a633bcaeb548267ab208b92e55931416b843340c5b2a528f3bcfbd72f430701513aeda02fcbfb2dc8602945456a94eb1ece4d
+  metadata.gz: 449aff2e37cf2abd954b38db5ee587401657f261b584097d4390ab23a76df991505561e7462e8683d950a126fb7675a2b0445a868fc0802ab7711d9a10cc9d2d
+  data.tar.gz: 5171d0b62826dd0d85371a7b81fcae4917a49a3cf1de2f4d19ed7e61f38e670c8bacc0e7afd2934aac060e889a44d7475013d031eaaed68d9d7006ab0b85f03f

data/.rubocop.yml

@@ -28,6 +28,7 @@ RSpec/ExampleLength:
 
 RSpec/FilePath:
   CustomTransform:
+    RSpecDescribeHttpEndpoint: rspec_describe_http_endpoint
     RSpecExamplesInSameGroup: rspec_examples_in_same_group
 
 RSpec/MultipleExpectations:

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    sevencop (0.17.2)
+    sevencop (0.18.4)
       rubocop
 
 GEM

data/README.md

@@ -39,6 +39,7 @@ Choose the cops you want to use and enable them on your .rubocop.yml.
 - [Sevencop/RailsUniquenessValidatorExplicitCaseSensitivity](lib/rubocop/cop/sevencop/rails_uniqueness_validator_explicit_case_sensitivity.rb)
 - [Sevencop/RailsWhereNot](lib/rubocop/cop/sevencop/rails_where_not.rb)
 - [Sevencop/RequireOrdered](lib/rubocop/cop/sevencop/require_ordered.rb)
+- [Sevencop/RSpecDescribeHttpEndpoint](lib/rubocop/cop/sevencop/rspec_describe_http_endpoint.rb)
 - [Sevencop/RSpecExamplesInSameGroup](lib/rubocop/cop/sevencop/rspec_examples_in_same_group.rb)
 
 Note that all cops are `Enabled: false` by default.

data/config/default.yml

@@ -68,6 +68,14 @@ Sevencop/RequireOrdered:
   Enabled: false
   VersionAdded: '0.16'
 
+Sevencop/RSpecDescribeHttpEndpoint:
+  Description: |
+    Pass HTTP endpoint identifier to top-level `describe` on request-specs.
+  Enabled: false
+  VersionAdded: '0.18'
+  Include:
+    - spec/requests/**/*.rb
+
 Sevencop/RSpecExamplesInSameGroup:
   Description: |
     Combine examples in the same groups in the time-consuming kinds of specs.

data/lib/rubocop/cop/sevencop/method_definition_ordered.rb

@@ -40,9 +40,7 @@ module RuboCop
           previous_older_sibling = find_previous_older_sibling(node)
           return unless previous_older_sibling
 
-          add_offense(
-            range_with_comments(node)
-          ) do |corrector|
+          add_offense(node) do |corrector|
             swap(
               range_with_comments_and_lines(previous_older_sibling),
               range_with_comments_and_lines(node),

data/lib/rubocop/cop/sevencop/require_ordered.rb

@@ -52,7 +52,7 @@ module RuboCop
           return unless previous_older_sibling
 
           add_offense(
-            range_with_comments(node),
+            node,
             message: "Sort `#{node.method_name}` in alphabetical order."
           ) do |corrector|
             swap(

data/lib/rubocop/cop/sevencop/rspec_describe_http_endpoint.rb

@@ -0,0 +1,64 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Sevencop
+      # Pass HTTP endpoint identifier to top-level `describe` on request-specs.
+      #
+      # @see https://github.com/r7kamura/rspec-request_describer
+      #
+      # @example
+      #   # bad
+      #   RSpec.describe 'Users'
+      #
+      #   # good
+      #   RSpec.describe 'GET /users'
+      class RSpecDescribeHttpEndpoint < Base
+        MSG = 'Pass HTTP endpoint identifier to top-level `describe` on request-specs.'
+
+        RESTRICT_ON_SEND = %i[
+          describe
+        ].freeze
+
+        SUPPORTED_HTTP_METHODS = %w[
+          DELETE
+          GET
+          PATCH
+          POST
+          PUT
+        ].freeze
+
+        DESCRIPTION_PATTERN = %r{\A#{::Regexp.union(SUPPORTED_HTTP_METHODS)} /}.freeze
+
+        # @param node [RuboCop::AST::SendNode]
+        # @return [void]
+        def on_send(node)
+          return unless top_level_describe?(node)
+          return unless node.first_argument
+          return if http_endpoint_identifier?(node.first_argument)
+
+          add_offense(node.first_argument)
+        end
+
+        private
+
+        # @!method top_level_describe?(node)
+        #   @param node [RuboCop::AST::SendNode]
+        #   @return [Boolean]
+        def_node_matcher :top_level_describe?, <<~PATTERN
+          (send
+            (const nil? :RSpec)
+            :describe
+            ...
+          )
+        PATTERN
+
+        # @param node [RuboCop::AST::Node, nil]
+        # @return [Boolean]
+        def http_endpoint_identifier?(node)
+          node&.str_type? && node.value.match?(DESCRIPTION_PATTERN)
+        end
+      end
+    end
+  end
+end

data/lib/sevencop/cop_concerns/ordered.rb

@@ -5,23 +5,30 @@ module Sevencop
     module Ordered
       private
 
+      # @param range1 [Paresr::Source::Range]
+      # @param range2 [Paresr::Source::Range]
+      # @return [Paresr::Source::Range]
+      def add_range(
+        range1,
+        range2
+      )
+        range1.with(
+          begin_pos: [range1.begin_pos, range2.begin_pos].min,
+          end_pos: [range1.end_pos, range2.end_pos].max
+        )
+      end
+
       # @param node [RuboCop::AST::Node]
       # @return [Paresr::Source::Range]
       def range_with_comments(node)
-        comment = processed_source.ast_with_comments[node].first
-        if comment
-          node.location.expression.with(
-            begin_pos: [
-              comment.location.expression.begin_pos,
-              node.location.expression.begin_pos
-            ].min,
-            end_pos: [
-              comment.location.expression.end_pos,
-              node.location.expression.end_pos
-            ].max
-          )
-        else
-          node.location.expression
+        ranges = [
+          node,
+          *processed_source.ast_with_comments[node]
+        ].map do |element|
+          element.location.expression
+        end
+        ranges.reduce do |result, range|
+          add_range(result, range)
         end
       end
 

data/lib/sevencop/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Sevencop
-  VERSION = '0.17.2'
+  VERSION = '0.18.4'
 end

data/lib/sevencop.rb

@@ -15,4 +15,5 @@ require_relative 'rubocop/cop/sevencop/rails_order_field'
 require_relative 'rubocop/cop/sevencop/rails_uniqueness_validator_explicit_case_sensitivity'
 require_relative 'rubocop/cop/sevencop/rails_where_not'
 require_relative 'rubocop/cop/sevencop/require_ordered'
+require_relative 'rubocop/cop/sevencop/rspec_describe_http_endpoint'
 require_relative 'rubocop/cop/sevencop/rspec_examples_in_same_group'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sevencop
 version: !ruby/object:Gem::Version
-  version: 0.17.2
+  version: 0.18.4
 platform: ruby
 authors:
 - Ryo Nakamura
-autorequire:
+autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-10-07 00:00:00.000000000 Z
+date: 2022-10-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -24,7 +24,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description:
+description: 
 email:
 - r7kamura@gmail.com
 executables: []
@@ -52,6 +52,7 @@ files:
 - lib/rubocop/cop/sevencop/rails_uniqueness_validator_explicit_case_sensitivity.rb
 - lib/rubocop/cop/sevencop/rails_where_not.rb
 - lib/rubocop/cop/sevencop/require_ordered.rb
+- lib/rubocop/cop/sevencop/rspec_describe_http_endpoint.rb
 - lib/rubocop/cop/sevencop/rspec_examples_in_same_group.rb
 - lib/sevencop.rb
 - lib/sevencop/config_loader.rb
@@ -67,7 +68,7 @@ metadata:
   homepage_uri: https://github.com/r7kamura/sevencop
   source_code_uri: https://github.com/r7kamura/sevencop
   changelog_uri: https://github.com/r7kamura/sevencop/releases
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -83,7 +84,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.3.7
-signing_key:
+signing_key: 
 specification_version: 4
 summary: Opinionated custom cops for RuboCop.
 test_files: []