diff options
Diffstat (limited to 'internal/federation/federatingprotocol_test.go')
| -rw-r--r-- | internal/federation/federatingprotocol_test.go | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/internal/federation/federatingprotocol_test.go b/internal/federation/federatingprotocol_test.go index 999569c85..f975cd7d6 100644 --- a/internal/federation/federatingprotocol_test.go +++ b/internal/federation/federatingprotocol_test.go @@ -21,6 +21,7 @@ import ( "bytes" "context" "encoding/json" + "errors" "io" "net/http" "net/http/httptest" @@ -30,6 +31,7 @@ import ( "github.com/stretchr/testify/suite" "github.com/superseriousbusiness/gotosocial/internal/ap" "github.com/superseriousbusiness/gotosocial/internal/gtscontext" + "github.com/superseriousbusiness/gotosocial/internal/gtserror" "github.com/superseriousbusiness/gotosocial/internal/gtsmodel" "github.com/superseriousbusiness/gotosocial/testrig" "github.com/superseriousbusiness/httpsig" @@ -99,7 +101,14 @@ func (suite *FederatingProtocolTestSuite) authenticatePostInbox( recorder := httptest.NewRecorder() newContext, authed, err := suite.federator.AuthenticatePostInbox(ctx, recorder, request) - if err != nil { + if withCode := new(gtserror.WithCode); (errors.As(err, withCode) && + (*withCode).Code() >= 500) || (err != nil && (*withCode) == nil) { + // NOTE: the behaviour here is a little strange as we have + // the competing code styles of the go-fed interface expecting + // that any err is a no-go, but authed bool is intended to be + // the main passer of whether failed auth occurred, but we in + // the gts codebase use errors to pass-back non-200 status codes, + // so we specifically have to check for an internal error code. suite.FailNow(err.Error()) } |