diff options
| author | Vincent Petry <pvince81@owncloud.com> | 2015-04-08 12:03:55 +0200 |
|---|---|---|
| committer | Vincent Petry <pvince81@owncloud.com> | 2015-04-08 12:45:38 +0200 |
| commit | f8cfc03f36e438f1874e9840ce69811112e15475 (patch) | |
| tree | 4b09a8e57c2c1492ee023e2adf9efa2a9acc709e /tests/lib/preview.php | |
| parent | 6c327f8331617652ef0b268d51edc6a23624e33c (diff) | |
| download | nextcloud-server-f8cfc03f36e438f1874e9840ce69811112e15475.tar.gz nextcloud-server-f8cfc03f36e438f1874e9840ce69811112e15475.zip | |
Replace originalStorage in tests with a proper teardown
The purpose of $originalStorage in unit tests was to remount the old
root.
However that storage itself is already wrapped by storage wrapper, so
remounting it would rewrap the storage several times.
This fix makes use of "loginAsUser()" and "logout()" from the TestCase
class to properly initialize and cleanup the FS as expected.
Diffstat (limited to 'tests/lib/preview.php')
| -rw-r--r-- | tests/lib/preview.php | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/tests/lib/preview.php b/tests/lib/preview.php index 003ecedb65a..ea9de9b777e 100644 --- a/tests/lib/preview.php +++ b/tests/lib/preview.php @@ -26,6 +26,8 @@ class Preview extends TestCase { protected function setUp() { parent::setUp(); + // FIXME: use proper tearDown with $this->loginAsUser() and $this->logout() + // (would currently break the tests for some reason) $this->originalStorage = \OC\Files\Filesystem::getStorage('/'); // create a new user with his own filesystem view |