CommandCollection and Editor.execute should return a result.

[tomalec]

📝 Provide a description of the improvement

editor.execute, CommandCollection.execute should/could return the command.execute value

ckeditor5/packages/ckeditor5-core/src/commandcollection.js

Line 68 in 1c44993

command.execute( ...args );

ckeditor5/packages/ckeditor5-core/src/editor/editor.js

Line 288 in 1c44993

this.commands.execute( ...args );

The current setup makes testing async commands complicated, as I cannot write

await buttonView.fire( 'execute' );

expect( something ).to.be.done.once.finally.executed;

It also introduces inconsistency, as I can do such things when calling command.execute directly
https://github.com/cksource/ckeditor5-internal/blob/233614fa4153c7cce92ff7f6b3304ba6b197aa8b/packages/ckeditor5-export-pdf/tests/exportpdfcommmand.js#L178-L180

📃 Other details

• Browser: n/a
• OS: n/a
• CKEditor version: master@1c449932924
• Installed CKEditor plugins: n/a

---

If you'd like to see this improvement implemented, add a 👍 reaction to this post.