I have a few tasks in mind for this issue:

  1. Investigate whether multiple tests can be consolidated into a single class, particularly for the migration tests.
  2. Flatten test class directories where possible, particularly for the migration tests.
  3. Replace docblocks with @covers annotations where possible.
CommentFileSizeAuthor
#2 3156787-2.patch25.35 KBdcam
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dcam created an issue. See original summary.

dcam’s picture

Status: Active » Needs review
FileSize
25.35 KB

I 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.

  • dcam committed fe40e49 on 8.x-2.x
    Issue #3156787 by dcam: Update tests
    
dcam’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.