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

Allow Xcode to highlight failing functional test expectations #72

Merged
merged 1 commit into from Mar 13, 2016

Conversation

briancroom
Copy link
Collaborator

As a part of our continuing attempts to improve the quality of the functional test suite, I was hoping to improve the way that the test failure output integrates with Xcode.

Here, I have amended failure output to include the file name and line number where the // CHECK: line came from. Xcode is able to use this information to mark the line in the editor as containing an error, like so:

screen shot 2016-03-11 at 11 13 37 pm

This makes it easier to diagnose failures in the editor itself, without needing to jump over to the build log to examine the raw output from xctest_checker

…ctional tests.

This allows Xcode to highlight the failing line in the editor.
@mike-ferris
Copy link

Very cool.

@modocache
Copy link
Collaborator

I tried this out locally, and it's really amazing!! 😍

I wonder if we can improve xctest_checker output such that all assertions have a corresponding file and line number. But that's a discussion for another pull request. Thanks, @briancroom!!

modocache added a commit that referenced this pull request Mar 13, 2016
Allow Xcode to highlight failing functional test expectations
@modocache modocache merged commit bd5e3f8 into apple:master Mar 13, 2016
@briancroom briancroom deleted the highlight-test-failures branch March 14, 2016 05:30
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.

None yet

3 participants