Open kreynen opened 1 year ago
Adding thorough testing for this module is difficult since it depends on a SAML service and usernames/passwords to actually authenticate, but even doing some basic testing similar to what is done in the D9 version of the project https://dispatcher.drupalci.org/job/drupal_contrib/606674/console would help catch issues like https://github.com/backdrop-contrib/simplesamlphp_auth/pull/3 and PHP version compatibility noise.
The process for testing Backdrop modules is a little different, but well documented at https://docs.backdropcms.org/change-records/information-about-tests-now-in-moduletestsinfo
Ideally we'd have at least a module install testing added before pushing a 1.0.0 release, but the D7 branches still have > 9K reported installs without any testing.
Adding thorough testing for this module is difficult since it depends on a SAML service and usernames/passwords to actually authenticate, but even doing some basic testing similar to what is done in the D9 version of the project https://dispatcher.drupalci.org/job/drupal_contrib/606674/console would help catch issues like https://github.com/backdrop-contrib/simplesamlphp_auth/pull/3 and PHP version compatibility noise.
The process for testing Backdrop modules is a little different, but well documented at https://docs.backdropcms.org/change-records/information-about-tests-now-in-moduletestsinfo
Ideally we'd have at least a module install testing added before pushing a 1.0.0 release, but the D7 branches still have > 9K reported installs without any testing.