#1890502: WYSIWYG: Add CKEditor module to core was committed with pretty solid test coverage; one of the things it tests are the available CKEditor plugins out-of-the-box. This uses PluginManagerBase::getDefinitions(), which does not provide any guarantees on the order in which the definitions are returned. For some unknown reason, it used to pass just fine at #1890502: WYSIWYG: Add CKEditor module to core, but now, that's no longer the case. (I don't know what the root cause is, but clearly plugin discovery is either stateful or non-deterministic.)

Patch test failures at:

CommentFileSizeAuthor
#2 ckeditor-1912026-1.patch1.14 KBtim.plunkett

Comments

wim leers’s picture

Status: Active » Closed (duplicate)

Double post — sorry.

tim.plunkett’s picture

Status: Closed (duplicate) » Needs review
StatusFileSize
new1.14 KB

They're sorted in the following hunk of tests, not sure why they weren't here.

tim.plunkett’s picture

Status: Needs review » Closed (duplicate)

I was beat to the punch.

wim leers’s picture

wim leers’s picture

Component: editor.module » ckeditor.module