rubocop-view_component 0.7.0 → 0.8.0

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

@@ -1,5 +1,8 @@
 # frozen_string_literal: true
 
+require "fileutils"
+require "tmpdir"
+
 RSpec.describe RuboCop::Cop::ViewComponent::PreferComposition, :config do
   let(:config) { RuboCop::Config.new }
 
@@ -55,6 +58,62 @@ RSpec.describe RuboCop::Cop::ViewComponent::PreferComposition, :config do
     end
   end
 
+  context "when the parent is a resolvable component with its own template" do
+    it "registers an offense" do
+      allow(File).to receive(:exist?).and_call_original
+      allow(File).to receive(:exist?).with("/test_source.html.erb").and_return(true)
+
+      expect_offense(<<~RUBY)
+        class BaseCardComponent < ViewComponent::Base
+        end
+
+        class UserCardComponent < BaseCardComponent
+                                  ^^^^^^^^^^^^^^^^^ ViewComponent/PreferComposition: Avoid inheriting from another ViewComponent.
+        end
+      RUBY
+    end
+  end
+
+  context "when the parent is a resolvable component without a template" do
+    it "does not register an offense" do
+      expect_no_offenses(<<~RUBY)
+        class BaseCardComponent < ViewComponent::Base
+        end
+
+        class UserCardComponent < BaseCardComponent
+        end
+      RUBY
+    end
+  end
+
+  context "when the parent resolves to a non-ViewComponent class" do
+    it "does not register an offense even when the name ends with Component" do
+      expect_no_offenses(<<~RUBY)
+        class WidgetComponent
+        end
+
+        class UserCardComponent < WidgetComponent
+        end
+      RUBY
+    end
+  end
+
+  context "when inheriting from a namespaced abstract base class" do
+    it "does not register an offense" do
+      expect_no_offenses(<<~RUBY)
+        module Shadcn
+          class ApplicationViewComponent < ViewComponent::Base
+          end
+
+          module Button
+            class Component < ApplicationViewComponent
+            end
+          end
+        end
+      RUBY
+    end
+  end
+
   context "when class has no parent" do
     it "does not register an offense" do
       expect_no_offenses(<<~RUBY)
@@ -88,4 +147,49 @@ RSpec.describe RuboCop::Cop::ViewComponent::PreferComposition, :config do
       RUBY
     end
   end
+
+  context "with AdditionalSourcePaths" do
+    let(:engine_dir) { Dir.mktmpdir("engine") }
+
+    let(:config) do
+      RuboCop::Config.new(
+        "ViewComponent/PreferComposition" => {
+          "AdditionalSourcePaths" => [File.join(engine_dir, "app", "components")]
+        }
+      )
+    end
+
+    before do
+      write_engine_component("engine_card", "EngineCard")
+      write_engine_component("engine_base", "EngineBase")
+    end
+
+    after { FileUtils.remove_entry(engine_dir) }
+
+    def write_engine_component(dir_name, class_name)
+      dir = File.join(engine_dir, "app", "components", dir_name)
+      FileUtils.mkdir_p(dir)
+      File.write(
+        File.join(dir, "component.rb"),
+        "module #{class_name}\n  class Component < ViewComponent::Base\n  end\nend\n"
+      )
+    end
+
+    it "resolves a parent through the additional paths and flags one with a template" do
+      File.write(File.join(engine_dir, "app", "components", "engine_card", "component.html.erb"), "<div></div>\n")
+
+      expect_offense(<<~RUBY)
+        class UserCardComponent < EngineCard::Component
+                                  ^^^^^^^^^^^^^^^^^^^^^ Avoid inheriting from another ViewComponent.
+        end
+      RUBY
+    end
+
+    it "does not flag a parent resolved through the additional paths without a template" do
+      expect_no_offenses(<<~RUBY)
+        class UserCardComponent < EngineBase::Component
+        end
+      RUBY
+    end
+  end
 end

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

@@ -1,5 +1,8 @@
 # frozen_string_literal: true
 
+require "fileutils"
+require "tmpdir"
+
 RSpec.describe RuboCop::Cop::ViewComponent::PreferPrivateMethods, :config do
   let(:config) { RuboCop::Config.new }
 
@@ -54,6 +57,77 @@ RSpec.describe RuboCop::Cop::ViewComponent::PreferPrivateMethods, :config do
     end
   end
 
