summaryrefslogtreecommitdiff
path: root/internal/rpn
diff options
context:
space:
mode:
authorPaul Buetow <paul@buetow.org>2026-03-26 09:46:26 +0200
committerPaul Buetow <paul@buetow.org>2026-03-26 09:46:26 +0200
commitb0f6b552ed0690e3fad98e9771bbd6a47fbc2e37 (patch)
treeb76762e50de49436cbe856d5714e640786abf059 /internal/rpn
parent0bf0381232019fbda419447166d021d8e0852317 (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.go8
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)
}