Add Unit test for Paste source from M365 Apps #1918
Conversation
BryanValverdeU
requested review from
flyingbee2012,
JiuqingSong,
ianeli1,
juliaroldi and
Andres-CT98
| editor.paste(clipboardData); | ||
|
|
||
| expect(editor.getContent()).toEqual( | ||
| Browser.isFirefox |
There was a problem hiding this comment.
Why Firefox can generate different HTML than others?
There was a problem hiding this comment.
Not sure about it. I think that is how Firefox handles the HTML. It seems they add family name style to paragraph element. That is the only difference I see.
There was a problem hiding this comment.
Can we also verify the content model inside? Then we may understand why they are different
There was a problem hiding this comment.
Added the content model tests to the new package.
There was a problem hiding this comment.
I am just going to check whether the function that process the content was called.
We already have other unit tests to make sure the content is sanitized correctly. These unit tests will just make sure we do not break the logic to call the function.
…u/bvalverde/addUnitTest
Add unit test using clipboard data from M365 Apps.