aboutsummaryrefslogtreecommitdiffstats
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
* Fix more testsRoeland Jago Douma2016-10-283-143/+4
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Proper DI of configRoeland Jago Douma2016-10-285-31/+61
| | | | | | * Fixed comments Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* introduce callForSeenUsers and countSeenUsers (#26361)Jörn Friedrich Dreyer2016-10-282-1/+66
| | | | | | | | | | * introduce callForSeenUsers and countSeenUsers * add tests * oracle should support not null on clob * since 9.2.0
* Merge pull request #1800 from nextcloud/nextcloud-rich-object-stringsMorris Jobke2016-10-272-29/+74
|\ | | | | Nextcloud rich object strings
| * Allow rich object subjects for NotificationsJoas Schilling2016-10-201-29/+21
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
| * Add Rich Object Definitions and a validatorJoas Schilling2016-10-201-0/+53
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Remove internal share mail functionRoeland Jago Douma2016-10-261-48/+0
| | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Merge pull request #1920 from ↵Roeland Jago Douma2016-10-261-0/+14
|\ \ | | | | | | | | | | | | nextcloud/legacy-pages-should-also-receive-the-nonce Add nonce also to legacy CSP
| * | Identify Chromium as ChromeJoas Schilling2016-10-261-0/+14
| | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Merge pull request #1738 from ↵Morris Jobke2016-10-263-6/+133
|\ \ \ | |/ / |/| | | | | | | | nextcloud/comments-provide-displaynames-with-mentions comment mentions: show displayname not uid
| * | add missing methods to test fake managerArthur Schiwon2016-10-201-0/+4
| | | | | | | | | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
| * | resolve displayname via manager and registerable resolversArthur Schiwon2016-10-191-0/+78
| | | | | | | | | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
| * | move mention extraction to (I)Comment and report mentions via DAVArthur Schiwon2016-10-191-6/+51
| | | | | | | | | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* | | Move browserSupportsCspV3 to CSPNonceManagerRoeland Jago Douma2016-10-252-6/+13
| | | | | | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | Merge pull request #1878 from nextcloud/downstream-25936Morris Jobke2016-10-252-7/+8
|\ \ \ | | | | | | | | Adjust unit tests to upstream
| * | | [9.2] Add missing unit tests (#25936)Thomas Müller2016-10-242-7/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Adjust unit test execution after folder rename * Adjust login controller tests to match current behavior * Fix broken unit tests
* | | | Merge pull request #1871 from nextcloud/use-csp-noncesMorris Jobke2016-10-255-1/+156
|\ \ \ \ | | | | | | | | | | Use CSP nonces
| * | | | Add ContentSecurityPolicyNonceManagerLukas Reschke2016-10-241-0/+57
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
| * | | | Add support for CSP noncesLukas Reschke2016-10-244-1/+99
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | CSP nonces are a feature available with CSP v2. Basically instead of saying "JS resources from the same domain are ok to be served" we now say "Ressources from everywhere are allowed as long as they add a `nonce` attribute to the script tag with the right nonce. At the moment the nonce is basically just a `<?php p(base64_encode($_['requesttoken'])) ?>`, we have to decode the requesttoken since `:` is not an allowed value in the nonce. So if somebody does on their own include JS files (instead of using the `addScript` public API, they now must also include that attribute.) IE does currently not implement CSP v2, thus there is a whitelist included that delivers the new CSP v2 policy to newer browsers. Check http://caniuse.com/#feat=contentsecuritypolicy2 for the current browser support list. An alternative approach would be to just add `'unsafe-inline'` as well as `'unsafe-inline'` is ignored by CSPv2 when a nonce is set. But this would make this security feature unusable at all in IE. Not worth it at the moment IMO. Implementing this offers the following advantages: 1. **Security:** As we host resources from the same domain by design we don't have to worry about 'self' anymore being in the whitelist 2. **Performance:** We can move oc.js again to inline JS. This makes the loading way quicker as we don't have to load on every load of a new web page a blocking dynamically non-cached JavaScript file. If you want to toy with CSP see also https://csp-evaluator.withgoogle.com/ Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* | | | Merge pull request #1884 from nextcloud/downstream-26292Lukas Reschke2016-10-251-34/+43
|\ \ \ \ | | | | | | | | | | Fix logClientIn for non-existing users (#26292)
| * | | | Fix and cleanup SessionTestRoeland Jago Douma2016-10-251-43/+26
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
| * | | | Fix logClientIn for non-existing users (#26292)Vincent Petry2016-10-251-0/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The check for two factor enforcement would return true for non-existing users. This fix makes it return false in order to be able to perform the regular login which will then fail and return false. This prevents throwing PasswordLoginForbidden for non-existing users.
* | | | | Merge pull request #1452 from nextcloud/appconfig-endpointJoas Schilling2016-10-252-3/+20
|\ \ \ \ \ | |/ / / / |/| | | | Appconfig endpoint
| * | | | Make AppConfig part of the public APIJoas Schilling2016-10-212-3/+20
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | | | App dependencies are now analysed on app enable as well - not only on app ↵Thomas Müller2016-10-241-26/+30
| |/ / / |/| | | | | | | | | | | install.
* | | | Merge pull request #1820 from nextcloud/4byte-filenamesRoeland Jago Douma2016-10-241-2/+8
|\ \ \ \ | | | | | | | | | | Allow 4byte unicode filenames on supported platforms
| * | | | Added method to DB and fix testRoeland Jago Douma2016-10-241-2/+8
| | |_|/ | |/| | | | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | Fix DateTime comparissonRoeland Jago Douma2016-10-212-5/+5
| |/ / |/| | | | | | | | | | | | | | | | | Datetime now returns microseconds. But if the database doesn't store those comparing them for equality most likely fails. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | Merge pull request #1835 from nextcloud/downstream-24948Morris Jobke2016-10-202-5/+5
|\ \ \ | | | | | | | | Move OC\Files\Storage\Shared to the right namespace
| * | | Fix shared storage namespace in DecryptAll classVincent Petry2016-10-201-1/+1
| | | |
| * | | Move OC\Files\Storage\Shared to the right namespaceJoas Schilling2016-10-201-4/+4
| | | |
* | | | Get rid of very old oc:// stream wrapper (#26381)Vincent Petry2016-10-201-36/+0
| | | |
* | | | Merge pull request #1824 from nextcloud/downstream-26423Morris Jobke2016-10-201-7/+8
|\ \ \ \ | |/ / / |/| | | Ensure $commands being an array
| * | | Ensure $commands being an array - fixes #26073Thomas Müller2016-10-201-0/+1
| | | |
| * | | use short array syntaxMorris Jobke2016-10-201-7/+7
| |/ / | | | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | | Restore all services when a tests failed/finished for better testcase isolationJoas Schilling2016-10-201-3/+26
| | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Make sure all tests use the TestCase method to overwrite servicesJoas Schilling2016-10-204-48/+20
|/ / | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Do not skip when mysql uses utf8mb4Joas Schilling2016-10-191-1/+2
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Fix test of repair stepJoas Schilling2016-10-191-6/+10
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Fix the testJoas Schilling2016-10-191-5/+5
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Adding docker based unit test execution for mysql utf8mb4Thomas Müller2016-10-192-0/+10
| |
* | only disable unicode test on mysqlMorris Jobke2016-10-191-7/+11
| |
* | Adding tests for 4 byte unicode charactersMorris Jobke2016-10-192-0/+46
|/ | | | | * success on SQLite and Postgres * failure on MySQL due to the limited charset that only supports up to 3 bytes
* Check the mimetype before reading the content and catch exceptionJoas Schilling2016-10-181-2/+46
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Merge pull request #1449 from nextcloud/comments-user-mentionMorris Jobke2016-10-172-5/+41
|\ | | | | Notifications for simple @-mentioning in comments
| * emit pre-update event for commentsArthur Schiwon2016-10-121-2/+2
| | | | | | | | | | | | | | * notifications can be cleaned up, no polluted DB * updating comments will re-notify users or remove notifications, depending on the message Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
| * Notificacations for simple @-mentioning in commentsArthur Schiwon2016-10-072-5/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (WIP) notify user when mentioned in comments Fix doc, and create absolute URL for as notification link. PSR-4 compatibility changes also move notification creation to comments app Do not notify yourself unit test for controller and application smaller fixes - translatable app name - remove doubles in mention array - micro perf optimization - display name: special label for deleted users, keep user id for users that could not be fetched from userManager Comment Notification-Listener Unit Test fix email adresses remove notification when triggering comment was deleted add and adjust tests add missing @license tags simplify NotificationsController registration appinfo simplification, php docs make string easier to translate adjust test replace dispatcher-based listeners with a registration method and interface safer to not pass optional data parameter to setSubject for marking as processed. ID and mention suffices Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de> update comment Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* | Merge pull request #1441 from nextcloud/getby-id-less-queriesLukas Reschke2016-10-147-117/+226
|\ \ | | | | | | optimize Folder::getById to use less queries
| * | optimize Folder::getById to use less queriesRobin Appelman2016-10-127-117/+226
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | Merge pull request #1037 from nextcloud/no-double-token-updateChristoph Wurst2016-10-141-27/+128
|\ \ \ | |/ / |/| | dont update the auth token twice