particle-os/debian-forge-composer
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

v2: update errors test for new error type

Browse source 
The oapi-codegen update to v2.5.0 changed the Details field of the main
Error type to be an empty interface. This requires updates to the error
handling tests to accommodate the new structure.
This commit is contained in:
Lukas Zapletal 2025-08-04 14:36:22 +02:00 • committed by Simon de Vlieger
parent e118df5dfd
commit a58a399008
1 changed files with 6 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

12
internal/cloudapi/v2/errors_test.go
View file

@ -114,7 +114,7 @@ func TestHTTPErrorHandler(t *testing.T) {
require.NotNil(t, apiErr)
require.Equal(t, "opid", apiErr.OperationId)
require.Equal(t, find(ErrorEnqueueingJob).reason, apiErr.Reason)
require.Empty(t, *apiErr.Details)
require.Empty(t, apiErr.Details)
}
// HTTPErrorWithInternal
@ -131,7 +131,7 @@ func TestHTTPErrorHandler(t *testing.T) {
require.NotNil(t, apiErr)
require.Equal(t, "opid", apiErr.OperationId)
require.Equal(t, find(ErrorEnqueueingJob).reason, apiErr.Reason)
require.Equal(t, err.Error(), *apiErr.Details)
require.Equal(t, err.Error(), apiErr.Details)
}
// HTTPErrorWithDetails
@ -149,7 +149,7 @@ func TestHTTPErrorHandler(t *testing.T) {
require.NotNil(t, apiErr)
require.Equal(t, "opid", apiErr.OperationId)
require.Equal(t, find(ErrorEnqueueingJob).reason, apiErr.Reason)
require.Equal(t, "even more extra details", *apiErr.Details)
require.Equal(t, "even more extra details", apiErr.Details)
}
// echo.HTTPError
@ -166,7 +166,7 @@ func TestHTTPErrorHandler(t *testing.T) {
require.NotNil(t, apiErr)
require.Equal(t, "opid", apiErr.OperationId)
require.Equal(t, find(ErrorUnspecified).reason, apiErr.Reason)
require.Equal(t, "code=500, message=some unexpected internal http error", *apiErr.Details)
require.Equal(t, "code=500, message=some unexpected internal http error", apiErr.Details)
}
// echo.HTTPError and internalErr is nil
@ -182,7 +182,7 @@ func TestHTTPErrorHandler(t *testing.T) {
require.NotNil(t, apiErr)
require.Equal(t, "opid", apiErr.OperationId)
require.Equal(t, find(ErrorUnspecified).reason, apiErr.Reason)
require.Equal(t, "code=500, message=<nil>", *apiErr.Details)
require.Equal(t, "code=500, message=<nil>", apiErr.Details)
}
// plain error
@ -199,6 +199,6 @@ func TestHTTPErrorHandler(t *testing.T) {
require.NotNil(t, apiErr)
require.Equal(t, "opid", apiErr.OperationId)
require.Equal(t, find(ErrorNotHTTPError).reason, apiErr.Reason)
require.Equal(t, "some unexpected internal error", *apiErr.Details)
require.Equal(t, "some unexpected internal error", apiErr.Details)
}
}