rubocop-view_component 0.7.0 → 0.8.0

data/lib/rubocop/cop/view_component/supplementary_index.rb

@@ -0,0 +1,142 @@
+# frozen_string_literal: true
+
+require "digest"
+require "rubydex"
+require "uri"
+
+module RuboCop
+  module Cop
+    module ViewComponent
+      # Extends the project index with `AdditionalSourcePaths`: source
+      # directories the cops index themselves, so a chain that passes through a
+      # gem RuboCop does not index is still resolvable. The motivating case is a
+      # Rails engine's `app/components`, which `AllCops/ProjectIndexIncludesGems`
+      # misses because it only indexes a gem's `lib`.
+      module SupplementaryIndex
+        # A minimal stand-in for the framework base class, indexed alongside the
+        # configured paths so ancestry there can still reach
+        # `ViewComponent::Base` without indexing the whole view_component gem.
+        BASE_URI = "rubydex:view_component_base"
+        BASE_SOURCE = <<~RUBY
+          module ViewComponent
+            class Base
+            end
+          end
+        RUBY
+
+        # RuboCop's cache key, extended with the supplementary files' mtimes:
+        # those files are not part of RuboCop's own index documents, so a change
+        # to one would otherwise not invalidate a cached result.
+        def external_dependency_checksum
+          base = super
+          extra = supplementary_files_checksum
+          return base if extra.nil?
+          return extra if base.nil?
+
+          Digest::SHA1.hexdigest("#{base}:#{extra}")
+        end
+
+        private
+
+        # Resolve through RuboCop's project index first, then the supplementary one.
+        def resolve_constant_in_index(const_node)
+          super || resolve_constant_in_graph(supplementary_index, const_node)
+        end
+
+        def resolve_constant_in_graph(graph, const_node)
+          return nil unless graph
+
+          segments = const_node.const_name.split("::")
+          # The inspected file's lexical scopes live in the project index, not
+          # in the supplementary one, so resolving through them can fail (a
+          # scope name like `Ui` is simply absent here). Try them first, then
+          # resolve from the top level.
+          nesting_candidates(const_node).each do |nesting|
+            declaration = resolve_segments(graph, segments, nesting)
+            return declaration if declaration
+          end
+          nil
+        end
+
+        def nesting_candidates(const_node)
+          return [[]] if const_node.absolute?
+
+          [lexical_nesting_of(const_node), []].uniq
+        end
+
+        def resolve_segments(graph, segments, nesting)
+          declaration = graph.resolve_constant(segments.first, nesting)
+          return nil unless declaration
+
+          segments.drop(1).each do |segment|
+            return nil unless declaration.is_a?(Rubydex::Namespace)
+
+            declaration = declaration.find_member(segment)
+            return nil unless declaration
+          end
+          declaration
+        end
+
+        # Built once per cop and only when the option is set.
+        def supplementary_index
+          return @supplementary_index if defined?(@supplementary_index)
+
+          @supplementary_index = build_supplementary_index
+        end
+
+        def build_supplementary_index
+          files = supplementary_source_files
+          return nil if files.empty?
+
+          graph = Rubydex::Graph.new
+          graph.index_source(BASE_URI, BASE_SOURCE, "ruby")
+          index_files(graph, files)
+          graph.resolve
+          graph
+        rescue StandardError
+          nil
+        end
+
+        def index_files(graph, files)
+          # File by file, not `index_all`: in a forked RuboCop worker
+          # `index_all` crashes once the parent has built its own graph, while
+          # `index_source` is safe.
+          files.each do |file|
+            graph.index_source(file_uri(file), File.read(file), "ruby")
+          rescue StandardError
+            next
+          end
+        end
+
+        def file_uri(file)
+          "file://#{URI::DEFAULT_PARSER.escape(file)}"
+        end
+
+        def supplementary_source_files
+          configured_source_paths.flat_map { |path| Dir.glob(File.join(path, "**", "*.rb")) }.uniq.sort
+        end
+
+        def configured_source_paths
+          Array(cop_config["AdditionalSourcePaths"]).filter_map do |path|
+            expanded = File.expand_path(path, config.base_dir_for_path_parameters)
+            expanded if File.directory?(expanded)
+          end
+        end
+
+        def supplementary_files_checksum
+          files = supplementary_source_files
+          return nil if files.empty?
+
+          Digest::SHA1.hexdigest(files.map { |file| file_signature(file) }.join("\n"))
+        end
+
+        def file_signature(file)
+          stat = File.stat(file)
+          "#{file}:#{stat.mtime.to_f}:#{stat.size}"
+        rescue StandardError
+          file
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/cop/view_component/test_rendered_output.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative "rspec_scope"
+
 module RuboCop
   module Cop
     module ViewComponent
