LockCommandTest::testExecute() - Code Metrics - Inspection of "NEW Add lock and unlock commands for member" - silverleague/silverstripe-console - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#56)

by Robbie

created 2018-04-30 06:32 UTC

LockCommandTest::testExecute() A

↳ Parent: LockCommandTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	11
Code Lines	7

Duplication

Lines	11
Ratio	100 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
dl	11
loc	11
rs	9.4285
c	1
b	0
f	0
cc	1
eloc	7
nc	1
nop	0

<?php

namespace SilverLeague\Console\Tests\Command\Member;

use SilverLeague\Console\Tests\Command\AbstractCommandTest;
use SilverStripe\Security\Member;

/**
 * Test the lock member command
 *
 * @package silverstripe-console
 * @author  Robbie Averill <[email protected]>
 */
class LockCommandTest extends AbstractCommandTest
{
    /**
     * Delete fixtured members after tests have run
     */
    protected function tearDown()
    {
        parent::tearDown();

        $testMember = Member::get()->filter(['Email' => '[email protected]'])->first();
        if ($testMember && $testMember->exists()) {
            $testMember->delete();
        }
    }

    protected function getTestCommand()
    {
        return 'member:lock';
    }

    public function testExecute()
    {
        $member = $this->createMember();
        $this->assertFalse($member->isLockedOut());

        $tester = $this->executeTest(['email' => '[email protected]']);
        /** @var Member $member */
        $member = Member::get()->byID($member->ID);
        $this->assertContains('Member [email protected] locked for', $tester->getDisplay());
        $this->assertTrue($member->isLockedOut());
    }

    public function testMemberNotFound()
    {
        $result = $this->executeTest(['email' => '[email protected]']);
        $this->assertContains('Member with email "[email protected]" was not found.', $result->getDisplay());
    }

    /**
     * Creates a dummy user for testing with
     *
     * @return Member
     */
    protected function createMember()
    {
        $member = Member::create();
        $member->Email = '[email protected]';
        $member->Password = 'Opensesame1';
        $member->write();
        return $member;
    }
}


1		<?php
2
3		namespace SilverLeague\Console\Tests\Command\Member;
4
5		use SilverLeague\Console\Tests\Command\AbstractCommandTest;
6		use SilverStripe\Security\Member;
7
8		/**
9		* Test the lock member command
10		*
11		* @package silverstripe-console
12		* @author Robbie Averill <[email protected]>
13		*/
14	View Code Duplication	class LockCommandTest extends AbstractCommandTest
15		{
16		/**
17		* Delete fixtured members after tests have run
18		*/
19		protected function tearDown()
20		{
21		parent::tearDown();
22
23		$testMember = Member::get()->filter(['Email' => '[email protected]'])->first();
24		if ($testMember && $testMember->exists()) {
25		$testMember->delete();
26		}
27		}
28
29		protected function getTestCommand()
30		{
31		return 'member:lock';
32		}
33
34		public function testExecute()
35		{
36		$member = $this->createMember();
37		$this->assertFalse($member->isLockedOut());
38
39		$tester = $this->executeTest(['email' => '[email protected]']);
40		/** @var Member $member */
41		$member = Member::get()->byID($member->ID);
42		$this->assertContains('Member [email protected] locked for', $tester->getDisplay());
43		$this->assertTrue($member->isLockedOut());
44		}
45
46		public function testMemberNotFound()
47		{
48		$result = $this->executeTest(['email' => '[email protected]']);
49		$this->assertContains('Member with email "[email protected]" was not found.', $result->getDisplay());
50		}
51
52		/**
53		* Creates a dummy user for testing with
54		*
55		* @return Member
56		*/
57		protected function createMember()
58		{
59		$member = Member::create();
60		$member->Email = '[email protected]';
61		$member->Password = 'Opensesame1';
62		$member->write();
63		return $member;
64		}
65		}
66

silverleague / silverstripe-console

Pull Request — master (#56)

LockCommandTest::testExecute() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like