change patch to put in feature flag endpoint

Author: davidgamez

docs/OperationsAPI.yaml

@@ -499,13 +499,13 @@ paths:
         "404":
           description: User not found.
   /v1/operations/users/{user_id}/feature-flags:
-    patch:
+    put:
       summary: Replace a user's feature flags
       description: |
         Replaces the user's complete set of feature flags in a single call.
         Flags in the list are assigned; any previously assigned flags not in the list are removed.
         All provided flag IDs must exist.
-      operationId: patchUserFeatureFlags
+      operationId: putUserFeatureFlags
       tags:
         - "users"
       security:
@@ -521,7 +521,7 @@ paths:
         content:
           application/json:
             schema:
-              $ref: "#/components/schemas/PatchUserFeatureFlagsRequest"
+              $ref: "#/components/schemas/PutUserFeatureFlagsRequest"
       responses:
         "200":
           description: Updated user profile with the new feature flag set.
@@ -2351,7 +2351,7 @@ components:
           items:
             $ref: "#/components/schemas/UserFeatureFlagState"
           default: []
-    PatchUserFeatureFlagsRequest:
+    PutUserFeatureFlagsRequest:
       x-operation: true
       type: object
       required:

functions-python/operations_api/.openapi-generator/FILES

@@ -46,10 +46,10 @@ src/feeds_gen/models/operation_feed.py
 src/feeds_gen/models/operation_gtfs_feed.py
 src/feeds_gen/models/operation_gtfs_rt_feed.py
 src/feeds_gen/models/operation_user_profile.py
-src/feeds_gen/models/patch_user_feature_flags_request.py
 src/feeds_gen/models/propagate_license_affected_feed.py
 src/feeds_gen/models/propagate_license_request.py
 src/feeds_gen/models/propagate_license_response.py
+src/feeds_gen/models/put_user_feature_flags_request.py
 src/feeds_gen/models/redirect.py
 src/feeds_gen/models/search_feed_item_result.py
 src/feeds_gen/models/source_info.py

functions-python/operations_api/src/feeds_operations/impl/user_feature_flags_impl.py

@@ -29,8 +29,8 @@
 )
 from feeds_gen.models.operation_feature_flag import OperationFeatureFlag
 from feeds_gen.models.operation_user_profile import OperationUserProfile
-from feeds_gen.models.patch_user_feature_flags_request import (
-    PatchUserFeatureFlagsRequest,
+from feeds_gen.models.put_user_feature_flags_request import (
+    PutUserFeatureFlagsRequest,
 )
 from feeds_gen.models.update_feature_flag_request import UpdateFeatureFlagRequest
 from feeds_operations.impl.models.operation_feature_flag_impl import (
@@ -196,10 +196,10 @@ def update_feature_flag(
         return OperationFeatureFlagImpl.from_orm(flag)
 
     @with_users_db_session
-    def patch_user_feature_flags(
+    def put_user_feature_flags(
         self,
         user_id: str,
-        patch_user_feature_flags_request: PatchUserFeatureFlagsRequest,
+        put_user_feature_flags_request: PutUserFeatureFlagsRequest,
         db_session=None,
     ) -> OperationUserProfile:
         user = (
@@ -211,7 +211,7 @@ def patch_user_feature_flags(
         if not user:
             raise HTTPException(status_code=404, detail="User not found.")
 
-        assignments = patch_user_feature_flags_request.assignments or []
+        assignments = put_user_feature_flags_request.assignments or []
         flag_ids = [a.feature_flag_id for a in assignments]
 
         # Validate all provided flag IDs exist before making any changes

functions-python/operations_api/tests/feeds_operations/impl/test_user_feature_flags.py

@@ -23,8 +23,8 @@
 from feeds_operations.impl.user_feature_flags_impl import UserFeatureFlagsApiImpl
 from feeds_gen.models.create_feature_flag_request import CreateFeatureFlagRequest
 from feeds_gen.models.feature_flag_assignment import FeatureFlagAssignment
-from feeds_gen.models.patch_user_feature_flags_request import (
-    PatchUserFeatureFlagsRequest,
+from feeds_gen.models.put_user_feature_flags_request import (
+    PutUserFeatureFlagsRequest,
 )
 from feeds_gen.models.update_feature_flag_request import UpdateFeatureFlagRequest
 from shared.users_database_gen.sqlacodegen_models import (
@@ -412,7 +412,7 @@ def test_raises_404_when_flag_not_found(self):
         self.session.delete.assert_not_called()
 
 
-class TestPatchUserFeatureFlags(unittest.TestCase):
+class TestPutUserFeatureFlags(unittest.TestCase):
     def setUp(self):
         self.api = UserFeatureFlagsApiImpl()
         self.session = MagicMock()
@@ -431,9 +431,9 @@ def _setup_flags_query(self, flag_ids):
     def test_raises_404_when_user_not_found(self):
         _mock_query(self.session, None)
 
-        req = PatchUserFeatureFlagsRequest(assignments=[])
+        req = PutUserFeatureFlagsRequest(assignments=[])
         with self.assertRaises(HTTPException) as ctx:
-            self.api.patch_user_feature_flags("missing", req, db_session=self.session)
+            self.api.put_user_feature_flags("missing", req, db_session=self.session)
 
         self.assertEqual(ctx.exception.status_code, 404)
 
@@ -451,11 +451,11 @@ def query_side_effect(model):
 
         self.session.query.side_effect = query_side_effect
 
-        req = PatchUserFeatureFlagsRequest(
+        req = PutUserFeatureFlagsRequest(
             assignments=[FeatureFlagAssignment(feature_flag_id="nonexistent")]
         )
         with self.assertRaises(HTTPException) as ctx:
-            self.api.patch_user_feature_flags("uid-1", req, db_session=self.session)
+            self.api.put_user_feature_flags("uid-1", req, db_session=self.session)
 
         self.assertEqual(ctx.exception.status_code, 404)
         self.assertIn("nonexistent", ctx.exception.detail)
@@ -472,8 +472,8 @@ def test_clears_flags_when_empty_list_provided(self):
         mock_q.all.return_value = []  # all-flags query returns no flags
         self.session.query.return_value = mock_q
 
-        req = PatchUserFeatureFlagsRequest(assignments=[])
-        self.api.patch_user_feature_flags("uid-1", req, db_session=self.session)
+        req = PutUserFeatureFlagsRequest(assignments=[])
+        self.api.put_user_feature_flags("uid-1", req, db_session=self.session)
 
         # delete() should be called on the query to clear all assignments
         mock_q.delete.assert_called_once()