-
Notifications
You must be signed in to change notification settings - Fork 50
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
Support for Xcode 11 #32
Comments
hi @thibault-ml thanks for bringing this up. I will take a look at this once I get some of these JSON files. |
Thanks! Google already parses them: And XCResultKit too: https://github.com/davidahouse/XCResultKit |
@thibault-ml Looking into! Will hopefully have something for you soon 😊 |
@thibault-ml Hey! Do you happen to have a xcresult file you could send me? 😊 It would be good to have a legit file that I can test on 🙏 |
@joshdholtz I'll have a look if we can, not sure 100% I'm at liberty to just yet. Will have a look :-) |
@thibault-ml If you don't want to send but are able to run my PR that would also be 💯 😊 The PR works for me on my simple test use cases (I don't have any huge test suites). I got a few fixes to do on it and then will merge. |
@joshdholtz Here's a Your PR fails with:
|
@juliangrosshauser Thanks for sending that over! I had a feeling my PR would need some tweaking for something a little more complex 🙃 I will make sure it works with you sent over ❤️ |
@juliangrosshauser Got it solved! Thanks sending this 🙌 |
@juliangrosshauser Alright, PR should be good to go again if you'd like to test when you get some time. I was able to get a successful output from the xcresults that you sent me 👌 I made sure all the data is getting pulled out safely now so that shouldn't happen anymore 😇 |
@joshdholtz Great work! Thank you so much, really appreciate it! I tested your PR and it works flawlessly now. The only problematic It contains 1008 test results. 1007 tests succeed, but the |
Oh boy 😱 I found the reason. The "producingTarget" : {
"_type" : {
"_name" : "String"
},
"_value" : "PSPDFKitTests"
},
"testCaseName" : {
"_type" : {
"_name" : "String"
},
"_value" : "-[PSPDFProcessorTests testStyledStringToPDFAsFileURL]"
} |
@juliangrosshauser Thanks for testing again! Here is a commit for fixing that 😊 3f88286 |
@joshdholtz I tested your newest changes with Objective-C and Swift failures. Looks like everything works now! 🎉 Thank you again! |
@juliangrosshauser Thanks so much for feeding me files needed to make this change 😊 I will merge the PR and get a new release out today at some point 💪🚀 |
Thank you guys for working on this! Is the correct way to use this to specify the
|
@vrutberg I think that should work! But you should also be able to do... scan(
result_bundle: true,
output_directory: "./" # optional
) |
Xcode 11 uses a new format for
xctest
(version 3), which doesn't have aTestSummary.plist
file, and uses Zstandard-compressed "database-like" files.Xcode 11 Release Notes state:
Basically, to obtain results from an Xcode 11
xcresult
, one would have to do the following:The text was updated successfully, but these errors were encountered: