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.
Problem/Motivation
We can improve one of our tests to resolve a @todo with testing the labels of the edit tabs.
Proposed resolution
Add tests.
Remaining tasks
User interface changes
API changes
Data model changes
Comment | File | Size | Author |
---|---|---|---|
#3 | 2843997-tab-label-coverage-3.patch | 3.09 KB | Sam152 |
#3 | interdiff.txt | 1.81 KB | Sam152 |
#2 | 2843997-tab-label-coverage-2.patch | 2.21 KB | Sam152 |
Comments
Comment #2
Sam152 CreditAttribution: Sam152 as a volunteer and at PreviousNext commentedComment #3
Sam152 CreditAttribution: Sam152 as a volunteer and at PreviousNext commentedMay as well get the other tab @todo while we're here.
Comment #4
timmillwoodComment #5
jeqqComment #7
timmillwoodComment #9
cilefen CreditAttribution: cilefen commentedWould it not be simpler to concatenate in this test code instead of sprintf? Let's open a novice task to replace this and other similar stuff in the test.
Comment #10
cilefen CreditAttribution: cilefen commented.. and by the way, we can still commit this as-is.
Comment #13
cilefen CreditAttribution: cilefen commentedThank you! We backported this to 8.3.x because it is a test improvement to an experimental module.
Comment #14
cilefen CreditAttribution: cilefen commentedComment #15
xjmComment #16
xjm(Since it was backported as experimental module tests).