@@ -8,6 +10,11 @@ module RuboCop
       #
       # This cop is only enabled for test files by default (see config).
       #
+      # A test is considered to instantiate a component when it builds one
+      # directly (`UserComponent.new`), through `described_class.new` inside a
+      # `describe UserComponent` block, or by referencing a `let`/`subject`
+      # helper whose body instantiates a component.
+      #
       # @example
       #   # bad
       #   def test_formatted_title
@@ -21,10 +28,40 @@ module RuboCop
       #     assert_text "HELLO"
       #   end
       #
+      #   # bad
+      #   describe TitleComponent do
+      #     it "formats the title" do
+      #       component = described_class.new("hello")
+      #       expect(component.formatted_title).to eq("HELLO")
+      #     end
+      #   end
+      #
+      #   # good
+      #   describe TitleComponent do
+      #     it "formats the title" do
+      #       render_inline described_class.new("hello")
+      #       expect(page).to have_text("HELLO")
+      #     end
+      #   end
+      #
+      #   # bad
+      #   let(:component) { TitleComponent.new("hello") }
+      #
+      #   it "formats the title" do
+      #     expect(component.formatted_title).to eq("HELLO")
+      #   end
+      #
       class TestRenderedOutput < RuboCop::Cop::Base
+        include ViewComponent::Base
+        include RSpecScope
+
         MSG = "Test instantiates a component but doesn't use `render_inline` or `render_preview`. " \
               "Test the rendered output instead of component methods directly."
 
+        # `render_in_view_context` renders through a view context (it is what
+        # form-builder tests need), so it is a rendered-output test too.
+        RENDER_METHODS = %i[render_inline render_preview render_in_view_context].freeze
+
         # Check Minitest-style test methods
         def on_def(node)
           return unless within_test_paths?
@@ -32,7 +69,7 @@ module RuboCop
           method_name = node.method_name.to_s
           return unless method_name.start_with?("test_")
           return unless instantiates_component?(node)
-          return if contains_render_method?(node)
+          return if renders_output?(node)
 
           add_offense(node)
         end
@@ -40,9 +77,9 @@ module RuboCop
         # Check RSpec-style it blocks
         def on_block(node)
           return unless within_test_paths?
-          return unless rspec_it_block?(node)
+          return unless rspec_example_block?(node)
           return unless instantiates_component?(node)
-          return if contains_render_method?(node)
+          return if renders_output?(node)
 
           add_offense(node)
         end
@@ -60,24 +97,82 @@ module RuboCop
         end
 
         def instantiates_component?(node)
-          node.each_descendant(:send).any? do |send_node|
+          component_new_call?(node) ||
+            referenced_helper?(node) { |body| component_new_call?(body) }
+        end
+
+        def renders_output?(node)
+          render_call?(node) ||
+            referenced_helper?(node) { |body| render_call?(body) }
+        end
+
+        # True when the node calls `.new` on a component constant or on
+        # `described_class` within a `describe` block.
+        def component_new_call?(node)
+          node.each_node(:send).any? do |send_node|
             next unless send_node.method?(:new)
+            next if inside_error_expectation?(send_node)
 
-            send_node.receiver&.const_type?
+            component_receiver?(send_node.receiver)
           end
         end
 
-        def contains_render_method?(node)
-          node.each_descendant(:send).any? do |send_node|
-            %i[render_inline render_preview].include?(send_node.method_name)
+        # A component built inside an error assertion is exercising
+        # initialisation behaviour rather than rendered output, so there is
+        # nothing to render. Covers RSpec's `expect { ... }.to raise_error` and
+        # Minitest's `assert_raises`.
+        def inside_error_expectation?(node)
+          node.each_ancestor(:block).any? do |ancestor|
+            error_assertion?(ancestor)
           end
         end
 
-        def rspec_it_block?(node)
-          send_node = node.send_node
+        def error_assertion?(block)
+          send_node = block.send_node
           return false unless send_node
