fix: map coder 429 errors to too many requests

Author: ThomasK33

internal/aggregated/coder/errors.go

@@ -45,6 +45,8 @@ func MapCoderError(err error, resource schema.GroupResource, name string) error
 		return apierrors.NewBadRequest(message)
 	case http.StatusUnauthorized:
 		return apierrors.NewUnauthorized(message)
+	case http.StatusTooManyRequests:
+		return apierrors.NewTooManyRequests(message, 0)
 	default:
 		if statusCode >= http.StatusBadRequest && statusCode < http.StatusInternalServerError {
 			return apierrors.NewBadRequest(message)

internal/aggregated/coder/errors_test.go

@@ -83,15 +83,15 @@ func TestMapCoderError(t *testing.T) {
 			},
 		},
 		{
-			name: "maps other client errors to bad request",
+			name: "maps too many requests",
 			err: withCoderMessage(
 				codersdk.NewTestError(http.StatusTooManyRequests, http.MethodGet, "https://coder.example.com"),
 				"rate limited",
 			),
 			assertMapping: func(t *testing.T, err error) {
 				t.Helper()
-				if !apierrors.IsBadRequest(err) {
-					t.Fatalf("expected BadRequest, got %v", err)
+				if !apierrors.IsTooManyRequests(err) {
+					t.Fatalf("expected TooManyRequests, got %v", err)
 				}
 			},
 		},