Fix memory leak in tests

Author: 0xTim

Tests/SteamPressTests/BlogTests/DisabledBlogTagTests.swift

@@ -5,10 +5,15 @@ class DisabledBlogTagTests: XCTestCase {
     func testDisabledBlogTagsPath() throws {
         var testWorld = try TestWorld.create(enableTagPages: false)
         _ = try testWorld.createTag("Engineering")
-        let tagResponse = try testWorld.getResponse(to: "/tags/Engineering")
-        let allTagsResponse = try testWorld.getResponse(to: "/tags")
+        var tagResponse: Response? = try testWorld.getResponse(to: "/tags/Engineering")
+        var allTagsResponse: Response? = try testWorld.getResponse(to: "/tags")
 
-        XCTAssertEqual(.notFound, tagResponse.http.status)
-        XCTAssertEqual(.notFound, allTagsResponse.http.status)
+        XCTAssertEqual(.notFound, tagResponse?.http.status)
+        XCTAssertEqual(.notFound, allTagsResponse?.http.status)
+        
+        tagResponse = nil
+        allTagsResponse = nil
+        
+        XCTAssertNoThrow(try testWorld.tryAsHardAsWeCanToShutdownApplication())
     }
 }

Tests/SteamPressTests/Helpers/TestWorld.swift

@@ -31,6 +31,10 @@ struct TestWorld {
 
     // To work around Vapor 3 dodgy lifecycle mess
     mutating func tryAsHardAsWeCanToShutdownApplication() throws {
+        if let threadPool = try context.app?.make(BlockingIOThreadPool.self) {
+            try threadPool.syncShutdownGracefully()
+        }
+        
         struct ApplicationDidNotGoAway: Error {
             var description: String
         }