+          return true if send_node.method?(:assert_raises)
+
+          send_node.method?(:expect) && error_expectation?(block)
+        end
+
+        def error_expectation?(expect_block)
+          parent = expect_block.parent
+          return false unless parent&.send_type?
+          return false unless %i[to not_to].include?(parent.method_name)
+
+          error_matcher?(parent.first_argument)
+        end
+
+        def error_matcher?(matcher)
+          matcher = matcher.send_node if matcher&.block_type?
+          matcher&.send_type? &&
+            %i[raise_error raise_exception].include?(matcher.method_name)
+        end
 
-          %i[it specify example].include?(send_node.method_name)
+        def component_receiver?(receiver)
+          return false unless receiver
+          return view_component_parent?(receiver) if receiver.const_type?
+          return false unless described_class_reference?(receiver)
+
+          described_class = enclosing_described_class(receiver)
+          described_class && view_component_parent?(described_class)
+        end
+
+        def render_call?(node)
+          node.each_node(:send).any? do |send_node|
+            RENDER_METHODS.include?(send_node.method_name)
+          end
+        end
+
+        # True when the node references a `let`/`subject` helper whose body
+        # satisfies the given block.
+        def referenced_helper?(node)
+          helper_bodies = helper_definitions(node)
+
+          referenced_helper_names(node).any? do |name|
+            body = helper_bodies[name]
+            body && yield(body)
+          end
         end
       end
     end

data/lib/rubocop/view_component/version.rb

@@ -2,6 +2,6 @@
 
 module RuboCop
   module ViewComponent
-    VERSION = "0.7.0"
+    VERSION = "0.8.0"
   end
 end

data/script/verify

@@ -49,7 +49,7 @@ def main
 
   Dir.chdir(config[:verification_dir]) do
     configure_rubocop(config[:config_file], config[:display_name])
-    add_gem_to_gemfile
+    replace_gemfile
 
     Bundler.with_unbundled_env do
       output = run_rubocop
@@ -89,33 +89,41 @@ def download_source(dir, tarball_url, display_name)
   system!("tar", "xz", "-C", dir, "--strip-components=1", "-f", "#{dir}/source.tar.gz")
 end
 
+# Replace the library's .rubocop.yml with a config that only runs our
+# ViewComponent cops. The library's own config may reference gems outside the
+# minimal bundle (see replace_gemfile), e.g. inherit_gem, require, or config
+# for cops from other RuboCop extensions.
 def configure_rubocop(config_file, display_name)
-  puts "Configuring .rubocop.yml with #{display_name} overrides..."
-  config = File.exist?(".rubocop.yml") ? YAML.load_file(".rubocop.yml") : {}
-  library_config = YAML.load_file(config_file)
-
-  # Merge the library config into the existing config
-  library_config.each do |key, value|
-    config[key] = if config[key].is_a?(Hash) && value.is_a?(Hash)
-                    config[key].merge(value)
-                  else
-                    value
-                  end
-  end
+  puts "Replacing .rubocop.yml with #{display_name} overrides..."
+  config = YAML.load_file(config_file)
+
+  # Silence warnings about unconfigured new cops - only ViewComponent cops run
+  config["AllCops"] ||= {}
+  config["AllCops"]["NewCops"] = "disable"
 
   File.write(".rubocop.yml", YAML.dump(config))
 end
 
-def add_gem_to_gemfile
-  puts "Adding rubocop-view_component gem to Gemfile..."
-  File.open("Gemfile", "a") { |f| f.puts "gem 'rubocop-view_component', path: '#{GEM_DIR}'" }
+# Replace the library's Gemfile with a minimal one. This avoids installing the
+# library's own dependencies, which are not needed for static analysis and can
+# fail to install or conflict with the rubocop version we require.
+# `view_component` must be present so the project index can resolve
+# `ViewComponent::Base` for ancestry-based detection.
+def replace_gemfile
+  puts "Replacing Gemfile with minimal setup..."
+  File.write("Gemfile", <<~RUBY)
+    source "https://rubygems.org"
+
+    gem "rubocop"
+    gem "rubocop-view_component", path: "#{GEM_DIR}"
+    gem "view_component"
+  RUBY
 end
 
 def run_rubocop
   FileUtils.rm_f("Gemfile.lock")
   puts "Running bundle install..."
