Fix golangci-lint issues: remove unused code

Author: pierre-b

liquid/drop_test.go

@@ -852,7 +852,6 @@ func TestGetInvokableMethodsEdgeCases(t *testing.T) {
 func TestInvokeDropOnMethodReturningNothing(t *testing.T) {
 	type testDropNoReturn struct {
 		Drop
-		called bool
 	}
 
 	// Add a method that returns nothing
@@ -1013,7 +1012,6 @@ func TestInvokeDropOnWithMethodReturningEmptyResults(t *testing.T) {
 	// Create a drop with a method that returns nothing
 	type testDropVoidMethod struct {
 		Drop
-		called bool
 	}
 
 	// Add a method with no return values using a function that matches the pattern
@@ -1188,18 +1186,6 @@ func TestInvokeDropOldNonStructValue(t *testing.T) {
 	}
 }
 
-// testDropPointerMethod has methods on pointer that won't be found on value
-type testDropPointerMethod struct {
-	Drop
-	TestField string
-}
-
-// PointerMethod is defined on *testDropPointerMethod, not testDropPointerMethod
-// This means v.MethodByName on the VALUE won't find it after v.Elem()
-func (t *testDropPointerMethod) PointerMethod() string {
-	return "pointer_method"
-}
-
 // TestInvokeDropOldPointerMethodNotFound tests InvokeDropOld when method is on pointer
 func TestInvokeDropOldPointerMethodNotFound(t *testing.T) {
 	// InvokeDropOld is a method on Drop, so when called, 'd' is *Drop, not the outer struct

liquid/tags/render_test.go

@@ -843,15 +843,6 @@ func TestRenderTagWithIterableObjectImplementation(t *testing.T) {
 	}
 }
 
-// mockPartialLoader is a mock file system that returns non-template objects
-type mockPartialLoader struct {
-	liquid.FileSystem
-}
-
-func (m *mockPartialLoader) ReadTemplateFile(templatePath string) (string, error) {
-	return "content", nil
-}
-
 // TestRenderTagPartialNotTemplate tests when LoadPartial returns non-template
 func TestRenderTagPartialNotTemplate(t *testing.T) {
 	env := liquid.NewEnvironment()