RemoveTest::validProvider() - Code Metrics - Inspection of "Add polls:share:* CLI commands" - nextcloud/polls - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#1516)

by Daniel

created 2021-05-01 21:00 UTC

RemoveTest::validProvider() A

↳ Parent: RemoveTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	68
Code Lines	40

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
eloc	40
c	2
b	0
f	0
dl	0
loc	68
rs	9.28
cc	1
nc	1
nop	0

How to fix Long Method

<?php
/**
 * @copyright Copyright (c) 2021 Daniel Rudolf <[email protected]>
 *
 * @author Daniel Rudolf <[email protected]>
 *
 * @license GNU AGPL version 3 or any later version
 *
 *  This program is free software: you can redistribute it and/or modify
 *  it under the terms of the GNU Affero General Public License as
 *  published by the Free Software Foundation, either version 3 of the
 *  License, or (at your option) any later version.
 *
 *  This program is distributed in the hope that it will be useful,
 *  but WITHOUT ANY WARRANTY; without even the implied warranty of
 *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 *  GNU Affero General Public License for more details.
 *
 *  You should have received a copy of the GNU Affero General Public License
 *  along with this program.  If not, see <http://www.gnu.org/licenses/>.
 *
 */

namespace OCA\Polls\Tests\Integration\Command\Share;

use OCA\Polls\Command\Share\Remove;
use OCA\Polls\Db\Poll;
use OCP\AppFramework\Db\DoesNotExistException;
use PHPUnit\Framework\TestCase;
use Symfony\Component\Console\Exception\RuntimeException as ConsoleRuntimeException;
use Symfony\Component\Console\Tester\CommandTester;

class RemoveTest extends TestCase {
	use TShareCommandTest;

	public function setUp(): void {
		parent::setUp();

		$this->setUpMocks();
	}

	public function testMissingArguments(): void {
		$this->pollMapper
			->expects($this->never())
			->method('find');

		$this->expectException(ConsoleRuntimeException::class);
		$this->expectExceptionMessage('Not enough arguments (missing: "id").');

		$command = new Remove(
			$this->pollMapper,
			$this->shareMapper,
			$this->shareService,
			$this->userManager,
			$this->groupManager
		);

		$tester = new CommandTester($command);
		$tester->execute([]);
	}

	public function testPollNotFound(): void {
		$pollId = 123;

		$this->pollMapper
			->expects($this->once())
			->method('find')
			->with($pollId)
			->willReturnCallback(static function (int $id): Poll {
			->willReturnCallback(static function (/** @scrutinizer ignore-unused */ int $id): Poll {
				throw new DoesNotExistException('');
			});

		$command = new Remove(
			$this->pollMapper,
			$this->shareMapper,
			$this->shareService,
			$this->userManager,
			$this->groupManager
		);

		$tester = new CommandTester($command);
		$tester->execute(['id' => $pollId]);

		$this->assertEquals("Poll not found.\n", $tester->getDisplay());
	}

	/**
	 * @dataProvider validProvider
	 */
	public function testValid(array $input, array $pollData): void {
		$initialShares = [];
		$expectedShareCount = 0;
		$expectedShareTokens = [];
		foreach ($pollData['initialShares'] ?? [] as $type => $shares) {
			foreach ($shares as $userId) {
				$initialShares[] = $this->createShareMock($pollData['pollId'], $type, $userId);

				if (in_array($userId, $pollData['expectedShares'][$type] ?? [])) {
					$expectedShareTokens[] = $this->getShareToken($pollData['pollId'], $type, $userId);
					$expectedShareCount++;
				}
			}
		}

		$this->pollMapper
			->expects($this->once())
			->method('find')
			->with($pollData['pollId'])
			->willReturnCallback([$this, 'createPollMock']);

		$this->shareMapper
			->method('findByPoll')
     method('findByPoll')
			->with($pollData['pollId'])
			->willReturn($initialShares);

		$this->shareService
			->expects($this->exactly($expectedShareCount))
			->method('delete')
			->with($this->logicalOr(...$expectedShareTokens));

		$command = new Remove(
			$this->pollMapper,
			$this->shareMapper,
			$this->shareService,
			$this->userManager,
			$this->groupManager
		);

		$tester = new CommandTester($command);
		$tester->execute($input);

		$this->assertEquals("Poll invitations successfully revoked.\n", $tester->getDisplay());
	}

	public function validProvider(): array {
		return [
			[
				[
					'id' => 1,
				],
				[
					'pollId' => 1,
				],
			],
			[
				[
					'id' => 123,
					'--user' => ['user1', 'user2'],
					'--group' => ['group1'],
					'--email' => ['[email protected]'],
				],
				[
					'pollId' => 123,
					'initialShares' => [
						'user' => ['user1', 'user2', 'user3'],
						'group' => ['group1'],
						'email' => ['[email protected]', '[email protected]'],
					],
					'expectedShares' => [
						'user' => ['user1', 'user2'],
						'group' => ['group1'],
						'email' => ['[email protected]'],
					],
				],
			],
			[
				[
					'id' => 456,
					'--user' => ['user1', 'user2', 'user3', 'user4'],
					'--email' => ['[email protected]', '[email protected]'],
				],
				[
					'pollId' => 456,
					'initialShares' => [
						'user' => ['user2', 'user3'],
						'email' => ['[email protected]', '[email protected]'],
						'contact' => ['[email protected]'],
					],
					'expectedShares' => [
						'user' => ['user2', 'user3'],
						'email' => ['[email protected]'],
						'contact' => ['[email protected]'],
					],
				]
			],
			[
				[
					'id' => 789,
					'--group' => ['group1'],
					'--email' => ['[email protected]', '[email protected]'],
				],
				[
					'pollId' => 789,
					'initialShares' => [
						'user' => ['user1', 'user2'],
						'email' => ['[email protected]'],
						'contact' => ['[email protected]'],
						'external' => ['[email protected]'],
					],
					'expectedShares' => [
						'email' => ['[email protected]'],
						'external' => ['[email protected]'],
					],
				]
			],
		];
	}
}


nextcloud / polls

Pull Request — master (#1516)

RemoveTest::validProvider() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like