nextcloud / server

tests/Core/Command/TwoFactorAuth/StateTest.php

Indentation +73 added lines, -73 removed lines

@@ -18,77 +18,77 @@
 use Test\TestCase;
 
 class StateTest extends TestCase {
-	/** @var IRegistry|MockObject */
-	private $registry;
-
-	/** @var IUserManager|MockObject */
-	private $userManager;
-
-	/** @var CommandTester|MockObject */
-	private $cmd;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->registry = $this->createMock(IRegistry::class);
-		$this->userManager = $this->createMock(IUserManager::class);
-
-		$cmd = new State($this->registry, $this->userManager);
-		$this->cmd = new CommandTester($cmd);
-	}
-
-	public function testWrongUID(): void {
-		$this->cmd->execute([
-			'uid' => 'nope',
-		]);
-
-		$output = $this->cmd->getDisplay();
-		$this->assertStringContainsString('Invalid UID', $output);
-	}
-
-	public function testStateNoProvidersActive(): void {
-		$user = $this->createMock(IUser::class);
-		$this->userManager->expects($this->once())
-			->method('get')
-			->with('eldora')
-			->willReturn($user);
-		$states = [
-			'u2f' => false,
-			'totp' => false,
-		];
-		$this->registry->expects($this->once())
-			->method('getProviderStates')
-			->with($user)
-			->willReturn($states);
-
-		$this->cmd->execute([
-			'uid' => 'eldora',
-		]);
-
-		$output = $this->cmd->getDisplay();
-		$this->assertStringContainsString('Two-factor authentication is not enabled for user eldora', $output);
-	}
-
-	public function testStateOneProviderActive(): void {
-		$user = $this->createMock(IUser::class);
-		$this->userManager->expects($this->once())
-			->method('get')
-			->with('mohamed')
-			->willReturn($user);
-		$states = [
-			'u2f' => true,
-			'totp' => false,
-		];
-		$this->registry->expects($this->once())
-			->method('getProviderStates')
-			->with($user)
-			->willReturn($states);
-
-		$this->cmd->execute([
-			'uid' => 'mohamed',
-		]);
-
-		$output = $this->cmd->getDisplay();
-		$this->assertStringContainsString('Two-factor authentication is enabled for user mohamed', $output);
-	}
+    /** @var IRegistry|MockObject */
+    private $registry;
+
+    /** @var IUserManager|MockObject */
+    private $userManager;
+
+    /** @var CommandTester|MockObject */
+    private $cmd;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->registry = $this->createMock(IRegistry::class);
+        $this->userManager = $this->createMock(IUserManager::class);
+
+        $cmd = new State($this->registry, $this->userManager);
+        $this->cmd = new CommandTester($cmd);
+    }
+
+    public function testWrongUID(): void {
+        $this->cmd->execute([
+            'uid' => 'nope',
+        ]);
+
+        $output = $this->cmd->getDisplay();
+        $this->assertStringContainsString('Invalid UID', $output);
+    }
+
+    public function testStateNoProvidersActive(): void {
+        $user = $this->createMock(IUser::class);
+        $this->userManager->expects($this->once())
+            ->method('get')
+            ->with('eldora')
+            ->willReturn($user);
+        $states = [
+            'u2f' => false,
+            'totp' => false,
+        ];
+        $this->registry->expects($this->once())
+            ->method('getProviderStates')
+            ->with($user)
+            ->willReturn($states);
+
+        $this->cmd->execute([
+            'uid' => 'eldora',
+        ]);
+
+        $output = $this->cmd->getDisplay();
+        $this->assertStringContainsString('Two-factor authentication is not enabled for user eldora', $output);
+    }
+
+    public function testStateOneProviderActive(): void {
+        $user = $this->createMock(IUser::class);
+        $this->userManager->expects($this->once())
+            ->method('get')
+            ->with('mohamed')
+            ->willReturn($user);
+        $states = [
+            'u2f' => true,
+            'totp' => false,
+        ];
+        $this->registry->expects($this->once())
+            ->method('getProviderStates')
+            ->with($user)
+            ->willReturn($states);
+
+        $this->cmd->execute([
+            'uid' => 'mohamed',
+        ]);
+
+        $output = $this->cmd->getDisplay();
+        $this->assertStringContainsString('Two-factor authentication is enabled for user mohamed', $output);
+    }
 }

tests/Core/Command/TwoFactorAuth/CleanupTest.php

Indentation +24 added lines, -24 removed lines

