CommentFileSizeAuthor
#2 simplenews.tests-broken-3045850-2.patch599 bytesadamps

Comments

AdamPS created an issue. See original summary.

adamps’s picture

Status: Active » Needs review
StatusFileSize
new599 bytes

The tests seem to work if I just delete that line of code

berdir’s picture

Yes, some bugs were already fixed other fixes are only in 8.7, make sure to do test runs against 8.6 at least as well.

See also #3043800: Clean applyUpdates to make test HEAD pass again

adamps’s picture

Issue tags: +Plan to commit

Thanks @Berdir - good news it that D8.6 passes too.

  • 4d4511c committed on 8.x-1.x
    Issue #3045850 by AdamPS: Test broken -...
adamps’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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

adamps’s picture

Issue tags: -Plan to commit