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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Lendude created an issue. See original summary.

Lendude’s picture

Lendude
Primary language Dutch
Location Amsterdam
CreditAttribution: Lendude at iO commented
Status: Active » Needs review
FileSize
3233480-2.patch1015 bytes

This should do it...

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia 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
CreditAttribution: Lendude at iO 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
CreditAttribution: Lendude at iO commented
FileSize
3233480-5-FAIL.patch1.39 KB
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
CreditAttribution: longwave at Full Fat Things 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
CreditAttribution: catch at Third and Grove 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.