@@ -17,36 +17,36 @@
 use Test\TestCase;
 
 class CleanupTest extends TestCase {
-	/** @var IRegistry|MockObject */
-	private $registry;
+    /** @var IRegistry|MockObject */
+    private $registry;
 
-	/** @var IUserManager|MockObject */
-	private $userManager;
+    /** @var IUserManager|MockObject */
+    private $userManager;
 
-	/** @var CommandTester */
-	private $cmd;
+    /** @var CommandTester */
+    private $cmd;
 
-	protected function setUp(): void {
-		parent::setUp();
+    protected function setUp(): void {
+        parent::setUp();
 
-		$this->registry = $this->createMock(IRegistry::class);
-		$this->userManager = $this->createMock(IUserManager::class);
+        $this->registry = $this->createMock(IRegistry::class);
+        $this->userManager = $this->createMock(IUserManager::class);
 
-		$cmd = new Cleanup($this->registry, $this->userManager);
-		$this->cmd = new CommandTester($cmd);
-	}
+        $cmd = new Cleanup($this->registry, $this->userManager);
+        $this->cmd = new CommandTester($cmd);
+    }
 
-	public function testCleanup(): void {
-		$this->registry->expects($this->once())
-			->method('cleanUp')
-			->with('u2f');
+    public function testCleanup(): void {
+        $this->registry->expects($this->once())
+            ->method('cleanUp')
+            ->with('u2f');
 
-		$rc = $this->cmd->execute([
-			'provider-id' => 'u2f',
-		]);
+        $rc = $this->cmd->execute([
+            'provider-id' => 'u2f',
+        ]);
 
-		$this->assertEquals(0, $rc);
-		$output = $this->cmd->getDisplay();
-		$this->assertStringContainsString('All user-provider associations for provider u2f have been removed', $output);
-	}
+        $this->assertEquals(0, $rc);
+        $output = $this->cmd->getDisplay();
+        $this->assertStringContainsString('All user-provider associations for provider u2f have been removed', $output);
+    }
 }

tests/Core/Command/TwoFactorAuth/EnableTest.php

Indentation +73 added lines, -73 removed lines

@@ -18,77 +18,77 @@
 use Test\TestCase;
 
 class EnableTest extends TestCase {
-	/** @var ProviderManager|MockObject */
-	private $providerManager;
-
-	/** @var IUserManager|MockObject */
-	private $userManager;
-
-	/** @var CommandTester */
-	private $command;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->providerManager = $this->createMock(ProviderManager::class);
-		$this->userManager = $this->createMock(IUserManager::class);
-
-		$cmd = new Enable($this->providerManager, $this->userManager);
-		$this->command = new CommandTester($cmd);
-	}
-
-	public function testInvalidUID(): void {
-		$this->userManager->expects($this->once())
-			->method('get')
-			->with('nope')
-			->willReturn(null);
-
-		$rc = $this->command->execute([
-			'uid' => 'nope',
-			'provider_id' => 'nope',
-		]);
-
-		$this->assertEquals(1, $rc);
-		$this->assertStringContainsString('Invalid UID', $this->command->getDisplay());
-	}
-
-	public function testEnableNotSupported(): void {
-		$user = $this->createMock(IUser::class);
-		$this->userManager->expects($this->once())
-			->method('get')
-			->with('belle')
-			->willReturn($user);
-		$this->providerManager->expects($this->once())
-			->method('tryEnableProviderFor')
-			->with('totp', $user)
-			->willReturn(false);
-
-		$rc = $this->command->execute([
-			'uid' => 'belle',
-			'provider_id' => 'totp',
-		]);
-
-		$this->assertEquals(2, $rc);
-		$this->assertStringContainsString('The provider does not support this operation', $this->command->getDisplay());
-	}
-
-	public function testEnabled(): void {
-		$user = $this->createMock(IUser::class);
-		$this->userManager->expects($this->once())
-			->method('get')
-			->with('belle')
-			->willReturn($user);
-		$this->providerManager->expects($this->once())
-			->method('tryEnableProviderFor')
-			->with('totp', $user)
-			->willReturn(true);
-
-		$rc = $this->command->execute([
-			'uid' => 'belle',
-			'provider_id' => 'totp',
-		]);
-
-		$this->assertEquals(0, $rc);
-		$this->assertStringContainsString('Two-factor provider totp enabled for user belle', $this->command->getDisplay());
-	}
+    /** @var ProviderManager|MockObject */
+    private $providerManager;
+
+    /** @var IUserManager|MockObject */
+    private $userManager;
+
+    /** @var CommandTester */
+    private $command;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->providerManager = $this->createMock(ProviderManager::class);
+        $this->userManager = $this->createMock(IUserManager::class);
+
+        $cmd = new Enable($this->providerManager, $this->userManager);
+        $this->command = new CommandTester($cmd);
+    }
+
+    public function testInvalidUID(): void {
+        $this->userManager->expects($this->once())
+            ->method('get')
+            ->with('nope')
+            ->willReturn(null);
+
+        $rc = $this->command->execute([
+            'uid' => 'nope',
+            'provider_id' => 'nope',
+        ]);
+
+        $this->assertEquals(1, $rc);
+        $this->assertStringContainsString('Invalid UID', $this->command->getDisplay());
+    }
+
+    public function testEnableNotSupported(): void {
+        $user = $this->createMock(IUser::class);
+        $this->userManager->expects($this->once())
+            ->method('get')
+            ->with('belle')
+            ->willReturn($user);
+        $this->providerManager->expects($this->once())
+            ->method('tryEnableProviderFor')
+            ->with('totp', $user)
+            ->willReturn(false);
+
+        $rc = $this->command->execute([
+            'uid' => 'belle',
+            'provider_id' => 'totp',
+        ]);
+
+        $this->assertEquals(2, $rc);
+        $this->assertStringContainsString('The provider does not support this operation', $this->command->getDisplay());
+    }
+
+    public function testEnabled(): void {
+        $user = $this->createMock(IUser::class);
+        $this->userManager->expects($this->once())
+            ->method('get')
+            ->with('belle')
+            ->willReturn($user);
+        $this->providerManager->expects($this->once())
+            ->method('tryEnableProviderFor')
+            ->with('totp', $user)
+            ->willReturn(true);
+
+        $rc = $this->command->execute([
+            'uid' => 'belle',
+            'provider_id' => 'totp',
+        ]);
+
+        $this->assertEquals(0, $rc);
+        $this->assertStringContainsString('Two-factor provider totp enabled for user belle', $this->command->getDisplay());
+    }
 }

