Skip to content

Add comprehensive test coverage for campaign_discovery.cjs #11552

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.

Sign up for GitHub

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

Merged
pelikhan merged 2 commits into from
Jan 23, 2026
Merged

Add comprehensive test coverage for campaign_discovery.cjs #11552

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
d73761f
Initial plan
Copilot Jan 23, 2026
fcbfb46
Add comprehensive tests for campaign_discovery.cjs
Copilot Jan 23, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Add comprehensive tests for campaign_discovery.cjs 
- Added 17 new tests (34 total, doubled from 17)
- Added tests for buildScopeParts helper function (6 tests)
- Added tests for searchItems helper function (8 tests)
- Added tests for discover edge cases (3 tests):
  * Budget exhaustion (items and pages)
  * Item deduplication across searches
  * Tracker label fallback behavior

All tests pass ✓
Formatting validated ✓
Linting validated ✓
Type checking validated ✓

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
  • Loading branch information
@pelikhan
Copilot and pelikhan committed Jan 23, 2026
commit fcbfb4630b3997b47ffcab3ba615b98e86bbbcef
Loading
Loading