+  context "when helper methods are made private with an explicit list" do
+    it "does not register offense for the listed methods" do
+      expect_no_offenses(<<~RUBY)
+        class CardComponent < ViewComponent::Base
+          def initialize(title)
+            @title = title
+          end
+
+          def formatted_title
+            @title.upcase
+          end
+
+          def helper
+            "helper"
+          end
+
+          private :formatted_title, :helper
+        end
+      RUBY
+    end
+
+    it "does not change the visibility of methods defined after the list" do
+      expect_offense(<<~RUBY)
+        class CardComponent < ViewComponent::Base
+          def formatted_title
+            "title"
+          end
+
+          private :formatted_title
+
+          def other_helper
+          ^^^^^^^^^^^^^^^^ ViewComponent/PreferPrivateMethods: Consider making `other_helper` private. Only ViewComponent interface methods should be public.
+            "other"
+          end
+        end
+      RUBY
+    end
+
+    it "flags a method made public again with an explicit list" do
+      expect_offense(<<~RUBY)
+        class CardComponent < ViewComponent::Base
+          private
+
+          def formatted_title
+          ^^^^^^^^^^^^^^^^^^^ ViewComponent/PreferPrivateMethods: Consider making `formatted_title` private. Only ViewComponent interface methods should be public.
+            "title"
+          end
+
+          public :formatted_title
+        end
+      RUBY
+    end
+  end
+
+  context "when a method is defined with `private def`" do
+    it "flags later public methods but not the `private def` method" do
+      expect_offense(<<~RUBY)
+        class CardComponent < ViewComponent::Base
+          private def formatted_title
+            "title"
+          end
+
+          def other_helper
+          ^^^^^^^^^^^^^^^^ ViewComponent/PreferPrivateMethods: Consider making `other_helper` private. Only ViewComponent interface methods should be public.
+            "other"
+          end
+        end
+      RUBY
+    end
+  end
+
   context "with allowed public interface methods" do
     it "allows initialize" do
       expect_no_offenses(<<~RUBY)
@@ -258,6 +332,56 @@ RSpec.describe RuboCop::Cop::ViewComponent::PreferPrivateMethods, :config do
     end
   end
 
+  context "with AdditionalSourcePaths" do
+    let(:engine_dir) { Dir.mktmpdir("engine") }
+
+    before do
+      FileUtils.mkdir_p(File.join(engine_dir, "app", "components"))
+      File.write(
+        File.join(engine_dir, "app", "components", "engine_base_component.rb"),
+        "class EngineBaseComponent < ViewComponent::Base\nend\n"
+      )
+    end
+
+    after { FileUtils.remove_entry(engine_dir) }
+
+    context "when the engine source path is configured" do
+      let(:config) do
+        RuboCop::Config.new(
+          "AllCops" => { "DisplayCopNames" => true },
+          "ViewComponent/PreferPrivateMethods" => {
+            "AdditionalSourcePaths" => [File.join(engine_dir, "app", "components")]
+          }
+        )
+      end
+
+      it "resolves ancestry through the additional source paths" do
+        expect_offense(<<~RUBY)
+          class CardComponent < EngineBaseComponent
+            def helper
+            ^^^^^^^^^^ ViewComponent/PreferPrivateMethods: Consider making `helper` private. Only ViewComponent interface methods should be public.
+              "helper"
+            end
+          end
+        RUBY
+      end
+    end
+
+    context "when the engine source path is not configured" do
+      let(:config) { RuboCop::Config.new }
+
+      it "does not detect the class as a ViewComponent" do
+        expect_no_offenses(<<~RUBY)
+          class CardComponent < EngineBaseComponent
+            def helper
+              "helper"
+            end
+          end
+        RUBY
+      end
+    end
+  end
+
   context "with template files" do
     let(:component_file) { "spec/fixtures/components/template_method_component.rb" }
 

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

@@ -1,7 +1,10 @@
 # frozen_string_literal: true
 
 RSpec.describe RuboCop::Cop::ViewComponent::TestRenderedOutput, :config do
+  include ViewComponent::ProjectIndexHelpers
+
   let(:config) { RuboCop::Config.new }
+  let(:project_index) { build_index }
 
   context "with Minitest-style tests" do
     context "when test instantiates a component but doesn't render" do
@@ -17,17 +20,37 @@ RSpec.describe RuboCop::Cop::ViewComponent::TestRenderedOutput, :config do
     end
 
     context "when test instantiates a non-Component-suffixed class" do
-      it "registers an offense" do
+      it "registers an offense for an indexed component" do
         expect_offense(<<~RUBY)
           def test_toggle
           ^^^^^^^^^^^^^^^ ViewComponent/TestRenderedOutput: Test instantiates a component but doesn't use `render_inline` or `render_preview`. Test the rendered output instead of component methods directly.
-            component = V2::Toggle.new
+            component = UserComponent.new
             assert component.active?
           end
         RUBY
       end
     end
 
+    context "when test instantiates a non-ViewComponent value object" do
+      let(:project_index) do
+        build_index(
+          "file:///month.rb" => <<~RUBY
+            class Month
+            end
+          RUBY
+        )
+      end
+
+      it "does not register an offense" do
+        expect_no_offenses(<<~RUBY)
+          def test_month
+            month = Month.new
+            assert month.active?
+          end
+        RUBY
+      end
+    end
+
     context "when test instantiates a component and uses render_inline" do
       it "does not register an offense" do
         expect_no_offenses(<<~RUBY)
@@ -50,6 +73,18 @@ RSpec.describe RuboCop::Cop::ViewComponent::TestRenderedOutput, :config do
       end
     end
 
+    context "when test instantiates a component and uses render_in_view_context" do
+      it "does not register an offense" do
+        expect_no_offenses(<<~RUBY)
+          def test_form_input
+            render_in_view_context do
+              render(UserComponent.new("hello"))
+            end
+          end
+        RUBY
+      end
+    end
+
     context "when test doesn't instantiate any constants" do
       it "does not register an offense" do
         expect_no_offenses(<<~RUBY)
@@ -84,16 +119,46 @@ RSpec.describe RuboCop::Cop::ViewComponent::TestRenderedOutput, :config do
     end
 
     context "when test instantiates a namespaced component" do
-      it "registers an offense" do
+      it "registers an offense for an indexed component" do
         expect_offense(<<~RUBY)
           def test_namespaced
           ^^^^^^^^^^^^^^^^^^^ ViewComponent/TestRenderedOutput: Test instantiates a component but doesn't use `render_inline` or `render_preview`. Test the rendered output instead of component methods directly.
-            component = Admin::UserComponent.new("hello")
+            component = UserComponent.new("hello")
             assert_equal "HELLO", component.formatted_title
           end
         RUBY
       end
     end
+
+    context "when the component is built to assert an error" do
+      it "does not register an offense for assert_raises" do
+        expect_no_offenses(<<~RUBY)
+          def test_raises_for_invalid_input
+            assert_raises(ArgumentError) { UserComponent.new(nil) }
+          end
+        RUBY
+      end
+
+      it "does not register an offense for a do/end assert_raises block" do
+        expect_no_offenses(<<~RUBY)
+          def test_raises_for_invalid_input
+            assert_raises(ArgumentError) do
+              UserComponent.new(nil)
+            end
+          end
+        RUBY
+      end
+
+      it "registers an offense when a component is built outside the assertion" do
+        expect_offense(<<~RUBY)
+          def test_mixed
+          ^^^^^^^^^^^^^^ ViewComponent/TestRenderedOutput: Test instantiates a component but doesn't use `render_inline` or `render_preview`. Test the rendered output instead of component methods directly.
+            component = UserComponent.new("hello")
+            assert_raises(ArgumentError) { component.invalid! }
+          end
+        RUBY
+      end
+    end
   end
 
   context "with TestPaths configured" do
@@ -154,6 +219,18 @@ RSpec.describe RuboCop::Cop::ViewComponent::TestRenderedOutput, :config do
       end
     end
 
+    context "when it block instantiates a component and uses render_in_view_context" do
+      it "does not register an offense" do
+        expect_no_offenses(<<~RUBY)
+          it "renders a form" do
+            render_in_view_context do
+              render(UserComponent.new("hello"))
+            end
+          end
+        RUBY
+      end
+    end
+
     context "when it block doesn't instantiate components" do
       it "does not register an offense" do
         expect_no_offenses(<<~RUBY)
@@ -164,5 +241,211 @@ RSpec.describe RuboCop::Cop::ViewComponent::TestRenderedOutput, :config do
         RUBY
       end
     end
