summaryrefslogtreecommitdiffstats
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #26281 from hosting-de/fix/emailRoeland Jago Douma2021-03-261-2/+2
|\ | | | | Change my email address
| * Change my email addressJohannes Leuker2021-03-241-2/+2
| | | | | | | | Signed-off-by: Johannes Leuker <j.leuker@hosting.de>
* | Cleanup unneeded code around database.xmlMorris Jobke2021-03-2419-3271/+29
|/ | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Fix language codes test result orderVincent Petry2021-03-241-0/+1
| | | | | | Fixes issue with Oracle by enforcing the order of the results to check. Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* Bring back forgotten testsVincent Petry2021-03-245-5/+5
| | | | | | | | Remove "ocs-provider" test folder reference as it doesn't exist any more. Added back "Test" test subdir and fixed the tests inside. Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* Fix more controller tests in Core subdirVincent Petry2021-03-242-8/+10
| | | | Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* Fix more tests in the Core subdirVincent Petry2021-03-243-13/+20
| | | | Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* Fix core tests command for app enabling/disablingVincent Petry2021-03-232-17/+17
| | | | Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* Fix casing of core test folderVincent Petry2021-03-231-2/+1
| | | | | | | | It seems Phpunit < 9 was case insensitive. Fixed the phpunit config to target the correct name for the "Core" test directory. Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* Merge pull request #26198 from nextcloud/unified-search-nodeMorris Jobke2021-03-221-108/+202
|\ | | | | Handle limit offset and sorting in files search
| * handle ordering in folder searchRobin Appelman2021-03-191-20/+34
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * folder filtering in sqlRobin Appelman2021-03-191-1/+0
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * cleanup fileinfo creationRobin Appelman2021-03-191-60/+30
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * handle limit and offset in folder file searchRobin Appelman2021-03-181-41/+157
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * only require user to be set in a query that handles tagsRobin Appelman2021-03-181-16/+11
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | Merge pull request #25529 from ↵Morris Jobke2021-03-221-0/+0
|\ \ | | | | | | | | | | | | nextcloud/fix-non-lgc-glyphs-in-avatars-and-txt-file-previews Fix non LGC glyphs in avatars and txt file previews
| * | Include additional scripts in Noto fontsDaniel Calviño Sánchez2021-03-011-0/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Generated avatars as well as text file previews are rendered using the "core/fonts/NotoSans-Regular.ttf" font. The file was the standard hinted "NotoSans-Regular.ttf" file from https://www.google.com/get/noto/. However that file does not cover some non LGC (Latin, Greek, Cyrillic) scripts, like Arabic, Devanagari or Hebrew, to name a few. Markdown file previews also use "core/fonts/NotoSans-Bold.ttf", which is in the same situation as the regular one. Due to limitations in the TTF format it is not possible to provide a single file for each style that includes all Noto fonts. However, it is possible to add more scripts to the standard "NotoSans-Regular.ttf" and "NotoSans-Bold.ttf" files (although no CJK (Chinese, Japanese, Korean) glyph can be included due to the aforementioned limitations). This commit replaces the standard files with an extended version created using the Noto Tools. The build script (as well as a patch for the Noto Tools) is also included for reference and to be able to update the font files in the future if needed. Due to the additional scripts added the font files are now much larger, although this does not seem to increase the time spent rendering LGC scripts. Note that the file for the bold style still contains less scripts than the regular one, as not all scripts supported by Noto have a bold weight. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | | Fix sharebymail testsJohn Molakvoæ (skjnldsv)2021-03-221-64/+64
| | | | | | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* | | Remove duplicated query for email sharesDaniel Calviño Sánchez2021-03-221-63/+79
| | | | | | | | | | | | | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com> Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>
* | | Log and continue when failing to update encryption keys during for ↵Julius Härtl2021-03-191-13/+17
| | | | | | | | | | | | | | | | | | individual files Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | | Merge pull request #23718 from nextcloud/already-shared-error-messageRoeland Jago Douma2021-03-191-4/+13
|\ \ \ | | | | | | | | expand 'path is already shared' error message
| * | | expand 'path is already shared' error messageRobin Appelman2021-03-151-4/+13
| | | | | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | | Merge pull request #26200 from ↵Vincent Petry2021-03-191-40/+40
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | nextcloud/bugfix/26197/fix-expiration-validation-confusion Update user share must use correct expiration validation
| * | | | Update user share must use correct expiration validationVincent Petry2021-03-181-40/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updating a user or group share now uses the correct method for the validation of the expiration date. Instead of using the one from links it uses the one for internal shares. To avoid future confusion, the method "validateExpirationDate" has been renamed to "validateExpirationDateLink". Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* | | | | log full expection during repair stepRobin Appelman2021-03-181-1/+2
|/ / / / | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | | Skip db migration simulation for core schema changesJulius Härtl2021-03-171-53/+0
| | | | | | | | | | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | | | Merge pull request #26161 from ↵Joas Schilling2021-03-171-13/+63
|\ \ \ \ | |_|_|/ |/| | | | | | | | | | | nextcloud/bugfix/noid/improve-matching-of-phonebook-searches Improve search results when only phonebook-matches can we autocompleted
| * | | Also limit to user group in case enumeration is enabled for groups and ↵Joas Schilling2021-03-171-5/+16
| | | | | | | | | | | | | | | | | | | | | | | | phonenumbers Signed-off-by: Joas Schilling <coding@schilljs.com>
| * | | Improve search results when only phonebook-matches can we autocompletedJoas Schilling2021-03-171-8/+47
| |/ / | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Merge pull request #25136 from nextcloud/cachejail-search-filterVincent Petry2021-03-152-3/+86
|\ \ \ | |/ / |/| | do cachejail search filtering in sql
| * | adjust testsRobin Appelman2021-01-261-3/+16
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | fix search in nested jailsRobin Appelman2021-01-261-0/+18
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | fix cachjail searching for rootRobin Appelman2021-01-261-2/+17
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | do cachejail search filtering in sqlRobin Appelman2021-01-261-0/+37
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | Merge pull request #25722 from nextcloud/objectstore-copy-crossRobin Appelman2021-03-121-0/+24
|\ \ \ | | | | | | | | apply object store copy optimization when 'cross storage' copy is wit…
| * | | apply object store copy optimization when 'cross storage' copy is within the ↵Robin Appelman2021-03-091-0/+24
| | | | | | | | | | | | | | | | | | | | | | | | same object store Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | | Fix Test class nameJoas Schilling2021-03-121-2/+2
| | | | | | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | | Change label also in the acceptance testsJoas Schilling2021-03-101-4/+4
| | | | | | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | | Add a setting to restrict returning a full match unless in phonebook or same ↵Joas Schilling2021-03-101-3/+90
| | | | | | | | | | | | | | | | | | | | | | | | group Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | | Restrict autocompletion also based on the phonebook known usersJoas Schilling2021-03-103-49/+351
|/ / / | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Allow overwriting isAuthenticatedRoeland Jago Douma2021-03-091-19/+35
| | | | | | | | | | | | | | | | | | | | | * Some implementations might check for different things * IT will not change how the current ones work Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | Merge pull request #25990 from ↵kesselb2021-03-081-7/+3
|\ \ \ | | | | | | | | | | | | | | | | nextcloud/do-not-try-to-add-an-existing-user-again-in-acceptance-tests Do not try to add an existing user again in acceptance tests
| * | | Remove unneeded steps to add already existing userDaniel Calviño Sánchez2021-03-071-5/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | User "user1" is added when installing and configuring the server, so it is already added in all tests. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
| * | | Fix acceptance test for searching just added users in contacts menuDaniel Calviño Sánchez2021-03-071-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | User "user1" is added when installing and configuring the server, so it is already added in all tests. As the test verifies that just added users can be searched in the contacts menu a new user should be actually added. The test did not fail because it assumed that "user1" did not exist and just checked that it existed after "adding" it, but not whether adding it failed. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | | | Merge pull request #25925 from nextcloud/techdept/psalm/ActionProviderStoreRoeland Jago Douma2021-03-081-3/+3
|\ \ \ \ | | | | | | | | | | Pimp ActionProviderStore
| * | | | Pimp ActionProviderStoreRoeland Jago Douma2021-03-041-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Fix paradoxale condition * Add types * Move away from ILogger Makes psalm happy, and thus me happy Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | | Find directly the label instead of falling back to itDaniel Calviño Sánchez2021-03-061-8/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The input element is always hidden, so the check always ended falling back to the label. Moreover, the label is the element that the user interacts with, so it must be the one used. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | | | | Assert also element visibility instead of just finding itDaniel Calviño Sánchez2021-03-061-4/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Although if the element could not be found an exception would be thrown and the test aborted if an element is in the DOM but hidden it would be found and the test would pass. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | | | | Add explicit locator for "Enable all" bundle buttonDaniel Calviño Sánchez2021-03-061-5/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Instead of looking for the bundle button and then checking its value now the expected value is included in the locator and the button is checked similarly to other elements. No "Disable all" locator was added as it was not currently needed anywhere. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | | | | Find elements through the actor rather than the driverDaniel Calviño Sánchez2021-03-061-4/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "Actor::find" is a more robust way to look for elements, as it handles some exceptions that may be thrown. Therefore, even if the elements are not actually used and it is only checked whether they exist or not using the actor is the preferred way when possible (and it also makes it consistent with the rest of the acceptance tests). Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>