aboutsummaryrefslogtreecommitdiffstats
path: root/apps/settings/tests
Commit message (Collapse)AuthorAgeFilesLines
* Rename setting name to reduce its lengthLouis Chemineau2022-06-271-4/+4
| | | | Signed-off-by: Louis Chemineau <louis@chmn.me>
* Remove deprecated at() matcher from settings testsCôme Chilliet2022-06-076-214/+129
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* Merge pull request #32443 from nextcloud/port/background-job-adminPytal2022-05-201-8/+7
|\
| * Port BackgroundJob admin settings to vueCarl Schwan2022-05-201-8/+7
| | | | | | | | Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* | Improve two factor admin settingsCarl Schwan2022-05-201-1/+3
|/ | | | | | | | - Port more of it to vue - Use new nextcloud vue components for the setting section - Add a bit of spacing between the elements Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* Add setting to ignore email during searchLouis Chemineau2022-05-031-0/+4
| | | | Signed-off-by: Louis Chemineau <louis@chmn.me>
* Fix migrators according to exceptionsVincent Petry2022-04-131-4/+2
| | | | | | | | Fixed syntax errors. Removed if condition for copyToFolder since it's void now. Change signature of setMigratorVersions to also be void. Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* Add settings to ignore second display name in searchLouis Chemineau2022-04-121-0/+4
| | | | Signed-off-by: Louis Chemineau <louis@chmn.me>
* Merge pull request #31927 from nextcloud/feat/dont_match_userid_testsVincent Petry2022-04-121-0/+4
|\ | | | | Tweak tests to include shareapi_restrict_user_enumeration_full_match_userid
| * Tweak tests to include shareapi_restrict_user_enumeration_full_match_useridLouis Chemineau2022-04-111-0/+4
| | | | | | | | Signed-off-by: Louis Chemineau <louis@chmn.me>
* | Merge pull request #31866 from nextcloud/test-avatar-migrationPytal2022-04-113-9/+55
|\ \
| * | Add comment explaining avatar basenameChristopher Ng2022-04-081-0/+1
| | | | | | | | | | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
| * | Remove unnecessary array_diffChristopher Ng2022-04-081-5/+1
| | | | | | | | | | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
| * | Test avatar migrationChristopher Ng2022-04-073-9/+58
| | | | | | | | | | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
| * | Improve strictness of account file regexChristopher Ng2022-04-071-1/+1
| |/ | | | | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
* / Adapt AccountMigratorTest to id changeCôme Chilliet2022-04-111-1/+1
|/ | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* Merge pull request #31852 from nextcloud/test-account-migratorPytal2022-04-063-0/+122
|\
| * Test migration of account informationChristopher Ng2022-04-063-0/+122
| | | | | | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
* | Allow to disable password policy enforcement for selected groupsCarl Schwan2022-04-051-0/+4
|/ | | | | | Signed-off-by: Carl Schwan <carl@carlschwan.eu> Co-authored-by: Vincent Petry <vincent@nextcloud.com> Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
* Add profile default setting for adminChristopher Ng2022-03-181-0/+11
| | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
* Add global profile toggle configChristopher Ng2022-03-181-0/+4
| | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
* fix drone tests on masterszaimen2022-03-111-0/+2
| | | | Signed-off-by: szaimen <szaimen@e.mail.de>
* fix testsszaimen2022-03-071-0/+12
| | | | Signed-off-by: szaimen <szaimen@e.mail.de>
* Add Nextcloud docs link to OPcache recommendsMichaIng2022-03-031-3/+0
| | | | | | | | A link to the Nextcloud documentation is currently only shown when the OPcache module is not loaded at all. This commit moves the link to the generic text above the individual recommendations list. Additionally remove the obsolete phpOpcacheDocumentation entry from test data arrays, which is not passed anymore by the backend. Signed-off-by: MichaIng <micha@dietpi.com>
* Setup warning for invalid LDAP user or group UUIDs.Arthur Schiwon2022-02-101-2/+19
| | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* fix testsszaimen2022-02-032-1/+12
| | | | Signed-off-by: szaimen <szaimen@e.mail.de>
* Fix tests with email templatesCarl Schwan2022-01-181-6/+6
| | | | Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* Enhance and complement OPcache setup checksMichaIng2021-12-191-11/+4
| | | | | | | | | | | | | | The current OPcache recommendations match the PHP defaults, but the values are much higher than required to run Nextcloud, even with a high number of installed apps. On the other hand, when other applications use the same OPcache instance, the recommended values might not be sufficient. Accurate recommendations need to take into account actual OPcache usage. With this commit, recommendations are shown to raise the config value if more than 90% of max cache size or number of keys is used. The checks whether the module is loaded and whether the OPcache is properly configured have been merged into a single function. This allowed to reduce the overhead of OPcache configuration checks when the module is not loaded. A check has been added whether Nextcloud is permitted to use the OPcache API. Without this, inconsistencies during core or app upgrades may cause errors and OPcache usage cannot be determined for the new usage based checks. OPcache usage based checks are skipped when Nextcloud is not permitted to use the API. Signed-off-by: MichaIng <micha@dietpi.com>
* Fix OCA\Settings\Tests\Settings\Admin\SharingTestCôme Chilliet2021-12-161-0/+2
| | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* More test fixingJoas Schilling2021-12-012-13/+13
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* ArgumentCountError: Too few arguments to function ↵Joas Schilling2021-11-091-18/+29
| | | | | | OCA\Settings\Controller\CheckSetupController::__construct() Signed-off-by: Joas Schilling <coding@schilljs.com>
* PHPUnit\Framework\MockObject\DuplicateMethodException: Cannot stub or mock ↵Joas Schilling2021-11-091-1/+0
| | | | | | using a method list that contains duplicates: Signed-off-by: Joas Schilling <coding@schilljs.com>
* Make sure trusted_proxies is an arrayJoas Schilling2021-11-091-4/+27
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* fix appstoreenabled check in Fetcher so it's coherent with AppSettingsControllerJulien Veyssier2021-10-181-2/+2
| | | | Signed-off-by: Julien Veyssier <eneiluj@posteo.net>
* Add admin privilege delegation for admin settingsCarl Schwan2021-09-295-5/+105
| | | | | | | This makes it possible for selected groups to access some settings pages. Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* Merge pull request #28837 from nextcloud/show-warning-if-sharing-app-disabledszaimen2021-09-201-9/+21
|\ | | | | Show warning on the share settings when the File Shares app is disabled
| * Show warning on the share settings when the File Shares app is disabledThomas Citharel2021-09-151-9/+21
| | | | | | | | | | | | Closes #3706 Signed-off-by: Thomas Citharel <tcit@tcit.fr>
* | Issue 16719: CheckSetupController.php: now injecting reference to ↵Stefan2021-09-171-0/+6
|/ | | | | | ITempManager, rather than obtaining using it manually Signed-off-by: Stefan <Stefan.Schilling@EXXETA.com>
* use specific email getter where necessaryArthur Schiwon2021-09-091-5/+4
| | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* Introduce ISecureRandom::CHAR_ALPHANUMERICJ0WI2021-07-081-5/+1
| | | | Signed-off-by: J0WI <J0WI@users.noreply.github.com>
* fix NewUserMailHelperTestszaimen2021-06-221-3/+3
| | | | Signed-off-by: szaimen <szaimen@e.mail.de>
* fix NewUserMailHelperTestszaimen2021-06-221-3/+3
| | | | Signed-off-by: szaimen <szaimen@e.mail.de>
* Update php licensesJohn Molakvoæ (skjnldsv)2021-06-0422-45/+28
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Fix translation phpunit testJohn Molakvoæ (skjnldsv)2021-06-021-3/+3
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* do not use private AccountManager in UsersControllerArthur Schiwon2021-05-251-151/+251
| | | | | | | | - extends IAccountProperty for verificationData getters and setters - implementation thereof ^ - and of course adaption of UsersController Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* load share settings from the share manager in more placesRobin Appelman2021-05-121-0/+6
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* Fix deleting properties of user settings when not given explicitlyDaniel Calviño Sánchez2021-04-231-0/+120
| | | | | | | | | | The controller can receive an optional subset of the properties of the user settings; values not given are set to "null" by default. However, those null values overwrote the previously existing values, so in practice any value not given was deleted from the user settings. Now only non null values overwrite the previous values. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Respect additional user settings not covered by the controllerDaniel Calviño Sánchez2021-04-231-10/+0
| | | | | | | | | | | | | | | | | | | | "AccountManager::updateUser()" wipes previous user data with whichever user data is given (except for some adjustments, like resetting the verified status when needed). As the controller overrode the properties those properties would lose some of their attributes even if they are not affected by the changes made by the controller. Now the controller only modifies the attributes set ("value" and "scope") to prevent that. Note that with this change the controller no longer removes the "verified" status, but this is not a problem because, as mentioned, "AccountManager::updateUser()" resets them when needed (for example, when the value of the website property changes). This change is a previous step to fix overwritting properties with null values, and it will prevent the controller from making unexpected changes if more attributes are added in the future. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Add more unit tests for setting user settingsDaniel Calviño Sánchez2021-04-231-0/+160
| | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Change default test data to values less similar to empty valuesDaniel Calviño Sánchez2021-04-231-12/+12
| | | | | | | Right now it makes no difference, but this should make future tests clearer, specially in case of failure. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>