autoreleasepool for AF specs

pcantrell · pcantrell · commit 1ce382d990b8 · 2018-08-28T15:12:40.000-05:00
diff --git a/Cartfile.private b/Cartfile.private
@@ -1,10 +1,10 @@
 # Optional integrations
 
-github "Alamofire/Alamofire" ~> 4.0
+github "Alamofire/Alamofire"
 # github "ReactiveCocoa/ReactiveCocoa" "master"  # add to Cartfile if/when it has tests
 
 # Testing
 
-github "Quick/Quick"
+github "pcantrell/Quick" "around-each"
 github "Quick/Nimble"
 github "pcantrell/Nocilla" "siesta"                     # fork adds delay() / go(), nullability annotations
diff --git a/Cartfile.resolved b/Cartfile.resolved
@@ -1,4 +1,4 @@
-github "Alamofire/Alamofire" "4.7.2"
-github "Quick/Nimble" "v7.1.2"
-github "Quick/Quick" "v1.3.0"
+github "Alamofire/Alamofire" "4.7.3"
+github "Quick/Nimble" "v7.1.3"
 github "pcantrell/Nocilla" "bd7ec7caa0576f08c00bbbf993a9204f93be16e3"
+github "pcantrell/Quick" "30b8c3f5f8b8b9eda6b6e75d1d8bcdef56908997"
diff --git a/Tests/Functional/ResourceSpecBase.swift b/Tests/Functional/ResourceSpecBase.swift
@@ -120,6 +120,12 @@ class ResourceSpecBase: SiestaSpec
         //
         // NB: This must come _after_ the specVars above, which use afterEach to clear the service and resource.
 
+        aroundEach
+            {
+            example in
+            autoreleasepool { example() }  // Alamofire relies on autorelease, so each spec needs its own pool for leak checking
+            }
+
         afterEach
             {
             awaitObserverCleanup()
@@ -134,8 +140,10 @@ class ResourceSpecBase: SiestaSpec
 
         // Run the actual specs
 
-        context("")  // Separate context for service and resource above, so they’re cleaned up before leak check below
-            { resourceSpec(service, resource) }
+        context("")  // Make specVars above run in a separate context so their afterEach cleans up _before_ the leak check
+            {
+            resourceSpec(service, resource)
+            }
         }
 
     var baseURL: String
