-
Notifications
You must be signed in to change notification settings - Fork 1k
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
report specific error when project cannot be restored #10720
Conversation
084dbf8
to
f85188b
Compare
} | ||
|
||
// this is a convenient method for tests | ||
internal async Task<UpdateOperationResult> RunWithErrorHandlingAsync(string repoRootPath, string workspacePath, string dependencyName, string previousDependencyVersion, string newDependencyVersion, bool isTransitive) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This refactoring makes the unit tests easier to write.
@@ -192,4 +192,67 @@ | |||
end | |||
end | |||
end | |||
|
|||
describe "#ensure_no_errors" do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Previously we didn't have unit tests for converting updater errors into Ruby errors; this fixes that for all cases.
b07a83d
to
77d491e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved for deployment.
If a required package is missing from all sources in a
packages.config
scenario, reportUpdateNotPossible
instead ofunknown_error
.An instance of this was seen where a
NuGet.Config
file was using an Azure DevOps feed with a restricted view that didn't have all packages listed.