From b0f6b552ed0690e3fad98e9771bbd6a47fbc2e37 Mon Sep 17 00:00:00 2001 From: Paul Buetow Date: Thu, 26 Mar 2026 09:46:26 +0200 Subject: fix: remove unused variable assignments in test files - internal/repl/repl_test.go: Remove unused state variable assignments - internal/rpn/rpn_test.go: Remove unused result/err variable assignments These changes address golangci-lint 'ineffectual assignment' warnings. --- internal/rpn/rpn_test.go | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) (limited to 'internal/rpn') diff --git a/internal/rpn/rpn_test.go b/internal/rpn/rpn_test.go index 8611cf8..0822125 100644 --- a/internal/rpn/rpn_test.go +++ b/internal/rpn/rpn_test.go @@ -698,8 +698,7 @@ func TestIncrementalAssignmentRPN(t *testing.T) { } // Now try x =: - should assign 2 to variable x - result, err = r.ParseAndEvaluate("x =:") - if err != nil { + if _, err := r.ParseAndEvaluate("x =:"); err != nil { t.Fatalf("Assignment failed: %v", err) } @@ -830,13 +829,12 @@ func TestBoundIdentifierPushesValue(t *testing.T) { r := NewRPN(vars) // First bind x to 5 - result, err := r.ParseAndEvaluate("x = 5") - if err != nil { + if _, err := r.ParseAndEvaluate("x = 5"); err != nil { t.Fatalf("ParseAndEvaluate('x = 5') returned error: %v", err) } // Now use x (bound) - should push value - result, err = r.ParseAndEvaluate("x") + result, err := r.ParseAndEvaluate("x") if err != nil { t.Fatalf("ParseAndEvaluate('x') after binding returned error: %v", err) } -- cgit v1.2.3