You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
And additional RPC tests can be added to ensure those underlying server diagnostics are correctly in the final response.
Bonus: While in the area, it might be good to update StopProviderError to StopProviderResponse instead for consistency.
The text was updated successfully, but these errors were encountered:
bflad
changed the title
Ensure ConfigureProvider and StopProvider Response Error/Diagnostics are Unit Tested
Ensure ConfigureProvider Response Diagnostics are Unit Tested
Jul 14, 2023
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.
terraform-plugin-mux version
Use cases
The
tf5muxserver
andtf6muxserver
implementations ofConfigureProvider
RPC includes logic which handles diagnostics from the underlying servers.Proposal
Implement additional unit testing for that RPC, similar to #177. The
TestServer
implementation can be updated to include additional response fields:And additional RPC tests can be added to ensure those underlying server diagnostics are correctly in the final response.
Bonus: While in the area, it might be good to update
StopProviderError
toStopProviderResponse
instead for consistency.The text was updated successfully, but these errors were encountered: