Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
I have a few tasks in mind for this issue:
- Investigate whether multiple tests can be consolidated into a single class, particularly for the migration tests.
- Flatten test class directories where possible, particularly for the migration tests.
- Replace docblocks with
@covers
annotations where possible.
Comment | File | Size | Author |
---|---|---|---|
#2 | 3156787-2.patch | 25.35 KB | dcam |
Comments
Comment #2
dcam CreditAttribution: dcam commentedI didn't end up consolidating the test classes as much as I originally hoped that I would. In particular I was hoping to consolidate migration test classes. I avoided it because each browser test method causes a new test environment to be generated. Consolidating classes would have meant consolidating setUp() methods. More complex test setups would have been run for simpler tests unnecessarily, resulting in a performance penalty.
Comment #4
dcam CreditAttribution: dcam commented