Would be nice to have some tests to make sure that user_password_reset_timeout variable is set correctly

Comments

shrop’s picture

shrop commented
Status: Active » Closed (won't fix)

Closing this. I really don't see any tests that will help. Seems that core test coverage handles any testing needs here. Let me know if I am off though. :)

shrop’s picture

shrop commented
Version: 7.x-1.0 » 7.x-1.x-dev
Status: Closed (won't fix) » Active

While I don't see a need for Drupal 8 tests, I did realize that there is a custom function to verify that the value for the timeout is a number. user_pwreset_timeout_validate(). Can probably test this one.

shrop’s picture

shrop commented
Version: 7.x-1.x-dev » 8.x-1.x-dev
Issue tags: +Add a test to show active config changes to confirm the password_reset_timeout config changes

Add a test to show active config changes to confirm the password_reset_timeout config changes. Once we get this for D8/9 could have an issue to setup a D7 test - but a bit different.

shrop’s picture

shrop commented
Title: Get some tests going for this module » Add a test to show active config changes to confirm the password_reset_timeout config changes
Issue tags: -Add a test to show active config changes to confirm the password_reset_timeout config changes
shrop’s picture

shrop commented
Version: 8.x-1.x-dev » 2.x-dev