From alexpott's comment (#24) in #2875279: Update core modules to use the new batch builder, the BatchBuilderTest class needs to be moved to core/tests/Drupal/Tests/Core/Batch.

CommentFileSizeAuthor
#2 batch.system-Batch-Builder-tests-in-the-wrong-location-2982983-2.patch2.75 KBsvetoslav.dragoev
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

John Cook created an issue. See original summary.

svetoslav.dragoev’s picture

svetoslav.dragoev CreditAttribution: svetoslav.dragoev at FFW commented
Status: Active » Needs review
FileSize
batch.system-Batch-Builder-tests-in-the-wrong-location-2982983-2.patch2.75 KB

I've moved and updated the batch test class file as described to the core/tests/Drupal/Tests/Core/Batch folder and I am providing the patch for review.

John Cook’s picture

John Cook CreditAttribution: John Cook at Creode commented
Status: Needs review » Reviewed & tested by the community

Thanks for the patch svetoslav.dragoev.

The patch looks good and all references to the old location have been changed.

Setting to RTBC.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Acro Commerce commented
Status: Reviewed & tested by the community » Fixed

Committed f35ef74 and pushed to 8.6.x. Thanks!

  • alexpott committed f35ef74 on 8.6.x 
    Issue #2982983 by svetoslav.dragoev, John Cook: Batch Builder tests in...

Status: Fixed » Closed (fixed)

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