This is an important security feature so it should have tests.

CommentFileSizeAuthor
#4 password_policy-7.x-1.x-test_superuser_expiration_setting-2527824-4.patch6.85 KBaohrvetpv
#3 password_policy-7.x-1.x-test_superuser_expiration_setting-2527824-3.patch6.94 KBaohrvetpv
#1 password_policy-7.x-1.x-test_superuser_expiration_setting-2527824-1.patch3.76 KBaohrvetpv

Comments

aohrvetpv’s picture

aohrvetpv commented
StatusFileSize
new password_policy-7.x-1.x-test_superuser_expiration_setting-2527824-1.patch3.76 KB

Initial, simplistic tests

Need to deduplicate code between test methods.

aohrvetpv’s picture

aohrvetpv commented
Status: Active » Needs review
aohrvetpv’s picture

aohrvetpv commented
StatusFileSize
new password_policy-7.x-1.x-test_superuser_expiration_setting-2527824-3.patch6.94 KB

Factored redundant code into new function.

aohrvetpv’s picture

aohrvetpv commented
StatusFileSize
new password_policy-7.x-1.x-test_superuser_expiration_setting-2527824-4.patch6.85 KB

Change tense of a verb in function comment for consistency.

  • AohRveTPV committed 269e4b7 on 7.x-1.x 
    Issue #2527824 by AohRveTPV: No tests for "Admin (UID=1) password...
aohrvetpv’s picture

aohrvetpv commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Needs review » Patch (to be ported)

7.x-1.x tests seem good enough for now.

aohrvetpv’s picture

aohrvetpv commented
Status: Patch (to be ported) » Fixed

6.x-1.x is no longer supported.

aohrvetpv’s picture

aohrvetpv commented
Status: Fixed » Closed (won't fix)