tests/Core/Command/TwoFactorAuth/DisableTest.php

Indentation +73 added lines, -73 removed lines

@@ -18,77 +18,77 @@
 use Test\TestCase;
 
 class DisableTest extends TestCase {
-	/** @var ProviderManager|MockObject */
-	private $providerManager;
-
-	/** @var IUserManager|MockObject */
-	private $userManager;
-
-	/** @var CommandTester */
-	private $command;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->providerManager = $this->createMock(ProviderManager::class);
-		$this->userManager = $this->createMock(IUserManager::class);
-
-		$cmd = new Disable($this->providerManager, $this->userManager);
-		$this->command = new CommandTester($cmd);
-	}
-
-	public function testInvalidUID(): void {
-		$this->userManager->expects($this->once())
-			->method('get')
-			->with('nope')
-			->willReturn(null);
-
-		$rc = $this->command->execute([
-			'uid' => 'nope',
-			'provider_id' => 'nope',
-		]);
-
-		$this->assertEquals(1, $rc);
-		$this->assertStringContainsString('Invalid UID', $this->command->getDisplay());
-	}
-
-	public function testEnableNotSupported(): void {
-		$user = $this->createMock(IUser::class);
-		$this->userManager->expects($this->once())
-			->method('get')
-			->with('ricky')
-			->willReturn($user);
-		$this->providerManager->expects($this->once())
-			->method('tryDisableProviderFor')
-			->with('totp', $user)
-			->willReturn(false);
-
-		$rc = $this->command->execute([
-			'uid' => 'ricky',
-			'provider_id' => 'totp',
-		]);
-
-		$this->assertEquals(2, $rc);
-		$this->assertStringContainsString('The provider does not support this operation', $this->command->getDisplay());
-	}
-
-	public function testEnabled(): void {
-		$user = $this->createMock(IUser::class);
-		$this->userManager->expects($this->once())
-			->method('get')
-			->with('ricky')
-			->willReturn($user);
-		$this->providerManager->expects($this->once())
-			->method('tryDisableProviderFor')
-			->with('totp', $user)
-			->willReturn(true);
-
-		$rc = $this->command->execute([
-			'uid' => 'ricky',
-			'provider_id' => 'totp',
-		]);
-
-		$this->assertEquals(0, $rc);
-		$this->assertStringContainsString('Two-factor provider totp disabled for user ricky', $this->command->getDisplay());
-	}
+    /** @var ProviderManager|MockObject */
+    private $providerManager;
+
+    /** @var IUserManager|MockObject */
+    private $userManager;
+
+    /** @var CommandTester */
+    private $command;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->providerManager = $this->createMock(ProviderManager::class);
+        $this->userManager = $this->createMock(IUserManager::class);
+
+        $cmd = new Disable($this->providerManager, $this->userManager);
+        $this->command = new CommandTester($cmd);
+    }
+
+    public function testInvalidUID(): void {
+        $this->userManager->expects($this->once())
+            ->method('get')
+            ->with('nope')
+            ->willReturn(null);
+
+        $rc = $this->command->execute([
+            'uid' => 'nope',
+            'provider_id' => 'nope',
+        ]);
+
+        $this->assertEquals(1, $rc);
+        $this->assertStringContainsString('Invalid UID', $this->command->getDisplay());
+    }
+
+    public function testEnableNotSupported(): void {
+        $user = $this->createMock(IUser::class);
+        $this->userManager->expects($this->once())
+            ->method('get')
+            ->with('ricky')
+            ->willReturn($user);
+        $this->providerManager->expects($this->once())
+            ->method('tryDisableProviderFor')
+            ->with('totp', $user)
+            ->willReturn(false);
+
+        $rc = $this->command->execute([
+            'uid' => 'ricky',
+            'provider_id' => 'totp',
+        ]);
+
+        $this->assertEquals(2, $rc);
+        $this->assertStringContainsString('The provider does not support this operation', $this->command->getDisplay());
+    }
+
+    public function testEnabled(): void {
+        $user = $this->createMock(IUser::class);
+        $this->userManager->expects($this->once())
+            ->method('get')
+            ->with('ricky')
+            ->willReturn($user);
+        $this->providerManager->expects($this->once())
+            ->method('tryDisableProviderFor')
+            ->with('totp', $user)
+            ->willReturn(true);
+
+        $rc = $this->command->execute([
+            'uid' => 'ricky',
+            'provider_id' => 'totp',
+        ]);
+
+        $this->assertEquals(0, $rc);
+        $this->assertStringContainsString('Two-factor provider totp disabled for user ricky', $this->command->getDisplay());
+    }
 }

tests/Core/Command/User/EnableTest.php

Indentation +60 added lines, -60 removed lines

@@ -14,64 +14,64 @@
 use Test\TestCase;
 
 class EnableTest extends TestCase {
-	/** @var IUserManager|\PHPUnit\Framework\MockObject\MockObject */
-	protected $userManager;
-	/** @var InputInterface|\PHPUnit\Framework\MockObject\MockObject */
-	protected $consoleInput;
-	/** @var OutputInterface|\PHPUnit\Framework\MockObject\MockObject */
-	protected $consoleOutput;
-
-	/** @var Disable */
-	protected $command;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->userManager = $this->createMock(IUserManager::class);
-		$this->consoleInput = $this->createMock(InputInterface::class);
-		$this->consoleOutput = $this->createMock(OutputInterface::class);
-
-		$this->command = new Enable($this->userManager);
-	}
-
-	public function testValidUser(): void {
-		$user = $this->createMock(IUser::class);
-		$user->expects($this->once())
-			->method('setEnabled')
-			->with(true);
-
-		$this->userManager
-			->method('get')
-			->with('user')
-			->willReturn($user);
-
-		$this->consoleInput
-			->method('getArgument')
-			->with('uid')
-			->willReturn('user');
-
-		$this->consoleOutput->expects($this->once())
-			->method('writeln')
-			->with($this->stringContains('The specified user is enabled'));
-
-		self::invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
-	}
-
-	public function testInvalidUser(): void {
-		$this->userManager->expects($this->once())
-			->method('get')
-			->with('user')
-			->willReturn(null);
-
-		$this->consoleInput
-			->method('getArgument')
-			->with('uid')
-			->willReturn('user');
-
-		$this->consoleOutput->expects($this->once())
-			->method('writeln')
-			->with($this->stringContains('User does not exist'));
-
-		self::invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
-	}
+    /** @var IUserManager|\PHPUnit\Framework\MockObject\MockObject */
+    protected $userManager;
+    /** @var InputInterface|\PHPUnit\Framework\MockObject\MockObject */
+    protected $consoleInput;
+    /** @var OutputInterface|\PHPUnit\Framework\MockObject\MockObject */
+    protected $consoleOutput;
+
+    /** @var Disable */
+    protected $command;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->userManager = $this->createMock(IUserManager::class);
+        $this->consoleInput = $this->createMock(InputInterface::class);
+        $this->consoleOutput = $this->createMock(OutputInterface::class);
+
+        $this->command = new Enable($this->userManager);
+    }
+
+    public function testValidUser(): void {
+        $user = $this->createMock(IUser::class);
+        $user->expects($this->once())
+            ->method('setEnabled')
+            ->with(true);
+
+        $this->userManager
+            ->method('get')
+            ->with('user')
+            ->willReturn($user);
+
+        $this->consoleInput
+            ->method('getArgument')
+            ->with('uid')
+            ->willReturn('user');
+
+        $this->consoleOutput->expects($this->once())
+            ->method('writeln')
+            ->with($this->stringContains('The specified user is enabled'));
+
+        self::invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
+    }
+
+    public function testInvalidUser(): void {
+        $this->userManager->expects($this->once())
+            ->method('get')
+            ->with('user')
+            ->willReturn(null);
+
+        $this->consoleInput
+            ->method('getArgument')
+            ->with('uid')
+            ->willReturn('user');
+
+        $this->consoleOutput->expects($this->once())
+            ->method('writeln')
+            ->with($this->stringContains('User does not exist'));
+
+        self::invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
+    }
 }

tests/Core/Command/User/AuthTokens/DeleteTest.php

Indentation +148 added lines, -148 removed lines

