Skip to content

Commit 75e7e49

Browse files
committed
move comment out of code
1 parent 1367935 commit 75e7e49

File tree

1 file changed

+0
-8
lines changed

1 file changed

+0
-8
lines changed

go/porcelain/deploy.go

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -541,14 +541,6 @@ func (n *Netlify) uploadFile(ctx context.Context, d *models.Deploy, f *FileBundl
541541
context.GetLogger(ctx).WithError(operationError).Errorf("Failed to upload file %v", f.Name)
542542
apiErr, ok := operationError.(apierrors.Error)
543543

544-
// In testing, we repeatedly get to this line, see "Failed to upload file", but no matter
545-
// what we try, ok is always `false`, and the code we're trying to test gets skipped.
546-
// There isn't a test case for the original code (the 401 error)
547-
548-
// The operation error we're receiving is "(*models.Error) is not supported by the TextConsumer, can be resolved by supporting TextUnmarshaler interface"
549-
// which seems like a problem with how we're stubbing the response body from the server in the test,
550-
// but it's similar to how it's stubbed in
551-
552544
if ok {
553545
if apiErr.Code() == 401 {
554546
sharedErr.mutex.Lock()

0 commit comments

Comments
 (0)