Adds dedicated exception to finding Azure CLI tools #369
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR closes issue #368
The changes in this PR are as follows:
Instead of a cryptic message " No such file or directory":
I've created a dedicated
AzureCLIToolsNotFound
exception with the message of:I have read the contributing guidelines and have completed the following:
If I haven't completed any of the tasks above, I include the reasons why here:
I tested this by directly changing
azCliPath
and running the tests insrc/Tests/AzCli.fs
.Given how the code is today, I'm unsure if I should write a test dedicated for this since we'd need to pass in the
azureCliPath
to theexecuteAz
function. Let me know if it's appropriate.I'm also unsure if this requires a documentation change. Let know if there's an appropriate place for the change.