Reported in #3121479: There's a typo in "mach" at tests/src/Kernel/Resource/CartAddResourceTest.php.

      if (!in_array($value['type'], $target_resource_type_names)) {
        throw new BadRequestHttpException(sprintf(
          'The provided type (%s) does not mach the destination resource types (%s).',
          $value['type'],
          implode(', ', $target_resource_type_names)
        ));
      }

The error should be "match" but instead it shows "match"

CommentFileSizeAuthor
#5 3127641-5.patch2.55 KBmglaman
#2 3127641-2.patch849 bytesmglaman

Comments

mglaman created an issue. See original summary.

mglaman’s picture

Status: Active » Needs review
StatusFileSize
new849 bytes

💪 typo tackled.

logickal’s picture

Status: Needs review » Reviewed & tested by the community

LGTM!

Status: Reviewed & tested by the community » Needs work

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

mglaman’s picture

Status: Needs work » Needs review
StatusFileSize
new2.55 KB

Fix test assertion which also had "mach"

longwave’s picture

Status: Needs review » Reviewed & tested by the community

These are the only instances of " mach " in the codebase.

wim leers’s picture

Issue tags: +API-First Initiative, +Trivial patch of the month

🦅👁👏

  • catch committed 6eda65d on 9.1.x
    Issue #3127641 by mglaman, longwave: Typo in...

  • catch committed 888080d on 9.0.x
    Issue #3127641 by mglaman, longwave: Typo in...
catch’s picture

Version: 9.1.x-dev » 9.0.x-dev
Status: Reviewed & tested by the community » Fixed

Committed 6eda65d and pushed to 9.1.x. Thanks!

Status: Fixed » Closed (fixed)

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