-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
tests: fix failing header acceptance test #20347
Conversation
The username is listed in a div with class `fullname` now. Signed-off-by: Azul <azul@riseup.net>
951ed57
to
62f7e6b
Compare
Looks like there are acceptance tests in 3 files failing right now:
This one fixes the last. I'm happy to continue working on these to get a green CI.
Is there a faster way that keeps the docker containers up or runs them in the main system? |
You're probably not going to get reliable green CI. There seem to be some agents on the CI system that are really slow for some reason so @nextcloud/infrastructure probably has to deal with that. Of course, nothing bad about trying to get the tests we have to be correct! BTW, I also host drone instances so let me know if I can be of help with that?
Take a look at the |
We improved a lot lately on lowering the load on our CI infrastructure :) Thanks a lot for the fix on header! Works nicely!! |
It's basically like a betting game. If you do it wrong, of course you lose. If you do it right, then maybe, just maybe, you'll get green ✔️ . :) |
Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22 |
The username is listed in a div with class
fullname
now.