YunoHost-Apps / mattermost_ynh

Mattermost package for YunoHost
http://www.mattermost.org
GNU General Public License v3.0
33 stars 19 forks source link

Testing #471

Closed ericgaspar closed 2 months ago

ericgaspar commented 2 months ago

Problem

Solution

PR Status

Automatic tests

Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

ericgaspar commented 2 months ago

!testme

yunohost-bot commented 2 months ago

:carousel_horse: Test Badge

yunohost-bot commented 2 months ago

:worm: Test Badge

kemenaran commented 2 months ago

The test in error is due to the test runner stating:

Error: There's quite a lot of warnings in the output ! If those warnings are coming from some app build step and ain't actual warnings, please redirect them to the standard output instead of the error output ...!

These are the warning printed when starting the Mattermost server. We could argue that of them should actually be "INFO" instead of "WARNING" – but anyway there legitimate. @ericgaspar any idea on how to make the test runner happy?

kemenaran commented 2 months ago

(Also the upgrade works properly on my server.)