commit 58f791207ea2dda67f76b6cac97dd57daf50a35d
author Ian Partridge <i.partridge@uk.ibm.com> Mon Nov 13 10:52:34 2017 +0000
committer GitHub <noreply@github.com> Mon Nov 13 10:52:34 2017 +0000
tree 9c75d78afd383aa4085a96c24ff9164e1712cea0
parent cc8ec6a57e66e98c1eb59c8bc50c3aacb942dcaf
parent d99556275b40fac9e4321386acbf14b0701a8ff1

Merge pull request #1316 from spevans/pr_cookie_tests

TestHTTPCookieStorage: Remove order dependancy of tests

TestFoundation/TestHTTPCookieStorage.swift

diff --git a/TestFoundation/TestHTTPCookieStorage.swift b/TestFoundation/TestHTTPCookieStorage.swift
index 757f64b..d355075 100644
--- a/TestFoundation/TestHTTPCookieStorage.swift
+++ b/TestFoundation/TestHTTPCookieStorage.swift
@@ -29,13 +29,18 @@
             ("test_BasicStorageAndRetrieval", test_BasicStorageAndRetrieval),
             ("test_deleteCookie", test_deleteCookie),
             ("test_removeCookies", test_removeCookies),
-            ("test_setCookiesForURL", test_setCookiesForURL),
-            ("test_getCookiesForURL", test_getCookiesForURL),
-            ("test_setCookiesForURLWithMainDocumentURL", test_setCookiesForURLWithMainDocumentURL),
+            ("test_cookiesForURL", test_cookiesForURL),
+            ("test_cookiesForURLWithMainDocumentURL", test_cookiesForURLWithMainDocumentURL),
             ("test_cookieInXDGSpecPath", test_cookieInXDGSpecPath),
         ]
     }
 
+    override func setUp() {
+        // Delete any cookies in the storage
+        getStorage(for: .shared).removeCookies(since: Date(timeIntervalSince1970: 0))
+        getStorage(for: .groupContainer("test")).removeCookies(since: Date(timeIntervalSince1970: 0))
+    }
+
     func test_sharedCookieStorageAccessedFromMultipleThreads() {
         let q = DispatchQueue.global()
         let syncQ = DispatchQueue(label: "TestHTTPCookieStorage.syncQ")
@@ -72,17 +77,15 @@
         removeCookies(with: .groupContainer("test"))
     }
 
-    func test_setCookiesForURL() {
+    func test_cookiesForURL() {
         setCookiesForURL(with: .shared)
-        setCookiesForURL(with: .groupContainer("test"))
-    }
-
-    func test_getCookiesForURL() {
         getCookiesForURL(with: .shared)
+
+        setCookiesForURL(with: .groupContainer("test"))
         getCookiesForURL(with: .groupContainer("test"))
     }
 
-    func test_setCookiesForURLWithMainDocumentURL() {
+    func test_cookiesForURLWithMainDocumentURL() {
         setCookiesForURLWithMainDocumentURL(with: .shared)
         setCookiesForURLWithMainDocumentURL(with: .groupContainer("test"))
     }
@@ -159,9 +162,11 @@
             .expires: Date(timeIntervalSince1970: Date().timeIntervalSince1970 + 1000)
             ])!
         storage.setCookie(simpleCookie)
+        storage.setCookie(simpleCookie2)
         XCTAssertEqual(storage.cookies!.count, 2)
 
         storage.deleteCookie(simpleCookie)
+        XCTAssertEqual(storage.cookies!.count, 1)
         storage.deleteCookie(simpleCookie2)
         XCTAssertEqual(storage.cookies!.count, 0)
     }
@@ -225,7 +230,7 @@
             .domain: "swift.org",
         ])!
         storage.setCookies([simpleCookie], for: url, mainDocumentURL: mainUrl)
-        XCTAssertEqual(storage.cookies(for: url!)!.count, 2)
+        XCTAssertEqual(storage.cookies(for: url!)!.count, 1)
 
         let url1 = URL(string: "https://dt.swift.org/downloads")
         let simpleCookie1 = HTTPCookie(properties: [
@@ -249,7 +254,7 @@
         ])!
         let storage = HTTPCookieStorage.shared
         storage.setCookie(testCookie)
-        XCTAssertEqual(storage.cookies!.count, 3)
+        XCTAssertEqual(storage.cookies!.count, 1)
         var destPath: String
         let bundlePath = Bundle.main.bundlePath
         var bundleName = "/" + bundlePath.components(separatedBy: "/").last!