summaryrefslogtreecommitdiffstats
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
* Explicitly check for portLukas Reschke2016-03-101-1/+9
| | | | | | The setup uses `\OCP\IRequest::getInsecureServerHost` which in some cases can also include a port. This makes the trusted domain check fail thus. I've decided to add this here that way because adjusting the setup would require parsing the host properly. This is not something that can be done very good in PHP. Check the following example for why `parse_url` is not our friend: https://3v4l.org/k501Z
* Merge pull request #23017 from owncloud/first-stop-the-dockersThomas Müller2016-03-101-0/+1
|\ | | | | docker containers should be stopped before being removed - refs https…
| * docker containers should be stopped before being removed - refs ↵Thomas Müller2016-03-091-0/+1
| | | | | | | | https://github.com/docker/docker/issues/18758
* | Added testsRoeland Jago Douma2016-03-071-0/+70
|/ | | | | * Unit tests for OC_Filechunking to verify the isComplete function * Intergration tests to show that shuffling chunks is all fine
* Keep "encryptedVersion" when calling `\OC\Files\View::copy`Lukas Reschke2016-03-031-6/+21
| | | | | | | | | | | | | | | | When calling `\OC\Files\View::copy` we should also keep the version to ensure that the file will always have the correct version attached and can be successfully decrypted. To test this the following steps are necessary (from https://github.com/owncloud/core/issues/22781#issuecomment-191328982): 1. setup a new ownCloud 9.0 beta2 2. enable encryption 2. upload a docx (5.7MB large) 3. upload the same file again and overwrite the existing file 4. I can download the original file and the first version 5. I restore the first version 6. restored version can no longer be downloaded with the error described above The manual cache operation in `\OCA\Files_Versions\Storage` is unfortunately necessary since `\OCA\Files_Versions\Storage::copyFileContents` is not using `\OCP\Files\Storage::moveFromStorage` in the case when an object storage is used. Due to the workaround added in https://github.com/owncloud/core/commit/54cea05271b887f1c8062c034741df869bc0f055 the stream is directly copied and thus bypassing the FS.
* Merge pull request #22742 from owncloud/ceph-keystone-readinessC. Montero Luque2016-03-013-290/+14
|\ | | | | Use readiness notification socket to be absolutely sure this works
| * local entrypoint.sh isn't needed - all fixes are upstreamMorris Jobke2016-03-012-276/+1
| |
| * Use sockets for startup notification in primary storage testsMorris Jobke2016-03-012-14/+13
| |
* | Use IQueryBuilder::PARAM_* instead of \PDO::PARAM_*Joas Schilling2016-02-294-5/+7
|/
* Merge pull request #22683 from owncloud/fix_22682Thomas Müller2016-02-291-0/+11
|\ | | | | Relax rootfolder check
| * Relax rootfolder checkRoeland Jago Douma2016-02-261-0/+11
| | | | | | | | | | * Updated unit tests * Added intergration test
* | Expiration date can only be enforced if default is enabledRoeland Jago Douma2016-02-261-0/+15
|/ | | | | | If the default expiration date is not enebaled it can not be enforced. * Added unit tests
* Merge pull request #22660 from owncloud/fix_22656Thomas Müller2016-02-261-0/+29
|\ | | | | Do not allow sharing of the users root folder
| * Do not allow sharing of the root folderRoeland Jago Douma2016-02-251-0/+29
| | | | | | | | | | | | | | | | | | | | | | Sharing of the users root folder should not be allowed as it is very weird UX. Also many of our clients have no proper way of displaying this. Added unit test Also added intergration tests to make sure we won't allow it in the future.
* | Merge pull request #22646 from owncloud/fix_22642Thomas Müller2016-02-261-6/+27
|\ \ | | | | | | Set default expiration date if none given on share creation
| * | Set default expiration date if none given on share creationRoeland Jago Douma2016-02-251-6/+27
| | | | | | | | | | | | | | | | | | | | | When we create a share for the first time we should set the default expiration date. If none is given. Fixes #22642
* | | Merge pull request #22650 from owncloud/fix_hookThomas Müller2016-02-261-2/+3
|\ \ \ | | | | | | | | Fix verifyExpirationDate passwordSet argument
| * | | Fix verifyExpirationDate passwordSet argumentRoeland Jago Douma2016-02-251-2/+3
| |/ / | | | | | | | | | | | | | | | Password set should be false if the password is null. Also updated the unit tests to tests this now.
* | / Fix formatting of responseLukas Reschke2016-02-251-52/+58
| |/ |/| | | | | The elements are expected to be within a `service` array as per specification.
* | Merge pull request #22579 from owncloud/fix_broken_unencrypted_sizeThomas Müller2016-02-251-1/+157
|\ \ | |/ |/| Heal unencrypted file sizes at download time (second approach)
| * recalculate unencrypted size if we assume that the size stored in the db is ↵Bjoern Schiessle2016-02-231-1/+157
| | | | | | | | not correct
* | Merge pull request #22573 from owncloud/issue-22568-allow-string-object-idsThomas Müller2016-02-241-37/+37
|\ \ | | | | | | Make sure we can store strings as per the interface
| * | Make sure we can store strings as per the interfaceJoas Schilling2016-02-221-37/+37
| | |
* | | Merge pull request #22569 from owncloud/issue-22566-too-much-mapping-entriesThomas Müller2016-02-231-0/+36
|\ \ \ | | | | | | | | Allow defining a limit and offset for getObjectIdsForTags
| * | | Allow defining a limit and offset for getObjectIdsForTagsJoas Schilling2016-02-231-0/+36
| |/ /
* | | Merge pull request #22551 from owncloud/make-exceptions-easier-to-debugThomas Müller2016-02-231-4/+13
|\ \ \ | |/ / |/| | Throw normal exceptions instead of eating them
| * | Throw normal exceptions instead of eating themLukas Reschke2016-02-221-4/+13
| | | | | | | | | | | | | | | | | | Partially addresses https://github.com/owncloud/core/issues/22550 Replaces https://github.com/owncloud/core/pull/20185
* | | We should check for exceptions when trying to get the avatarRoeland Jago Douma2016-02-221-0/+26
|/ / | | | | | | | | | | | | Fixes #22550 * Updated phpdoc of avatatmanager * Add unit test
* | Merge pull request #22503 from owncloud/issue_22500Thomas Müller2016-02-191-9/+88
|\ \ | | | | | | When (re-)sharing an incomming federated share set the corrent owner
| * | When (re-)sharing an incomming federated share set the corrent ownerRoeland Jago Douma2016-02-191-9/+88
| |/ | | | | | | | | | | | | | | | | | | | | | | Incomming federated shares are a special kind. We mount them as normal webdav shares but we do supply owner info with the federated cloud id of the share owner. Since we do not yet have the new resharing behaviour on federated shares we need to set the correct owner. Which will allow sharing and proper mounting for other users. fixes #22500
* | Merge pull request #22410 from owncloud/fix_22119Thomas Müller2016-02-191-31/+20
|\ \ | |/ |/| Do not copy skeleton on avatar access
| * Do not copy skeleton on avatar accessRoeland Jago Douma2016-02-161-31/+20
| | | | | | | | | | | | | | | | Fixes #22119 Just try to get the folder of the user. If it is not there a NotFoundException will be thrown. Which will be handled by the avatar endpoint.
* | fix unit testv1r0x2016-02-181-1/+1
| |
* | Fix coverage reportingThomas Müller2016-02-161-11/+12
| |
* | Merge pull request #22416 from owncloud/fix_22402Thomas Müller2016-02-161-31/+10
|\ \ | | | | | | Return proper error string if sharing for this user is disabled
| * | Return proper error string if sharing for this user is disabledRoeland Jago Douma2016-02-161-31/+10
| | | | | | | | | | | | Fixes #22402
* | | Merge pull request #22338 from owncloud/fix_22247Thomas Müller2016-02-161-7/+7
|\ \ \ | |/ / |/| | Allow more characters in filenames
| * | Fix unit testsRoeland Jago Douma2016-02-121-7/+7
| | |
* | | Remove unneeded parameter from OC_App::getEnabledAppsMorris Jobke2016-02-151-5/+5
| |/ |/|
* | Merge pull request #22345 from ↵Thomas Müller2016-02-131-0/+19
|\ \ | | | | | | | | | | | | owncloud/make-note-if-appcodechecker-is-not-enabled Add note if integrity check is disabled
| * | Add note if integrity check is disabledLukas Reschke2016-02-121-0/+19
| |/ | | | | | | | | | | | | | | Our issue template states that users should post the output of `/index.php/settings/integrity/failed`, at the moment it displays that all passes have been passed if the integrity checker has been disabled. This is however a wrong approach considering that some distributions are gonna package Frankenstein releases and makes it harder for us to detect such issues. Thus if the integrity code checker is disabled (using the config switch) it displays now: `Appcode checker has been disabled. Integrity cannot be verified.` This is not displayed anywhere else in the UI except these URL used for us for debugging purposes.
* | Merge pull request #22331 from owncloud/dav-nonexisting-partThomas Müller2016-02-131-0/+6
|\ \ | | | | | | fix getNodeForPath for non existing part files
| * | fix getNodeForPath for non existing part filesRobin Appelman2016-02-111-0/+6
| | |
* | | Merge pull request #22309 from owncloud/infinite-recursion-on-expired-linkThomas Müller2016-02-121-12/+2
|\ \ \ | | | | | | | | Do not getShare in deleteShare, it's already there when deleting
| * | | Fix unit testsRoeland Jago Douma2016-02-111-12/+2
| | | |
* | | | Merge pull request #22317 from owncloud/fix_invisible_linksharesThomas Müller2016-02-121-0/+4
|\ \ \ \ | |_|_|/ |/| | | Do not allow invisible link shares
| * | | Add parent for invisible link sharesRoeland Jago Douma2016-02-111-0/+4
| |/ /
* | | Merge pull request #22314 from owncloud/fix-mariadb-autotestThomas Müller2016-02-111-0/+5
|\ \ \ | |_|/ |/| | Wait until mariadb is up and running
| * | Use official mariadb dockerThomas Müller2016-02-111-0/+5
| |/
* | Merge pull request #22315 from owncloud/update-info-checker-requirementsThomas Müller2016-02-111-1/+1
|\ \ | | | | | | Update info checker requirements