Add plural finder methods to shoes-spec (buttons, paras, drawables, etc.)

- Add pluralize_dsl_name helper for proper English pluralization
- Singular finders (button, para, etc.) still raise if not exactly one match
- Plural finders (buttons, paras, etc.) return arrays of all matches
- Add drawables() as plural of drawable()
- Works in both niente and webview display services
- Add tests for plural finders in lacci/test/

Author: Nick Schwaderer

lacci/lib/scarpe/niente/shoes_spec.rb

@@ -36,9 +36,21 @@ def self.run_shoes_spec_test_code(code, class_name: nil, test_name: nil)
 end
 
 class Niente::ShoesSpecTest < Minitest::Test
+  # Helper to pluralize DSL names
+  def self.pluralize_dsl_name(name)
+    # Handle common English pluralization rules
+    case name
+    when /box$/
+      name + "es"
+    else
+      name + "s"
+    end
+  end
+
   Shoes::Drawable.drawable_classes.each do |drawable_class|
     finder_name = drawable_class.dsl_name
 
+    # Singular finder - raises if not exactly one match
     define_method(finder_name) do |*args|
       drawables = Shoes::App.find_drawables_by(drawable_class, *args)
 
@@ -47,13 +59,27 @@ class Niente::ShoesSpecTest < Minitest::Test
 
       Niente::ShoesSpecProxy.new(drawables[0])
     end
+
+    # Plural finder - returns array of all matches
+    plural_name = pluralize_dsl_name(finder_name)
+    define_method(plural_name) do |*args|
+      drawables = Shoes::App.find_drawables_by(drawable_class, *args)
+      drawables.map { |d| Niente::ShoesSpecProxy.new(d) }
+    end
   end
 
+  # Singular drawable finder
   def drawable(*specs)
-    drawables = Shoes::App.find_drawables_by(*specs)
-    raise Shoes::Errors::MultipleDrawablesFoundError, "Found more than one #{finder_name} matching #{args.inspect}!" if drawables.size > 1
-    raise Shoes::Errors::NoDrawablesFoundError, "Found no #{finder_name} matching #{args.inspect}!" if drawables.empty?
-    Niente::ShoesSpecProxy.new(drawables[0])
+    found = Shoes::App.find_drawables_by(*specs)
+    raise Shoes::Errors::MultipleDrawablesFoundError, "Found more than one drawable matching #{specs.inspect}!" if found.size > 1
+    raise Shoes::Errors::NoDrawablesFoundError, "Found no drawable matching #{specs.inspect}!" if found.empty?
+    Niente::ShoesSpecProxy.new(found[0])
+  end
+
+  # Plural drawables finder - returns array of all matches
+  def drawables(*specs)
+    found = Shoes::App.find_drawables_by(*specs)
+    found.map { |d| Niente::ShoesSpecProxy.new(d) }
   end
 end
 

lacci/test/test_plural_finders.rb