+
+    context "when it block uses described_class inside a describe block" do
+      it "registers an offense" do
+        expect_offense(<<~RUBY)
+          describe UserComponent do
+            it "formats the title" do
+            ^^^^^^^^^^^^^^^^^^^^^^^^^ ViewComponent/TestRenderedOutput: Test instantiates a component but doesn't use `render_inline` or `render_preview`. Test the rendered output instead of component methods directly.
+              component = described_class.new("hello")
+              expect(component.formatted_title).to eq("HELLO")
+            end
+          end
+        RUBY
+      end
+
+      it "registers an offense for RSpec.describe" do
+        expect_offense(<<~RUBY)
+          RSpec.describe UserComponent do
+            it "formats the title" do
+            ^^^^^^^^^^^^^^^^^^^^^^^^^ ViewComponent/TestRenderedOutput: Test instantiates a component but doesn't use `render_inline` or `render_preview`. Test the rendered output instead of component methods directly.
+              expect(described_class.new("hello").formatted_title).to eq("HELLO")
+            end
+          end
+        RUBY
+      end
+
+      it "registers an offense inside a nested context" do
+        expect_offense(<<~RUBY)
+          describe UserComponent do
+            context "when rendered" do
+              it "formats the title" do
+              ^^^^^^^^^^^^^^^^^^^^^^^^^ ViewComponent/TestRenderedOutput: Test instantiates a component but doesn't use `render_inline` or `render_preview`. Test the rendered output instead of component methods directly.
+                component = described_class.new("hello")
+                expect(component.formatted_title).to eq("HELLO")
+              end
+            end
+          end
+        RUBY
+      end
+
+      it "does not register an offense when using render_inline" do
+        expect_no_offenses(<<~RUBY)
+          describe UserComponent do
+            it "formats the title" do
+              render_inline described_class.new("hello")
+              expect(page).to have_text("HELLO")
+            end
+          end
+        RUBY
+      end
+    end
+
+    context "when the component is built to assert an error" do
+      it "does not register an offense for described_class" do
+        expect_no_offenses(<<~RUBY)
+          describe UserComponent do
+            it "raises for invalid input" do
+              expect { described_class.new(nil) }.to raise_error(ArgumentError)
+            end
+          end
+        RUBY
+      end
+
+      it "does not register an offense for a component constant" do
+        expect_no_offenses(<<~RUBY)
+          it "raises for invalid input" do
+            expect { UserComponent.new(nil) }.to raise_error(ArgumentError)
+          end
+        RUBY
+      end
+
+      it "does not register an offense for not_to raise_error" do
+        expect_no_offenses(<<~RUBY)
+          it "does not raise for valid input" do
+            expect { UserComponent.new("hello") }.not_to raise_error
+          end
+        RUBY
+      end
+
+      it "does not register an offense when raise_error takes a block" do
+        expect_no_offenses(<<~RUBY)
+          it "raises with a message" do
+            expect { UserComponent.new(nil) }.to raise_error(ArgumentError) do |error|
+              expect(error.message).to eq("invalid")
+            end
+          end
+        RUBY
+      end
+    end
+
+    context "when a component is built by a let helper" do
+      it "registers an offense" do
+        expect_offense(<<~RUBY)
+          describe UserComponent do
+            let(:component) { UserComponent.new("hello") }
+
+            it "formats the title" do
+            ^^^^^^^^^^^^^^^^^^^^^^^^^ ViewComponent/TestRenderedOutput: Test instantiates a component but doesn't use `render_inline` or `render_preview`. Test the rendered output instead of component methods directly.
+              expect(component.formatted_title).to eq("HELLO")
+            end
+          end
+        RUBY
+      end
+
+      it "registers an offense for a subject using described_class" do
+        expect_offense(<<~RUBY)
+          describe UserComponent do
+            subject(:component) { described_class.new("hello") }
+
+            it "formats the title" do
+            ^^^^^^^^^^^^^^^^^^^^^^^^^ ViewComponent/TestRenderedOutput: Test instantiates a component but doesn't use `render_inline` or `render_preview`. Test the rendered output instead of component methods directly.
+              expect(component.formatted_title).to eq("HELLO")
+            end
+          end
+        RUBY
+      end
+
+      it "registers an offense when the let is defined in an outer group" do
+        expect_offense(<<~RUBY)
+          describe UserComponent do
+            let(:component) { UserComponent.new("hello") }
+
+            context "when rendered" do
+              it "formats the title" do
+              ^^^^^^^^^^^^^^^^^^^^^^^^^ ViewComponent/TestRenderedOutput: Test instantiates a component but doesn't use `render_inline` or `render_preview`. Test the rendered output instead of component methods directly.
+                expect(component.formatted_title).to eq("HELLO")
+              end
+            end
+          end
+        RUBY
+      end
+
+      it "does not register an offense when the helper renders output" do
+        expect_no_offenses(<<~RUBY)
+          describe UserComponent do
+            let(:rendered) { render_inline(UserComponent.new("hello")) }
+
+            it "formats the title" do
+              expect(rendered).to have_text("HELLO")
+            end
+          end
+        RUBY
+      end
+
+      it "does not register an offense when the helper builds another object" do
+        expect_no_offenses(<<~RUBY)
+          describe UserComponent do
+            let(:component) { double("component") }
+
+            it "formats the title" do
+              expect(component.formatted_title).to eq("HELLO")
+            end
+          end
+        RUBY
+      end
+
+      it "does not register an offense when the helper is rendered with render_inline" do
+        expect_no_offenses(<<~RUBY)
+          describe UserComponent do
+            let(:component) { UserComponent.new("hello") }
+
+            it "formats the title" do
+              render_inline(component)
+              expect(page).to have_text("HELLO")
+            end
+          end
+        RUBY
+      end
+
+      it "prefers the innermost helper definition" do
+        expect_no_offenses(<<~RUBY)
+          describe UserComponent do
+            let(:component) { UserComponent.new("hello") }
+
+            context "with a stub" do
+              let(:component) { double("component") }
+
+              it "formats the title" do
+                expect(component).to be_present
+              end
+            end
+          end
+        RUBY
+      end
+    end
+  end
+
+  context "when the described class is not a ViewComponent" do
+    let(:project_index) do
+      build_index(
+        "file:///month.rb" => <<~RUBY
+          class Month
+          end
+        RUBY
+      )
+    end
+
+    it "does not register an offense" do
+      expect_no_offenses(<<~RUBY)
+        describe Month do
+          it "formats the month" do
+            month = described_class.new
+            expect(month.active?).to be(true)
+          end
+        end
+      RUBY
+    end
   end
 end

