lookbook_visual_tester 0.5.1 → 0.5.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 865c3eeb875ed336338493ccbd620fcfeba15be5410c7d605cf73d94e79e3756
-  data.tar.gz: '081022c43fe54c60a95be903181d956223bb34800700a44a45baf3c69d7cc4a8'
+  metadata.gz: 61d8ae85e5897ffcb0dd53078b753a02cfc118e9cf5990ebbf27b40c5778bc4a
+  data.tar.gz: afcd716247590085bf7dbb99c36e146f45a350f349c1c2fda5dad9d7c8441192
 SHA512:
-  metadata.gz: 4991161dee4f699f25517991abf504cc92ac8c229e4e61a985da90195d0238815423c9129f51cc5abf5ad0818f0bacfdf65a4aea087e93ad309fbe69ed3cbd9d
-  data.tar.gz: 9e3d4f144424de70fc3754b5939d65ee3f327cab3ad2dcecc6dd1923adc77d88db834bfaacbf059deb051fe66546248346f01d90f20593d25cea01e0f1449860
+  metadata.gz: e22889eacb81f638d7ba65ab1e8682d0e1ec7e22903f67b950912ef5f670eb3ec13dca31ba21e52ef6cac3e73ce6da11179a7ec35e37eb44b5392e4f99ac6004
+  data.tar.gz: e5f18347bff022b59a7b6504c2a2cd0b64e279b587b9dc92f4a0001f7056cc63e8affa58167799cac8ef2fa434718d973ea6fbd375f54a6697240f5b1bc050bf

data/CHANGELOG.md

@@ -1,4 +1,15 @@
 # Changelog
+## [0.5.3] - 2026-01-04
+
+### Fixed
+- **Deep Check**: `rake lookbook:deep_check` now correctly detects and fails when a preview returns `nil` (implicit rendering) but the corresponding template is missing (`ViewComponent::MissingPreviewTemplateError`).
+- **Deep Check**: `rake lookbook:deep_check` now exits with status 1 if any errors or failures are detected.
+
+## [0.5.2] - 2026-01-04
+
+### Fixed
+- **Preview Checker**: Fixed a `NoMethodError` crash in `PreviewChecker` when encountering template-only Lookbook scenarios (scenarios without a corresponding method in the preview class).
+
 ## [0.5.1] - 2026-01-03
 
 ### Fixed

data/README.md

@@ -29,7 +29,7 @@ sudo apt-get install imagemagick xclip
 Add to your application's Gemfile:
 ```ruby
 group :test do
-  gem 'lookbook_visual_tester'
+  gem 'lookbook_visual_tester', '~> 0.5.3'
 end
 ```
 
@@ -153,7 +153,7 @@ bundle exec rake lookbook:check
 ```
 
 #### Deep Check (Render)
-effectively renders all previews to catch runtime and template errors.
+effectively renders all previews to catch runtime errors, missing templates, and other failures. Exits with status 1 if any errors are found.
 ```bash
 bundle exec rake lookbook:deep_check
 ```

data/lib/lookbook_visual_tester/preview_checker.rb

@@ -111,6 +111,11 @@ module LookbookVisualTester
 
       begin
         preview_instance = preview_class.new
+        unless preview_instance.respond_to?(example_name)
+          return CheckResult.new(preview_name: preview.name, example_name: example_name,
+                                 status: :passed)
+        end
+
         preview_instance.public_send(example_name)
 
         # We don't render, just verify we can call it.
@@ -126,8 +131,21 @@ module LookbookVisualTester
       example_name = example.name
 
       begin
-        preview_instance = preview_class.new
-        result = preview_instance.public_send(example_name)
+        # Use ViewComponent::Preview logic to get the component to render
+        # This handles the case where the method returns nil (implicit template)
+        # and returns a component that renders the template.
+        if preview_class.respond_to?(:preview_example)
+          result = preview_class.preview_example(example_name)
+          puts "DEBUG: preview_example('#{example_name}') returned: #{result.inspect} (class: #{result.class})"
+        else
+          # Fallback for older VC or non-standard setups
+          preview_instance = preview_class.new
+          unless preview_instance.respond_to?(example_name)
+            return CheckResult.new(preview_name: preview.name, example_name: example_name,
+                                   status: :passed)
+          end
+          result = preview_instance.public_send(example_name)
+        end
 
         if result.respond_to?(:render_in)
           # Mock current_user/pundit if needed on the component itself if possible
@@ -147,6 +165,36 @@ module LookbookVisualTester
           result.render_in(view_context)
         elsif result.is_a?(String)
           # Rendered string, good.
+        elsif result.nil?
+          # If result is nil, it implies an implicit template rendering.
+          # We need to verify that the template exists.
+
+          # Try to find the template file based on conventions
+          # Convention: preview_file_dir/preview_file_name/example_name.html.erb
+
+          # We need the preview file path. Lookbook::Preview might not expose it easily
+          # but the class might have source_location.
+
+          method = preview_class.instance_method(example_name)
+          source_file = method.source_location&.first
+
+          found_template = false
+          if source_file
+            dir = File.dirname(source_file)
+            filename = File.basename(source_file, '.rb')
+            template_dir = File.join(dir, filename)
+
+            extensions = ['.html.erb', '.html.haml', '.html.slim']
+            path = File.join(template_dir, "#{example_name}")
+
+            found_template = extensions.any? { |ext| File.exist?("#{path}#{ext}") }
+          end
+
+          unless found_template
+            raise ViewComponent::MissingPreviewTemplateError.new("Preview #{example_name} returned nil and no template found at #{path}.* (checked erb, haml, slim)") if defined?(ViewComponent::MissingPreviewTemplateError)
+
+            raise "Preview returned nil and no template found at #{path}.*"
+          end
         end
 
         CheckResult.new(preview_name: preview.name, example_name: example_name, status: :passed)

data/lib/lookbook_visual_tester/runner.rb

@@ -179,6 +179,8 @@ module LookbookVisualTester
           if result[:error] == 'Baseline not found'
             # First run, maybe auto-approve or just report
             puts '  [NEW] Baseline not found. Saved current as potential baseline.'
+            FileUtils.mkdir_p(File.dirname(baseline_path))
+            FileUtils.cp(current_path, baseline_path)
             status = :new
           else
             puts "  [ERROR] #{result[:error]}"

data/lib/lookbook_visual_tester/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module LookbookVisualTester
-  VERSION = '0.5.1'
+  VERSION = '0.5.3'
 end

data/lib/tasks/lookbook_visual_tester.rake

@@ -21,6 +21,7 @@ namespace :lookbook do
     checker = LookbookVisualTester::PreviewChecker.new
     results = checker.deep_check
     LookbookVisualTester::CheckReporter.start([results])
+    exit 1 if results.any? { |r| r.status == :failed }
   end
 
   desc 'Find components missing previews'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lookbook_visual_tester
 version: !ruby/object:Gem::Version
-  version: 0.5.1
+  version: 0.5.3
 platform: ruby
 authors:
 - Murilo Vasconcelos
@@ -9,6 +9,20 @@ bindir: exe
 cert_chain: []
 date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: benchmark
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: chunky_png
   requirement: !ruby/object:Gem::Requirement
@@ -173,7 +187,6 @@ files:
 - lib/lookbook_visual_tester/configuration.rb
 - lib/lookbook_visual_tester/driver.rb
 - lib/lookbook_visual_tester/drivers/ferrum_driver.rb
-- lib/lookbook_visual_tester/image_comparator.rb
 - lib/lookbook_visual_tester/json_output_handler.rb
 - lib/lookbook_visual_tester/preview_checker.rb
 - lib/lookbook_visual_tester/railtie.rb

data/lib/lookbook_visual_tester/image_comparator.rb

@@ -1,49 +0,0 @@
-require 'lookbook_visual_tester/configuration'
-
-module LookbookVisualTester
-  class ImageComparator
-    attr_reader :baseline_dir, :current_dir, :diff_dir
-
-    def initialize
-      @baseline_dir = LookbookVisualTester.config.baseline_dir
-      @current_dir = LookbookVisualTester.config.current_dir
-      @diff_dir = LookbookVisualTester.config.diff_dir
-    end
-
-    def compare(scenario_run)
-      filename = scenario_run.filename
-
-      current_path = current_dir.join(filename)
-      baseline_path = baseline_dir.join(filename)
-      diff_path = diff_dir.join(scenario_run.diff_filename)
-
-      if baseline_path.exist?
-        baseline_image = MiniMagick::Image.open(baseline_path)
-        current_image  = MiniMagick::Image.open(current_path)
-
-        unless baseline_image.dimensions == current_image.dimensions
-          puts '    Image dimensions do not match. Skipping comparison.'
-          return
-        end
-
-        begin
-          compare_command = "compare -metric AE \"#{baseline_path}\" \"#{current_path}\" \"#{diff_path}\" 2>&1"
-          result = `#{compare_command}`
-          distortion = result.strip.to_i
-
-          if distortion > 0
-            puts "    Differences found! Diff image saved to #{diff_path}"
-          else
-            puts '    No differences detected.'
-            File.delete(diff_path) if diff_path.exist?
-          end
-        rescue StandardError => e
-          puts "    Error comparing images: #{e.message}"
-        end
-      else
-        FileUtils.cp(current_path, baseline_path)
-        puts "    Baseline image created at #{baseline_path}"
-      end
-    end
-  end
-end