Skip to content

[SR-5247] Add the new spellings for the *WithAccuracy assertion functions #193

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

Conversation

briancroom
Copy link
Contributor

This is to bring Corelibs XCTest in line with changes made in the main XCTest Swift overlay for Xcode 9 (see here).

XCTAssertEqualWithAccuracy and XCTAssertNotEqualWithAccuracy are now deprecated in favor of XCTAssertEqual(_:_:accuracy:file:line:) and XCTAssertNotEqual(_:_:accuracy:file:line:).

This fixes an inconsistency where XCTAssertEqualWithAccuracy had accuracy: as a named parameter, but XCTAssertNotEqualWithAccuracy had it unnamed.

…ions

This is to bring Corelibs XCTest in line with changes made in the
main XCTest Swift overlay for Xcode 9.

XCTAssertEqualWithAccuracy and XCTAssertNotEqualWithAccuracy are now
deprecated in favor of XCTAssertEqual(_:_:accuracy:file:line:) and
XCTAssertNotEqual(_:_:accuracy:file:line:)

This fixes an inconsistency where XCTAssertEqualWithAccuracy had
accuracy: as a named parameter, but XCTAssertNotEqualWithAccuracy
had it unnamed.
@briancroom briancroom self-assigned this Jun 17, 2017
@briancroom
Copy link
Contributor Author

@swift-ci please test

Copy link
Contributor

@modocache modocache left a comment

Choose a reason for hiding this comment

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

LGTM! I like the new spellings, too.

@alblue
Copy link
Contributor

alblue commented Jun 21, 2017

The OSX build failed with:

directory/Users/buildnode/jenkins/workspace/swift-corelibs-xctest-PR-osx/swift-corelibs-xctest
<unknown>:0: error: unknown argument: '-index-store-path'
Command /Users/buildnode/jenkins/workspace/swift-corelibs-xctest-PR-osx/Ninja-DebugAssert/swift-macosx-x86_64/bin/swiftc failed with exit code 1

I'm not sure if this is to do with the updated version of Swift used on the build server - I'll try kicking it off again.

@swift-ci please test macOS

@briancroom
Copy link
Contributor Author

@swift-ci please test macOS

@briancroom
Copy link
Contributor Author

@swift-ci please test

1 similar comment
@alblue
Copy link
Contributor

alblue commented Jun 22, 2017

@swift-ci please test

@briancroom
Copy link
Contributor Author

Alright, it passed!

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