Skip to content

Conversation

@jmschonfeld
Copy link
Contributor

This is the final step in our swift-testing migration. Now that all tests use swift-testing we can:

  • Remove all remaining imports of XCTest and XCTest-based helper functions
  • Remove typealias declarations used to disambiguate between FoundationEssentials and Foundation now that the ambiguity is gone (due to the lack of an imported Foundation)
  • Move the TestSupport module to the tests folder and remove its dependencies on FoundationEssentials/FoundationInternationalization to remove FoundationEssentialsTests's dependency on FoundationInternationalization
  • Fix or move a few tests in FoundationEssentials that depended on FoundationInternationalization and add a new test to validate that FoundationInternationalization upcalls are not enabled in FoundationEssentialsTests

@jmschonfeld
Copy link
Contributor Author

@swift-ci please test

test(.yearForWeekOfYear, date, expectedStart: nil, end: nil)
}

@Test func testDateInterval_DST() {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This test actually already existed in the GregorianCalendarInternationalizationTests suite, so it's just removed from here since it requires a non-GMT timezone

@@ -0,0 +1,266 @@
//===----------------------------------------------------------------------===//
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file was just moved from the old TestSupport folder, but the XCTest helpers in it were removed

@jmschonfeld
Copy link
Contributor Author

@swift-ci please test

@jmschonfeld jmschonfeld merged commit 348ba1c into swiftlang:main Jul 7, 2025
16 checks passed
@jmschonfeld jmschonfeld deleted the swift-testing/cleanup branch July 7, 2025 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants