diff options
| author | Paul Buetow <paul@buetow.org> | 2026-03-26 09:46:26 +0200 |
|---|---|---|
| committer | Paul Buetow <paul@buetow.org> | 2026-03-26 09:46:26 +0200 |
| commit | b0f6b552ed0690e3fad98e9771bbd6a47fbc2e37 (patch) | |
| tree | b76762e50de49436cbe856d5714e640786abf059 /internal/rpn | |
| parent | 0bf0381232019fbda419447166d021d8e0852317 (diff) | |
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.
Diffstat (limited to 'internal/rpn')
| -rw-r--r-- | internal/rpn/rpn_test.go | 8 |
1 files changed, 3 insertions, 5 deletions
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) } |
