Skip to content

Add a README for the xctest_checker tool in the functional test suite #70

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 11, 2016

Conversation

briancroom
Copy link
Contributor

I was trying to make some minor changes to xctest_checker but didn't know how to get started with the tests, so I documented what I learned.

It's quite possible @modocache will come along and tell me my example invocation is all wrong. That'd be fine by me! 😆

It includes unit tests which can be run with the following command in this directory:

```sh
PYTHONPATH=. python tests/test_compare.py
Copy link
Contributor

Choose a reason for hiding this comment

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

There's a better way:

$ python -m unittest discover -s Tests/Functional/xctest_checker

This works with both relative and absolute paths.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the tip!

I've updated the file, though I excluded the -s option because the context states that you'd be invoking the command from the correct directory already.

@modocache
Copy link
Contributor

Looks great, just the one comment. Thanks, @briancroom!

@briancroom briancroom force-pushed the xctest_checker-readme branch from 66fb40d to 1a8a8e8 Compare March 11, 2016 11:55
modocache added a commit that referenced this pull request Mar 11, 2016
Add a README for the xctest_checker tool in the functional test suite
@modocache modocache merged commit 5a564c5 into swiftlang:master Mar 11, 2016
@modocache
Copy link
Contributor

Thanks! 👋

@briancroom briancroom deleted the xctest_checker-readme branch March 11, 2016 20:16
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.

2 participants