While #3042591: Drupal 9 Deprecated Code Report was pretty useless regarding deprecated code, the report correctly pointed out that the \Drupal\tests\search_api\Kernel\Views\TaxonomyTermArgumentTest has an incorrect namespace – the \tests\ part should be capitalized.

CommentFileSizeAuthor
#4 3044867-4--correct_TaxonomyTermArgumentTest_namespace.patch1.05 KBdrunken monkey
#4 3044867-4--correct_TaxonomyTermArgumentTest_namespace--interdiff.txt847 bytesdrunken monkey
#2 3044867-2--correct_TaxonomyTermArgumentTest_namespace.patch492 bytesdrunken monkey

Comments

drunken monkey created an issue. See original summary.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Active » Needs review
StatusFileSize
new 3044867-2--correct_TaxonomyTermArgumentTest_namespace.patch492 bytes

Status: Needs review » Needs work

The last submitted patch, 2: 3044867-2--correct_TaxonomyTermArgumentTest_namespace.patch, failed testing. View results

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Needs work » Needs review
StatusFileSize
new 3044867-4--correct_TaxonomyTermArgumentTest_namespace--interdiff.txt847 bytes
new 3044867-4--correct_TaxonomyTermArgumentTest_namespace.patch1.05 KB
borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
commented
Status: Needs review » Reviewed & tested by the community

Sure - simple enough change.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Reviewed & tested by the community » Fixed

Excellent, thanks for reviewing!
Committed.

  • drunken monkey committed 9a13de4 on 8.x-1.x 
    Issue #3044867 by drunken monkey, borisson_: Fixed namespace of...

Status: Fixed » Closed (fixed)

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