BUGFIX: 'Connect' command tab completion does not list purchased servers

[Nerdpie]

Fixes #1904

Repro steps:

1. Purchase a server
2. Connect to a non-purchased server (e.g. connect n00dles
3. Type connect and press Tab
4. The hostname of the server from step 1 is shown in the tab completion list

Note: Format and lint have been run

[Nerdpie]

... yeah, that tracks that I'd overlook an associated test case.

Let me know if this fix makes sense, and I can adjust the test to match.

Edit: Upon closer inspection, the test is failing because the autocomplete code is now including home, even though that is the current server connected.

One possible fix would be to add

.filter(server => server !== currServ)

into the chain, or to directly add that condition to the existing filter, though it would push the length out a bit.

I'm not sure what approach would be most desirable.

[d0sboots]

I'm really torn about this whole concept.

On one hand, yes, the player-purchased servers are valid targets to connect to, from anywhere on the network.

On the other hand, with the exception of home, almost none of them are ever useful targets to connect to, so adding them to completion would bloat the list of useful possibilities. And it does it for every server in the network, making it harder to read the useful servers.