Skip to content
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

Jump to bottom

[SR-12575] assertForOverFulfill not working on Linux XCTest #351

Closed
ahoppen opened this issue Apr 11, 2020 · 3 comments
Closed

[SR-12575] assertForOverFulfill not working on Linux XCTest #351

ahoppen opened this issue Apr 11, 2020 · 3 comments
Assignees
@ahoppen
Labels
bug XCTest

Comments

@ahoppen
Copy link

ahoppen commented Apr 11, 2020

Previous ID SR-12575
Radar rdar://problem/62202297
Original Reporter @ahoppen
Type Bug
Status Resolved
Resolution Done

Attachment: Download

Additional Detail from JIRA
Votes 0
Component/s XCTest
Labels Bug
Assignee @ahoppen
Priority Medium

md5: 8b3b42fc1b73078c5df7f1b5508ff3cf

Issue Description:

Setting assertForOverFulfill on XCTestExpectation on Linux seems to have no effect. The following test case will fail with the following error message:

let myExpectation = expectation(description: "will be fulfilled twice")
myExpectation.assertForOverFulfill = false
myExpectation.fulfill()
myExpectation.fulfill() // error: homeTests.testExample : API violation - multiple calls made to XCTestExpectation.fulfill() for will be fulfilled twice.
self.waitForExpectations(timeout: 1, handler: nil)

To reproduce, mount the sample project in the Docker container and run swift test

docker run -it -v ~/path/to/sample/project:/home swiftlang/swift:nightly-bionic
cd /home
swift test
@beccadax
Copy link
Contributor

@swift-ci create

@stmontgomery
Copy link
Contributor

#320

@stmontgomery
Copy link
Contributor

Fixed in e24d665

@swift-ci swift-ci transferred this issue from apple/swift-issues Apr 25, 2022
@shahmishal shahmishal transferred this issue from apple/swift May 9, 2022
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ahoppen ahoppen

Labels
bug XCTest
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@stmontgomery @beccadax @ahoppen