-  system!({ "BUNDLE_WITHOUT" => "development" }, "bundle", "install")
-  system!("bundle", "add", "rubocop")
+  system!("bundle", "install")
 
   puts "Running RuboCop (ViewComponent cops only)..."
   output, status = Open3.capture2(
@@ -128,6 +136,9 @@ def run_rubocop
 
   puts "RuboCop exit status: #{status.exitstatus}"
 
+  # Exit status 0 means no offenses, 1 means offenses found (expected, since
+  # the libraries have known offenses). 2 means RuboCop itself failed.
+  abort "ERROR: RuboCop failed (exit status: #{status.exitstatus})" if status.exitstatus == 2
   abort "ERROR: RuboCop produced no output (exit status: #{status.exitstatus})" if output.strip.empty?
 
   output

data/spec/expected_flowbite_failures.yml

@@ -0,0 +1,64 @@
+# This file is auto-generated by `script/verify --regenerate`.
+# It captures known offenses in the library source.
+---
+ViewComponent/MissingPreview:
+  - 'app/components/flowbite/badge/dot.rb'
+  - 'app/components/flowbite/badge/pill.rb'
+  - 'app/components/flowbite/breadcrumb/home_icon.rb'
+  - 'app/components/flowbite/breadcrumb/item/current.rb'
+  - 'app/components/flowbite/breadcrumb/item/first.rb'
+  - 'app/components/flowbite/breadcrumb/separator_icon.rb'
+  - 'app/components/flowbite/button/outline.rb'
+  - 'app/components/flowbite/button/pill.rb'
+  - 'app/components/flowbite/card/title.rb'
+  - 'app/components/flowbite/input/date.rb'
+  - 'app/components/flowbite/input/date_time.rb'
+  - 'app/components/flowbite/input/email.rb'
+  - 'app/components/flowbite/input/file.rb'
+  - 'app/components/flowbite/input/hint.rb'
+  - 'app/components/flowbite/input/label.rb'
+  - 'app/components/flowbite/input/number.rb'
+  - 'app/components/flowbite/input/password.rb'
+  - 'app/components/flowbite/input/phone.rb'
+  - 'app/components/flowbite/input/textarea.rb'
+  - 'app/components/flowbite/input/url.rb'
+  - 'app/components/flowbite/input/validation_error.rb'
+  - 'app/components/flowbite/input_field/date.rb'
+  - 'app/components/flowbite/input_field/date_time.rb'
+  - 'app/components/flowbite/input_field/email.rb'
+  - 'app/components/flowbite/input_field/file.rb'
+  - 'app/components/flowbite/input_field/number.rb'
+  - 'app/components/flowbite/input_field/password.rb'
+  - 'app/components/flowbite/input_field/phone.rb'
+  - 'app/components/flowbite/input_field/text.rb'
+  - 'app/components/flowbite/input_field/textarea.rb'
+  - 'app/components/flowbite/input_field/url.rb'
+  - 'app/components/flowbite/pagination/chevron_left_icon.rb'
+  - 'app/components/flowbite/pagination/chevron_right_icon.rb'
+  - 'app/components/flowbite/pagination/gap.rb'
+  - 'app/components/flowbite/pagination/next_link.rb'
+  - 'app/components/flowbite/pagination/previous_link.rb'
+  - 'app/components/flowbite/sidebar/item.rb'
+  - 'app/components/flowbite/sidebar/navigation.rb'
+  - 'app/components/flowbite/toast/icon.rb'
+ViewComponent/PreferPrivateMethods:
+  - 'app/components/flowbite/badge.rb'
+  - 'app/components/flowbite/input.rb'
+  - 'app/components/flowbite/input/checkbox.rb'
+  - 'app/components/flowbite/input/date.rb'
+  - 'app/components/flowbite/input/date_time.rb'
+  - 'app/components/flowbite/input/email.rb'
+  - 'app/components/flowbite/input/file.rb'
+  - 'app/components/flowbite/input/hint.rb'
+  - 'app/components/flowbite/input/label.rb'
+  - 'app/components/flowbite/input/number.rb'
+  - 'app/components/flowbite/input/password.rb'
+  - 'app/components/flowbite/input/phone.rb'
+  - 'app/components/flowbite/input/radio_button.rb'
+  - 'app/components/flowbite/input/select.rb'
+  - 'app/components/flowbite/input/url.rb'
+  - 'app/components/flowbite/input_field.rb'
+  - 'app/components/flowbite/input_field/select.rb'
+  - 'app/components/flowbite/pagination/link.rb'
+  - 'app/components/flowbite/pagination/next_link.rb'
+  - 'app/components/flowbite/pagination/previous_link.rb'

data/spec/expected_govuk_failures.yml

@@ -3,24 +3,20 @@
 ---
 ViewComponent/ComponentSuffix:
   - 'app/components/govuk_component/base.rb'
-  - 'app/components/govuk_component/header_component.rb'
-  - 'app/components/govuk_component/notification_banner_component.rb'
+  - 'app/components/govuk_component/header_component.rb' # due to nested class
+  - 'app/components/govuk_component/language_navigation_component.rb' # due to nested class
+  - 'app/components/govuk_component/notification_banner_component.rb' # due to nested class
   - 'app/components/govuk_component/pagination_component/adjacent_page.rb'
   - 'app/components/govuk_component/pagination_component/item.rb'
-  - 'app/components/govuk_component/panel_component.rb'
-  - 'app/components/govuk_component/tab_component.rb'
+  - 'app/components/govuk_component/pagination_component/next_page.rb'
+  - 'app/components/govuk_component/pagination_component/previous_page.rb'
+  - 'app/components/govuk_component/panel_component.rb' # due to nested class
+  - 'app/components/govuk_component/tab_component.rb' # due to nested class
 ViewComponent/PreferPrivateMethods:
   - 'app/components/govuk_component/base.rb'
+  - 'app/components/govuk_component/language_navigation_component.rb'
   - 'app/components/govuk_component/notification_banner_component.rb'
+  - 'app/components/govuk_component/pagination_component/next_page.rb'
   - 'app/components/govuk_component/panel_component.rb'
   - 'app/components/govuk_component/service_navigation_component.rb'
   - 'app/components/govuk_component/tab_component.rb'
-ViewComponent/TestRenderedOutput:
-  - 'spec/components/govuk_component/accordion_component_spec.rb'
-  - 'spec/components/govuk_component/breadcrumbs_component_spec.rb'
-  - 'spec/components/govuk_component/configuration/footer_component_configuration_spec.rb'
-  - 'spec/components/govuk_component/footer_component_spec.rb'
-  - 'spec/components/govuk_component/notification_banner_component_spec.rb'
-  - 'spec/components/govuk_component/pagination_component_spec.rb'
-  - 'spec/components/govuk_component/task_list_component_spec.rb'
-  - 'spec/components/govuk_component/warning_text_component_spec.rb'

data/spec/expected_polaris_failures.yml

@@ -17,6 +17,8 @@ ViewComponent/MissingPreview:
   - 'app/components/polaris/inline_code_component.rb'
   - 'app/components/polaris/label_component.rb'
   - 'app/components/polaris/labelled_component.rb'
+  - 'app/components/polaris/layout/annotated_section.rb'
+  - 'app/components/polaris/layout/section.rb'
   - 'app/components/polaris/list_component.rb'
   - 'app/components/polaris/page_component.rb'
   - 'app/components/polaris/placeholder_component.rb'

data/spec/expected_primer_failures.yml

@@ -2,9 +2,66 @@
 # It captures known offenses in the library source.
 ---
 ViewComponent/MissingPreview:
+  - 'app/components/primer/alpha/action_bar/divider.rb'
+  - 'app/components/primer/alpha/action_bar/item.rb'
+  - 'app/components/primer/alpha/action_list/form_wrapper.rb'
+  - 'app/components/primer/alpha/action_menu/group.rb'
+  - 'app/components/primer/alpha/action_menu/heading.rb'
+  - 'app/components/primer/alpha/action_menu/list.rb'
+  - 'app/components/primer/alpha/action_menu/list_wrapper.rb'
+  - 'app/components/primer/alpha/auto_complete/item.rb'
+  - 'app/components/primer/alpha/dialog/body.rb'
+  - 'app/components/primer/alpha/dialog/footer.rb'
+  - 'app/components/primer/alpha/dialog/header.rb'
+  - 'app/components/primer/alpha/dropdown/menu.rb'
+  - 'app/components/primer/alpha/file_tree_view/directory_node.rb'
+  - 'app/components/primer/alpha/file_tree_view/file_node.rb'
+  - 'app/components/primer/alpha/form_button.rb'
+  - 'app/components/primer/alpha/image.rb'
+  - 'app/components/primer/alpha/layout.rb'
+  - 'app/components/primer/alpha/nav_list.rb'
+  - 'app/components/primer/alpha/nav_list/divider.rb'
+  - 'app/components/primer/alpha/nav_list/group.rb'
+  - 'app/components/primer/alpha/nav_list/heading.rb'
+  - 'app/components/primer/alpha/nav_list/item.rb'
+  - 'app/components/primer/alpha/overlay/body.rb'
+  - 'app/components/primer/alpha/overlay/footer.rb'
+  - 'app/components/primer/alpha/overlay/header.rb'
+  - 'app/components/primer/alpha/segmented_control/item.rb'
+  - 'app/components/primer/alpha/select_panel.rb'
+  - 'app/components/primer/alpha/submit_button.rb'
+  - 'app/components/primer/alpha/tab_container.rb'
+  - 'app/components/primer/alpha/tree_view/icon.rb'
+  - 'app/components/primer/alpha/tree_view/icon_pair.rb'
+  - 'app/components/primer/alpha/tree_view/leading_action.rb'
+  - 'app/components/primer/alpha/tree_view/loading_failure_message.rb'
+  - 'app/components/primer/alpha/tree_view/node.rb'
+  - 'app/components/primer/alpha/tree_view/skeleton_loader.rb'
+  - 'app/components/primer/alpha/tree_view/spinner_loader.rb'
+  - 'app/components/primer/alpha/tree_view/sub_tree.rb'
+  - 'app/components/primer/alpha/tree_view/sub_tree_container.rb'
+  - 'app/components/primer/alpha/tree_view/visual.rb'
+  - 'app/components/primer/beta/auto_complete/item.rb'
+  - 'app/components/primer/beta/auto_complete/no_result_item.rb'
+  - 'app/components/primer/beta/border_box/header.rb'
+  - 'app/components/primer/beta/breadcrumbs.rb'
+  - 'app/components/primer/beta/button_group.rb'
+  - 'app/components/primer/beta/clipboard_copy_button.rb'
+  - 'app/components/primer/beta/timeline_item.rb'
+  - 'app/components/primer/beta/truncate.rb'
   - 'app/components/primer/blankslate_component.rb'
   - 'app/components/primer/button_component.rb'
+  - 'app/components/primer/conditional_wrapper.rb'
+  - 'app/components/primer/content.rb'
+  - 'app/components/primer/icon_button.rb'
   - 'app/components/primer/layout_component.rb'
+  - 'app/components/primer/tooltip.rb'
+  - 'app/components/primer/truncate.rb'
+  - 'test/components/beta/nav_list_test.rb'
+  - 'test/lib/primer/audited/dsl_test.rb'
+  - 'test/lib/primer/forms/form_test_component.rb'
+  - 'test/lib/primer/status/dsl_test.rb'
+  - 'test/test_helpers/components/primer/deny_component.rb'
 ViewComponent/PreferPrivateMethods:
   - 'app/components/primer/alpha/action_list.rb'
   - 'app/components/primer/alpha/action_list/divider.rb'
@@ -18,5 +75,3 @@ ViewComponent/PreferPrivateMethods:
   - 'app/components/primer/beta/nav_list/divider.rb'
   - 'app/components/primer/beta/nav_list/group.rb'
   - 'app/components/primer/beta/nav_list/item.rb'
-ViewComponent/TestRenderedOutput:
-  - 'test/components/alpha/action_list_test.rb'

data/spec/rubocop/cop/view_component/base_spec.rb

@@ -89,4 +89,20 @@ RSpec.describe RuboCop::Cop::ViewComponent::Base do
       )
     end
   end
+
+  describe "#resolved_index_declaration" do
+    it "raises when project_index is not set" do
+      source = parse_source(<<~RUBY)
+        class Foo < ViewComponent::Base
+        end
+      RUBY
+
+      parent = source.ast.parent_class
+
+      expect { cop.send(:resolved_index_declaration, parent) }.to raise_error(
+        RuntimeError,
+        /UseProjectIndex/
+      )
+    end
+  end
 end