@@ -0,0 +1,82 @@
+# frozen_string_literal: true
+
+require_relative "test_helper"
+
+class TestPluralFinders < NienteTest
+  def test_buttons_returns_all_buttons
+    run_test_niente_code(<<~SHOES_APP, app_test_code: <<~SHOES_SPEC)
+      Shoes.app do
+        @b1 = button "Button One"
+        @b2 = button "Button Two"
+        @b3 = button "Button Three"
+      end
+    SHOES_APP
+      all_buttons = buttons
+      assert_equal 3, all_buttons.size
+      assert all_buttons.all? { |b| b.is_a?(Niente::ShoesSpecProxy) }
+    SHOES_SPEC
+  end
+
+  def test_paras_returns_all_paras
+    run_test_niente_code(<<~SHOES_APP, app_test_code: <<~SHOES_SPEC)
+      Shoes.app do
+        @p1 = para "Para One"
+        @p2 = para "Para Two"
+      end
+    SHOES_APP
+      all_paras = paras
+      assert_equal 2, all_paras.size
+    SHOES_SPEC
+  end
+
+  def test_plural_finder_with_variable_filter
+    run_test_niente_code(<<~SHOES_APP, app_test_code: <<~SHOES_SPEC)
+      Shoes.app do
+        @b1 = button "Button One"
+        @b2 = button "Other"
+      end
+    SHOES_APP
+      # Filter by variable name - should return just that one
+      filtered = buttons("@b1")
+      assert_equal 1, filtered.size
+      assert_equal "Button One", filtered[0].text
+    SHOES_SPEC
+  end
+
+  def test_plural_finder_empty_result_bad_variable
+    run_test_niente_code(<<~SHOES_APP, app_test_code: <<~SHOES_SPEC)
+      Shoes.app do
+        @b1 = button "Button One"
+      end
+    SHOES_APP
+      # Non-existent variable returns empty array
+      no_match = buttons("@nonexistent")
+      assert_equal 0, no_match.size
+      assert_equal [], no_match
+    SHOES_SPEC
+  end
+
+  def test_drawables_plural
+    run_test_niente_code(<<~SHOES_APP, app_test_code: <<~SHOES_SPEC)
+      Shoes.app do
+        @b1 = button "One"
+        @b2 = button "Two"
+        @p1 = para "Text"
+      end
+    SHOES_APP
+      all_buttons = drawables(Shoes::Button)
+      assert_equal 2, all_buttons.size
+    SHOES_SPEC
+  end
+
+  def test_singular_still_works
+    run_test_niente_code(<<~SHOES_APP, app_test_code: <<~SHOES_SPEC)
+      Shoes.app do
+        @b1 = button "Only One"
+      end
+    SHOES_APP
+      b = button("@b1")
+      assert_equal "Only One", b.text
+    SHOES_SPEC
+  end
+end

lib/scarpe/shoes_spec.rb

@@ -254,9 +254,21 @@ def respond_to_missing?(method_name, include_private = false)
 module Scarpe::ShoesSpecTest
   include Scarpe::Test::HTMLAssertions
 
+  # Helper to pluralize DSL names
+  def self.pluralize_dsl_name(name)
+    # Handle common English pluralization rules
+    case name
+    when /box$/
+      name + "es"
+    else
+      name + "s"
+    end
+  end
+
   Shoes::Drawable.drawable_classes.each do |drawable_class|
     finder_name = drawable_class.dsl_name
 
+    # Singular finder - raises if not exactly one match
     define_method(finder_name) do |*args|
       # Scarpe-Webview only supports a single Shoes::App instance
       app = Shoes.APPS[0]
@@ -267,13 +279,31 @@ module Scarpe::ShoesSpecTest
 
       Scarpe::ShoesSpecProxy.new(drawables[0])
     end
+
+    # Plural finder - returns array of all matches
+    plural_name = pluralize_dsl_name(finder_name)
+    define_method(plural_name) do |*args|
+      app = Shoes.APPS[0]
+      drawables = app.find_drawables_by(drawable_class, *args)
+      drawables.map { |d| Scarpe::ShoesSpecProxy.new(d) }
+    end
   end
+
+  # Singular drawable finder
   def drawable(*specs)
-    drawables = app.find_drawables_by(*specs)
-    raise Scarpe::MultipleDrawablesFoundError, "Found more than one #{finder_name} matching #{args.inspect}!" if drawables.size > 1
-    raise Scarpe::NoDrawablesFoundError, "Found no #{finder_name} matching #{args.inspect}!" if drawables.empty?
+    app = Shoes.APPS[0]
+    found = app.find_drawables_by(*specs)
+    raise Shoes::Errors::MultipleDrawablesFoundError, "Found more than one drawable matching #{specs.inspect}!" if found.size > 1
+    raise Shoes::Errors::NoDrawablesFoundError, "Found no drawable matching #{specs.inspect}!" if found.empty?
+
+    Scarpe::ShoesSpecProxy.new(found[0])
+  end
 
-    Scarpe::ShoesSpecProxy.new(drawables[0])
+  # Plural drawables finder - returns array of all matches
+  def drawables(*specs)
+    app = Shoes.APPS[0]
+    found = app.find_drawables_by(*specs)
+    found.map { |d| Scarpe::ShoesSpecProxy.new(d) }
   end
 
   def catscradle_dsl(&block)