Change expected onError behaviour

Java-WebSocket will call onError when an exception occurs, then onClose.
Changing the pusher-java-client behaviour so that it doesn't change the
connection state to DISCONNECTED onError, but waits for onClose.

src/main/java/com/pusher/client/connection/websocket/WebSocketConnection.java

@@ -269,7 +269,9 @@ public void onError(final Exception ex) {
 
 		Factory.getEventQueue().execute(new Runnable() {
 			public void run() {
-				updateState(ConnectionState.DISCONNECTED);
+				// Do not change connection state as Java_WebSocket will also call onClose.
+				// See: https://github.com/leggetter/pusher-java-client/issues/8#issuecomment-16128590
+				//updateState(ConnectionState.DISCONNECTED);
 				sendErrorToAllListeners("An exception was thrown by the websocket",
 						null, ex);
 			}

src/test/java/com/pusher/client/connection/websocket/WebSocketConnectionTest.java

@@ -265,17 +265,14 @@ public void testOnCloseCallbackDoesNotCallListenerIfItIsNotBoundToDisconnectedEv
 	}
 
 	@Test
-	public void testOnErrorCallbackUpdatesStateToDisconnectedAndRaisesErrorEvent() {
+	public void testOnErrorCallbackRaisesErrorEvent() {
 		connection.connect();
 		verify(mockEventListener).onConnectionStateChange(
 				new ConnectionStateChange(ConnectionState.DISCONNECTED,
 						ConnectionState.CONNECTING));
 
 		Exception e = new Exception();
 		connection.onError(e);
-		verify(mockEventListener).onConnectionStateChange(
-				new ConnectionStateChange(ConnectionState.CONNECTING,
-						ConnectionState.DISCONNECTED));
 		verify(mockEventListener).onError(
 				"An exception was thrown by the websocket", null, e);
 	}