As your change only modifies the `checkNodeHasIps` method (and not the check socket logic), perhaps I'd make that method visible for testing and just test that concrete method, with the different inputs, instead of going through the entire socket finder path. That would lead to simpler tests and reduce the indererminism the timeouts/mock executors may introduce.