[Regression] Widget undo test failure

[Olek Nowodziński]

• Where: ​http://tests.ckeditor.dev:1030/tests/plugins/widget/undo (​http://tests.ckeditor.dev:1030/tests/plugins/widget/undo#tests%2Fplugins%2Fwidget%2Fundo%20-%20test%20d%26d%20inline%20widget)
• First bad commit ​git:bfdd04262f9.
• Firefox, Chrome

[Artur Delura]

Works for me.

[Jakub Ś]

Can this be related to #13334? If yes, please also see other ticket from this user.

[Piotrek Koszuliński]

There's a small mess in the dnd tests, because usually all three events - dragstart, drop and dragend are using the same mocked native event. It meant that e.g. all 3 events were fired with the same target. Moreover, this target was totally unrealistic because it was a typeless dom.domObject.

I made small cleanup here and there. Most importantly, the target is now set to more realistic values in the widget tests. And the widgets check the target the most, because it is important in dragstart and drop (hence test failure).

Changes in ​branch:t/13346.

[Piotr Jasiun]

Replying to Reinmar:

Most importantly, the target is now set to more realistic values in the widget tests.

No, it was not ;P dragstart and dragend should have the same target which is dragged element, while drop target in the element on which we drop. Also if test emulate external drag and drop, so it has no dragstart, it should have no dragend too. I fixed this and push changes to the same branch. But I agree, there was a mess and the unrealistic target.

Check if changes makes sense and close this ticket if they do.

[Piotrek Koszuliński]

Merged to major with ​git:ebc9132.