Skip to content
This repository has been archived by the owner on Jun 12, 2020. It is now read-only.

Adds tests for Codable #12

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Adds tests for Codable #12

wants to merge 6 commits into from

Conversation

simonbs
Copy link

@simonbs simonbs commented Jun 9, 2017

Adds tests for parsing and mapping using Apples JSON decoder and Codable protocol. The tests were run on an iPhone 6S.

README.md Outdated
@@ -217,6 +217,7 @@ JSONCodable | 4.363 sec | 0.510 sec
Unbox | 3.102 sec | 0.372 sec
Decodable | 1.642 sec | 0.215 sec
Marshal | 0.528 sec | 0.096 sec
Codable | 1.233 sec | 0.204 sec
Copy link
Author

Choose a reason for hiding this comment

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

These numbers were obtained from running the tests on an iPhone 6S.

@simonbs
Copy link
Author

simonbs commented Jun 9, 2017

Naturally these numbers are obtained using Xcode 9 beta 1, so it may be worth to run the tests again during and after the beta to ensure that they are up to date.

@dominik-hadl
Copy link

First of all, thanks for the PR! 🎉

It's worth noting that this change also changes the Swift version of these tests to Swift 4. We haven't yet looked into what are the consequences of upgrading our frameworks/repos to Swift 4, although I am pretty sure this repository would be the one least affected. 😄

For now, I'll leave the PR open and either merge it in soon, or wait until there's a release version of Xcode.

Thanks again 👏

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants