Change Details

`TestNode()` already takes in the majority of CServiceResult. Passing in the whole struct would allow removal of the return value and will be useful in the future for more complex banning logic.

`TestNode()` already takes in the majority of CServiceResult and only has one call site. In addition, having it located in `seeder/bitcoin.*` when the call site is in `seeder/main.cpp` make banning logic more complex and difficult to follow.

`TestNode()` already takes in the majority of CServiceResult. Passing and only has one call site. in the whole struct would allow removal ofIn addition, having it located in `seeder/bitcoin.*` when the return value and will becall site is in `seeder/main.cpp` useful in the future formake banning logic more complex banning logicand difficult to follow.