Skip to content

test(ci): run audit-unwired-trackers tests before cron audit (#6929)#7873

Merged
mrveiss merged 1 commit into
Dev_new_guifrom
issue-6929
May 16, 2026
Merged

test(ci): run audit-unwired-trackers tests before cron audit (#6929)#7873
mrveiss merged 1 commit into
Dev_new_guifrom
issue-6929

Conversation

@mrveiss

@mrveiss mrveiss commented May 16, 2026

Copy link
Copy Markdown
Owner

Summary

  • Adds pytest pipeline-scripts/test_audit_unwired_trackers.py step to .github/workflows/unwired-tracker-audit.yml, running before the audit step
  • The 76-test suite (512 lines) already exists in the repo — it covers regex shapes, is_test_path, scan boundaries, dedup, and end-to-end fixtures
  • Test failure breaks the workflow fail-fast before the audit runs

Problem solved

pipeline-scripts/audit_unwired_trackers.py is the Tier-3 cron defense for the #6836 closure-gate process. It had 387 LOC and zero automated tests. Silent regressions in the regex, scan, or dedup paths would surface as "no findings" rather than a clear test failure.

Acceptance criteria

  • Test file exists with ≥6 test cases per function — 76 tests total ✅
  • Tests pass on clean checkout ✅
  • Tests run in the cron workflow before the audit step ✅

Test plan

🤖 Generated with Claude Code

Wire the 76-test suite in pipeline-scripts/test_audit_unwired_trackers.py
into the unwired-tracker-audit workflow so a regression in the regex,
scan, or dedup paths breaks CI before the audit step runs. Addresses the
Tier-3 dogfooding gap identified in #6929: the audit script had 387 LOC
and zero automated tests.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
@mrveiss mrveiss merged commit 04f9697 into Dev_new_gui May 16, 2026
0 of 5 checks passed
@mrveiss mrveiss deleted the issue-6929 branch May 16, 2026 19:40
@github-actions

Copy link
Copy Markdown
Contributor

⚠️ SSOT Configuration Compliance: Violations Found

Metric Count
Total Violations 7
SSOT Violations (high priority) 7
Other Violations 0

⚠️ 7 values have SSOT config equivalents!

These should be replaced with SSOT config imports:

Python:

from src.config.ssot_config import config
# Use: config.vm.main, config.port.backend, config.backend_url

TypeScript:

import config from '@/config/ssot-config'
// Use: config.vm.main, config.port.backend, config.backendUrl

📖 See SSOT_CONFIG_GUIDE.md for documentation.

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.

1 participant