data/verification/flowbite_rubocop_config.yml

@@ -0,0 +1,12 @@
+AllCops:
+  UseProjectIndex: true
+  ProjectIndexIncludesGems: true
+  Exclude:
+    - demo/**/*
+
+ViewComponent/ComponentSuffix:
+  Enabled: false
+
+ViewComponent/MissingPreview:
+  PreviewPaths:
+    - demo/test/components/previews

data/verification/libraries.yml

@@ -9,3 +9,7 @@ govuk:
 polaris:
   tarball_url: https://github.com/baoagency/polaris_view_components/archive/refs/heads/main.tar.gz
   display_name: baoagency/polaris_view_components
+
+flowbite:
+  tarball_url: https://github.com/substancelab/flowbite-components/archive/refs/heads/main.tar.gz
+  display_name: substancelab/flowbite-components

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rubocop-view_component
 version: !ruby/object:Gem::Version
-  version: 0.7.0
+  version: 0.8.0
 platform: ruby
 authors:
 - Andy Waite
@@ -102,12 +102,13 @@ extra_rdoc_files: []
 files:
 - ".rspec"
 - ".rubocop.yml"
+- AGENTS.md
 - Appraisals
-- CLAUDE.md
 - LICENSE.txt
 - README.md
 - Rakefile
 - config/default.yml
+- docs/rubydex.md
 - gemfiles/rubocop_minimum.gemfile
 - gemfiles/rubocop_minimum.gemfile.lock
 - lib/rubocop-view_component.rb
@@ -118,6 +119,8 @@ files:
 - lib/rubocop/cop/view_component/prefer_composition.rb
 - lib/rubocop/cop/view_component/prefer_private_methods.rb
 - lib/rubocop/cop/view_component/prefer_slots.rb
+- lib/rubocop/cop/view_component/rspec_scope.rb
+- lib/rubocop/cop/view_component/supplementary_index.rb
 - lib/rubocop/cop/view_component/template_analyzer.rb
 - lib/rubocop/cop/view_component/test_rendered_output.rb
 - lib/rubocop/cop/view_component_cops.rb
@@ -125,6 +128,7 @@ files:
 - lib/rubocop/view_component/plugin.rb
 - lib/rubocop/view_component/version.rb
 - script/verify
+- spec/expected_flowbite_failures.yml
 - spec/expected_govuk_failures.yml
 - spec/expected_polaris_failures.yml
 - spec/expected_primer_failures.yml
@@ -140,6 +144,7 @@ files:
 - spec/rubocop/cop/view_component/test_rendered_output_spec.rb
 - spec/spec_helper.rb
 - spec/support/project_index_helpers.rb
+- verification/flowbite_rubocop_config.yml
 - verification/govuk_rubocop_config.yml
 - verification/libraries.yml
 - verification/polaris_rubocop_config.yml
@@ -166,7 +171,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.16
+rubygems_version: 4.0.20
 specification_version: 4
 summary: RuboCop extension for ViewComponent best practices
 test_files: []