Don't deallocate channel twice

Author: yim-lee

Sources/AsyncDNSResolver/Channel.swift

@@ -27,7 +27,6 @@ class AresChannel {
     private let semaphore = DispatchSemaphore(value: 1)
 
     deinit {
-        self.pointer?.deallocate()
         ares_destroy(self._ares_channel)
         ares_library_cleanup()
     }

Tests/AsyncDNSResolverTests/AsyncDNSResolverTests.swift

@@ -126,19 +126,18 @@ final class AsyncDNSResolverTests: XCTestCase {
         self.untilFinishOrTimeout(timeout: .seconds(3))
     }
 
-    // FIXME: this crashes tests
-//    func test_queryTXT() throws {
-//        self.resolver.query(.TXT(name: "apple.com") { result in
-//            switch result {
-//            case .success(let r):
-//                print("TXT records: \(r)")
-//            case .failure(let error):
-//                print("Error: \(error)")
-//            }
-//        })
-//
-//        self.untilFinishOrTimeout(timeout: .seconds(3))
-//    }
+    func test_queryTXT() throws {
+        self.resolver.query(.TXT(name: "apple.com") { result in
+            switch result {
+            case .success(let r):
+                print("TXT records: \(r)")
+            case .failure(let error):
+                print("Error: \(error)")
+            }
+        })
+
+        self.untilFinishOrTimeout(timeout: .seconds(3))
+    }
 
     func test_querySRV() throws {
         self.resolver.query(.SRV(name: "_caldavs._tcp.google.com") { result in
@@ -153,19 +152,18 @@ final class AsyncDNSResolverTests: XCTestCase {
         self.untilFinishOrTimeout(timeout: .seconds(3))
     }
 
-    // FIXME: this crashes tests
-//    func test_queryNAPTR() throws {
-//        self.resolver.query(.NAPTR(name: "apple.com") { result in
-//            switch result {
-//            case .success(let r):
-//                print("NAPTR records: \(r)")
-//            case .failure(let e):
-//                print("Error: \(e)")
-//            }
-//        })
-//
-//        self.untilFinishOrTimeout(timeout: .seconds(3))
-//    }
+    func test_queryNAPTR() throws {
+        self.resolver.query(.NAPTR(name: "apple.com") { result in
+            switch result {
+            case .success(let r):
+                print("NAPTR records: \(r)")
+            case .failure(let e):
+                print("Error: \(e)")
+            }
+        })
+
+        self.untilFinishOrTimeout(timeout: .seconds(3))
+    }
 
     func test_concurrency() throws {
         func run(times: Int = 100, timeout: DispatchTimeInterval = .seconds(5), _ query: (_ index: Int) -> Void) {
@@ -210,23 +208,21 @@ final class AsyncDNSResolverTests: XCTestCase {
                 print("[MX] Run #\(i) result: \(result)")
             })
         }
-        // FIXME: this crashes tests
-//        run { i in
-//            self.resolver.query(.TXT(name: "apple.com") { result in
-//                print("[TXT] Run #\(i) result: \(result)")
-//            })
-//        }
+        run { i in
+            self.resolver.query(.TXT(name: "apple.com") { result in
+                print("[TXT] Run #\(i) result: \(result)")
+            })
+        }
         run { i in
             self.resolver.query(.SRV(name: "_caldavs._tcp.google.com") { result in
                 print("[SRV] Run #\(i) result: \(result)")
             })
         }
-        // FIXME: this crashes tests
-//        run { i in
-//            self.resolver.query(.NAPTR(name: "apple.com") { result in
-//                print("[NAPTR] Run #\(i) result: \(result)")
-//            })
-//        }
+        run { i in
+            self.resolver.query(.NAPTR(name: "apple.com") { result in
+                print("[NAPTR] Run #\(i) result: \(result)")
+            })
+        }
     }
 
     /// Waits until channel has no more pending queries or times out.