Fix broken testcases

Author: cbaker6

Sources/ParseSwift/LiveQuery/ParseLiveQuery.swift

@@ -211,11 +211,6 @@ public final class ParseLiveQuery: NSObject {
         close(useDedicatedQueue: false)
         authenticationDelegate = nil
         receiveDelegate = nil
-        if task != nil {
-            URLSession.liveQuery.delegates.removeValue(forKey: task)
-        } else {
-            task = nil
-        }
     }
 }
 
@@ -518,8 +513,8 @@ extension ParseLiveQuery {
                         if error == nil {
                             self.isConnecting = true
                         }
+                        completion(error)
                     }
-                    completion(nil)
                 } catch {
                     completion(error)
                 }

Tests/ParseSwiftTests/ParseLiveQueryTests.swift

@@ -316,14 +316,15 @@ class ParseLiveQueryTests: XCTestCase {
         }
         client.isSocketEstablished = true // Socket neets to be true
         client.isConnecting = true
-        client.attempts = 50
         client.isConnected = true
+        client.attempts = 5
         client.clientId = "yolo"
+        client.isDisconnectedByUser = false
 
         XCTAssertEqual(client.isSocketEstablished, true)
         XCTAssertEqual(client.isConnecting, false)
         XCTAssertEqual(client.clientId, "yolo")
-        XCTAssertEqual(client.attempts, 1)
+        XCTAssertEqual(client.attempts, 5)
     }
 
     func testDisconnectedState() throws {
@@ -384,7 +385,7 @@ class ParseLiveQueryTests: XCTestCase {
         XCTAssertEqual(client.clientId, "yolo")
         client.close()
 
-        XCTAssertEqual(client.isSocketEstablished, true)
+        XCTAssertEqual(client.isSocketEstablished, false)
         XCTAssertEqual(client.isConnected, false)
         XCTAssertEqual(client.isConnecting, false)
         XCTAssertNil(client.clientId)