-
-
Notifications
You must be signed in to change notification settings - Fork 12
Adopt Swift Testing #41
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
Conversation
b750536
to
757d7c3
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #41 +/- ##
==========================================
Coverage 100.00% 100.00%
==========================================
Files 9 11 +2
Lines 666 1060 +394
==========================================
+ Hits 666 1060 +394
|
757d7c3
to
7393888
Compare
3fb6c7b
to
f9163f9
Compare
5c584c3
to
fab5c47
Compare
19eb1ef
to
17f0730
Compare
|
||
import Testing | ||
|
||
public actor Expectation { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is quite similar to the Expectation
in the dfed/swift-testing-expectation repo. I figure it is best to keep this repo having zero dependencies, and a little copy/paste never hurt anyone.
Migrates from XCTest to Swift Testing. I ended up having to create an
Expectation
type to mimic what XCTest has, since confirmations require theconfirmation
to be confirmed before the confirmation closure exits, which doesn't work terribly well in a highly async environment.