diff options
| author | Paul Buetow <paul@buetow.org> | 2026-05-23 20:17:01 +0300 |
|---|---|---|
| committer | Paul Buetow <paul@buetow.org> | 2026-05-23 20:17:01 +0300 |
| commit | d02c4ff623141704489f41088ca861e75a632b87 (patch) | |
| tree | d7de9adc38f6b68df3ad9ab5a29f645d68794dba /internal/repl/concurrent_test.go | |
| parent | 8988ba5e03a61825f644841789b8654c233e1936 (diff) | |
refactor: remove Calculator/RPNCalculator passthrough adapter (KISS)
Delete internal/repl/calculator.go which defined the Calculator
interface and RPNCalculator adapter. Every method delegated directly
to *rpn.RPN with zero transformation — unnecessary indirection.
RPNState now holds *rpn.RPN directly instead of the Calculator
interface. Updated NewRPNState, NewREPL, handleRatCommand, and
RPNHandler.Handle() to access *rpn.RPN directly.
Removes 71 net lines, zero behavioral changes.
Diffstat (limited to 'internal/repl/concurrent_test.go')
| -rw-r--r-- | internal/repl/concurrent_test.go | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/internal/repl/concurrent_test.go b/internal/repl/concurrent_test.go index d4065ff..501b363 100644 --- a/internal/repl/concurrent_test.go +++ b/internal/repl/concurrent_test.go @@ -16,13 +16,12 @@ func TestConcurrentExecutor(t *testing.T) { defer wg.Done() vars := rpn.NewVariables() rpnCalc := rpn.NewRPN(vars) - calculator := NewRPNCalculator(rpnCalc) rpl := &REPL{ ttyChecker: &TTYChecker{}, historyMgr: NewHistoryManager(".gt_history"), signalHandler: NewSignalHandler(), commandChain: NewCommandChain(), - rpnState: &RPNState{vars: vars, calculator: calculator}, + rpnState: &RPNState{vars: vars, rpnCalc: rpnCalc}, } defaultExecutor(rpl, "20% of 150") }(i) @@ -54,13 +53,12 @@ func TestConcurrentRatModeToggle(t *testing.T) { defer wg.Done() vars := rpn.NewVariables() rpnCalc := rpn.NewRPN(vars) - calculator := NewRPNCalculator(rpnCalc) rpl := &REPL{ ttyChecker: &TTYChecker{}, historyMgr: NewHistoryManager(".gt_history"), signalHandler: NewSignalHandler(), commandChain: NewCommandChain(), - rpnState: &RPNState{vars: vars, calculator: calculator}, + rpnState: &RPNState{vars: vars, rpnCalc: rpnCalc}, } defaultExecutor(rpl, "rat toggle") }(i) @@ -77,13 +75,12 @@ func TestConcurrentExecutorAndRPN(t *testing.T) { defer wg.Done() vars := rpn.NewVariables() rpnCalc := rpn.NewRPN(vars) - calculator := NewRPNCalculator(rpnCalc) rpl := &REPL{ ttyChecker: &TTYChecker{}, historyMgr: NewHistoryManager(".gt_history"), signalHandler: NewSignalHandler(), commandChain: NewCommandChain(), - rpnState: &RPNState{vars: vars, calculator: calculator}, + rpnState: &RPNState{vars: vars, rpnCalc: rpnCalc}, } defaultExecutor(rpl, "20% of 150") }(i) |