diff options
author | Julius Härtl <jus@bitgrid.net> | 2023-02-09 20:44:07 +0100 |
---|---|---|
committer | Julius Härtl <jus@bitgrid.net> | 2023-02-17 19:18:38 +0100 |
commit | 20058eb9dea035ea541ff351d9a1e8a0d777a7c7 (patch) | |
tree | b1263771c288c79ddd0dc3edbacf9d8aa5bd6a69 /apps/files_sharing | |
parent | 5d4efb4d5fd8e4389856df5d94c3b92c7019e603 (diff) | |
download | nextcloud-server-20058eb9dea035ea541ff351d9a1e8a0d777a7c7.tar.gz nextcloud-server-20058eb9dea035ea541ff351d9a1e8a0d777a7c7.zip |
tests: Fix test isolation on object storage
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Diffstat (limited to 'apps/files_sharing')
-rw-r--r-- | apps/files_sharing/tests/ApiTest.php | 6 | ||||
-rw-r--r-- | apps/files_sharing/tests/TestCase.php | 8 |
2 files changed, 13 insertions, 1 deletions
diff --git a/apps/files_sharing/tests/ApiTest.php b/apps/files_sharing/tests/ApiTest.php index d7661297e9e..d9b61f837eb 100644 --- a/apps/files_sharing/tests/ApiTest.php +++ b/apps/files_sharing/tests/ApiTest.php @@ -73,6 +73,7 @@ class ApiTest extends TestCase { \OC::$server->getConfig()->setAppValue('core', 'shareapi_exclude_groups', 'no'); \OC::$server->getConfig()->setAppValue('core', 'shareapi_expire_after_n_days', '7'); + \OC::$server->getConfig()->setAppValue('core', 'shareapi_enforce_links_password', 'no'); $this->folder = self::TEST_FOLDER_NAME; $this->subfolder = '/subfolder_share_api_test'; @@ -80,6 +81,11 @@ class ApiTest extends TestCase { $this->filename = '/share-api-test.txt'; + // Initialize view again as we delete all filecache/mount entries in tearDown + // Otherwise those tests fail on object storage as the filecache is missing the user home + $this->view = new \OC\Files\View('/' . self::TEST_FILES_SHARING_API_USER1 . '/files'); + $this->view2 = new \OC\Files\View('/' . self::TEST_FILES_SHARING_API_USER2 . '/files'); + // save file with content $this->view->file_put_contents($this->filename, $this->data); $this->view->mkdir($this->folder); diff --git a/apps/files_sharing/tests/TestCase.php b/apps/files_sharing/tests/TestCase.php index 2bd83d6c9c2..abdb6a2d60f 100644 --- a/apps/files_sharing/tests/TestCase.php +++ b/apps/files_sharing/tests/TestCase.php @@ -33,6 +33,7 @@ namespace OCA\Files_Sharing\Tests; use OC\Files\Filesystem; +use OC\Files\View; use OCA\Files_Sharing\AppInfo\Application; use OCA\Files_Sharing\External\MountProvider as ExternalMountProvider; use OCA\Files_Sharing\MountProvider; @@ -211,7 +212,12 @@ abstract class TestCase extends \Test\TestCase { \OC::$server->getUserSession()->setUser(null); \OC\Files\Filesystem::tearDown(); \OC::$server->getUserSession()->login($user, $password); - \OC::$server->getUserFolder($user); + // We need to get the directory listing to trigger the lazy user folder + // to create the files directory. Since the filecache might get cleared + // in the cache, any follow up test case may fail as with object storage + // the filecache represents the file structure + Filesystem::initMountPoints($user); + \OC::$server->getUserFolder($user)->getDirectoryListing(); \OC_Util::setupFS($user); } |