Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
As a follow up of https://www.drupal.org/project/shield/issues/2953625, we should add more tests to cover the basic features of shield.
Suggestion:
- Enable checkbox
- Auth message
- Shield credential provider
Comment | File | Size | Author |
---|---|---|---|
#4 | shield-tests-3133349-3.patch | 3.5 KB | vbouchet |
| |||
#2 | shield-tests-3133349-2.patch | 2.82 KB | vbouchet |
|
Comments
Comment #2
vbouchetPlease find a patch which extends the test coverage.
Comment #4
vbouchetExisting test is failing because I was enabling/configuring shield in the setup() method. I updated the setup() so it prepare everything but does not enable shield feature. I updated my test cases so it does not assume shield feature is enabled.
Updated patch.
Comment #5
vbouchetComment #7
geek-merlinThanks, that's brilliant. Can you estimate what parts are missing in coverage?