The RangeQueryTest extends Drupal\KernelTests\Core\Database\DatabaseTestBase and that class has the module "database_test" enabled. So the RangeQueryTest does not have to do that. Lets remove the adding of the module "database_test" from the class RangeQueryTest.

CommentFileSizeAuthor
#3 2855927-3.patch612 bytesjofitz
range_query_test_remove_module_database_test.patch609 bytesdaffie

Comments

daffie created an issue. See original summary.

chi’s picture

chi commented
Priority: Normal » Minor
Status: Needs review » Needs work
Issue tags: +Needs reroll, +Novice
jofitz’s picture

jofitz
he/him
commented
Status: Needs work » Needs review
Issue tags: -Needs reroll, -Novice
StatusFileSize
new 2855927-3.patch612 bytes

Re-rolled.

The last submitted patch, range_query_test_remove_module_database_test.patch, failed testing.

chi’s picture

chi commented
Status: Needs review » Reviewed & tested by the community

This works for me.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 3: 2855927-3.patch, failed testing.

goz’s picture

goz commented
Status: Needs work » Needs review

No reasons to tests to fail. I re-launch tests.

daffie’s picture

daffie commented
Status: Needs review » Reviewed & tested by the community

Back to RTBC.

alexpott’s picture

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

Committed and pushed 4976fb5 to 8.4.x and 303b9ae to 8.3.x. Thanks!

  • alexpott committed 4976fb5 on 8.4.x 
    Issue #2855927 by daffie, Jo Fitzgerald: Drupal\KernelTests\Core\...

  • alexpott committed 303b9ae on 8.3.x 
    Issue #2855927 by daffie, Jo Fitzgerald: Drupal\KernelTests\Core\...

Status: Fixed » Closed (fixed)

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