Skip to content

test rewrite: legacy intg test_pam_responder.py - first batch - #9004

Open
danlavu wants to merge 1 commit into
SSSD:masterfrom
danlavu:rewrite-test_pam_responder.py1
Open

test rewrite: legacy intg test_pam_responder.py - first batch#9004
danlavu wants to merge 1 commit into
SSSD:masterfrom
danlavu:rewrite-test_pam_responder.py1

Conversation

@danlavu

@danlavu danlavu commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Port sssd/src/tests/intg/test_pam_responder.py to test_smartcard.py, test_authentication.py

  • test_smartcard__login_fails_when_wrong_pin_is_entered
  • test_smartcard__login_fails_when_card_is_not_mapped
  • test_smartcard__cert_auth_limited_to_allowed_pam_services
  • test_smartcard__login_succeeds_when_cert_auth_required
  • test_smartcard__login_fails_when_cert_auth_required_without_card
  • test_authentication__custom_password_prompt_is_shown_at_login

Peeling out the reviewed test cases into it's own PR, from #8873

Model used: Claude Sonnet 4.6

Comment thread src/tests/system/tests/test_authentication.py Fixed
Comment thread src/tests/system/tests/test_authentication.py Fixed
Comment thread src/tests/system/tests/test_authentication.py Fixed
Port sssd/src/tests/intg/test_pam_responder.py to test_smartcard.py,
test_authentication.py

- test_smartcard__login_fails_when_wrong_pin_is_entered
- test_smartcard__login_fails_when_card_is_not_mapped
- test_smartcard__cert_auth_limited_to_allowed_pam_services
- test_smartcard__login_succeeds_when_cert_auth_required
- test_smartcard__login_fails_when_cert_auth_required_without_card
- test_authentication__custom_password_prompt_is_shown_at_login

Peeling out the reviewed test cases into it's own PR, from
SSSD#8873

Co-authored-by: Cursor <cursoragent@cursor.com>
Model used: Claude Sonnet 4.6
@danlavu
danlavu force-pushed the rewrite-test_pam_responder.py1 branch from 0822131 to 5245b03 Compare July 24, 2026 18:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants