SimpleTest's unit tests needs some annotation love.

CommentFileSizeAuthor
#4 drupal8-improve_unit_test_annotation_for_simpletest-2216531-4.patch3.22 KBJalandhar
#1 2216531.patch3.21 KBMile23
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Mile23’s picture

Mile23
Primary language English
Location Seattle, WA
CreditAttribution: Mile23 commented
FileSize
2216531.patch3.21 KB

Adding some @covers and @group updates.

Mile23’s picture

Mile23
Primary language English
Location Seattle, WA
CreditAttribution: Mile23 commented
Status: Active » Needs review
ParisLiakos’s picture

ParisLiakos CreditAttribution: ParisLiakos commented
Status: Needs review » Needs work
Issue tags: +Needs reroll
Jalandhar’s picture

Jalandhar CreditAttribution: Jalandhar commented
Status: Needs work » Needs review
FileSize
drupal8-improve_unit_test_annotation_for_simpletest-2216531-4.patch3.22 KB

Updating rerolled patch.

ParisLiakos’s picture

ParisLiakos CreditAttribution: ParisLiakos commented
Status: Needs review » Reviewed & tested by the community
Issue tags: -Needs reroll

thank you!

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

  • Commit 93d73d5 on 8.x by webchick: 
    Issue #2216531 by Jalandhar, Mile23: Improve unit test annotation for...

Status: Fixed » Closed (fixed)

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