Skip to content

Fix cocoapods check and add unit tests for doctor service #1248

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
Nov 26, 2015

Conversation

rosen-vladimirov
Copy link
Contributor

When cocoapods are not installed on Mac machine, doctor is failing.
Fix this by skipping match check in case pods are not installed.
Add basic unit tests for doctor service.

Add unit test for the failing cocoapods check.

@rosen-vladimirov rosen-vladimirov self-assigned this Nov 25, 2015
@rosen-vladimirov rosen-vladimirov added this to the 1.5.1 milestone Nov 25, 2015
@ns-bot
Copy link

ns-bot commented Nov 25, 2015

Test FAILed.

@rosen-vladimirov rosen-vladimirov force-pushed the vladimirov/fix-cocoapods-check branch 2 times, most recently from 44b19bd to 1722709 Compare November 25, 2015 16:05
@ns-bot
Copy link

ns-bot commented Nov 25, 2015

Test FAILed.

1 similar comment
@ns-bot
Copy link

ns-bot commented Nov 25, 2015

Test FAILed.

@rosen-vladimirov rosen-vladimirov force-pushed the vladimirov/fix-cocoapods-check branch from 1722709 to a428275 Compare November 25, 2015 18:14
@ns-bot
Copy link

ns-bot commented Nov 25, 2015

Test FAILed.

@rosen-vladimirov rosen-vladimirov force-pushed the vladimirov/fix-cocoapods-check branch from a428275 to d78a885 Compare November 25, 2015 18:24
@ns-bot
Copy link

ns-bot commented Nov 25, 2015

Test PASSed.

@rosen-vladimirov rosen-vladimirov force-pushed the vladimirov/fix-cocoapods-check branch from d78a885 to a16713c Compare November 26, 2015 07:13
@ns-bot
Copy link

ns-bot commented Nov 26, 2015

Test PASSed.

@teobugslayer
Copy link
Contributor

👍

When cocoapods are not installed on Mac machine, doctor is failing.
Fix this by skipping match check in case pods are not installed.
Add basic unit tests for doctor service.

Add unit test for the failing cocoapods check.
@rosen-vladimirov rosen-vladimirov force-pushed the vladimirov/fix-cocoapods-check branch from a16713c to cea0f36 Compare November 26, 2015 10:23
@ns-bot
Copy link

ns-bot commented Nov 26, 2015

Test PASSed.

rosen-vladimirov added a commit that referenced this pull request Nov 26, 2015
…heck

Fix cocoapods check and add unit tests for doctor service
@rosen-vladimirov rosen-vladimirov merged commit db97db7 into release Nov 26, 2015
@rosen-vladimirov rosen-vladimirov deleted the vladimirov/fix-cocoapods-check branch November 26, 2015 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants