Skip to content
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

Open
PVince81 opened this issue Jul 4, 2016 · 4 comments
Open

Automated tests for updater app #268

PVince81 opened this issue Jul 4, 2016 · 4 comments

Comments

@PVince81
Copy link
Contributor

PVince81 commented Jul 4, 2016

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.

@PVince81
Copy link
Contributor Author

PVince81 commented Jul 4, 2016

@jvillafanez do we have such tests ? Do they check integrity at the end ?

@jvillafanez
Copy link
Member

Nope. I don't think we have any automatic tests with the updater app. The tests we have are for the occ upgrade command; these ones include the integrity check at the beginning and at the end of the upgrade process

@PVince81 PVince81 changed the title Updater tests from 9.0.x -> latest Automated tests for updater app Nov 10, 2016
@PVince81
Copy link
Contributor Author

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.

@PVince81
Copy link
Contributor Author

cc @davitol

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants