Fixed bug where pendingPromiseDeallocated event was incorrectly repor… (

mxcl#972)

* Fixed bug where pendingPromiseDeallocated event was incorrectly reported as a waitOnMainThread event. Added test which should have caught this bug in the first place.

* Kick Travis CI

Sources/Resolver.swift

@@ -8,7 +8,7 @@ public final class Resolver<T> {
 
     deinit {
         if case .pending = box.inspect() {
-            conf.logHandler(.waitOnMainThread)
+            conf.logHandler(.pendingPromiseDeallocated)
         }
     }
 }

Tests/CorePromise/LoggingTests.swift

@@ -153,5 +153,27 @@ class LoggingTests: XCTestCase {
         XCTAssertEqual(logOutput!, "waitOnMainThread")
     }
 
+    // Verify pendingPromiseDeallocated is logged
+    func testPendingPromiseDeallocatedIsLogged() {
+
+        var logOutput: String? = nil
+        conf.logHandler = { event in
+            switch event {
+            case .waitOnMainThread:
+                logOutput = "\(event)"
+            case .pendingPromiseDeallocated:
+                logOutput = "\(event)"
+            case .cauterized:
+                // Using an enum with associated value does not convert to a string properly in
+                // earlier versions of swift
+                logOutput = "cauterized"
+            }
+        }
+        do {
+            let _ = Promise<Int>.pending()
+        }
+        XCTAssertEqual ("pendingPromiseDeallocated", logOutput!)
+    }
+
     //TODO Verify pending promise deallocation is logged
 }