Skip to content

Commit

Permalink
MNT Remove webauthn module from behat tests
Browse files Browse the repository at this point in the history
  • Loading branch information
emteknetnz committed Nov 25, 2024
1 parent 49b480d commit f7da6b1
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 12 deletions.
1 change: 0 additions & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@
"squizlabs/php_codesniffer": "^3",
"silverstripe/documentation-lint": "^1",
"silverstripe/recipe-testing": "^4",
"silverstripe/webauthn-authenticator": "^6",
"silverstripe/frameworktest": "^2",
"silverstripe/standards": "^1",
"phpstan/extension-installer": "^1.3"
Expand Down
12 changes: 1 addition & 11 deletions tests/behat/features/mfa-totp.feature
Original file line number Diff line number Diff line change
Expand Up @@ -61,12 +61,9 @@ Feature: Use MFA TOTP
And I log in with "[email protected]" and "Secret!123" without skipping MFA
Then I should not see "Add extra security to your account"

Scenario: User can select a verification method, handles incorrect TOTP codes
Scenario: Handles incorrect TOTP codes
When I log in with "[email protected]" and "Secret!123" without skipping MFA
And I press the "Get started" button
And I should see "Select a verification method"
When I click on the "img[alt='Authenticator app']" element
And I press the "Next" button
# QR code screen
And I press the "Next" button
# Enter invalid code
Expand All @@ -87,13 +84,6 @@ Feature: Use MFA TOTP
Then I should see "Authenticator app: Registered"
And I should see "Recovery codes"

# Add another MFA method
When I press the "Add another MFA method" button
And I wait for 3 seconds
Then I should see "Select a verification method"
# close the modal
When I press the "Back" button

# Reset method - will show a QR code - should still manually test
When I press the TOTP authenticator app "Reset" button with javascript
And I wait for 3 seconds
Expand Down

0 comments on commit f7da6b1

Please sign in to comment.