diff options
author | Christoph Wurst <christoph@winzerhof-wurst.at> | 2020-12-29 11:44:40 +0300 |
---|---|---|
committer | Christoph Wurst <christoph@winzerhof-wurst.at> | 2020-12-29 12:04:20 +0300 |
commit | 2d20e4c37de724f6c95cf7cb487e34d1645bb6ae (patch) | |
tree | 36491adc9f661d7450bcc572052183eb5021e9c0 /tests | |
parent | 87e241a60b12cff07e3b3b50317a968b3667610f (diff) |
Drop php7.2 and add the nextcloud coding standard
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/Acceptance/TOTPAcceptanceTest.php | 2 | ||||
-rw-r--r-- | tests/Unit/Activity/ProviderTest.php | 8 | ||||
-rw-r--r-- | tests/Unit/Activity/SettingTest.php | 2 | ||||
-rw-r--r-- | tests/Unit/Controller/SettingsControllerTest.php | 2 | ||||
-rw-r--r-- | tests/Unit/Event/StateChangedTest.php | 4 | ||||
-rw-r--r-- | tests/Unit/Listener/StateChangeActivityTest.php | 3 | ||||
-rw-r--r-- | tests/Unit/Listener/StateChangeRegistryUpdaterTest.php | 1 | ||||
-rw-r--r-- | tests/Unit/Provider/AtLoginProviderTest.php | 1 | ||||
-rw-r--r-- | tests/Unit/Provider/TotpProviderTest.php | 1 | ||||
-rw-r--r-- | tests/bootstrap.php | 2 |
10 files changed, 6 insertions, 20 deletions
diff --git a/tests/Acceptance/TOTPAcceptanceTest.php b/tests/Acceptance/TOTPAcceptanceTest.php index 4ba5193..691202d 100644 --- a/tests/Acceptance/TOTPAcceptanceTest.php +++ b/tests/Acceptance/TOTPAcceptanceTest.php @@ -46,7 +46,6 @@ use PHPUnit_Framework_AssertionFailedError; * @group Acceptance */ class TOTPAcceptenceTest extends TestCase { - use TestUser; use Selenium; @@ -174,5 +173,4 @@ class TOTPAcceptenceTest extends TestCase { $this->assertEquals('http://localhost:8080/index.php/login/challenge/totp', $this->webDriver->getCurrentURL()); } - } diff --git a/tests/Unit/Activity/ProviderTest.php b/tests/Unit/Activity/ProviderTest.php index 05e47c6..b1a086c 100644 --- a/tests/Unit/Activity/ProviderTest.php +++ b/tests/Unit/Activity/ProviderTest.php @@ -32,7 +32,6 @@ use OCP\IURLGenerator; use OCP\L10N\IFactory; class ProviderTest extends TestCase { - private $l10n; private $urlGenerator; private $logger; @@ -63,9 +62,9 @@ class ProviderTest extends TestCase { public function subjectData() { return [ - ['totp_enabled_subject'], - ['totp_disabled_subject'], - ['totp_disabled_by_admin'], + ['totp_enabled_subject'], + ['totp_disabled_subject'], + ['totp_disabled_by_admin'], ]; } @@ -103,5 +102,4 @@ class ProviderTest extends TestCase { $this->provider->parse($lang, $event); } - } diff --git a/tests/Unit/Activity/SettingTest.php b/tests/Unit/Activity/SettingTest.php index 76debf0..fbcbad5 100644 --- a/tests/Unit/Activity/SettingTest.php +++ b/tests/Unit/Activity/SettingTest.php @@ -27,7 +27,6 @@ use OCA\TwoFactorTOTP\Activity\Setting; use OCP\IL10N; class SettingTest extends TestCase { - private $l10n; /** @var Setting */ @@ -54,5 +53,4 @@ class SettingTest extends TestCase { $this->assertEquals(true, $this->setting->isDefaultEnabledMail()); $this->assertEquals(true, $this->setting->isDefaultEnabledStream()); } - } diff --git a/tests/Unit/Controller/SettingsControllerTest.php b/tests/Unit/Controller/SettingsControllerTest.php index 88eef84..5963448 100644 --- a/tests/Unit/Controller/SettingsControllerTest.php +++ b/tests/Unit/Controller/SettingsControllerTest.php @@ -34,7 +34,6 @@ use OCP\IUserSession; use PHPUnit\Framework\TestCase; class SettingsControllerTest extends TestCase { - private $request; private $userSession; private $totp; @@ -134,5 +133,4 @@ class SettingsControllerTest extends TestCase { $this->expectException(InvalidArgumentException::class); $this->controller->enable(17); } - } diff --git a/tests/Unit/Event/StateChangedTest.php b/tests/Unit/Event/StateChangedTest.php index aca5a0e..f55d0f6 100644 --- a/tests/Unit/Event/StateChangedTest.php +++ b/tests/Unit/Event/StateChangedTest.php @@ -29,7 +29,6 @@ use OCA\TwoFactorTOTP\Event\StateChanged; use OCP\IUser; class StateChangedTest extends TestCase { - public function testEnabled() { $user = $this->createMock(IUser::class); $event = new StateChanged($user, true); @@ -47,5 +46,4 @@ class StateChangedTest extends TestCase { $this->assertFalse($enabled); } - -}
\ No newline at end of file +} diff --git a/tests/Unit/Listener/StateChangeActivityTest.php b/tests/Unit/Listener/StateChangeActivityTest.php index 540f864..626f053 100644 --- a/tests/Unit/Listener/StateChangeActivityTest.php +++ b/tests/Unit/Listener/StateChangeActivityTest.php @@ -115,5 +115,4 @@ class StateChangeActivityTest extends TestCase { $this->listener->handle($event); } - -}
\ No newline at end of file +} diff --git a/tests/Unit/Listener/StateChangeRegistryUpdaterTest.php b/tests/Unit/Listener/StateChangeRegistryUpdaterTest.php index 0c48d4b..603bfaf 100644 --- a/tests/Unit/Listener/StateChangeRegistryUpdaterTest.php +++ b/tests/Unit/Listener/StateChangeRegistryUpdaterTest.php @@ -65,5 +65,4 @@ class StateChangeRegistryUpdaterTest extends TestCase { $this->listener->handle($event); } - } diff --git a/tests/Unit/Provider/AtLoginProviderTest.php b/tests/Unit/Provider/AtLoginProviderTest.php index ac610f8..bc225a3 100644 --- a/tests/Unit/Provider/AtLoginProviderTest.php +++ b/tests/Unit/Provider/AtLoginProviderTest.php @@ -52,5 +52,4 @@ class AtLoginProviderTest extends TestCase { $this->addToAssertionCount(1); } - } diff --git a/tests/Unit/Provider/TotpProviderTest.php b/tests/Unit/Provider/TotpProviderTest.php index 411ca59..d379831 100644 --- a/tests/Unit/Provider/TotpProviderTest.php +++ b/tests/Unit/Provider/TotpProviderTest.php @@ -156,5 +156,4 @@ class TotpProviderTest extends TestCase { $this->assertSame($provider, $result); } - } diff --git a/tests/bootstrap.php b/tests/bootstrap.php index f0aa03a..2eb4a87 100644 --- a/tests/bootstrap.php +++ b/tests/bootstrap.php @@ -23,4 +23,4 @@ require_once __DIR__ . '/../../../lib/base.php'; require_once __DIR__ . '/../../../tests/bootstrap.php'; -\OC_App::loadApp('twofactor_totp');
\ No newline at end of file +\OC_App::loadApp('twofactor_totp'); |