-
Notifications
You must be signed in to change notification settings - Fork 14
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
Automated tests for updater app #268
Comments
@jvillafanez do we have such tests ? Do they check integrity at the end ? |
Nope. I don't think we have any automatic tests with the updater app. The tests we have are for the |
Some ideas: write behat tests similar to the ones we have in core but which are independent from the install. This idea could build up on top of this other idea: #329 So if the test is able to setup a tarball, it can also use OCC commands to change config.php's link to a test update server (if needed). And for the "lastupdatedat" refresh the value can be deleted using the "occ config:*" commands. Not sure if behat would be overkill in this case. We might want to add additional assertion cases to check after the update and also some preconditions (ex: LDAP was setup + some users + encryption) before running the update. |
cc @davitol |
Don't only automatically test 9.0.2->9.0.3 (the latest version) but also from previous versions to 9.0.3.
Could help avoid discovering issues like owncloud/updater#358 too later.
The text was updated successfully, but these errors were encountered: