Skip to content

[Maintenance] Add auth status, logout, DWD subject, and compatibility tests - #94

Open
terrancedejesus wants to merge 2 commits into
mainfrom
add-cred-store-compat-tests
Open

[Maintenance] Add auth status, logout, DWD subject, and compatibility tests#94
terrancedejesus wants to merge 2 commits into
mainfrom
add-cred-store-compat-tests

Conversation

@terrancedejesus

@terrancedejesus terrancedejesus commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Keep the existing swat CLI, auth session / auth list, creds, store keys (default / external), pickle store, and emulation session_key arguments unchanged for forks and current operators.
  • Add auth status and auth logout (local session clear only) plus optional auth session --service-account --subject USER@DOMAIN for domain-wide delegation.
  • Lock the public command, emulation, config, and import surface with mocked tests. No live tenant or real credentials.

Test plan

  • poetry run pytest tests/ -v passes locally (70 tests)
  • Existing auth session / auth list / creds add|remove|list invocations still work
  • auth status does not print client secrets or private keys
  • auth logout default clears the session and leaves the stored credential
  • auth session --subject without --service-account is rejected
  • CI pytest job passes

…ands.

Co-authored-by: Cursor <cursoragent@cursor.com>
…aking existing commands.

Co-authored-by: Cursor <cursoragent@cursor.com>
@terrancedejesus terrancedejesus added the documentation Improvements or additions to documentation label Aug 17, 2026
@terrancedejesus terrancedejesus changed the title [Maintenance] Add mocked compatibility tests for credential store and auth commands [Maintenance] Add auth status, logout, DWD subject, and compatibility tests Aug 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation maintenance testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant