updates order and comments for test__validate_paged_field_size_type

Author: chalmerlowe

tests/unit/schema/wrappers/test_method.py

@@ -1015,10 +1015,12 @@ def test_mixin_rule():
 @pytest.mark.parametrize(
     "field_type, _type, expected",
     [
+        # valid paged_result_field candidate
         (int, 5, True), # 5 = int
-        (float, 1, None), # 1 = float
         (wrappers_pb2.UInt32Value, 11, True), # 11 = MessageType
         (wrappers_pb2.Int32Value, 11, True), # 11 = MessageType
+        # invalid paged_result_field candidate
+        (float, 1, None), # 1 = float
         (wrappers_pb2.UInt32Value, 1, None), # 1 = float
         (wrappers_pb2.Int32Value, 1, None), # 1 = float
     ],
@@ -1027,7 +1029,10 @@ def test__validate_paged_field_size_type(field_type, _type, expected):
     """Test _validate_paged_field_size_type with wrapper types and type indicators."""
 
     # Setup
-    if _type == 11:
+    if _type in {5, 1}:
+        # The _type values represent int (5) and float (1) 
+        page_size = make_field(name="page_size", type=_type)
+    else: 
         # _type 11 is MESSAGETYPE
         # See: https://github.com/googleapis/gapic-generator-python/blob/c8b7229ba2865d6a2f5966aa151be121de81f92d/gapic/schema/wrappers.py#L378C1-L411C10
         # and is only associated with *Int32Value in legacy APIs such as BigQuery.
@@ -1036,9 +1041,6 @@ def test__validate_paged_field_size_type(field_type, _type, expected):
             type=_type,
             message=make_message(name=field_type.DESCRIPTOR.name)
         )
-    else:
-        # The other _type values represent int (5) and float (1) 
-        page_size = make_field(name="page_size", type=_type)
 
     parent = make_field(name="parent", type=9)  # str
     page_token = make_field(name="page_token", type=9)  # str