Skip to content
This repository has been archived by the owner on Feb 15, 2022. It is now read-only.

Add error handling for bad directory #247

Merged
merged 3 commits into from
Aug 23, 2019
Merged

Conversation

shubham1172
Copy link
Member

Solves #156.

If the directory doesn't exist or the user doesn't have permissions to access it, an error will be logged and returned to the caller.

@shubham1172
Copy link
Member Author

Should I write a test case to cover it too? The generator can simply fail if the directory is missing.

@timfpark
Copy link
Contributor

Thank you @shubham1172. Yes, please add a test to cover this functionality.

@msftclas
Copy link

msftclas commented Aug 23, 2019

CLA assistant check
All CLA requirements met.

@shubham1172
Copy link
Member Author

Added a test!

@timfpark
Copy link
Contributor

Looks good to me - thanks so much for your contribution - its much appreciated.

@timfpark timfpark merged commit 56d5968 into microsoft:master Aug 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants