Skip to content

Commit 1ffc36b

Browse files
pftgclaude
andcommitted
refactor: inline ScreenshotNamerDSL into DSL module
8-line module in a separate file with pure delegation. Move screenshot_section, screenshot_group, and screenshot_namer directly into DSL. Delete the file. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent a3359dc commit 1ffc36b

4 files changed

Lines changed: 13 additions & 38 deletions

File tree

docs/simplest_comparator.rb

Whitespace-only changes.

lib/capybara/screenshot/diff/screenshot_namer_dsl.rb

Lines changed: 0 additions & 35 deletions
This file was deleted.

lib/capybara_screenshot_diff.rb

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55
require "capybara/screenshot/diff/utils"
66
require "capybara/screenshot/diff/image_compare"
77
require "capybara_screenshot_diff/snap_manager"
8-
require "capybara/screenshot/diff/screenshot_namer_dsl"
98
require "capybara/screenshot/diff/screenshoter"
109
require "capybara/screenshot/diff/reporters/default"
1110

lib/capybara_screenshot_diff/dsl.rb

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
require "capybara/screenshot/diff/drivers"
55
require "capybara/screenshot/diff/image_compare"
66
require "capybara/screenshot/diff/screenshot_matcher"
7-
require "capybara/screenshot/diff/screenshot_namer_dsl"
7+
require "capybara_screenshot_diff/screenshot_namer"
88
require "capybara_screenshot_diff/screenshot_assertion"
99

1010
module CapybaraScreenshotDiff
@@ -16,7 +16,14 @@ module CapybaraScreenshotDiff
1616
# to provide screenshot comparison capabilities.
1717
module DSL
1818
include Capybara::DSL
19-
include Capybara::Screenshot::Diff::ScreenshotNamerDSL
19+
20+
def screenshot_section(name)
21+
screenshot_namer.section = name
22+
end
23+
24+
def screenshot_group(name)
25+
screenshot_namer.group = name
26+
end
2027

2128
# Takes a screenshot and optionally compares it against a baseline image.
2229
#
@@ -87,5 +94,9 @@ def build_screenshot_assertion(name, options, skip_stack_frames: 0)
8794
.new(name, options)
8895
.build_screenshot_assertion(skip_stack_frames: skip_stack_frames + 1)
8996
end
97+
98+
def screenshot_namer
99+
CapybaraScreenshotDiff.screenshot_namer
100+
end
90101
end
91102
end

0 commit comments

Comments
 (0)