PartialUserFormVerifyControllerTest::testDoValidate() - Code Metrics - Inspection of "Add password protection selection to the form" - Firesphere/silverstripe-partial-userforms - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#12)

by Simon

created 2019-08-29 02:11 UTC

testDoValidate() A

↳ Parent: PartialUserFormVerifyControllerTest

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	38
rs	9.312
c	0
b	0
f	0
cc	1
nc	1
nop	0

<?php

namespace Firesphere\PartialUserforms\Tests;

use Firesphere\PartialUserforms\Controllers\PartialSubmissionController;
use Firesphere\PartialUserforms\Controllers\PartialUserFormVerifyController;
use Firesphere\PartialUserforms\Forms\PasswordForm;
use Firesphere\PartialUserforms\Models\PartialFormSubmission;
use SilverStripe\Control\NullHTTPRequest;
use SilverStripe\Control\Session;
use SilverStripe\Dev\SapphireTest;

class PartialUserFormVerifyControllerTest extends SapphireTest
{
    protected static $fixture_file = '../fixtures/partialformtest.yml';

    /**
     * @expectedException  \SilverStripe\Control\HTTPResponse_Exception
     */
    public function testInit()
    {
        $request = new NullHTTPRequest();
        $session = new Session(['hi' => 'bye']);
        $request->setSession($session);
        $controller = PartialUserFormVerifyController::create();
        $controller->setRequest($request);

        $controller->init();
    }

    public function testInitSuccess()
    {
        $partialForm = $this->objFromFixture(PartialFormSubmission::class, 'submission1');
        $request = new NullHTTPRequest();
        $session = new Session(['hi' => 'bye', PartialSubmissionController::SESSION_KEY => $partialForm->ID]);
        $request->setSession($session);
        $controller = PartialUserFormVerifyController::create();
        $controller->setRequest($request);

        $controller->init();
        $this->assertEquals(200, $controller->getResponse()->getStatusCode());
    }

    public function testDoValidate()
    {
        $request = new NullHTTPRequest();
        /** @var PartialFormSubmission $partialForm */
        $partialForm = $this->objFromFixture(PartialFormSubmission::class, 'submission1');
        $parent = $partialForm->getParent();
        $parent->PasswordProtected = true;
        $parent->write();
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
        $parent->publishRecursive();
        $partialForm->ParentID = $parent->ID;
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
        $partialForm->write();
        $session = new Session(['hi' => 'bye', PartialSubmissionController::SESSION_KEY => $partialForm->ID]);
        $request->setSession($session);
        $controller = PartialUserFormVerifyController::create();
        $controller->setRequest($request);

        $this->assertNotNull($partialForm->Password);

        $testPwd = hash_pbkdf2('SHA256', '1234567890', $partialForm->TokenSalt, 1000);

        $partialForm->Password = $testPwd;
        $partialForm->write();

        $controller->init();
        $form = new PasswordForm($controller, __FUNCTION__);
        $result = $controller->doValidate(['Password' => '1234567890'], $form);

        $this->assertEquals($partialForm->ID, $session->get('PartialFormSession'));
        $this->assertContains('/partial/', $result->getHeader('Location'));

        $controller = new PartialUserFormVerifyController();
        $controller->setRequest($request);
        $controller->init();
        $result = $controller->doValidate(['Password' => '0987654321'], $form);

        // It uses redirect back, so it's unclear what the 'back' is at this stage
        $this->assertNotContains('/partial', $result->getHeader('Location'));
    }
}


1			<?php
2
3			namespace Firesphere\PartialUserforms\Tests;
4
5			use Firesphere\PartialUserforms\Controllers\PartialSubmissionController;
6			use Firesphere\PartialUserforms\Controllers\PartialUserFormVerifyController;
7			use Firesphere\PartialUserforms\Forms\PasswordForm;
8			use Firesphere\PartialUserforms\Models\PartialFormSubmission;
9			use SilverStripe\Control\NullHTTPRequest;
10			use SilverStripe\Control\Session;
11			use SilverStripe\Dev\SapphireTest;
12
13			class PartialUserFormVerifyControllerTest extends SapphireTest
14			{
15			protected static $fixture_file = '../fixtures/partialformtest.yml';
16
17			/**
18			* @expectedException \SilverStripe\Control\HTTPResponse_Exception
19			*/
20			public function testInit()
21			{
22			$request = new NullHTTPRequest();
23			$session = new Session(['hi' => 'bye']);
24			$request->setSession($session);
25			$controller = PartialUserFormVerifyController::create();
26			$controller->setRequest($request);
27
28			$controller->init();
29			}
30
31			public function testInitSuccess()
32			{
33			$partialForm = $this->objFromFixture(PartialFormSubmission::class, 'submission1');
34			$request = new NullHTTPRequest();
35			$session = new Session(['hi' => 'bye', PartialSubmissionController::SESSION_KEY => $partialForm->ID]);
36			$request->setSession($session);
37			$controller = PartialUserFormVerifyController::create();
38			$controller->setRequest($request);
39
40			$controller->init();
41			$this->assertEquals(200, $controller->getResponse()->getStatusCode());
42			}
43
44			public function testDoValidate()
45			{
46			$request = new NullHTTPRequest();
47			/** @var PartialFormSubmission $partialForm */
48			$partialForm = $this->objFromFixture(PartialFormSubmission::class, 'submission1');
49			$parent = $partialForm->getParent();
50			$parent->PasswordProtected = true;
51			$parent->write();
			0 ignored issues – show Bug introduced 2019-08-28 04:06 UTC by Report Bug Copy Issue Report The method `write` does only exist in `SilverStripe\ORM\DataObject`, but not in `SilverStripe\UserForms\Model\UserDefinedForm`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
52			$parent->publishRecursive();
53			$partialForm->ParentID = $parent->ID;
			0 ignored issues – show Documentation introduced 2019-08-28 04:06 UTC by Report Bug Copy Issue Report The property `ParentID` does not exist on `object<Firesphere\Partia...\PartialFormSubmission>`. Since you implemented `__set`, maybe consider adding a @property annotation. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
54			$partialForm->write();
55			$session = new Session(['hi' => 'bye', PartialSubmissionController::SESSION_KEY => $partialForm->ID]);
56			$request->setSession($session);
57			$controller = PartialUserFormVerifyController::create();
58			$controller->setRequest($request);
59
60			$this->assertNotNull($partialForm->Password);
61
62			$testPwd = hash_pbkdf2('SHA256', '1234567890', $partialForm->TokenSalt, 1000);
63
64			$partialForm->Password = $testPwd;
65			$partialForm->write();
66
67			$controller->init();
68			$form = new PasswordForm($controller, __FUNCTION__);
69			$result = $controller->doValidate(['Password' => '1234567890'], $form);
70
71			$this->assertEquals($partialForm->ID, $session->get('PartialFormSession'));
72			$this->assertContains('/partial/', $result->getHeader('Location'));
73
74			$controller = new PartialUserFormVerifyController();
75			$controller->setRequest($request);
76			$controller->init();
77			$result = $controller->doValidate(['Password' => '0987654321'], $form);
78
79			// It uses redirect back, so it's unclear what the 'back' is at this stage
80			$this->assertNotContains('/partial', $result->getHeader('Location'));
81			}
82			}
83

Firesphere / silverstripe-partial-userforms

Pull Request — master (#12)

testDoValidate() A

Complexity

Size

Duplication

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like