@@ -14,152 +14,152 @@
 use Test\TestCase;
 
 class DeleteTest extends TestCase {
-	/** @var \PHPUnit\Framework\MockObject\MockObject */
-	protected $tokenProvider;
-	/** @var \PHPUnit\Framework\MockObject\MockObject */
-	protected $consoleInput;
-	/** @var \PHPUnit\Framework\MockObject\MockObject */
-	protected $consoleOutput;
-
-	/** @var \Symfony\Component\Console\Command\Command */
-	protected $command;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$tokenProvider = $this->tokenProvider = $this->getMockBuilder(IProvider::class)
-			->disableOriginalConstructor()
-			->getMock();
-		$this->consoleInput = $this->getMockBuilder(InputInterface::class)->getMock();
-		$this->consoleOutput = $this->getMockBuilder(OutputInterface::class)->getMock();
-
-		/** @var \OC\Authentication\Token\IProvider $tokenProvider */
-		$this->command = new Delete($tokenProvider);
-	}
-
-	public function testDeleteTokenById(): void {
-		$this->consoleInput->expects($this->exactly(2))
-			->method('getArgument')
-			->willReturnMap([
-				['uid', 'user'],
-				['id', '42']
-			]);
-
-		$this->consoleInput->expects($this->once())
-			->method('getOption')
-			->with('last-used-before')
-			->willReturn(null);
-
-		$this->tokenProvider->expects($this->once())
-			->method('invalidateTokenById')
-			->with('user', 42);
-
-		$result = self::invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
-		$this->assertSame(Command::SUCCESS, $result);
-	}
-
-	public function testDeleteTokenByIdRequiresTokenId(): void {
-		$this->consoleInput->expects($this->exactly(2))
-			->method('getArgument')
-			->willReturnMap([
-				['uid', 'user'],
-				['id', null]
-			]);
-
-		$this->consoleInput->expects($this->once())
-			->method('getOption')
-			->with('last-used-before')
-			->willReturn(null);
-
-		$this->expectException(RuntimeException::class);
-
-		$this->tokenProvider->expects($this->never())->method('invalidateTokenById');
-
-		$result = self::invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
-		$this->assertSame(Command::FAILURE, $result);
-	}
-
-	public function testDeleteTokensLastUsedBefore(): void {
-		$this->consoleInput->expects($this->exactly(2))
-			->method('getArgument')
-			->willReturnMap([
-				['uid', 'user'],
-				['id', null]
-			]);
-
-		$this->consoleInput->expects($this->once())
-			->method('getOption')
-			->with('last-used-before')
-			->willReturn('946684800');
-
-		$this->tokenProvider->expects($this->once())
-			->method('invalidateLastUsedBefore')
-			->with('user', 946684800);
-
-		$result = self::invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
-		$this->assertSame(Command::SUCCESS, $result);
-	}
-
-	public function testLastUsedBeforeAcceptsIso8601Expanded(): void {
-		$this->consoleInput->expects($this->exactly(2))
-			->method('getArgument')
-			->willReturnMap([
-				['uid', 'user'],
-				['id', null]
-			]);
-
-		$this->consoleInput->expects($this->once())
-			->method('getOption')
-			->with('last-used-before')
-			->willReturn('2000-01-01T00:00:00Z');
-
-		$this->tokenProvider->expects($this->once())
-			->method('invalidateLastUsedBefore')
-			->with('user', 946684800);
-
-		$result = self::invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
-		$this->assertSame(Command::SUCCESS, $result);
-	}
-
-	public function testLastUsedBeforeAcceptsYmd(): void {
-		$this->consoleInput->expects($this->exactly(2))
-			->method('getArgument')
-			->willReturnMap([
-				['uid', 'user'],
-				['id', null]
-			]);
-
-		$this->consoleInput->expects($this->once())
-			->method('getOption')
-			->with('last-used-before')
-			->willReturn('2000-01-01');
-
-		$this->tokenProvider->expects($this->once())
-			->method('invalidateLastUsedBefore')
-			->with('user', 946684800);
-
-		$result = self::invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
-		$this->assertSame(Command::SUCCESS, $result);
-	}
-
-	public function testIdAndLastUsedBeforeAreMutuallyExclusive(): void {
-		$this->consoleInput->expects($this->exactly(2))
-			->method('getArgument')
-			->willReturnMap([
-				['uid', 'user'],
-				['id', '42']
-			]);
-
-		$this->consoleInput->expects($this->once())
-			->method('getOption')
-			->with('last-used-before')
-			->willReturn('946684800');
-
-		$this->expectException(RuntimeException::class);
-
-		$this->tokenProvider->expects($this->never())->method('invalidateLastUsedBefore');
-
-		$result = self::invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
-		$this->assertSame(Command::SUCCESS, $result);
-	}
+    /** @var \PHPUnit\Framework\MockObject\MockObject */
+    protected $tokenProvider;
+    /** @var \PHPUnit\Framework\MockObject\MockObject */
+    protected $consoleInput;
+    /** @var \PHPUnit\Framework\MockObject\MockObject */
+    protected $consoleOutput;
+
+    /** @var \Symfony\Component\Console\Command\Command */
+    protected $command;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $tokenProvider = $this->tokenProvider = $this->getMockBuilder(IProvider::class)
+            ->disableOriginalConstructor()
+            ->getMock();
+        $this->consoleInput = $this->getMockBuilder(InputInterface::class)->getMock();
+        $this->consoleOutput = $this->getMockBuilder(OutputInterface::class)->getMock();
+
+        /** @var \OC\Authentication\Token\IProvider $tokenProvider */
+        $this->command = new Delete($tokenProvider);
+    }
+
+    public function testDeleteTokenById(): void {
+        $this->consoleInput->expects($this->exactly(2))
+            ->method('getArgument')
+            ->willReturnMap([
+                ['uid', 'user'],
+                ['id', '42']
+            ]);
+
+        $this->consoleInput->expects($this->once())
+            ->method('getOption')
+            ->with('last-used-before')
+            ->willReturn(null);
+
+        $this->tokenProvider->expects($this->once())
+            ->method('invalidateTokenById')
+            ->with('user', 42);
+
+        $result = self::invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
+        $this->assertSame(Command::SUCCESS, $result);
+    }
+
+    public function testDeleteTokenByIdRequiresTokenId(): void {
+        $this->consoleInput->expects($this->exactly(2))
+            ->method('getArgument')
+            ->willReturnMap([
+                ['uid', 'user'],
+                ['id', null]
+            ]);
+
+        $this->consoleInput->expects($this->once())
+            ->method('getOption')
+            ->with('last-used-before')
+            ->willReturn(null);
+
+        $this->expectException(RuntimeException::class);
+
+        $this->tokenProvider->expects($this->never())->method('invalidateTokenById');
+
+        $result = self::invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
+        $this->assertSame(Command::FAILURE, $result);
+    }
+
+    public function testDeleteTokensLastUsedBefore(): void {
+        $this->consoleInput->expects($this->exactly(2))
+            ->method('getArgument')
+            ->willReturnMap([
+                ['uid', 'user'],
+                ['id', null]
+            ]);
+
+        $this->consoleInput->expects($this->once())
+            ->method('getOption')
+            ->with('last-used-before')
+            ->willReturn('946684800');
+
+        $this->tokenProvider->expects($this->once())
+            ->method('invalidateLastUsedBefore')
+            ->with('user', 946684800);
+
+        $result = self::invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
+        $this->assertSame(Command::SUCCESS, $result);
+    }
+
+    public function testLastUsedBeforeAcceptsIso8601Expanded(): void {
+        $this->consoleInput->expects($this->exactly(2))
+            ->method('getArgument')
+            ->willReturnMap([
+                ['uid', 'user'],
+                ['id', null]
+            ]);
+
+        $this->consoleInput->expects($this->once())
+            ->method('getOption')
+            ->with('last-used-before')
+            ->willReturn('2000-01-01T00:00:00Z');
+
+        $this->tokenProvider->expects($this->once())
+            ->method('invalidateLastUsedBefore')
+            ->with('user', 946684800);
+
+        $result = self::invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
+        $this->assertSame(Command::SUCCESS, $result);
+    }
+
+    public function testLastUsedBeforeAcceptsYmd(): void {
+        $this->consoleInput->expects($this->exactly(2))
+            ->method('getArgument')
+            ->willReturnMap([
+                ['uid', 'user'],
+                ['id', null]
+            ]);
+
+        $this->consoleInput->expects($this->once())
+            ->method('getOption')
+            ->with('last-used-before')
+            ->willReturn('2000-01-01');
+
+        $this->tokenProvider->expects($this->once())
+            ->method('invalidateLastUsedBefore')
+            ->with('user', 946684800);
+
+        $result = self::invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
+        $this->assertSame(Command::SUCCESS, $result);
+    }
+
+    public function testIdAndLastUsedBeforeAreMutuallyExclusive(): void {
+        $this->consoleInput->expects($this->exactly(2))
+            ->method('getArgument')
+            ->willReturnMap([
+                ['uid', 'user'],
+                ['id', '42']
+            ]);
+
+        $this->consoleInput->expects($this->once())
+            ->method('getOption')
+            ->with('last-used-before')
+            ->willReturn('946684800');
+
+        $this->expectException(RuntimeException::class);
+
+        $this->tokenProvider->expects($this->never())->method('invalidateLastUsedBefore');
+
+        $result = self::invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
+        $this->assertSame(Command::SUCCESS, $result);
+    }
 }

tests/Core/Command/User/DisableTest.php

Indentation +60 added lines, -60 removed lines

@@ -14,64 +14,64 @@
 use Test\TestCase;
 
 class DisableTest extends TestCase {
-	/** @var IUserManager|\PHPUnit\Framework\MockObject\MockObject */
-	protected $userManager;
-	/** @var InputInterface|\PHPUnit\Framework\MockObject\MockObject */
-	protected $consoleInput;
-	/** @var OutputInterface|\PHPUnit\Framework\MockObject\MockObject */
-	protected $consoleOutput;
-
-	/** @var Disable */
-	protected $command;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->userManager = $this->createMock(IUserManager::class);
-		$this->consoleInput = $this->createMock(InputInterface::class);
-		$this->consoleOutput = $this->createMock(OutputInterface::class);
-
-		$this->command = new Disable($this->userManager);
-	}
-
-	public function testValidUser(): void {
-		$user = $this->createMock(IUser::class);
-		$user->expects($this->once())
-			->method('setEnabled')
-			->with(false);
-
-		$this->userManager
-			->method('get')
-			->with('user')
-			->willReturn($user);
-
-		$this->consoleInput
-			->method('getArgument')
-			->with('uid')
-			->willReturn('user');
-
-		$this->consoleOutput->expects($this->once())
-			->method('writeln')
-			->with($this->stringContains('The specified user is disabled'));
-
-		self::invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
-	}
-
-	public function testInvalidUser(): void {
-		$this->userManager->expects($this->once())
-			->method('get')
-			->with('user')
-			->willReturn(null);
-
-		$this->consoleInput
-			->method('getArgument')
-			->with('uid')
-			->willReturn('user');
-
-		$this->consoleOutput->expects($this->once())
-			->method('writeln')
-			->with($this->stringContains('User does not exist'));
-
-		self::invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
-	}
+    /** @var IUserManager|\PHPUnit\Framework\MockObject\MockObject */
+    protected $userManager;
+    /** @var InputInterface|\PHPUnit\Framework\MockObject\MockObject */
+    protected $consoleInput;
+    /** @var OutputInterface|\PHPUnit\Framework\MockObject\MockObject */
+    protected $consoleOutput;
+
+    /** @var Disable */
+    protected $command;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->userManager = $this->createMock(IUserManager::class);
+        $this->consoleInput = $this->createMock(InputInterface::class);
+        $this->consoleOutput = $this->createMock(OutputInterface::class);
+
+        $this->command = new Disable($this->userManager);
+    }
+
+    public function testValidUser(): void {
+        $user = $this->createMock(IUser::class);
+        $user->expects($this->once())
+            ->method('setEnabled')
+            ->with(false);
+
+        $this->userManager
+            ->method('get')
+            ->with('user')
+            ->willReturn($user);
+
+        $this->consoleInput
+            ->method('getArgument')
+            ->with('uid')
+            ->willReturn('user');
+
+        $this->consoleOutput->expects($this->once())
+            ->method('writeln')
+            ->with($this->stringContains('The specified user is disabled'));
+
+        self::invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
+    }
+
+    public function testInvalidUser(): void {
+        $this->userManager->expects($this->once())
+            ->method('get')
+            ->with('user')
+            ->willReturn(null);
+
+        $this->consoleInput
+            ->method('getArgument')
+            ->with('uid')
+            ->willReturn('user');
+
+        $this->consoleOutput->expects($this->once())
+            ->method('writeln')
+            ->with($this->stringContains('User does not exist'));
+
+        self::invokePrivate($this->command, 'execute', [$this->consoleInput, $this->consoleOutput]);
+    }
 }

tests/Core/Command/Group/AddUserTest.php

Indentation +81 added lines, -81 removed lines

@@ -16,85 +16,85 @@
 use Test\TestCase;
 
 class AddUserTest extends TestCase {
-	/** @var IGroupManager|\PHPUnit\Framework\MockObject\MockObject */
-	private $groupManager;
-
-	/** @var IUserManager|\PHPUnit\Framework\MockObject\MockObject */
-	private $userManager;
-
-	/** @var AddUser */
-	private $command;
-
-	/** @var InputInterface|\PHPUnit\Framework\MockObject\MockObject */
-	private $input;
-
-	/** @var OutputInterface|\PHPUnit\Framework\MockObject\MockObject */
-	private $output;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->groupManager = $this->createMock(IGroupManager::class);
-		$this->userManager = $this->createMock(IUserManager::class);
-		$this->command = new AddUser($this->userManager, $this->groupManager);
-
-		$this->input = $this->createMock(InputInterface::class);
-		$this->input->method('getArgument')
-			->willReturnCallback(function ($arg) {
-				if ($arg === 'group') {
-					return 'myGroup';
-				} elseif ($arg === 'user') {
-					return 'myUser';
-				}
-				throw new \Exception();
-			});
-		$this->output = $this->createMock(OutputInterface::class);
-	}
-
-	public function testNoGroup(): void {
-		$this->groupManager->method('get')
-			->with('myGroup')
-			->willReturn(null);
-
-		$this->output->expects($this->once())
-			->method('writeln')
-			->with('<error>group not found</error>');
-
-		$this->invokePrivate($this->command, 'execute', [$this->input, $this->output]);
-	}
-
-	public function testNoUser(): void {
-		$group = $this->createMock(IGroup::class);
-		$this->groupManager->method('get')
-			->with('myGroup')
-			->willReturn($group);
-
-		$this->userManager->method('get')
-			->with('myUser')
-			->willReturn(null);
-
-		$this->output->expects($this->once())
-			->method('writeln')
-			->with('<error>user not found</error>');
-
-		$this->invokePrivate($this->command, 'execute', [$this->input, $this->output]);
-	}
-
-	public function testAdd(): void {
-		$group = $this->createMock(IGroup::class);
-		$this->groupManager->method('get')
-			->with('myGroup')
-			->willReturn($group);
-
-		$user = $this->createMock(IUser::class);
-		$this->userManager->method('get')
-			->with('myUser')
-			->willReturn($user);
-
-		$group->expects($this->once())
-			->method('addUser')
-			->with($user);
-
-		$this->invokePrivate($this->command, 'execute', [$this->input, $this->output]);
-	}
+    /** @var IGroupManager|\PHPUnit\Framework\MockObject\MockObject */
+    private $groupManager;
+
+    /** @var IUserManager|\PHPUnit\Framework\MockObject\MockObject */
+    private $userManager;
+
+    /** @var AddUser */
+    private $command;
+
+    /** @var InputInterface|\PHPUnit\Framework\MockObject\MockObject */
+    private $input;
+
+    /** @var OutputInterface|\PHPUnit\Framework\MockObject\MockObject */
+    private $output;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->groupManager = $this->createMock(IGroupManager::class);
+        $this->userManager = $this->createMock(IUserManager::class);
+        $this->command = new AddUser($this->userManager, $this->groupManager);
+
+        $this->input = $this->createMock(InputInterface::class);
+        $this->input->method('getArgument')
+            ->willReturnCallback(function ($arg) {
+                if ($arg === 'group') {
+                    return 'myGroup';
+                } elseif ($arg === 'user') {
+                    return 'myUser';
+                }
+                throw new \Exception();
+            });
+        $this->output = $this->createMock(OutputInterface::class);
+    }
+
+    public function testNoGroup(): void {
+        $this->groupManager->method('get')
+            ->with('myGroup')
+            ->willReturn(null);
+
+        $this->output->expects($this->once())
+            ->method('writeln')
+            ->with('<error>group not found</error>');
+
+        $this->invokePrivate($this->command, 'execute', [$this->input, $this->output]);
+    }
+
+    public function testNoUser(): void {
+        $group = $this->createMock(IGroup::class);
+        $this->groupManager->method('get')
+            ->with('myGroup')
+            ->willReturn($group);
+
+        $this->userManager->method('get')
+            ->with('myUser')
+            ->willReturn(null);
+
+        $this->output->expects($this->once())
+            ->method('writeln')
+            ->with('<error>user not found</error>');
+
+        $this->invokePrivate($this->command, 'execute', [$this->input, $this->output]);
+    }
+
+    public function testAdd(): void {
+        $group = $this->createMock(IGroup::class);
+        $this->groupManager->method('get')
+            ->with('myGroup')
+            ->willReturn($group);
+
+        $user = $this->createMock(IUser::class);
+        $this->userManager->method('get')
+            ->with('myUser')
+            ->willReturn($user);
+
+        $group->expects($this->once())
+            ->method('addUser')
+            ->with($user);
+
+        $this->invokePrivate($this->command, 'execute', [$this->input, $this->output]);
+    }
 }

