Taking another direction and jumping straight into the Server Development and doing the connect message first
the returned remaining length is 2
Close the client connection if a CONNECT message is not sent in a reasonable time frame
Close the connection if the protocol name does not equal MQTT
Refactoring to encapsulate the logic for each of the command packets
Adding read index back in.  This is going to change but for now this is what I have
Major refactor  to the server and now have a ConnectPacketHandler.  Also got rid of the readIndex thing for now
fixed the failing tests and removed functions which are no longer required.  They were created because I was too hasty with the spec implementation
Made a ConnAckMessageParser
Fixed all the tests and got this grunting again