test(playwright): add basic tests for editor API

[mejo-]

🏁 Checklist

• Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
• Sign-off message is added to all commits
• Tests (unit, integration and/or end-to-end) passing and the changes are covered with tests

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[max-nextcloud]

All in all looks good. The create...Editor functions could be moved into fixtures for easier reuse. But we can also do that later, just need to make sure not to copy them anywhere else / rewrite them.

[mejo-]

All in all looks good. The create...Editor functions could be moved into fixtures for easier reuse. But we can also do that later, just need to make sure not to copy them anywhere else / rewrite them.

That's a good point. I refactored the tests with help of copilot and think they're much cleaner structured now. Let me know what you think 😊

[max-nextcloud]

Just some comments. I think the AI artifacts are showing. 😉

[max-nextcloud]

You're not using the containerId fixture from below. So I think you can either do away with it or add it above to the { page }.

[mejo-]

I don't see a reason to make it configurable. At the same time it needs to be exposed by the fixture as the tests need it in order to use it in CSS selectors.