Skip to content

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

Merged
rosen-vladimirov merged 1 commit into
releasefrom
vladimirov/fix-cocoapods-check
Nov 26, 2015
Merged

Fix cocoapods check and add unit tests for doctor service#1248
rosen-vladimirov merged 1 commit into
releasefrom
vladimirov/fix-cocoapods-check

Conversation

@rosen-vladimirov

Copy link
Copy Markdown
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-vladimirovrosen-vladimirov self-assigned this Nov 25, 2015
@rosen-vladimirovrosen-vladimirov added this to the 1.5.1 milestone Nov 25, 2015
@ns-bot

Copy link
Copy Markdown

Test FAILed.

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

Copy link
Copy Markdown

Test FAILed.

1 similar comment
@ns-bot

Copy link
Copy Markdown

Test FAILed.

@rosen-vladimirov
rosen-vladimirovforce-pushed the vladimirov/fix-cocoapods-check branch from 1722709 to a428275CompareNovember 25, 2015 18:14
@ns-bot

Copy link
Copy Markdown

Test FAILed.

@rosen-vladimirov
rosen-vladimirovforce-pushed the vladimirov/fix-cocoapods-check branch from a428275 to d78a885CompareNovember 25, 2015 18:24
@ns-bot

Copy link
Copy Markdown

Test PASSed.

@rosen-vladimirov
rosen-vladimirovforce-pushed the vladimirov/fix-cocoapods-check branch from d78a885 to a16713cCompareNovember 26, 2015 07:13
@ns-bot

Copy link
Copy Markdown

Test PASSed.

@teobugslayer

Copy link
Copy Markdown
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-vladimirovforce-pushed the vladimirov/fix-cocoapods-check branch from a16713c to cea0f36CompareNovember 26, 2015 10:23
@ns-bot

Copy link
Copy Markdown

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 releaseNov 26, 2015
@rosen-vladimirov
rosen-vladimirov deleted the vladimirov/fix-cocoapods-check branch November 26, 2015 10:30
Sign up for freeto 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

@rosen-vladimirov@ns-bot@teobugslayer