fix(tests): Resolve AttributeError for pb method in ad group bid modifier tests

I've updated `test_get_ad_group_bid_modifiers.py` to fix the
`AttributeError: type object 'MagicMock' has no attribute 'pb'`.

This was achieved by:
1. Defining a `MockAdGroupBidModifierModel` class at the module level.
   This class includes a `pb` class method that returns a mock
   configured with a `WhichOneof` method, simulating the behavior of
   the Google Ads library's model objects.
2. Initializing instances of `MockAdGroupBidModifierModel` to represent
   the `ad_group_bid_modifier` objects in the mocked search results
   provided by the `GoogleAdsService`.
3. Ensuring that the mock enum values (e.g., `DeviceEnum.MOBILE`)
   accessed via `mock_google_ads_client.enums` are themselves
   `MagicMock` instances with a `.name` attribute, as required by
   the script's print statements.
4. Setting the `_active_criterion_field` on `MockAdGroupBidModifierModel`
   instances and configuring their specific criterion attributes (e.g.,
   `device.type_`) using the mocked enums.

Author: google-labs-jules[bot]

examples/advanced_operations/tests/conftest.py

@@ -20,12 +20,12 @@ def mock_google_ads_client():
     # For now, we'll make it a MagicMock. If specific enums are needed,
     # they can be added as properties to this mock_enums object.
     mock_enums = MagicMock()
-    
+
     # Example of mocking a specific enum if needed for a test:
     # mock_campaign_status_enum = MagicMock()
     # mock_campaign_status_enum.PAUSED = "PAUSED_ENUM_VALUE" # Or an int if that's what the API uses
     # type(mock_enums).CampaignStatusEnum = PropertyMock(return_value=mock_campaign_status_enum)
-    
+
     mock_client.enums = mock_enums
 
     return mock_client

examples/advanced_operations/tests/test_add_ad_customizer.py

@@ -30,7 +30,7 @@ def test_main_runs_successfully(mock_google_ads_client: MagicMock) -> None:
     mock_ad_group_customizer_result.resource_name = "mock_resource_name_ad_group_customizer"
     mock_mutate_ad_group_customizers_response.results = [mock_ad_group_customizer_result]
     mock_ad_group_customizer_service.mutate_ad_group_customizers.return_value = mock_mutate_ad_group_customizers_response
-    
+
     mock_googleads_service = mock_google_ads_client.get_service("GoogleAdsService")
     mock_googleads_service.ad_group_path.return_value = "mock/ad_group/path"
 
@@ -40,7 +40,7 @@ def test_main_runs_successfully(mock_google_ads_client: MagicMock) -> None:
     mock_ad_group_ad_result.resource_name = "mock_resource_name_ad_group_ad"
     mock_mutate_ad_group_ads_response.results = [mock_ad_group_ad_result]
     mock_ad_group_ad_service.mutate_ad_group_ads.return_value = mock_mutate_ad_group_ads_response
-    
+
     # Mock enums used by the script
     mock_enums = mock_google_ads_client.enums
     mock_enums.CustomizerAttributeTypeEnum.TEXT = "TEXT"

examples/advanced_operations/tests/test_add_ad_group_bid_modifier.py

@@ -24,7 +24,7 @@ def test_main_runs_successfully(mock_google_ads_client: MagicMock) -> None:
     mock_device_enum = MagicMock()
     mock_device_enum.MOBILE = "MOBILE_DEVICE" # Or actual enum value if known/needed
     mock_google_ads_client.enums.DeviceEnum = mock_device_enum
-    
+
     # Mock types
     # The get_type method in conftest returns a MagicMock by default.
     # We can configure its "create" attribute if necessary, but often not needed

examples/advanced_operations/tests/test_add_app_campaign.py

