Problem/Motivation

InstallerExistingConfigSyncDriectoryProfileMismatchTest should be InstallerExistingConfigSyncDirectoryProfileMismatchTest

Proposed resolution

Fix the typo

CommentFileSizeAuthor
#5 3233480-2.patch1015 byteslendude
#5 3233480-5-FAIL.patch1.39 KBlendude
#2 3233480-2.patch1015 byteslendude

Comments

Lendude created an issue. See original summary.

lendude’s picture

lendude
Primary language Dutch
Location Amsterdam
commented
Status: Active » Needs review
StatusFileSize
new 3233480-2.patch1015 bytes

This should do it...

xjm’s picture

xjm
she/her
Primary language English
commented

Took me a minute to spot it; "Driectory". Nice find.

It's hard to tell from https://dispatcher.drupalci.org/job/drupal_patches/97711/consoleFull if the test is actually running because the output is truncated at Drupal\FunctionalTests\Installer\InstallerExistingConfigSync. It might be good to post a fail-patch or such to prove the test is running?

lendude’s picture

lendude
Primary language Dutch
Location Amsterdam
commented

Since both the file name and the class name contain the typo, I would assume it is running....

lendude’s picture

lendude
Primary language Dutch
Location Amsterdam
commented
StatusFileSize
new 3233480-5-FAIL.patch1.39 KB
new 3233480-2.patch1015 bytes

Here we go, with a fail build in! and the fix patch again

The last submitted patch, 5: 3233480-5-FAIL.patch, failed testing. View results

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Needs review » Reviewed & tested by the community

Shame that cspell did not pick up on this, maybe it doesn't check camel-cased words or this string is too long or something like that.

  • catch committed 4ce6de6 on 9.3.x 
    Issue #3233480 by Lendude, xjm: Fix typo in...
catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed 4ce6de6 and pushed to 9.3.x. Thanks!

Status: Fixed » Closed (fixed)

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