[10:45:21]
<Yunohost Git/Infra notifications> [uptime-kuma_ynh] DeMiro5001 merged [pull request #186](https://github.com/YunoHost-Apps/uptime-kuma_ynh/pull/186) : Upgrade to v2.5.8
[10:45:30]
<Yunohost Git/Infra notifications> [uptime-kuma_ynh] DeMiro5001 deleted branch ci-auto-update-2.5.8
[10:45:44]
<Yunohost Git/Infra notifications> [uptime-kuma_ynh] DeMiro5001 opened [pull request #188](https://github.com/YunoHost-Apps/uptime-kuma_ynh/pull/188) : Testing
[10:46:01]
<Yunohost Git/Infra notifications> [uptime-kuma_ynh] DeMiro5001 merged [pull request #188](https://github.com/YunoHost-Apps/uptime-kuma_ynh/pull/188) : Testing
[11:08:14]
<Yunohost Git/Infra notifications> [peertube_ynh] orhtej2 [commented](https://github.com/YunoHost-Apps/peertube_ynh/pull/573#issuecomment-6467407403) on [issue #573](https://github.com/YunoHost-Apps/peertube_ynh/pull/573) Upgrade to v8.3.2: changelog This release contains important security fixes, refer to [upstream changelog](https://github.com/Chocobozzz/P...
[11:08:29]
<Yunohost Git/Infra notifications> [peertube_ynh] orhtej2 merged [pull request #573](https://github.com/YunoHost-Apps/peertube_ynh/pull/573) : Upgrade to v8.3.2
[11:08:37]
<Yunohost Git/Infra notifications> [baikal_ynh] robosatsfan [commented](https://github.com/YunoHost-Apps/baikal_ynh/issues/105#issuecomment-6467415828) on [issue #105](https://github.com/YunoHost-Apps/baikal_ynh/issues/105) Log in as yunohost user to Baikal: Some context that might help here (and a docs suggestion): the package uses LDAP-UserBind, so DAV logins are checked aga...
[11:08:44]
<Yunohost Git/Infra notifications> [peertube_ynh] orhtej2 opened [pull request #574](https://github.com/YunoHost-Apps/peertube_ynh/pull/574) : Testing
[11:08:53]
<Yunohost Git/Infra notifications> [peertube_ynh] orhtej2 merged [pull request #574](https://github.com/YunoHost-Apps/peertube_ynh/pull/574) : Testing
[12:03:39]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #3682](https://github.com/YunoHost/apps/pull/3682) : Add Parenthèse to wishlist
[12:05:10]
<Yunohost Git/Infra notifications> [apps] Limezy [commented](https://github.com/YunoHost/apps/pull/3682#issuecomment-6468508830) on [issue #3682](https://github.com/YunoHost/apps/pull/3682) Add Parenthèse to wishlist: Je ne suis pas sûr pour la license malheureusement, à voir
[13:55:40]
<sitegui> Coucou ^^
I've recently fixed 2 problems in the packaging of PeerTube that could have been caught by UI end-to-end tests, using something like playwright for example.
Was it ever discussed before?
I guess two clear downsides are:
- the additional work to write and maintain these tests
- the additional cost of running CI
But the clear upside is increased app quality:
- reduced time to detect problems
- updates that are less likely to break
I know we have a very good CI already, which runs install, backup, restore, etc.
But when it comes to testing if the app is "working", it can only do some curls, right? (doc: https://github.com/YunoHost/package_check#teststoml-syntax)
[14:01:32]
<sitegui> I would like to do an experiment and write some e2e tests to the top 5 most-popular apps (nextcloud, vaultwarden, my_webapp, jellyfin, roundcube). Together, they make up 15% of all stars.
[14:07:55]
<sitegui> What do you think?
[14:48:05]
<m606> hello, other than me should answer, but I agree that blindly trusting the CI to merge a PR regularly results in some troubles (e.g. https://github.com/YunoHost-Apps/kanboard_ynh/issues/233). And I haven't spent time to write specific tests for the app packages I maintain and resort to some manual test (obviously these packages are much simpler than most of the ones you are referring to). But I'm wondering what kind of things you think should be done with playwright that could not be done with curl?
[15:17:59]
<sitegui> For example, in Nextcloud with playwright one test would be:
1. user logins in, uploads a file and generates a public sharing link
2. another anonymous user checks if the link loads
Doing it with curl seems harder and more prone to false breakage, since we would need to reverse engineer the app
[15:18:28]
<sitegui> A concrete example, in my app "nidimages" I have a test for adding comments:
https://git.sitegui.dev/sitegui/nidimages/src/commit/7f818a8f2c66a614eba66b062b1ae6c711c92f32/e2e-tests/tests/media_item_comment.spec.js#L4
[16:15:21]
<m606> hmm on the basis of the example you mention, regarding JS/Playwrite vs. Python/Curl I am not sure that playwright is that simpler than pycurl. I mean, maybe more of a question of being used to it than actual difference of complexity?
Because basically I am under the impression that the imports [you do there](https://git.sitegui.dev/sitegui/nidimages/src/commit/7f818a8f2c66a614eba66b062b1ae6c711c92f32/e2e-tests/tests/media_item_comment.spec.js#L2) is what you would find in [this lib file](https://github.com/YunoHost/package_check/blob/main/lib/curl_tests.py) (at least once they would have been written, as custom login, upload, and link generation helpers do not exist yet - here's [how to submit a basic form with pycurl](https://stackoverflow.com/questions/38393830/uploading-a-file-via-pycurl), not that complex) and then you have to instantiate them in the app's `tests.toml` as in this example with [SSO login and file existence check](https://github.com/YunoHost/package_check/blob/f7d32cadac3cfced282decede024707fba9e6ec6/README.md?plain=1#L208-L210).
[16:19:19]
<m606> In current `package_check` implementation, I notice it takes to write generic helpers for the feature you need, get them merged in that repo, and then only be able to use them at app package level via `tests.toml`.
[16:23:22]
<m606> please tell me how you see it, but it seems to me that a similar amount of "reverse engineering" is required in both case (to write helpers, be it `commons.js` or `curl_tests.py`).
[16:23:39]
<m606> please tell me how you see it, but it seems to me that a similar amount of "reverse engineering" is required in both case (to write helpers, be it in `commons.js` or in `curl_tests.py`).
[17:21:44]
<Yunohost Git/Infra notifications> [nextcloud_ynh] Thatoo opened [issue #921](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/921) : WebSocket Server for Real-time Collaboration
[17:41:07]
<Yunohost Git/Infra notifications> [nextcloud_ynh] Thovi98 [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/921#issuecomment-6476237189) on [issue #921](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/921) WebSocket Server for Real-time Collaboration: Isnt it the purpose of https://apps.yunohost.org/app/nextcloud-whiteboard?
[19:12:32]
<Yunohost Git/Infra notifications> [nextcloud_ynh] Thatoo [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/921#issuecomment-6478211331) on [issue #921](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/921) WebSocket Server for Real-time Collaboration: I love you yunohost
[19:12:32]
<Yunohost Git/Infra notifications> [nextcloud_ynh] Thatoo closed [issue #921](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/921) : WebSocket Server for Real-time Collaboration
[20:30:21]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #3683](https://github.com/YunoHost/apps/pull/3683) : Add tootik to wishlist