tests/Core/Command/Group/RemoveUserTest.php

Indentation +81 added lines, -81 removed lines

@@ -16,85 +16,85 @@
 use Test\TestCase;
 
 class RemoveUserTest extends TestCase {
-	/** @var IGroupManager|\PHPUnit\Framework\MockObject\MockObject */
-	private $groupManager;
-
-	/** @var IUserManager|\PHPUnit\Framework\MockObject\MockObject */
-	private $userManager;
-
-	/** @var RemoveUser */
-	private $command;
-
-	/** @var InputInterface|\PHPUnit\Framework\MockObject\MockObject */
-	private $input;
-
-	/** @var OutputInterface|\PHPUnit\Framework\MockObject\MockObject */
-	private $output;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->groupManager = $this->createMock(IGroupManager::class);
-		$this->userManager = $this->createMock(IUserManager::class);
-		$this->command = new RemoveUser($this->userManager, $this->groupManager);
-
-		$this->input = $this->createMock(InputInterface::class);
-		$this->input->method('getArgument')
-			->willReturnCallback(function ($arg) {
-				if ($arg === 'group') {
-					return 'myGroup';
-				} elseif ($arg === 'user') {
-					return 'myUser';
-				}
-				throw new \Exception();
-			});
-		$this->output = $this->createMock(OutputInterface::class);
-	}
-
-	public function testNoGroup(): void {
-		$this->groupManager->method('get')
-			->with('myGroup')
-			->willReturn(null);
-
-		$this->output->expects($this->once())
-			->method('writeln')
-			->with('<error>group not found</error>');
-
-		$this->invokePrivate($this->command, 'execute', [$this->input, $this->output]);
-	}
-
-	public function testNoUser(): void {
-		$group = $this->createMock(IGroup::class);
-		$this->groupManager->method('get')
-			->with('myGroup')
-			->willReturn($group);
-
-		$this->userManager->method('get')
-			->with('myUser')
-			->willReturn(null);
-
-		$this->output->expects($this->once())
-			->method('writeln')
-			->with('<error>user not found</error>');
-
-		$this->invokePrivate($this->command, 'execute', [$this->input, $this->output]);
-	}
-
-	public function testAdd(): void {
-		$group = $this->createMock(IGroup::class);
-		$this->groupManager->method('get')
-			->with('myGroup')
-			->willReturn($group);
-
-		$user = $this->createMock(IUser::class);
-		$this->userManager->method('get')
-			->with('myUser')
-			->willReturn($user);
-
-		$group->expects($this->once())
-			->method('removeUser')
-			->with($user);
-
-		$this->invokePrivate($this->command, 'execute', [$this->input, $this->output]);
-	}
+    /** @var IGroupManager|\PHPUnit\Framework\MockObject\MockObject */
+    private $groupManager;
+
+    /** @var IUserManager|\PHPUnit\Framework\MockObject\MockObject */
+    private $userManager;
+
+    /** @var RemoveUser */
+    private $command;
+
+    /** @var InputInterface|\PHPUnit\Framework\MockObject\MockObject */
+    private $input;
+
+    /** @var OutputInterface|\PHPUnit\Framework\MockObject\MockObject */
+    private $output;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->groupManager = $this->createMock(IGroupManager::class);
+        $this->userManager = $this->createMock(IUserManager::class);
+        $this->command = new RemoveUser($this->userManager, $this->groupManager);
+
+        $this->input = $this->createMock(InputInterface::class);
+        $this->input->method('getArgument')
+            ->willReturnCallback(function ($arg) {
+                if ($arg === 'group') {
+                    return 'myGroup';
+                } elseif ($arg === 'user') {
+                    return 'myUser';
+                }
+                throw new \Exception();
+            });
+        $this->output = $this->createMock(OutputInterface::class);
+    }
+
+    public function testNoGroup(): void {
+        $this->groupManager->method('get')
+            ->with('myGroup')
+            ->willReturn(null);
+
+        $this->output->expects($this->once())
+            ->method('writeln')
+            ->with('<error>group not found</error>');
+
+        $this->invokePrivate($this->command, 'execute', [$this->input, $this->output]);
+    }
+
+    public function testNoUser(): void {
+        $group = $this->createMock(IGroup::class);
+        $this->groupManager->method('get')
+            ->with('myGroup')
+            ->willReturn($group);
+
+        $this->userManager->method('get')
+            ->with('myUser')
+            ->willReturn(null);
+
+        $this->output->expects($this->once())
+            ->method('writeln')
+            ->with('<error>user not found</error>');
+
+        $this->invokePrivate($this->command, 'execute', [$this->input, $this->output]);
+    }
+
+    public function testAdd(): void {
+        $group = $this->createMock(IGroup::class);
+        $this->groupManager->method('get')
+            ->with('myGroup')
+            ->willReturn($group);
+
+        $user = $this->createMock(IUser::class);
+        $this->userManager->method('get')
+            ->with('myUser')
+            ->willReturn($user);
+
+        $group->expects($this->once())
+            ->method('removeUser')
+            ->with($user);
+
+        $this->invokePrivate($this->command, 'execute', [$this->input, $this->output]);
+    }
 }