summaryrefslogtreecommitdiffstats
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'master' into contactsmenuJan-Christoph Borchardt2017-04-262-1/+457
|\ | | | | | | Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>
| * Merge pull request #4499 from nextcloud/downstream-26984Morris Jobke2017-04-251-1/+49
| |\ | | | | | | Trigger change when a user is enabled/disabled
| | * Trigger change when a user is enabled/disabledJoas Schilling2017-04-251-1/+49
| | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
| * | AppToken to 72 charsRoeland Jago Douma2017-04-251-2/+2
| | | | | | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
| * | Add new auth flowLukas Reschke2017-04-251-0/+408
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This implements the basics for the new app-password based authentication flow for our clients. The current implementation tries to keep it as simple as possible and works the following way: 1. Unauthenticated client opens `/index.php/login/flow` 2. User will be asked whether they want to grant access to the client 3. If accepted the user has the chance to do so using existing App Token or automatically generate an app password. If the user chooses to use an existing app token then that one will simply be redirected to the `nc://` protocol handler. While we can improve on that in the future, I think keeping this smaller at the moment has its advantages. Also, in the near future we have to think about an automatic migration endpoint so there's that anyways :-) If the user chooses to use the regular login the following happens: 1. A session state token is written to the session 2. User is redirected to the login page 3. If successfully authenticated they will be redirected to a page redirecting to the POST controller 4. The POST controller will check if the CSRF token as well as the state token is correct, if yes the user will be redirected to the `nc://` protocol handler. This approach is quite simple but also allows to be extended in the future. One could for example allow external websites to consume this authentication endpoint as well. Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* | Adjust entry unit test to newly added avatar propertyChristoph Wurst2017-04-251-0/+1
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Use absolute URI for action iconsChristoph Wurst2017-04-251-3/+16
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Don't show contacts an entry for themselvesChristoph Wurst2017-04-252-10/+49
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Let apps register contact menu provider via info.xmlChristoph Wurst2017-04-253-13/+73
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Contacts menuChristoph Wurst2017-04-258-0/+715
|/ | | | | | | * load list of contacts from the server * show last message of each contact Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Merge pull request #3531 from nextcloud/theming-scssMorris Jobke2017-04-251-23/+33
|\ | | | | Theming using SCSS variables
| * Implement scss variable injection by OC_DefaultsJulius Haertl2017-04-251-23/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Julius Haertl <jus@bitgrid.net> Add Scss variables to example theme and theming app Signed-off-by: Julius Haertl <jus@bitgrid.net> Use SCSSCacher to build theming css Signed-off-by: Julius Härtl <jus@bitgrid.net> Update theming.scss Signed-off-by: Julius Härtl <jus@bitgrid.net> Code cleanup Signed-off-by: Julius Härtl <jus@bitgrid.net> Fix tests Signed-off-by: Julius Härtl <jus@bitgrid.net> Inject SCSSCacher for easier testing Signed-off-by: Julius Härtl <jus@bitgrid.net> Fix typehint Signed-off-by: Lukas Reschke <lukas@statuscode.ch> Generate absolute URLs Signed-off-by: Lukas Reschke <lukas@statuscode.ch> Fix tests to always use absolute urls for theming images Signed-off-by: Julius Härtl <jus@bitgrid.net> MailheaderColor -> ColorPrimary Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Merge pull request #3195 from nextcloud/settings-apps-tabularMorris Jobke2017-04-251-4/+9
|\ \ | | | | | | Make apps settings tabular
| * | Add installed category in AppSettingsControlerTestFelix A. Epp2017-04-251-4/+9
| |/ | | | | | | Signed-off-by: Felix A. Epp <work@felixepp.de>
* | Merge pull request #4462 from danxuliu/fix-sharing-password-protected-linkRoeland Jago Douma2017-04-257-5/+403
|\ \ | | | | | | Fix sharing a password protected link
| * | Add missing unit test for updateShare with email shareDaniel Calviño Sánchez2017-04-241-0/+75
| | | | | | | | | | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
| * | Add missing hook checkDaniel Calviño Sánchez2017-04-241-2/+13
| | | | | | | | | | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
| * | Fix double hashing of shared link passwordsDaniel Calviño Sánchez2017-04-241-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The plain text password for a shared links was hashed and, then, the hashed password was hashed again and set as the final password. Due to this the password introduced in the "Authenticate" page for the shared link was always a wrong password, and thus the file could not be accessed. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
| * | Add acceptance tests for sharing password protected linksDaniel Calviño Sánchez2017-04-244-0/+280
| | | | | | | | | | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
| * | Add system to share data between acceptance test stepsDaniel Calviño Sánchez2017-04-242-3/+29
| | | | | | | | | | | | | | | | | | | | | | | | The data storage (the "notebook") is shared between all the actors, so the data can be stored and retrieved between different steps by any actor in the same scenario. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | | Fix remember redirect_url on failed login attemptsChristoph Wurst2017-04-251-4/+6
| |/ |/| | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Merge pull request #4477 from danxuliu/acceptance-macos-nitpickingRoeland Jago Douma2017-04-241-27/+32
|\ \ | | | | | | Fix minor code style issues in acceptance test runner
| * | Fix minor code style issues (also known as nitpicking)Daniel Calviño Sánchez2017-04-221-27/+32
| |/ | | | | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | Add unit tests for mounts of delete usersJoas Schilling2017-04-241-0/+24
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Merge pull request #1023 from GitHubUser4234/ldap_password_renew_prblizzz2017-04-241-0/+2
|\ \ | |/ |/| Handle password expiry in user_ldap
| * unit test adjustmentRoger Szabo2017-04-071-0/+2
| | | | | | | | Signed-off-by: Roger Szabo <roger.szabo@web.de>
* | Run acceptance tests on macOSMorris Jobke2017-04-211-4/+30
| | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | Merge pull request #4208 from danxuliu/add-basic-acceptance-test-systemRoeland Jago Douma2017-04-2123-0/+2108
|\ \ | | | | | | Add basic acceptance test system
| * | Make test passwords valid for the password_policy appDaniel Calviño Sánchez2017-04-213-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | As requested by Morris Jobke, the passwords in the acceptance tests were modified to make them valid both for a clean Nextcloud server and one with the password_policy app enabled. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
| * | Move acceptance tests from build/acceptance to tests/acceptanceDaniel Calviño Sánchez2017-04-2123-0/+2108
| | | | | | | | | | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | | Make sure we use a new encryption module all the timeJoas Schilling2017-04-201-7/+16
| | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Don't assume the admin didn't configure Opcache correctly...Joas Schilling2017-04-201-3/+5
| | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Use the correct classJoas Schilling2017-04-201-1/+1
| | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Import some classesJoas Schilling2017-04-201-80/+51
| | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Use the correct Dummy and Backend classJoas Schilling2017-04-201-2/+2
| | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Kill dead codeJoas Schilling2017-04-201-125/+0
| | | | | | | | | | | | | | | | | | > No tests found in class "Test\Share\MailNotificationsTest". Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Fix warning with undefined methodJoas Schilling2017-04-201-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | Trying to configure method "getRemember" which cannot be configured because it does not exist, has not been specified, is final, or is static Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Don't use deprecated getMock() anymoreJoas Schilling2017-04-203-16/+18
| | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Restoring the error handler within the error handler causes unexpected resultsJoas Schilling2017-04-201-3/+10
| | | | | | | | | | | | | | | | | | | | | See http://php.net/manual/en/function.restore-error-handler.php#120879 for more information. Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Delete the correct config valueJoas Schilling2017-04-201-1/+1
| | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Merge pull request #4398 from nextcloud/fix_accesslistcodeJoas Schilling2017-04-201-2/+1
|\ \ \ | | | | | | | | Get proper accesslist for userFolder
| * | | Get proper accesslist for userFolderRoeland Jago Douma2017-04-201-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | If the accesslist is requested for a users root folder we should properly construct the path Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | Merge pull request #4212 from individual-it/masterJoas Schilling2017-04-201-1/+2
|\ \ \ \ | | | | | | | | | | validate file name before uploading in upload only folder
| * | | | JS tests for upload only functionArtur Neumann2017-04-181-1/+2
| | | | | | | | | | | | | | | Signed-off-by: Artur Neumann <info@individual-it.net>
* | | | | Rename renderHTML to renderHtmlMorris Jobke2017-04-193-10/+10
| |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | * fixes #4383 * improves consistency Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | | | Merge pull request #4350 from ↵Roeland Jago Douma2017-04-192-55/+23
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | nextcloud/adjust-old-bruteforce-protection-annotations Adjust existing bruteforce protection code
| * | | Adjust existing bruteforce protection codeLukas Reschke2017-04-142-55/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | - Moves code to annotation - Adds the `throttle()` call on the responses on existing annotations Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* | | | Merge pull request #4371 from nextcloud/dont-allow-dot-usernamesMorris Jobke2017-04-181-0/+43
|\ \ \ \ | | | | | | | | | | Better validation of allowed user names
| * | | | Better validation of allowed user namesJoas Schilling2017-04-181-0/+43
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | | | Merge pull request #4369 from nextcloud/fix-translationsMorris Jobke2017-04-185-9/+9
|\ \ \ \ \ | | | | | | | | | | | | Fix translations