tests passing

lib/liquid/tags/render.rb

@@ -69,15 +69,16 @@ module Liquid
       # Inline snippets take precedence over external snippets
       if (inline_snippet = context.registers[:inline_snippet][template_name])
         inner_context = context.new_isolated_subcontext
-        # binding.irb
+
         snippet_body = inline_snippet[:body]
         snippet_args = inline_snippet[:args]
-
         # Validate and set the arguments in the inner context
         @attributes.each do |key, value|
-          if snippet_args.include?(key)
+          unless snippet_args.include?(key)
-            inner_context[key] = context.evaluate(value)
+            raise Liquid::ArgumentError, "Invalid argument `#{key}` for snippet `#{template_name}`"
           end
+
+          inner_context[key] = context.evaluate(value)
         end
 
         return output << snippet_body.render(inner_context)

test/integration/tags/snippet_test.rb

@@ -104,6 +104,31 @@ class SnippetTest < Minitest::Test
     assert_template_result(expected, template)
   end
 
+  def test_render_inline_snippets_using_same_argument_name
+    template = <<~LIQUID.strip
+      {% snippet "input" |type| %}
+      <input type="{{ type }}" />
+      {% endsnippet %}
+
+      {% snippet "inputs" |type, value| %}
+        <input type="{{ type }}" value="{{ value }}" />
+      {% endsnippet %}
+
+      {%- render "input", type: "text" -%}
+      {%- render "inputs", type: "password", value: "pass" -%}
+    LIQUID
+    expected = <<~OUTPUT
+
+
+
+      <input type="text" />
+
+        <input type="password" value="pass" />
+    OUTPUT
+
+    assert_template_result(expected, template)
+  end
+
   def test_render_inline_snippet_empty_string_when_missing_argument
     template = <<~LIQUID.strip
       {% snippet "input" |type| %}
@@ -145,19 +170,19 @@ class SnippetTest < Minitest::Test
       {% snippet "input" |type| %}
       <input type="{{ type }}" />
       {% endsnippet %}
-
+      {% snippet "no_leak" %}
-      {% snippet "banner"%}
+      <input type="{{ type }}" />
-        {{ type }}
       {% endsnippet %}
 
       {%- render "input", type: "text" -%}
-      {%- render "banner" -%}
+      {%- render "no_leak" -%}
     LIQUID
-    expected = <<~OUTPUT.strip
+    expected = <<~OUTPUT
+
 
       <input type="text" />
 
-
+      <input type="" />
     OUTPUT
 
     assert_template_result(expected, template)