Skip to content

[StdLib] Refactor DictionaryKeyValueTypes test input data #968

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
merged 1 commit into from
Mar 5, 2016
Merged

[StdLib] Refactor DictionaryKeyValueTypes test input data #968

merged 1 commit into from
Mar 5, 2016

Conversation

ezephir
Copy link

@ezephir ezephir commented Jan 14, 2016

Split the input file into a pure Swift and Objective-C interop files in
anticipation of refactoring tests to remove XFAILs on Linux.

Tested on OS X 10.10 and Ubuntu 14.04.

@gparker42
Copy link
Contributor

This change splits resetLeaksOfDictionaryKeysValues() and expectNoLeaksOfDictionaryKeysValues() into ObjC and non-ObjC functions. That is fine, but it also needs to add calls to the new ObjC functions to the existing tests.

@gparker42 gparker42 self-assigned this Jan 20, 2016
@ezephir
Copy link
Author

ezephir commented Jan 20, 2016

Thanks. I've updated the PR with those calls.

@swift-ci
Copy link
Contributor

Can one of the admins verify this patch?

@tkremenek
Copy link
Member

@swift-ci Please test

@gottesmm
Copy link
Contributor

The linux test here failed because of a check out failure (not sure why). Rerunning the linux platform.

@swift-ci Please test Linux Platform

@dabrahams
Copy link
Contributor

@ezephir Can you update the PR so it merges cleanly and we can retest it? Thanks.

Split the input file into a pure Swift and Objective-C interop files in
anticipation of refactoring tests to remove XFAILs on Linux.
@ezephir
Copy link
Author

ezephir commented Mar 1, 2016

@dabrahams Updated! Thanks.

@dabrahams
Copy link
Contributor

@swift-ci Please test

@gottesmm
Copy link
Contributor

gottesmm commented Mar 5, 2016

Passes tests... Merging.

gottesmm added a commit that referenced this pull request Mar 5, 2016
[StdLib] Refactor DictionaryKeyValueTypes test input data
@gottesmm gottesmm merged commit 9a5c3ea into swiftlang:master Mar 5, 2016
@gribozavr
Copy link
Contributor

@dabrahams Did you review?

@gottesmm
Copy link
Contributor

gottesmm commented Mar 5, 2016

@gribozavr Sorry if that was premature. I can revert if/when you want me to.

@gribozavr
Copy link
Contributor

I reviewed it, LGTM.

kateinoigakukun pushed a commit to kateinoigakukun/swift that referenced this pull request Sep 7, 2020
…-nightly

Update BASE_URL in install-nightly-toolchain.sh
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.

7 participants