Client.setNetwork() does not propagate address book

[steven-sheehy]

Similar to #1995, the address book support is still not fully propagated in some scenarios. When adjusting the network, the certificate info inside the NodeAddressBook parameter is not passed through to Node.setAddressBookEntry() so TLS does not work unless we also specify Client.setVerifyCertificates(false), which is less secure. Consider the below:

var client = Client.forMainnet();
var network = client.getNetwork();
client.setNetwork(Map.of("1.2.3.4:50211", "0.0.3"));
client.setNetwork(network);

So essentially the problem is you start with a valid address book and N nodes, then call setNetwork() with some subset of N and later add back any of those nodes, the nodes added back do not retain the address book information despite it being present in the Network class. This scenario may seem unconventional but mirror node does this at the start of its acceptance tests to validate each specific endpoints before test execution.

The fix is to follow the invariant then whenever a Node is created to lookup and set its corresponding address book entry. Probably Network.createNodeFromNetworkEntry() just needs to call node.setAddressBookEntry() if addressBook is present.

[0xivanov]

Fixed in #2322