diff options
| author | Joas Schilling <coding@schilljs.com> | 2021-04-20 17:53:00 +0300 |
|---|---|---|
| committer | Joas Schilling <coding@schilljs.com> | 2021-04-20 17:53:00 +0300 |
| commit | fc6fbb649999c39968c88721f51112d0a0c67131 (patch) | |
| tree | eb346bbc7c7aef19456e4bed55ef74d1ac1652db /apps | |
| parent | 94322971a5a052becb92d4f81a158fd1cf43c873 (diff) | |
Fix broken Expiration test
15) Warning
The data provider specified for OCA\Files_Trashbin\Tests\ExpirationTest::testGetMaxAgeAsTimestamp is invalid.
PHPUnit\Util\Exception: Method timestampTestData does not exist
Signed-off-by: Joas Schilling <coding@schilljs.com>
Diffstat (limited to 'apps')
| -rw-r--r-- | apps/files_trashbin/tests/ExpirationTest.php | 20 |
1 files changed, 11 insertions, 9 deletions
diff --git a/apps/files_trashbin/tests/ExpirationTest.php b/apps/files_trashbin/tests/ExpirationTest.php index adef194f34f..d33c502f419 100644 --- a/apps/files_trashbin/tests/ExpirationTest.php +++ b/apps/files_trashbin/tests/ExpirationTest.php @@ -24,6 +24,8 @@ * */ +namespace OCA\Files_Trashbin\Tests; + use OCA\Files_Trashbin\Expiration; use OCP\AppFramework\Utility\ITimeFactory; use OCP\IConfig; @@ -118,16 +120,16 @@ class ExpirationTest extends \Test\TestCase { } - public function configData() { + public function timestampTestData(): array { return [ - [ 'disabled', null, null, null], - [ 'auto', Expiration::DEFAULT_RETENTION_OBLIGATION, Expiration::NO_OBLIGATION, true ], - [ 'auto,auto', Expiration::DEFAULT_RETENTION_OBLIGATION, Expiration::NO_OBLIGATION, true ], - [ 'auto, auto', Expiration::DEFAULT_RETENTION_OBLIGATION, Expiration::NO_OBLIGATION, true ], - [ 'auto, 3', Expiration::NO_OBLIGATION, 3, true ], - [ '5, auto', 5, Expiration::NO_OBLIGATION, true ], - [ '3, 5', 3, 5, false ], - [ '10, 3', 10, 10, false ], + [ 'disabled', false], + [ 'auto', false ], + [ 'auto,auto', false ], + [ 'auto, auto', false ], + [ 'auto, 3', self::FAKE_TIME_NOW - (3 * self::SECONDS_PER_DAY) ], + [ '5, auto', false ], + [ '3, 5', self::FAKE_TIME_NOW - (5 * self::SECONDS_PER_DAY) ], + [ '10, 3', self::FAKE_TIME_NOW - (10 * self::SECONDS_PER_DAY) ], ]; } |