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

Disable test to unblock swift PR testing. #1466

Merged
merged 1 commit into from
Mar 10, 2018

Conversation

gottesmm
Copy link
Contributor

It looks like the order in which things are getting printed was flipped. So we are getting this error in the swift build:

20:53:48 TestFoundation/TestNSDictionary.swift:49: error: TestNSDictionary.test_description : XCTAssertEqual failed: ("{
20:53:48 foo = bar;
20:53:48 baz = qux;
20:53:48 }") is not equal to ("{
20:53:48 baz = qux;
20:53:48 foo = bar;
20:53:48 }") -

I would just fix this, but I don't work often in this codebase so I think
disabling is the right way to go.

https://bugs.swift.org/browse/SR-7166

It looks like the order in which things are getting printed was flipped. So we are getting this error in the swift build:

20:53:48 TestFoundation/TestNSDictionary.swift:49: error: TestNSDictionary.test_description : XCTAssertEqual failed: ("{
20:53:48     foo = bar;
20:53:48     baz = qux;
20:53:48 }") is not equal to ("{
20:53:48     baz = qux;
20:53:48     foo = bar;
20:53:48 }") -

I would just fix this, but I don't work often in this codebase so I think
disabling is the right way to go.

https://bugs.swift.org/browse/SR-7166
@gottesmm
Copy link
Contributor Author

@gottesmm
Copy link
Contributor Author

@swift-ci test and merge

@parkera
Copy link
Contributor

parkera commented Mar 12, 2018

This seems related to #1468. cc @lorentey

@lorentey
Copy link
Member

@parkera #1468 updates this test and the NSCache test disabled in #1465 to support randomly seeded hashing, and reenables both. Sorry for not linking them together! (And also for the surprise regressions.)

@lorentey
Copy link
Member

Both of these tests were broken when swiftlang/swift#14913 landed on Swift master.

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.

4 participants