Removed client.connect calls, unit tests now run.

    I previously thought that using client.connect fucks up the shutdown signal
    passing, as I was getting tons of TCP sockets hanging in CLOSE_WAIT state,
    plus the associated bin/dbwriter and bin/browserid procs were also hanging.

    Oddly I can't repro this behavior today. At any rate, removing the
    redundant client.connect calls simplifies the code and so we'll call this a
    net win overall. Grr sockety heisenbug.