SilverLeague\Console\Tests\Command\Member\ChangePasswordCommandTest

Complexity

Total Complexity

5

Size/Duplication

Total Lines	72
Duplicated Lines	36.11 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	setUp()	0	10	1
A	getTestCommand()	0	4	1
A	testExecute()	13	13	1
A	testErrorWhenEmailNotFound()	13	13	1
A	testConfigure()	0	4	1

<?php

namespace SilverLeague\Console\Tests\Command\Member;

use SilverLeague\Console\Tests\Command\AbstractCommandTest;
use SilverStripe\Security\Member;
use Symfony\Component\Console\Tester\CommandTester;

/**
 * @coversDefaultClass \SilverLeague\Console\Command\Member\ChangePasswordCommand
 * @package silverstripe-console
 * @author  Robbie Averill <robbie@averill.co.nz>
 */
class ChangePasswordCommandTest extends AbstractCommandTest
{
    /**
     * Create a Member to play with
     */
    public function setUp()
    {
        parent::setUp();

        Member::get()->removeAll();
        $member = Member::create();
        $member->Email = 'unittest@example.com';
        $member->Password = 'notrelevant';
        $member->write();
    }

    /**
     * {@inheritDoc}
     */
    public function getTestCommand()
    {
        return 'member:change-password';
    }

    /**
     * Test that our existing Member's password can be changed
     *
     * @covers ::execute
     */
    public function testExecute()

This method seems to be duplicated in your project.

    {
        $tester = new CommandTester($this->command);
        $tester->execute(
            [
                'command'  => $this->command->getName(),
                'email'    => 'unittest@example.com',
                'password' => 'newpassword'
            ]
        );

        $this->assertContains('Password updated', $tester->getDisplay());
    }

    /**
     * Test that if an email does not match a Member, then an error is returned
     *
     * @covers ::execute
     */
    public function testErrorWhenEmailNotFound()

This method seems to be duplicated in your project.

    {
        $tester = new CommandTester($this->command);
        $tester->execute(
            [
                'command'  => $this->command->getName(),
                'email'    => 'joe.unpopular@example.com',
                'password' => 'helloworld'
            ]
        );

        $this->assertContains('Member with email "joe.unpopular@example.com" was not found', $tester->getDisplay());
    }

    /**
     * Ensure that the InputArgument for at least one of the arguments has been added
     *
     * @covers ::configure
     */
    public function testConfigure()
    {
        $this->assertTrue($this->command->getDefinition()->hasArgument('password'));
    }
}