I guess this is where `shouldClose()` should have been used to initialize `closeChannel`. But still, if your tests pass without it, I'd leave the SPI unchanged for now.