Skip to content

test: add CI test coverage for core workflows - #15

Open
chestercheng wants to merge 1 commit into
sciwork:mainfrom
chestercheng:test/ci-coverage-workflows
Open

test: add CI test coverage for core workflows#15
chestercheng wants to merge 1 commit into
sciwork:mainfrom
chestercheng:test/ci-coverage-workflows

Conversation

@chestercheng

@chestercheng chestercheng commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Changes

  • Add locked GitHub Actions checks for Ruff lint, Ruff formatting, pytest, and a downloadable coverage report.
  • Add an OIDC login scenario using a local provider:
    • chester@example.com is allowed.
    • steve@example.com is denied.
  • Add a report delivery scenario:
    • Seed an in-memory database with an active event and tickets.
    • Run report.send_report().
    • Verify the rendered Discord payload and delivery timestamp behavior.
    • Verify failed Discord delivery leaves the report pending for retry.
  • Add a saved Sciwork KKTIX page fragment for parser coverage.
  • Verify webhook logs redact authorization credentials and contact PII.
  • Move Discord payload text and schema into the packaged report.json.j2 production template.

Verification

  • uv run --locked --group dev ruff check src tests
  • uv run --locked --group dev ruff format --check src tests
  • uv run --locked --group dev pytest tests/ -W error --cov=argus --cov-report=term-missing --cov-report=xml
  • uv lock --check

@chestercheng
chestercheng requested a review from iefiru August 13, 2026 21:13
@chestercheng
chestercheng force-pushed the test/ci-coverage-workflows branch 2 times, most recently from d47f760 to 82e4135 Compare August 14, 2026 00:22
Add CI checks for lint, formatting, tests, and coverage artifacts.

Cover Google OIDC allowlist login, Discord report delivery from stored event data, KKTIX event-page parsing, and webhook-log PII redaction.

Signed-off-by: chestercheng <hi@chester.ch>
@chestercheng
chestercheng force-pushed the test/ci-coverage-workflows branch from 82e4135 to 3835573 Compare August 14, 2026 00:36
@codecov

codecov Bot commented Aug 14, 2026

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@chestercheng chestercheng changed the title test: add CI coverage for dashboard and report workflows test: add CI test coverage for core workflows Aug 14, 2026
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