filters + more tests

Author: pierre-b

integration/detailed_forloop_test.go

@@ -0,0 +1,64 @@
+package integration
+
+import (
+	"testing"
+
+	"github.com/Notifuse/liquidgo/liquid"
+	"github.com/Notifuse/liquidgo/liquid/tags"
+)
+
+func TestForloopLastDetailed(t *testing.T) {
+	tests := []struct {
+		name     string
+		template string
+		want     string
+	}{
+		{
+			name:     "simple forloop.last",
+			template: `{% for item in items %}{{ item }}{% if forloop.last %}!{% endif %}{% endfor %}`,
+			want:     "123!",
+		},
+		{
+			name:     "forloop.last with trailing comma",
+			template: `{% for item in items %}{{ item }},{% endfor %}`,
+			want:     "1,2,3,",
+		},
+		{
+			name:     "forloop.last controlling comma",
+			template: `{% for item in items %}{{ item }}{% unless forloop.last %},{% endunless %}{% endfor %}`,
+			want:     "1,2,3",
+		},
+		{
+			name:     "forloop.last with content then comma",
+			template: `{% for item in items %}{{ item }}{% if forloop.last %} last{% endif %},{% endfor %}`,
+			want:     "1,2,3 last,",
+		},
+		{
+			name:     "forloop.last debug",
+			template: `{% for item in items %}{{ forloop.last }}{% endfor %}`,
+			want:     "falsefalsetrue",
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			env := liquid.NewEnvironment()
+			tags.RegisterStandardTags(env)
+			tmpl := liquid.NewTemplate(&liquid.TemplateOptions{Environment: env})
+			
+			err := tmpl.Parse(tt.template, nil)
+			if err != nil {
+				t.Fatalf("Parse failed: %v", err)
+			}
+
+			vars := map[string]interface{}{"items": []interface{}{1, 2, 3}}
+			got := tmpl.Render(vars, nil)
+
+			if got != tt.want {
+				t.Errorf("FAILED\nWant: %q\nGot:  %q", tt.want, got)
+			} else {
+				t.Logf("PASS: %q", got)
+			}
+		})
+	}
+}

integration/test_array_last.go

@@ -0,0 +1,62 @@
+package integration
+
+import (
+	"testing"
+
+	"github.com/Notifuse/liquidgo/liquid"
+	"github.com/Notifuse/liquidgo/liquid/tags"
+)
+
+func TestArrayCommandMethods(t *testing.T) {
+	env := liquid.NewEnvironment()
+	tags.RegisterStandardTags(env)
+	
+	tests := []struct {
+		name     string
+		template string
+		vars     map[string]interface{}
+		want     string
+	}{
+		{
+			name:     "array.last",
+			template: `{{ items.last }}`,
+			vars:     map[string]interface{}{"items": []interface{}{1, 2, 3}},
+			want:     "3",
+		},
+		{
+			name:     "array.first",
+			template: `{{ items.first }}`,
+			vars:     map[string]interface{}{"items": []interface{}{1, 2, 3}},
+			want:     "1",
+		},
+		{
+			name:     "array.size",
+			template: `{{ items.size }}`,
+			vars:     map[string]interface{}{"items": []interface{}{1, 2, 3}},
+			want:     "3",
+		},
+		{
+			name:     "string.size",
+			template: `{{ str.size }}`,
+			vars:     map[string]interface{}{"str": "hello"},
+			want:     "5",
+		},
+	}
+	
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			tmpl := liquid.NewTemplate(&liquid.TemplateOptions{Environment: env})
+			err := tmpl.Parse(tt.template, nil)
+			if err != nil {
+				t.Fatalf("Parse failed: %v", err)
+			}
+			
+			got := tmpl.Render(tt.vars, nil)
+			if got != tt.want {
+				t.Errorf("FAILED\nTemplate: %s\nWant: %q\nGot:  %q", tt.template, tt.want, got)
+			} else {
+				t.Logf("PASS: %q", got)
+			}
+		})
+	}
+}

integration/trace_forloop_rendering_test.go

@@ -0,0 +1,49 @@
+package integration
+
+import (
+	"testing"
+
+	"github.com/Notifuse/liquidgo/liquid"
+	"github.com/Notifuse/liquidgo/liquid/tags"
+)
+
+func TestTraceForloopRendering(t *testing.T) {
+	env := liquid.NewEnvironment()
+	tags.RegisterStandardTags(env)
+	
+	// Test direct variable lookup in context
+	ctx := liquid.BuildContext(liquid.ContextConfig{Environment: env})
+	
+	// Create a forloop drop and add it to context
+	drop := liquid.NewForloopDrop("test", 3, nil)
+	ctx.Set("forloop", drop)
+	
+	// Test 1: Direct lookup
+	forloopVar := ctx.FindVariable("forloop", false)
+	t.Logf("FindVariable('forloop') = %#v (type: %T)", forloopVar, forloopVar)
+	
+	// Test 2: Evaluate a VariableLookup
+	vl := liquid.VariableLookupParse("forloop.last", liquid.NewStringScanner(""), nil)
+	result := vl.Evaluate(ctx)
+	t.Logf("VariableLookup.Evaluate('forloop.last') = %#v (type: %T)", result, result)
+	
+	// Test 3: ToS on the result
+	resultStr := liquid.ToS(result, nil)
+	t.Logf("ToS(result) = %q", resultStr)
+	
+	// Test 4: Full template rendering
+	template := `{{ forloop.last }}`
+	tmpl := liquid.NewTemplate(&liquid.TemplateOptions{Environment: env})
+	err := tmpl.Parse(template, nil)
+	if err != nil {
+		t.Fatalf("Parse failed: %v", err)
+	}
+	
+	vars := map[string]interface{}{"forloop": drop}
+	got := tmpl.Render(vars, nil)
+	t.Logf("Template render result = %q", got)
+	
+	if got == "" {
+		t.Error("Template rendered empty string instead of 'false'")
+	}
+}