Skip to content

Gold tests for shopping ads#1002

Closed
bobhancockg wants to merge 1 commit intomainfrom
feat/shopping-ads-tests
Closed

Gold tests for shopping ads#1002
bobhancockg wants to merge 1 commit intomainfrom
feat/shopping-ads-tests

Conversation

@bobhancockg
Copy link
Contributor

  • Created a new test directory examples/shopping_ads/tests/.
  • Added initial placeholder test files for each example script in examples/shopping_ads/.
  • Updated noxfile.py to ensure tests in the new directory are discovered by changing the start directory for unittest discover from tests to . (project root).

All shopping_ads examples were already using Google Ads API v19, so no changes were needed for that.

- Created a new test directory `examples/shopping_ads/tests/`.
- Added initial placeholder test files for each example script in `examples/shopping_ads/`.
- Updated `noxfile.py` to ensure tests in the new directory are discovered by changing the start directory for `unittest discover` from `tests` to `.` (project root).

All shopping_ads examples were already using Google Ads API v19, so no changes were needed for that.
@bobhancockg bobhancockg requested a review from a team as a code owner June 2, 2025 10:17
@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
@BenRKarl
Copy link
Contributor

BenRKarl commented Aug 1, 2025

Closing this because we're going to approach example tests in a different way.

@BenRKarl BenRKarl closed this Aug 1, 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