-
Notifications
You must be signed in to change notification settings - Fork 561
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
Don't require a test case subclass #79
Conversation
tookSnapshot.fulfill() | ||
} | ||
let result = XCTWaiter.wait(for: [tookSnapshot], timeout: timeout) | ||
switch result { |
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.
We need to manually fail based on the XCTWaiter
result now.
tookSnapshot.fulfill() | ||
} | ||
#if os(Linux) | ||
self.waitForExpectations(timeout: timeout) |
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.
I'm hoping this doesn't fail. Linux got the additional APIs in recent snapshots...
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.
Dang, merged less than a week ago: apple/swift-corelibs-xctest#228
Guess we can't merge and support Linux unless we lock Linux to a recent snapshot.
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.
We decided to support both subclassless asserts (for iOS) and subclass asserts (for Linux) until Linux catches up with the XCTest support.
Bless you both @stephencelis @mbrandonw, very excited to give this a test! |
Fixes #71.
Hoping that a serial queue makes things reliable.