@@ -46,7 +46,7 @@ def test_main_runs_successfully(mock_uuid4: MagicMock, mock_google_ads_client: M
     mock_criterion_result_lang2 = MagicMock()
     mock_criterion_result_lang2.resource_name = "criterion_lang2"
     mock_criterion_response.results = [
-        mock_criterion_result_loc1, 
+        mock_criterion_result_loc1,
         mock_criterion_result_loc2,
         mock_criterion_result_lang1,
         mock_criterion_result_lang2
@@ -79,7 +79,7 @@ def test_main_runs_successfully(mock_uuid4: MagicMock, mock_google_ads_client: M
     mock_enums.AppCampaignBiddingStrategyGoalTypeEnum.OPTIMIZE_INSTALLS_TARGET_INSTALL_COST = "OPTIMIZE_INSTALLS_TARGET_INSTALL_COST"
     mock_enums.AdGroupStatusEnum.ENABLED = "ENABLED"
     mock_enums.AdGroupAdStatusEnum.ENABLED = "ENABLED"
-    
+
     # --- Mock types ---
     # Most types are simple containers, MagicMock default behavior is often sufficient.
     # For AdTextAsset, it's created and then its `text` attribute is set.

examples/advanced_operations/tests/test_add_bidding_data_exclusion.py

@@ -23,7 +23,7 @@ def test_main_runs_successfully(mock_uuid4: MagicMock, mock_google_ads_client: M
     mock_enums.SeasonalityEventScopeEnum.CHANNEL = "CHANNEL"
     mock_enums.AdvertisingChannelTypeEnum.SEARCH = "SEARCH"
     # The script also uses DeviceEnum, but it's commented out, so not strictly needed for this test pass
-    # mock_enums.DeviceEnum.MOBILE = "MOBILE" 
+    # mock_enums.DeviceEnum.MOBILE = "MOBILE"
 
     try:
         main(

examples/advanced_operations/tests/test_add_demand_gen_campaign.py

@@ -48,7 +48,7 @@ def test_main_runs_successfully(
     mock_mutate_response.mutate_operation_responses.append(MagicMock(campaign_criterion_result=MagicMock(resource_name="campaign_criterion_1"))) # Campaign Criterion
 
     mock_googleads_service.mutate.return_value = mock_mutate_response
-    
+
     # Mock SearchStream for finding existing assets (assumed not found for simplicity of this test)
     mock_search_response = MagicMock()
     mock_search_response.results = [] # No existing assets found

examples/advanced_operations/tests/test_add_display_upload_ad.py

@@ -25,7 +25,7 @@ def test_main_runs_successfully(mock_requests_get: MagicMock, mock_google_ads_cl
     # Mock AdGroupService for ad_group_path
     mock_ad_group_service_path_helper = mock_google_ads_client.get_service("AdGroupService")
     mock_ad_group_service_path_helper.ad_group_path.return_value = f"customers/{mock_customer_id}/adGroups/{mock_ad_group_id}"
-    
+
     # Mock AdGroupAdService
     mock_ad_group_ad_service = mock_google_ads_client.get_service("AdGroupAdService")
     mock_ad_mutate_response = MagicMock()
@@ -39,7 +39,7 @@ def test_main_runs_successfully(mock_requests_get: MagicMock, mock_google_ads_cl
     mock_enums.AssetTypeEnum.MEDIA_BUNDLE = "MEDIA_BUNDLE"
     mock_enums.AdGroupAdStatusEnum.PAUSED = "PAUSED"
     mock_enums.DisplayUploadProductTypeEnum.HTML5_UPLOAD_AD = "HTML5_UPLOAD_AD"
-    
+
     # Mock types
     # mock_asset_operation = mock_google_ads_client.get_type("AssetOperation")
     # mock_ad_group_ad_operation = mock_google_ads_client.get_type("AdGroupAdOperation")

examples/advanced_operations/tests/test_add_dynamic_page_feed_asset.py

@@ -49,7 +49,7 @@ def test_main_runs_successfully(mock_get_printable_datetime: MagicMock, mock_goo
     mock_campaign_asset_set_result.resource_name = "customers/123/campaignAssetSets/campaign_asset_set123"
     mock_campaign_asset_set_response.results = [mock_campaign_asset_set_result]
     mock_campaign_asset_set_service.mutate_campaign_asset_sets.return_value = mock_campaign_asset_set_response
-    
+
     # Mock AdGroupCriterionService
     mock_ad_group_criterion_service = mock_google_ads_client.get_service("AdGroupCriterionService")
     mock_ad_group_criterion_response = MagicMock()

examples/advanced_operations/tests/test_add_dynamic_search_ads.py

@@ -47,7 +47,7 @@ def test_main_runs_successfully(mock_uuid4: MagicMock, mock_google_ads_client: M
     mock_criterion_result.resource_name = f"customers/{mock_customer_id}/adGroupCriteria/criterion_testuuid"
     mock_criterion_response.results = [mock_criterion_result]
     mock_ad_group_criterion_service.mutate_ad_group_criteria.return_value = mock_criterion_response
-    
+
     # Mock GoogleAdsService for campaign_path (used in _create_campaign_level_targeting)
     mock_googleads_service = mock_google_ads_client.get_service("GoogleAdsService")
     mock_googleads_service.campaign_path.return_value = f"customers/{mock_customer_id}/campaigns/campaign_testuuid"

examples/advanced_operations/tests/test_add_performance_max_campaign.py

@@ -36,7 +36,7 @@ def test_main_runs_successfully(
 
     # Mock CampaignService (similar to budget service, paths are via GoogleAdsService)
     # mock_campaign_service = mock_google_ads_client.get_service("CampaignService")
-    
+
     # Mock AssetGroupService (paths via GoogleAdsService)
     # mock_asset_group_service = mock_google_ads_client.get_service("AssetGroupService")