Skip to content

Commit e1ceed8

Browse files
authored
Merge pull request #2758 from apple/revert-2757-pr_reenable_testurlsession
Revert "TestURLSession: Re-enable tests"
2 parents bc6fb15 + e6028ed commit e1ceed8

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

Tests/Foundation/Tests/TestURLSession.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1742,7 +1742,7 @@ class TestURLSession: LoopbackServerTest {
17421742
("test_simpleUploadWithDelegate", test_simpleUploadWithDelegate),
17431743
("test_requestWithEmptyBody", test_requestWithEmptyBody),
17441744
("test_requestWithNonEmptyBody", test_requestWithNonEmptyBody),
1745-
/* ⚠️ */ ("test_concurrentRequests", testExpectedToFail(test_concurrentRequests, "Fails about 4% of the time")),
1745+
("test_concurrentRequests", test_concurrentRequests),
17461746
("test_disableCookiesStorage", test_disableCookiesStorage),
17471747
("test_cookiesStorage", test_cookiesStorage),
17481748
("test_cookieStorageForEphemeralConfiguration", test_cookieStorageForEphemeralConfiguration),
@@ -1756,7 +1756,7 @@ class TestURLSession: LoopbackServerTest {
17561756
("test_checkErrorTypeAfterInvalidateAndCancel", test_checkErrorTypeAfterInvalidateAndCancel),
17571757
("test_taskCountAfterInvalidateAndCancel", test_taskCountAfterInvalidateAndCancel),
17581758
("test_sessionDelegateAfterInvalidateAndCancel", test_sessionDelegateAfterInvalidateAndCancel),
1759-
/* ⚠️ */ ("test_getAllTasks", testExpectedToFail(test_getAllTasks, "This test causes later ones to crash")),
1759+
("test_getAllTasks", test_getAllTasks),
17601760
("test_getTasksWithCompletion", test_getTasksWithCompletion),
17611761
/* ⚠️ */ ("test_invalidResumeDataForDownloadTask",
17621762
/* ⚠️ */ testExpectedToFail(test_invalidResumeDataForDownloadTask, "This test crashes nondeterministically: https://bugs.swift.org/browse/SR-11353")),

Tests/Foundation/main.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ var allTestCases = [
9595
testCase(TestURLRequest.allTests),
9696
testCase(TestURLResponse.allTests),
9797
testCase(TestHTTPURLResponse.allTests),
98-
testCase(TestURLSession.allTests),
98+
testCaseExpectedToFail(TestURLSession.allTests, "URLSession test interdependencies are causing intermittent CI issues."),
9999
testCase(TestNSUUID.allTests),
100100
testCase(TestUUID.allTests),
101101
testCase(TestNSValue.allTests),

0 commit comments

Comments
 (0)