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.
Tests! Moar tests!
Comment | File | Size | Author |
---|---|---|---|
#4 | add_tests-2832671-4.patch | 3.18 KB | tlyngej |
|
Comments
Comment #2
jrockowitz CreditAttribution: jrockowitz as a volunteer commented@tlyngej The lack of tests was one of the first things I noticed when I reviewed your module. Your solution makes sense and adding tests would confirm that it is working as expected.
Comment #3
tlyngej CreditAttribution: tlyngej at Annertech commented@jrockowitz, you are right, and I am ashamed by the lack of tests!
I'll get something up here soon.
Comment #4
tlyngej CreditAttribution: tlyngej at Annertech commentedYay! Tests has arrived.
I have restored my honour :-)
Comment #5
jrockowitz CreditAttribution: jrockowitz as a volunteer commentedLooks good to me.
Comment #6
tlyngej CreditAttribution: tlyngej at Annertech commentedPatch committed. Thanks a lot for pushing me ;-)
Comment #8
tlyngej CreditAttribution: tlyngej at Annertech commentedComment #9
tlyngej CreditAttribution: tlyngej at Annertech commentedComment #10
tlyngej CreditAttribution: tlyngej at Annertech commented