Fix typing for POST user endpoint (#31767)

* Fix typing for POST user endpoint

This should raise exception not return it.

* fixup! Fix typing for POST user endpoint

(cherry picked from commit f7ed87823afb2871d8f11e615d96693db3a6c2b8)

GitOrigin-RevId: cc3cd20dce547843125701b2cbc0ef2660c4e452

airflow/api_connexion/endpoints/user_endpoint.py

@@ -119,7 +119,7 @@ def post_user() -> APIResponse:
     user = security_manager.add_user(role=roles_to_add, **data)
     if not user:
         detail = f"Failed to add user `{username}`."
-        return Unknown(detail=detail)
+        raise Unknown(detail=detail)
 
     return user_schema.dump(user)
 

tests/api_connexion/endpoints/test_user_endpoint.py

@@ -525,6 +525,20 @@ def test_invalid_payload(self, autoclean_user_payload, payload_converter, error_
             "type": EXCEPTIONS_LINK_MAP[400],
         }
 
+    def test_internal_server_error(self, autoclean_user_payload):
+        with unittest.mock.patch.object(self.app.appbuilder.sm, "add_user", return_value=None):
+            response = self.client.post(
+                "/api/v1/users",
+                json=autoclean_user_payload,
+                environ_overrides={"REMOTE_USER": "test"},
+            )
+            assert response.json == {
+                "detail": "Failed to add user `example_user`.",
+                "status": 500,
+                "title": "Internal Server Error",
+                "type": EXCEPTIONS_LINK_MAP[500],
+            }
+
 
 class TestPatchUser(TestUserEndpoint):
     @pytest.mark.usefixtures("autoclean_admin_user")