Skip to content

Commit

Permalink
Revert "add comments to describe the reason for disabled test cases […
Browse files Browse the repository at this point in the history
…skip ci]"

This reverts commit 119cd49.
  • Loading branch information
briskt committed May 15, 2024
1 parent c4afaf1 commit c651a0d
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions features/mfa.feature
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,6 @@ Feature: Prompt for MFA credentials
| | TOTP | | supports WebAuthn | TOTP |
| | TOTP | , backup codes | supports WebAuthn | TOTP |
| | | backup codes | supports WebAuthn | backup code |
# The following cases are disabled due to lack of test support for changing web client user agent
# | WebAuthn | | | does not support WebAuthn | WebAuthn |
# | WebAuthn | , TOTP | | does not support WebAuthn | TOTP |
# | WebAuthn | | , backup codes | does not support WebAuthn | backup code |
Expand All @@ -164,7 +163,6 @@ Feature: Prompt for MFA credentials
| TOTP | WebAuthn | supports WebAuthn | WebAuthn |
| TOTP | backup code | supports WebAuthn | backup code |
| backup code | TOTP | supports WebAuthn | TOTP |
# The following case is disabled due to lack of test support for changing web client user agent
# | TOTP | WebAuthn | does not support WebAuthn | TOTP |

Scenario: Defaulting to the manager code despite having a used mfa
Expand All @@ -182,7 +180,6 @@ Feature: Prompt for MFA credentials
Examples:
| supports WebAuthn or not | should or not |
| supports WebAuthn | should not |
# The following case is disabled due to lack of test support for changing web client user agent
# | does not support WebAuthn | should |

Scenario Outline: When to show the link to send a manager rescue code
Expand Down

0 comments on commit c651a0d

Please sign in to comment.