[stable9] Backport flat reshare unit tests #26926
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…therfilelists [stable9] Fix search result link for file results outside default list
* Fix Decrypt message via occ * Comments fixed * Fixed reviews * Originally: commit 2304e4b Author: Joas Schilling <nickvergessen@owncloud.com> Date: Tue Jun 7 09:13:11 2016 +0200 Allow to decrypt user '0' files only * Fix uid comparison
* Initial web executor * Fix PHPDoc Fix broken integration test OccControllerTests do not require database access - moch them all! Kill unused sprintf
This reverts commit 58ed5b9.
Makes sure that the share owner is set before entering the checks that need it. Partial backport of afa37d3
Add integration test for the use case where a group share exists and then the same owner creates a direct share to a user in that group.
[stable9] occ web executor
…ble9 [stable9] get only vcards which match both the address book id and the vcard uri
[stable9] Change order of share creation validation
Revert "Remove repair steps for broken updater repair"
…8e99e379449792eeaf8 [stable9] Ignore invalid paths in the JS file list
* Wait for socket to be open * Fix call on null * Allow DB access for MountProviderTest Makes unit tests pass when using object store, since their FS access is actually oc_filecache DB access. It is currently not possible to mock or bypass the logic from "SharedMount::verifyMountPoint()" triggered by this test.
* Filelist change dir auto-prepend slash Prepend a slash to directories in case it was missing since many places assume that it's there. * Fix js unit test in filelist spec
* [stable9] Contacts API: replace raw image data with url * fix a ImageExportPlugin Test
[stable9] Additional perm check in Webdav
* Workaround to check htaccess in case of redirects In some setups, the web server will redirect any call to "data/" to the main page. This causes the XHR to return the 200 HTTP status code and the body contains the HTML page of the main page / files app. This fix improves the htaccess failure detection by adding a known string inside the test file "htaccesstest.txt". If we are able to find this string, it means that the web server didn't block access to that file. * Fix setup check unit test (#25439)
…certificate is revoked (#25469) * Adding certificate revocation list and validate if the app certificate is revoked * Check integrity of a signed app in any case on installation
[stable9] Use the correct realm for basic authentication - fixes #23427
404 errors were not properly cached due to catching the wrong exception. Now catching ClientHttpException and checking the error code. In case of 404, adjust the stat cache accordingly.
Conflicts: build/integration/features/webdav-related.feature
…ages [stable9] Remove obsolete RepairLegacyStorages repair step
Stable9: Do not validate return code if set was successful
[stable9] Fix for app groups setting preservation
…-colons [Stable9] Add integration test for creating folder with colon
[stable9] Fix DAV stat cache to properly cache 404
|
@PVince81, thanks for your PR! By analyzing the history of the files in this pull request, we identified @DeepDiver1975 and @Xenopathic to be potential reviewers. |
|
Wrong branch |
|
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Since stable9 already has flat reshares, the unit tests need to also use the new sharing API when creating shares to make sure we're testing the correct results.
Also this is required by #26912 which adds a performance fix that uses a shortcut that limits recursion based on the idea that recursion is not needed any more due to flat reshares.
Required for #26912
Please review @SergioBertolinSG @jvillafanez @DeepDiver1975 @VicDeo