Skip to content

Continued swift testing adoption #1350

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 12 commits into from
Jun 12, 2025

Conversation

jmschonfeld
Copy link
Contributor

These are cherry-picked parts of #908. In particular, I pulled in each file that applied cleanly or almost cleanly (i.e. files that haven't really added any or more than 1-2 new tests since my original porting work last Fall) for tests for FoundationEssentials. Since we still have some XCTests present I moved the existing checkHashable-related functions aside (prefixed with XCT for XCTest) and added the new checkHashable functions based on swift-testing. This covers most of the FoundationEssentials tests, but leaves out some suites that have changed a lot (ex. String/Data/FileManager) and some suites that are new.

@jmschonfeld
Copy link
Contributor Author

@swift-ci please test

@jmschonfeld
Copy link
Contributor Author

@swift-ci please test

@jmschonfeld jmschonfeld merged commit ff0459d into swiftlang:main Jun 12, 2025
14 of 15 checks passed
@jmschonfeld jmschonfeld deleted the continued-swift-testing branch June 12, 2025 23:36
chloe-yeo pushed a commit to chloe-yeo/swift-foundation that referenced this pull request Jun 16, 2025
* Convert buffer view tests to swift-testing

* Convert BuiltInUnicodeScalarSet tests

* Convert ErrorTests

* Convert LockedStateTests

* Move aside old XCTest-based equatable/hashable utilities and define new swift-testing utilities

* Convert UUID tests

* Convert DateInterval tests

* Convert date tests

* Convert SortComparator tests

* Convert decimal tests

* Convert IndexPath tests

* Fix build failures
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.

2 participants