Fix 9 spelling words in tests

spellcheck tested in the core folder for the above 9 words which were not found and resolved.

Looks Good.

Below is the report generated by spellcheck on my local machine.

Issues found:
drupalci
corepack
Validatable
validatable
drush
Drush
SIMPLETEST
simpletest
lullabot
langcode
umami
Langcodes
Langcode
dblog
tempstore
langcodes
Umami
olivero
LANGCODE
Olivero
Simpletest
Bartik
dependees
Dependees
Dependee
dependee
dependee's
bartik
druplicon
Dblog
DBLOG
Druplicon
DRUPLICON
Tempstore
Olivero's
UMAMI
CSpell: Files checked: 15264, Issues found: 7385 in 2685 files.

Moving the issue with RTBC+1

Not sure about changing webassert, that is the name of a class in a dependency. If we have tests related to it then they should refer to it by its name, it makes it easier to find all references when needed.

@amanbtr72, thanks for testing this change. These changes also change comments and code so a review needs to include reading the MR and confirming the change make sense. Sometimes a review will reject the proposed change and the 'misspelling' will be kept. For example, we allow the words dependee, dependee's, and dependees even though they are not valid words by cspell or American English dictionaries.

@mstrelan, can you point out the instances you think should not be changed?

Should webassert be moved to Drupal dictionary?

I would be on board with webassert being in the Drupal dictionary.

@quietone mind generating that please?

Since this is about JsWebAssert, I think that is the word to put in the drupal dictionary. Trying that now. What do you think?

Think that’s a good compromise

This all makes sense but needs a rebase.

There were no conflicts with the rebase so I am restoring the RTBC.

Oh I probably could have tried /rebase on the MR, will try to remember that next time.

Committed/pushed to 11.x, thanks!

catch → closed merge request !9112

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