-
Notifications
You must be signed in to change notification settings - Fork 86
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
Update to Xcode 8.0 / Swift 2.3 #95
Conversation
…into swift-2.3 # Please enter a commit message to explain why this merge is necessary, # especially if it merges an updated upstream into a topic branch. # # Lines starting with '#' will be ignored, and an empty message aborts # the commit.
@SusannProszak instead of disabling tests, what do you think about forcing them to run on iOS 9? My PR changed this (#94). |
describe("func storeAccessToken(accessToken: OAuthAccessToken?)") { | ||
|
||
// Since there is a bug with writing to the keychain within the iOS10 simulator we had to | ||
// disbale some test until the bug is fixed by apple. Radar: https://openradar.appspot.com/27844971 |
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.
Typo: disable
Regarding the Keychain bug: One more thing we might think of is adding a test application and move the failing test into the tests of the test application (kind of meta). |
Updated:
Mac
is nowmacOS
Regarding the failing tests: there is currently a bug with the latest iOS 10 simulators and the keychain. We agreed on disable the tests until the bug is fixed by apple.