Skip to content

Gold test for examples/billing #992

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

bobhancockg
Copy link
Contributor

…ingdirectory. Specifically,get_invoices.py, add_account_budget_proposal.py, and add_billing_setup.pynow have corresponding tests inexamples/billing/tests/`.

These tests use unittest.mock to simulate interactions with the Google Ads API. This helps ensure your example scripts are making the correct service calls and handling responses as you expect.

I also confirmed that all examples in examples/billing already explicitly initialize the GoogleAdsClient with version="v19".

During the development of these tests, I made minor corrections to get_invoices.py to fix a NameError and an attribute error on a mocked object.

…ing` directory. Specifically, `get_invoices.py`, `add_account_budget_proposal.py`, and `add_billing_setup.py` now have corresponding tests in `examples/billing/tests/`.

These tests use `unittest.mock` to simulate interactions with the Google Ads API. This helps ensure your example scripts are making the correct service calls and handling responses as you expect.

I also confirmed that all examples in `examples/billing` already explicitly initialize the `GoogleAdsClient` with `version="v19"`.

During the development of these tests, I made minor corrections to `get_invoices.py` to fix a `NameError` and an attribute error on a mocked object.
@bobhancockg bobhancockg requested a review from a team as a code owner May 31, 2025 18:26
@bobhancockg bobhancockg requested review from Raibaz and dorasun and removed request for Raibaz and dorasun May 31, 2025 18:26
@BenRKarl BenRKarl added the kokoro:run This label is required to run a kokoro presubmit if an external contributor sends a PR. label Jun 5, 2025
@kokoro-team kokoro-team removed the kokoro:run This label is required to run a kokoro presubmit if an external contributor sends a PR. label Jun 5, 2025
@BenRKarl BenRKarl added the kokoro:force-run This label is required to run a kokoro presubmit if an external contributor sends a PR. label Jun 5, 2025
@kokoro-team kokoro-team removed the kokoro:force-run This label is required to run a kokoro presubmit if an external contributor sends a PR. label Jun 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants