summaryrefslogtreecommitdiffstats
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #12185 from nextcloud/cleanup/dicontainerMorris Jobke2018-11-052-17/+7
|\ | | | | Cleanup the DIContainer
| * Fix testsRoeland Jago Douma2018-11-022-17/+7
| | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Merge pull request #12071 from nextcloud/addressbook-uid-check-migrationRoeland Jago Douma2018-11-021-0/+121
|\ \ | | | | | | Addressbook uid check migration
| * \ Merge branch 'addressbook-uid-check-migration' of ↵John Molakvoæ (skjnldsv)2018-11-0112-41/+369
| |\ \ | | | | | | | | | | | | | | | | | | | | https://github.com/nextcloud/server into addressbook-uid-check-migration Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
| | * | TestsJohn Molakvoæ (skjnldsv)2018-11-011-0/+121
| | | | | | | | | | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
| * | | Fix phpunitJohn Molakvoæ (skjnldsv)2018-11-011-20/+20
| | | | | | | | | | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
| * | | TestsJohn Molakvoæ (skjnldsv)2018-10-301-0/+121
| | | | | | | | | | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* | | | Merge pull request #12215 from ↵Roeland Jago Douma2018-11-021-0/+8
|\ \ \ \ | |_|_|/ |/| | | | | | | | | | | nextcloud/wait-for-the-link-share-menu-to-open-in-acceptance-tests Wait for the link share menu to open in acceptance tests
| * | | Wait for the link share menu to open in acceptance testsDaniel Calviño Sánchez2018-11-021-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the acceptance tests the link share menu is automatically opened if needed before interacting with an item in the menu; if the menu is not open it is opened by clicking on its toggle. However, since a recent change the link share menu is automatically opened by the regular UI after the link share is created. This causes that, sometimes, after the creation of a link share the acceptance tests check whether the menu is shown or not before the menu was automatically opened; as the menu is not open then the acceptance tests proceed to click on the toggle, but in the meantime the link share was created and the menu opened, so clicking on the toggle now closes it. As the menu is closed it is not possible to interact with its items and the test fails. To prevent that now the acceptance tests wait for the link share menu to open after a link share is created before continuing with the other steps. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | | | Merge pull request #11404 from flokli/sendmail-pathRoeland Jago Douma2018-11-021-1/+7
|\ \ \ \ | | | | | | | | | | Mailer: discover sendmail path instead of hardcoding it to /usr/sbin/sendmail
| * | | | Fallback to default path is sendmail can't be foundRoeland Jago Douma2018-11-021-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the sendmail binary can't be found at all we fallback to the default path. It most likely is not there but then at least a proper error message pops up. Updated the tests to also properly pass. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | | Merge pull request #11875 from ↵Roeland Jago Douma2018-11-023-2/+156
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | nextcloud/add-support-for-sending-the-password-for-a-link-share-by-nextcloud-talk Add support for sending the password for a link share by Nextcloud Talk
| * | | | Add acceptance test steps to be used from Talk acceptance testsDaniel Calviño Sánchez2018-11-021-0/+49
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
| * | | | Test that "Password protect by Talk" is not shown if Talk is not enabledDaniel Calviño Sánchez2018-11-022-0/+48
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
| * | | | Store "sendPasswordByTalk" property of link shares in the databaseDaniel Calviño Sánchez2018-11-021-0/+9
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
| * | | | Fix update share testsDaniel Calviño Sánchez2018-11-021-2/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The update share tests only checked that the share returned by "update()" had the expected values. However, as "update()" returns the same share that was given as a parameter the tests were not really verifying that the values were updated in the database. In a similar way, the test that checked that a password was removed did not set a password first, so even if the database returned null it could be simply returning the default value for the share; a password must be set first to ensure that it is removed. Besides that, a typo was fixed too that made the checks on the original share instead of on the one returned by "update()"; right now it is the same share, so the change makes no difference, but it is how the check should be done anyway. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | | | | Merge pull request #12200 from nextcloud/tech-debt/noid/cleanup-legacy-sharingMorris Jobke2018-11-021-1/+6
|\ \ \ \ \ | |_|/ / / |/| | | | Cleanup some unused sharing methods from the old sharing code
| * | | | Cleanup some unused sharing methods from the old sharing codeMorris Jobke2018-11-021-1/+6
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | | | | Merge pull request #12177 from nextcloud/fix/apptoken_login_flow_proper_hostMorris Jobke2018-11-021-2/+8
|\ \ \ \ \ | |_|/ / / |/| | | | Use the proper server for the apptoken flow login
| * | | | Use the proper server for the apptoken flow loginRoeland Jago Douma2018-11-011-2/+8
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If a user can't authenticate normally (because they have 2FA that is not available on their devices for example). The redirect that is generated should be of the proper format. This means 1. Include the protocol 2. Include the possible subfolder Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | Merge pull request #11844 from nextcloud/multiple-link-sharesMorris Jobke2018-11-011-21/+18
|\ \ \ \ | |_|/ / |/| | | allow to create multiple link shares via share api
| * | | Adjust acceptance tests to changes in link sharesDaniel Calviño Sánchez2018-11-011-21/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Although now it is possible to create several link shares the acceptance tests currently handles only the first link share; this first link share is now created by clicking an "Add new share" button instead of a checkbox. Besides that, the "Copy link" button has been moved from the menu to the row, next to the menu trigger. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | | | Revert "Wait for cron to finish before running upgrade command"Morris Jobke2018-11-011-8/+1
| |/ / |/| | | | | | | | | | | | | | This reverts commit 18e9631810ad1d3d72c2b4bbee330169808108ad. Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | | fix writeStream for jail wrapperRobin Appelman2018-10-311-0/+17
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | extend storage api to allow directly writing a stream to storageRobin Appelman2018-10-311-0/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | this removes the need for temporary storages with some external storage backends. The new method is added to a separate interface to maintain compatibility with storage backends implementing the storage interface directly (without inheriting common) Currently the interface is implemented for objectstorage based storages and local storage and used by webdav uploads Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | Only generate previews in powers of 4 and set minRoeland Jago Douma2018-10-311-8/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before we'd round up all preview request to their nearest power of two. This resulted still in a lot of possible images. Generating a lot of server load and taking up a lot of space. This moves it to previews to be powers of 4: 64, 256, 1024 and 4096 Also the first two powers are always skipped (4, 16) as it doesn't make sense to generate previews for that. We cache preview pretty agressively and I feel this is a better tradeoff. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | Merge pull request #12089 from Rayn0r/masterDennis19932018-10-311-6/+14
|\ \ \ | |/ / |/| | added possibility to disable autocomplete in login form
| * | adapted automated test for autocomplete changesRayn0r2018-10-301-6/+14
| | | | | | | | | | | | Signed-off-by: Rayn0r <Andre.Weidemann@web.de>
* | | Move ExpiredTokenException to the correct namespaceRoeland Jago Douma2018-10-302-5/+2
| | | | | | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | Merge pull request #11899 from nextcloud/enhancement/noid/sharing-consolidationRoeland Jago Douma2018-10-302-21/+145
|\ \ \ | | | | | | | | Sharing user consolidation
| * | | Fix injection to get the user idJulius Härtl2018-10-301-1/+11
| | | | | | | | | | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
| * | | Add uuid/name entry to Remote/MailPlugin testsJulius Härtl2018-10-302-20/+104
| | | | | | | | | | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
| * | | Make enhancing entries with type property optionalJulius Härtl2018-10-301-6/+30
| | | | | | | | | | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
| * | | Filter out local users from address book remote searchesJulius Härtl2018-10-301-1/+7
| | | | | | | | | | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | | | Add acceptance tests for hiding download in link sharesDaniel Calviño Sánchez2018-10-303-0/+127
| |_|/ |/| | | | | | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | | Merge pull request #12036 from olivermg/masterMorris Jobke2018-10-301-0/+115
|\ \ \ | |/ / |/| | Add capability of specifying "trusted_proxies" entries in CIDR notation (IPv4)
| * | Adding handling of CIDR notation to trusted_proxies for IPv4Oliver Wegner2018-10-301-0/+115
| | | | | | | | | | | | Signed-off-by: Oliver Wegner <void1976@gmail.com>
* | | Merge pull request #12052 from ↵Morris Jobke2018-10-291-24/+28
|\ \ \ | | | | | | | | | | | | | | | | nextcloud/bugfix/11594/fix-setup-check-trusted-proxies Fix setup check trusted proxies
| * | | Update logic for forwardedForHeadersWorkingDaniel Kesselberg2018-10-251-24/+28
| | |/ | |/| | | | | | | | | | | | | | | | | | | As discussed in https://github.com/nextcloud/server/issues/11594 when discovering if x-forwarded-for is working properly its not possible to use getRemoteAddr because the "client ip" is returned. For this check the ip of the last hop would be required. Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* | | Merge pull request #12019 from nextcloud/icons-svg-api-fixesJohn Molakvoæ2018-10-291-2/+2
|\ \ \ | | | | | | | | Move svg api endpoint
| * | | Fix testsJulius Härtl2018-10-291-2/+2
| | | | | | | | | | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | | | Move acceptance tests that crash the PHP built-in server to ApacheDaniel Calviño Sánchez2018-10-293-0/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The PHP built-in server can crash when certain actions are performed in Nextcloud (but although the crash is triggered by Nextcloud it does not seem to be a Nextcloud bug), which can lead to failures in the acceptance tests that would have otherwise passed. A crash of the PHP built-in server during an acceptance test can be identified by the message "sh: 1: kill: No such process" in the acceptance tests output; as the PHP built-in server crashed its process does no longer exist when it is tried to be killed when the scenario ends. Although the crash has been observed in other tests too it is more prevalent in the tests for tags and the theming app. In order to reduce the false positives those tests are now run on Apache instead of on the PHP built-in sever. However, the rest of tests are still run on the PHP built-in server due to its lower resource consumption. In order to run a feature or just a scenario using Apache it has to be tagged with "@apache"; features or scenarios without that tag (the default) will run on the PHP built-in server instead. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | | | Link the default Apache directory to the Nextcloud server in all runnersDaniel Calviño Sánchez2018-10-292-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In order to run the acceptance tests in Apache "/var/www/html" has to be linked to the root directory of the Nextcloud server. Before this was automatically done when launching the acceptance tests through "./run.sh", but an explicit command was needed when run in Drone. Now the linking was moved from "run.sh" to "run-local.sh", so it is automatically done when run through "./run.sh" and in Drone, including when running the tests for an app instead of for the server. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | | | Extract acceptance tests for tags to their own feature fileDaniel Calviño Sánchez2018-10-292-88/+89
|/ / / | | | | | | | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* / / Move the browser window of an actor to the foreground when acting as himDaniel Calviño Sánchez2018-10-262-0/+17
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Each time a new actor appears in a scenario the browser window of the new actor is put in front of the browser windows of the previous actors. Before, when acting again as a previous actor his browser window stayed in the background; in most cases everything worked fine even if the window was in the background, but due to a bug in the Firefox driver of Selenium and/or maybe in Firefox itself when the window was in the background it was not possible to set the value of an input field that had a range selected. Now, when acting again as a previous actor his browser window is brought to the foreground. This prevents the bug from manifesting, but also reflects better how a user would interact with the browser in real life. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* / Keep list of icons in a separate file for use in the accessibility appJulius Härtl2018-10-252-8/+10
|/ | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* skip already decrypted files on decrypt all commandBjoern Schiessle2018-10-241-10/+31
| | | | Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
* Merge pull request #11396 from nextcloud/wellknown-webfingerMorris Jobke2018-10-241-0/+1
|\ | | | | adding .well-known/webfinger
| * adding .well-known/webfingerMaxence Lange2018-10-101-0/+1
| | | | | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
* | Merge pull request #11847 from iPaat/Fix/DeleteCookiesThroughClearSiteDataMorris Jobke2018-10-231-2/+2
|\ \ | | | | | | Remove cookies from Clear-Site-Data Header
generated by cgit v1.2.3 (git 2.39.1) at 2025-01-29 01:26:45 +0000