Jules was unable to complete the task in time. Please review the work done so far and provide feedback for Jules to continue.

Author: google-labs-jules[bot]

examples/advanced_operations/add_app_campaign.py

@@ -284,6 +284,7 @@ def create_app_ad(client, customer_id, ad_group_resource_name):
         "Ad Group App Ad created with resource name:"
         f'"{ad_group_ad_resource_name}".'
     )
+    return ad_group_ad_resource_name
 
 
 def create_ad_text_asset(client, text):

examples/advanced_operations/tests/__init__.py

@@ -0,0 +1 @@
+# This file intentionally left blank.

examples/advanced_operations/tests/test_add_ad_customizer.py

@@ -0,0 +1,194 @@
+# Copyright 2020 Google LLC
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     https://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+"""Tests for add_ad_customizer.py."""
+
+from unittest import mock
+from unittest import TestCase
+
+from google.ads.googleads.client import GoogleAdsClient
+from google.ads.googleads.errors import GoogleAdsException
+
+from examples.advanced_operations.add_ad_customizer import main
+from examples.advanced_operations.add_ad_customizer import create_text_customizer_attribute
+from examples.advanced_operations.add_ad_customizer import create_price_customizer_attribute
+from examples.advanced_operations.add_ad_customizer import link_customizer_attributes
+from examples.advanced_operations.add_ad_customizer import create_ad_with_customizations
+
+
+_CUSTOMER_ID = "1234567890"
+_AD_GROUP_ID = "9876543210"
+_TEXT_CUSTOMIZER_NAME = "Planet_test"
+_PRICE_CUSTOMIZER_NAME = "Price_test"
+_TEXT_CUSTOMIZER_RESOURCE_NAME = "customers/1234567890/customizerAttributes/1"
+_PRICE_CUSTOMIZER_RESOURCE_NAME = "customers/1234567890/customizerAttributes/2"
+
+
+@mock.patch("examples.advanced_operations.add_ad_customizer.GoogleAdsClient.load_from_storage")
+class AddAdCustomizerTest(TestCase):
+    def setUp(self):
+        self.client_mock = mock.MagicMock(spec=GoogleAdsClient)
+        # Mock the enums attribute
+        self.client_mock.enums = mock.MagicMock()
+        self.client_mock.enums.CustomizerAttributeTypeEnum = mock.MagicMock()
+        self.client_mock.enums.ServedAssetFieldTypeEnum = mock.MagicMock()
+        # It's also good practice to assign specific mock objects or values
+        # if the code relies on specific enum members, for example:
+        # self.client_mock.enums.CustomizerAttributeTypeEnum.TEXT = "TEXT_ENUM_VALUE"
+        # self.client_mock.enums.CustomizerAttributeTypeEnum.PRICE = "PRICE_ENUM_VALUE"
+        # self.client_mock.enums.ServedAssetFieldTypeEnum.HEADLINE_1 = "HEADLINE_1_ENUM_VALUE"
+        # However, for now, MagicMock should suffice as the code seems to only access them.
+
+        self.googleads_service_mock = self.client_mock.get_service("GoogleAdsService")
+        self.customizer_attribute_service_mock = self.client_mock.get_service("CustomizerAttributeService")
+        self.ad_group_customizer_service_mock = self.client_mock.get_service("AdGroupCustomizerService")
+        self.ad_group_ad_service_mock = self.client_mock.get_service("AdGroupAdService")
+
+        # Mock successful responses for service calls
+        # This will be the default return value for mutate_customizer_attributes.
+        # Individual tests that call it once will need to adjust this.
+        self.customizer_attribute_service_mock.mutate_customizer_attributes.return_value.results = [
+            mock.Mock(resource_name=_TEXT_CUSTOMIZER_RESOURCE_NAME),
+            mock.Mock(resource_name=_PRICE_CUSTOMIZER_RESOURCE_NAME)
+        ]
+        self.ad_group_customizer_service_mock.mutate_ad_group_customizers.return_value.results = [
+            mock.Mock(resource_name="adGroupCustomizer1"),
+            mock.Mock(resource_name="adGroupCustomizer2")
+        ]
+        self.ad_group_ad_service_mock.mutate_ad_group_ads.return_value.results = [
+            mock.Mock(resource_name="adGroupAd1")
+        ]
+
+        self.googleads_service_mock.ad_group_path.return_value = f"customers/{_CUSTOMER_ID}/adGroups/{_AD_GROUP_ID}"
+
+
+    def test_create_text_customizer_attribute(self, mock_load_client):
+        mock_load_client.return_value = self.client_mock
+        # Adjust return_value for a single call to mutate_customizer_attributes
+        self.customizer_attribute_service_mock.mutate_customizer_attributes.return_value.results = [
+            mock.Mock(resource_name=_TEXT_CUSTOMIZER_RESOURCE_NAME)
+        ]
+
+        resource_name = create_text_customizer_attribute(
+            self.client_mock, _CUSTOMER_ID, _TEXT_CUSTOMIZER_NAME
+        )
+        self.assertEqual(resource_name, _TEXT_CUSTOMIZER_RESOURCE_NAME)
+        self.customizer_attribute_service_mock.mutate_customizer_attributes.assert_called_once()
+
+
+    def test_create_price_customizer_attribute(self, mock_load_client):
+        mock_load_client.return_value = self.client_mock
+        # Adjust return_value for a single call to mutate_customizer_attributes
+        self.customizer_attribute_service_mock.mutate_customizer_attributes.return_value.results = [
+            mock.Mock(resource_name=_PRICE_CUSTOMIZER_RESOURCE_NAME)
+        ]
+
+        resource_name = create_price_customizer_attribute(
+            self.client_mock, _CUSTOMER_ID, _PRICE_CUSTOMIZER_NAME
+        )
+        self.assertEqual(resource_name, _PRICE_CUSTOMIZER_RESOURCE_NAME)
+        self.customizer_attribute_service_mock.mutate_customizer_attributes.assert_called_once()
+
+
+    def test_link_customizer_attributes(self, mock_load_client):
+        mock_load_client.return_value = self.client_mock
+        link_customizer_attributes(
+            self.client_mock,
+            _CUSTOMER_ID,
+            _AD_GROUP_ID,
+            _TEXT_CUSTOMIZER_RESOURCE_NAME,
+            _PRICE_CUSTOMIZER_RESOURCE_NAME,
+        )
+        self.ad_group_customizer_service_mock.mutate_ad_group_customizers.assert_called_once()
+
+
+    def test_create_ad_with_customizations(self, mock_load_client):
+        mock_load_client.return_value = self.client_mock
+        create_ad_with_customizations(
+            self.client_mock,
+            _CUSTOMER_ID,
+            _AD_GROUP_ID,
+            _TEXT_CUSTOMIZER_NAME,
+            _PRICE_CUSTOMIZER_NAME,
+        )
+        self.ad_group_ad_service_mock.mutate_ad_group_ads.assert_called_once()
+
+
+    @mock.patch("examples.advanced_operations.add_ad_customizer.create_text_customizer_attribute")
+    @mock.patch("examples.advanced_operations.add_ad_customizer.create_price_customizer_attribute")
+    @mock.patch("examples.advanced_operations.add_ad_customizer.link_customizer_attributes")
+    @mock.patch("examples.advanced_operations.add_ad_customizer.create_ad_with_customizations")
+    def test_main_success(
+        self,
+        mock_create_ad,
+        mock_link_attributes,
+        mock_create_price_attr,
+        mock_create_text_attr,
+        mock_load_client
+    ):
+        mock_load_client.return_value = self.client_mock
+        mock_create_text_attr.return_value = _TEXT_CUSTOMIZER_RESOURCE_NAME
+        mock_create_price_attr.return_value = _PRICE_CUSTOMIZER_RESOURCE_NAME
+
+        main(self.client_mock, _CUSTOMER_ID, _AD_GROUP_ID)
+
+        mock_create_text_attr.assert_called_once_with(
+            self.client_mock, _CUSTOMER_ID, mock.ANY
+        )
+        mock_create_price_attr.assert_called_once_with(
+            self.client_mock, _CUSTOMER_ID, mock.ANY
+        )
+        mock_link_attributes.assert_called_once_with(
+            self.client_mock,
+            _CUSTOMER_ID,
+            _AD_GROUP_ID,
+            _TEXT_CUSTOMIZER_RESOURCE_NAME,
+            _PRICE_CUSTOMIZER_RESOURCE_NAME,
+        )
+        mock_create_ad.assert_called_once_with(
+            self.client_mock,
+            _CUSTOMER_ID,
+            _AD_GROUP_ID,
+            mock.ANY,
+            mock.ANY,
+        )
+
+    @mock.patch("examples.advanced_operations.add_ad_customizer.create_text_customizer_attribute")
+    def test_main_google_ads_exception(
+        self, mock_create_text_attr, mock_load_client
+    ):
+        # We don't need mock_load_client here as main directly uses the passed client
+        # However, it's included by the class-level decorator, so we accept it.
+        mock_failure = mock.Mock()
+        mock_failure.errors = [mock.Mock(message="Test Error")]
+        # Add a mock for the 'call' attribute if it's accessed by GoogleAdsException
+        mock_call = mock.Mock()
+
+        mock_create_text_attr.side_effect = GoogleAdsException(
+            error=mock.Mock(),
+            failure=mock_failure,
+            call=mock_call,  # Add the missing 'call' argument
+            request_id="test_request_id"
+        )
+
+        # The main function in the script catches the exception and calls sys.exit(1)
+        # and prints to stdout. For this test, we'll assert that the exception is raised
+        # when calling main with our mock client.
+        with self.assertRaises(GoogleAdsException):
+            main(self.client_mock, _CUSTOMER_ID, _AD_GROUP_ID)
+
+# It's good practice to also create an __init__.py in the tests directory
+# if it doesn't exist, to make it a Python package.
+# This subtask will also create that file.
+# Create the file examples/advanced_operations/tests/__init__.py if it doesn't exist.
+# If it exists, leave it as is.

examples/advanced_operations/tests/test_add_ad_group_bid_modifier.py

@@ -0,0 +1,107 @@
+# Copyright 2020 Google LLC
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     https://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+"""Tests for add_ad_group_bid_modifier.py."""
+
+from unittest import mock
+from unittest import TestCase
+
+from google.ads.googleads.client import GoogleAdsClient
+from google.ads.googleads.errors import GoogleAdsException
+# Attempt to import DeviceEnum from the versioned enum path
+from google.ads.googleads.v20.enums import DeviceEnum
+
+from examples.advanced_operations.add_ad_group_bid_modifier import main
+
+
+_CUSTOMER_ID = "1234567890"
+_AD_GROUP_ID = "9876543210"
+_BID_MODIFIER_VALUE = 1.5
+# Using the known value for DeviceEnum.MOBILE (which is 3) directly
+# to avoid issues with enum access at module load time.
+# DeviceEnum.MOBILE.value is 3.
+_MODIFIER_RESOURCE_NAME = (
+    f"customers/{_CUSTOMER_ID}/adGroupBidModifiers/"
+    f"{_AD_GROUP_ID}~3"
+)
+
+
+@mock.patch("examples.advanced_operations.add_ad_group_bid_modifier.GoogleAdsClient.load_from_storage")
+class AddAdGroupBidModifierTest(TestCase):
+    def setUp(self):
+        self.client_mock = mock.MagicMock(spec=GoogleAdsClient)
+        self.ad_group_service_mock = self.client_mock.get_service("AdGroupService")
+        self.ad_group_bm_service_mock = self.client_mock.get_service("AdGroupBidModifierService")
+
+        # Mock the ad group path
+        self.ad_group_service_mock.ad_group_path.return_value = (
+            f"customers/{_CUSTOMER_ID}/adGroups/{_AD_GROUP_ID}"
+        )
+
+        # Mock the response from mutate_ad_group_bid_modifiers
+        mock_result = mock.Mock()
+        mock_result.resource_name = _MODIFIER_RESOURCE_NAME
+        self.ad_group_bm_service_mock.mutate_ad_group_bid_modifiers.return_value.results = [
+            mock_result
+        ]
+
+        # Mock enums
+        self.client_mock.enums = mock.MagicMock() # Ensure enums attribute exists
+        # Create a mock for the DeviceEnum type itself
+        mock_device_enum = mock.MagicMock()
+        # Set the MOBILE attribute on this mock to return the integer value 3
+        mock_device_enum.MOBILE = 3
+        self.client_mock.enums.DeviceEnum = mock_device_enum
+
+
+    def test_main_success(self, mock_load_client):
+        mock_load_client.return_value = self.client_mock
+
+        main(self.client_mock, _CUSTOMER_ID, _AD_GROUP_ID, _BID_MODIFIER_VALUE)
+
+        self.ad_group_service_mock.ad_group_path.assert_called_once_with(
+            _CUSTOMER_ID, _AD_GROUP_ID
+        )
+
+        # Check that mutate_ad_group_bid_modifiers was called
+        self.ad_group_bm_service_mock.mutate_ad_group_bid_modifiers.assert_called_once()
+
+        # Get the actual operation passed to the mock
+        call_args = self.ad_group_bm_service_mock.mutate_ad_group_bid_modifiers.call_args
+        operation = call_args[1]["operations"][0].create # Accessing create from AdGroupBidModifierOperation
+
+        self.assertEqual(
+            operation.ad_group,
+            f"customers/{_CUSTOMER_ID}/adGroups/{_AD_GROUP_ID}",
+        )
+        self.assertEqual(operation.bid_modifier, _BID_MODIFIER_VALUE)
+        # Assert against the integer value since DeviceEnum.MOBILE might be problematic
+        self.assertEqual(operation.device.type_, 3)
+
+
+    def test_main_google_ads_exception(self, mock_load_client):
+        mock_load_client.return_value = self.client_mock
+
+        # Configure the service mock to raise GoogleAdsException
+        self.ad_group_bm_service_mock.mutate_ad_group_bid_modifiers.side_effect = GoogleAdsException(
+            error=mock.Mock(),
+            failure=mock.Mock(errors=[mock.Mock(message="Test Error")]),
+            request_id="test_request_id",
+            call=mock.Mock() # Added missing call argument
+        )
+
+        with self.assertRaises(GoogleAdsException):
+            main(self.client_mock, _CUSTOMER_ID, _AD_GROUP_ID, _BID_MODIFIER_VALUE)
+
+if __name__ == "__main__":
+    TestCase.main()