EditProfileFormTest::getUser() - Code Metrics - CorWatts/fasterscale - Measure and Improve Code Quality continuously with Scrutinizer

EditProfileFormTest::getUser() A
last analyzed 2021-01-22 05:57 UTC

↳ Parent: EditProfileFormTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	22
Code Lines	20

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
eloc	20
c	2
b	0
f	0
dl	0
loc	22
rs	9.6
cc	1
nc	1
nop	0

<?php

namespace site\tests\unit\models;

use Yii;
use \site\models\EditProfileForm;

class EditProfileFormTest extends \Codeception\Test\Unit {
  public $graph;
  public $values = [
      'timezone' => 'America/Los_Angeles',
      'send_email' => true,
      'email_category' => 4,
      'partner_email1' => '[email protected]',
      'partner_email2' => '[email protected]',
      'partner_email3' => '[email protected]',
      'expose_graph' => true,
    ];

  public function setUp() {
    $this->graph = $this->getMockBuilder(common\components\Graph::class)

      ->setMethods(['create', 'destroy'])
      ->getMock();
    parent::setUp();
  }

  public function testLoadUser() {
    $user = $this->getUser();
    $form = new \site\models\EditProfileForm($user);
    $form->loadUser();
    expect('timezone should be equal', $this->assertEquals($form->timezone, $user->timezone));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
    expect('partner_email1 should be equal', $this->assertEquals($form->partner_email1, $user->partner_email1));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
    expect('partner_email2 should be equal', $this->assertEquals($form->partner_email2, $user->partner_email2));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
    expect('partner_email3 should be equal', $this->assertEquals($form->partner_email3, $user->partner_email3));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
    expect('expose_graph should be equal', $this->assertEquals($form->expose_graph, $user->expose_graph));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
    expect('send_email should be true', $this->assertTrue($form->send_email));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {

    $user->send_email = false;

    $form->loadUser();
    expect('send_email should be false', $this->assertFalse($form->send_email));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {

    $user->partner_email1 = null;
    $user->partner_email2 = null;
    $user->partner_email3 = null;
    $form->loadUser();
    expect('send_email should be false', $this->assertFalse($form->send_email));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
  }

  public function testSaveProfile() {
    $user = $this->getUser();
    $form = new \site\models\EditProfileForm($user);
    $form->loadUser();

    // Set up dependency injections
    Yii::$container
      ->set(\common\interfaces\TimeInterface::class, function () { return new \common\components\Time('UTC'); });
    Yii::$container
      ->set(\common\components\Graph::class, function () { return $this->graph; });
    Yii::$container
      ->set(\common\interfaces\UserBehaviorInterface::class, function () { return new FakeUserBehavior(); });

    // Actually begin testing
    $form->attributes = $this->values;
    expect('saveProfile should return the user', $this->assertEquals($user, $form->saveProfile()));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
    expect('saveProfile should set the user\'s attributes to be the form values', $this->assertEquals($this->values, $user->attributes));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {

    $form->send_email = 'not_a_boolean';
    expect('saveProfile should return the user with the partner-related settings equal to null', $this->assertNull($form->saveProfile()));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {

    $null_vals = [
      'partner_email1'  => null,
      'partner_email2'  => null,
      'partner_email3'  => null,
      'send_email'      => false,
      'email_category'  => 4,
      'timezone'        => 'America/Los_Angeles',
      'expose_graph'    => true,
    ];
    $form->send_email = false;
    $ret = $form->saveProfile();

    expect('saveProfile should return the user with the partner-related settings equal to null', $this->assertEquals($null_vals, $user->attributes));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {

    $this->graph
      ->expects($this->once())
      ->method('destroy');
    $form->expose_graph = false;
    $ret = $form->saveProfile();
    expect('if form->expose_graph is set to false, destroy the graph and set user->expose_graph to false', $this->assertFalse($ret->expose_graph));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {

    $this->graph
      ->expects($this->once())
      ->method('create');
    $form->expose_graph = true;
    $ret = $form->saveProfile();
    expect('if form->expose_graph is set to true, create the graph and set user->expose_graph to true', $this->assertTrue($ret->expose_graph));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {

    $form->send_email = 'not_a_boolean';
    expect('saveProfile should return null if the form validation fails', $this->assertNull($form->saveProfile()));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
  }

  private function getUser() {
    $user = $this->getmockbuilder('\common\models\User')
      ->disableoriginalconstructor()
      ->setmethods(['getisnewrecord', 'attributes', 'save', 'findbypasswordresettoken', 'removepasswordresettoken', 'getid'])
      ->getmock();
    $user->method('attributes')->willReturn([
      'timezone',
      'partner_email1',
      'partner_email2',
      'partner_email3',
      'send_email',
      'email_category',
      'expose_graph',
    ]);
    $user->timezone = 'America/Los_Angeles';

    $user->partner_email1 = '[email protected]';

    $user->partner_email2 = '[email protected]';

    $user->partner_email3 = '[email protected]';

    $user->send_email = true;

    $user->email_category = 4;

    $user->expose_graph = true;

    return $user;
  }
}

class FakeUserBehavior {
  public $scores = true;
  public function getCheckInBreakdown() { return $this->scores; }
}


1			<?php
2
3			namespace site\tests\unit\models;
4
5			use Yii;
6			use \site\models\EditProfileForm;
7
8			class EditProfileFormTest extends \Codeception\Test\Unit {
9			public $graph;
10			public $values = [
11			'timezone' => 'America/Los_Angeles',
12			'send_email' => true,
13			'email_category' => 4,
14			'partner_email1' => '[email protected]',
15			'partner_email2' => '[email protected]',
16			'partner_email3' => '[email protected]',
17			'expose_graph' => true,
18			];
19
20			public function setUp() {
21			$this->graph = $this->getMockBuilder(common\components\Graph::class)
			0 ignored issues – show Bug introduced 2018-12-14 02:57 UTC by Report Bug Copy Issue Report The type `site\tests\unit\models\common\components\Graph` was not found. Did you mean `common\components\Graph`? If so, make sure to prefix the type with `\`. Loading history...
22			->setMethods(['create', 'destroy'])
23			->getMock();
24			parent::setUp();
25			}
26
27			public function testLoadUser() {
28			$user = $this->getUser();
29			$form = new \site\models\EditProfileForm($user);
30			$form->loadUser();
31			expect('timezone should be equal', $this->assertEquals($form->timezone, $user->timezone));
			0 ignored issues – show Bug introduced 2018-08-15 06:29 UTC by Report Bug Copy Issue Report Accessing `timezone` on the interface `PHPUnit\Framework\MockObject\MockObject` suggest that you code against a concrete implementation. How about adding an `instanceof` check? Loading history... Bug introduced 2018-08-15 06:29 UTC by Report Bug Copy Issue Report Are you sure the usage of `$this->assertEquals($for...ezone, $user->timezone)` targeting `PHPUnit\Framework\Assert::assertEquals()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
32			expect('partner_email1 should be equal', $this->assertEquals($form->partner_email1, $user->partner_email1));
			0 ignored issues – show Bug introduced 2018-08-15 06:29 UTC by Report Bug Copy Issue Report Are you sure the usage of `$this->assertEquals($for... $user->partner_email1)` targeting `PHPUnit\Framework\Assert::assertEquals()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history... Bug introduced 2018-08-15 06:29 UTC by Report Bug Copy Issue Report Accessing `partner_email1` on the interface `PHPUnit\Framework\MockObject\MockObject` suggest that you code against a concrete implementation. How about adding an `instanceof` check? Loading history...
33			expect('partner_email2 should be equal', $this->assertEquals($form->partner_email2, $user->partner_email2));
			0 ignored issues – show Bug introduced 2018-08-15 06:29 UTC by Report Bug Copy Issue Report Accessing `partner_email2` on the interface `PHPUnit\Framework\MockObject\MockObject` suggest that you code against a concrete implementation. How about adding an `instanceof` check? Loading history... Bug introduced 2018-08-15 06:29 UTC by Report Bug Copy Issue Report Are you sure the usage of `$this->assertEquals($for... $user->partner_email2)` targeting `PHPUnit\Framework\Assert::assertEquals()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
34			expect('partner_email3 should be equal', $this->assertEquals($form->partner_email3, $user->partner_email3));
			0 ignored issues – show Bug introduced 2018-08-15 06:29 UTC by Report Bug Copy Issue Report Accessing `partner_email3` on the interface `PHPUnit\Framework\MockObject\MockObject` suggest that you code against a concrete implementation. How about adding an `instanceof` check? Loading history... Bug introduced 2018-08-15 06:29 UTC by Report Bug Copy Issue Report Are you sure the usage of `$this->assertEquals($for... $user->partner_email3)` targeting `PHPUnit\Framework\Assert::assertEquals()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
35			expect('expose_graph should be equal', $this->assertEquals($form->expose_graph, $user->expose_graph));
			0 ignored issues – show Bug introduced 2018-08-15 06:29 UTC by Report Bug Copy Issue Report Are you sure the usage of `$this->assertEquals($for...h, $user->expose_graph)` targeting `PHPUnit\Framework\Assert::assertEquals()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history... Bug introduced 2018-08-15 06:29 UTC by Report Bug Copy Issue Report Accessing `expose_graph` on the interface `PHPUnit\Framework\MockObject\MockObject` suggest that you code against a concrete implementation. How about adding an `instanceof` check? Loading history...
36			expect('send_email should be true', $this->assertTrue($form->send_email));
			0 ignored issues – show Bug introduced 2018-08-15 06:29 UTC by Report Bug Copy Issue Report Are you sure the usage of `$this->assertTrue($form->send_email)` targeting `PHPUnit\Framework\Assert::assertTrue()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
37
38			$user->send_email = false;
			0 ignored issues – show Bug introduced 2019-03-11 03:54 UTC by Report Bug Copy Issue Report Accessing `send_email` on the interface `PHPUnit\Framework\MockObject\MockObject` suggest that you code against a concrete implementation. How about adding an `instanceof` check? Loading history...
39			$form->loadUser();
40			expect('send_email should be false', $this->assertFalse($form->send_email));
			0 ignored issues – show Bug introduced 2018-08-15 06:29 UTC by Report Bug Copy Issue Report Are you sure the usage of `$this->assertFalse($form->send_email)` targeting `PHPUnit\Framework\Assert::assertFalse()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
41
42			$user->partner_email1 = null;
43			$user->partner_email2 = null;
44			$user->partner_email3 = null;
45			$form->loadUser();
46			expect('send_email should be false', $this->assertFalse($form->send_email));
			0 ignored issues – show Bug introduced 2018-08-15 06:29 UTC by Report Bug Copy Issue Report Are you sure the usage of `$this->assertFalse($form->send_email)` targeting `PHPUnit\Framework\Assert::assertFalse()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
47			}
48
49			public function testSaveProfile() {
50			$user = $this->getUser();
51			$form = new \site\models\EditProfileForm($user);
52			$form->loadUser();
53
54			// Set up dependency injections
55			Yii::$container
56			->set(\common\interfaces\TimeInterface::class, function () { return new \common\components\Time('UTC'); });
57			Yii::$container
58			->set(\common\components\Graph::class, function () { return $this->graph; });
59			Yii::$container
60			->set(\common\interfaces\UserBehaviorInterface::class, function () { return new FakeUserBehavior(); });
61
62			// Actually begin testing
63			$form->attributes = $this->values;
64			expect('saveProfile should return the user', $this->assertEquals($user, $form->saveProfile()));
			0 ignored issues – show Bug introduced 2018-12-14 02:57 UTC by Report Bug Copy Issue Report Are you sure the usage of `$this->assertEquals($user, $form->saveProfile())` targeting `PHPUnit\Framework\Assert::assertEquals()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
65			expect('saveProfile should set the user\'s attributes to be the form values', $this->assertEquals($this->values, $user->attributes));
			0 ignored issues – show Bug introduced 2018-12-14 02:57 UTC by Report Bug Copy Issue Report Are you sure the usage of `$this->assertEquals($thi...ues, $user->attributes)` targeting `PHPUnit\Framework\Assert::assertEquals()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history... Bug introduced 2018-12-14 02:57 UTC by Report Bug Copy Issue Report Accessing `attributes` on the interface `PHPUnit\Framework\MockObject\MockObject` suggest that you code against a concrete implementation. How about adding an `instanceof` check? Loading history...
66
67			$form->send_email = 'not_a_boolean';
68			expect('saveProfile should return the user with the partner-related settings equal to null', $this->assertNull($form->saveProfile()));
			0 ignored issues – show Bug introduced 2019-02-02 18:41 UTC by Report Bug Copy Issue Report Are you sure the usage of `$this->assertNull($form->saveProfile())` targeting `PHPUnit\Framework\Assert::assertNull()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
69
70			$null_vals = [
71			'partner_email1' => null,
72			'partner_email2' => null,
73			'partner_email3' => null,
74			'send_email' => false,
75			'email_category' => 4,
76			'timezone' => 'America/Los_Angeles',
77			'expose_graph' => true,
78			];
79			$form->send_email = false;
80			$ret = $form->saveProfile();
			0 ignored issues – show Unused Code introduced 2018-12-14 02:57 UTC by Report Bug Copy Issue Report The assignment to `$ret` is dead and can be removed. Loading history...
81			expect('saveProfile should return the user with the partner-related settings equal to null', $this->assertEquals($null_vals, $user->attributes));
			0 ignored issues – show Bug introduced 2018-12-14 02:57 UTC by Report Bug Copy Issue Report Are you sure the usage of `$this->assertEquals($nul...als, $user->attributes)` targeting `PHPUnit\Framework\Assert::assertEquals()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
82
83			$this->graph
84			->expects($this->once())
85			->method('destroy');
86			$form->expose_graph = false;
87			$ret = $form->saveProfile();
88			expect('if form->expose_graph is set to false, destroy the graph and set user->expose_graph to false', $this->assertFalse($ret->expose_graph));
			0 ignored issues – show Bug introduced 2018-12-14 02:57 UTC by Report Bug Copy Issue Report Are you sure the usage of `$this->assertFalse($ret->expose_graph)` targeting `PHPUnit\Framework\Assert::assertFalse()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
89
90			$this->graph
91			->expects($this->once())
92			->method('create');
93			$form->expose_graph = true;
94			$ret = $form->saveProfile();
95			expect('if form->expose_graph is set to true, create the graph and set user->expose_graph to true', $this->assertTrue($ret->expose_graph));
			0 ignored issues – show Bug introduced 2018-12-14 02:57 UTC by Report Bug Copy Issue Report Are you sure the usage of `$this->assertTrue($ret->expose_graph)` targeting `PHPUnit\Framework\Assert::assertTrue()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
96
97			$form->send_email = 'not_a_boolean';
98			expect('saveProfile should return null if the form validation fails', $this->assertNull($form->saveProfile()));
			0 ignored issues – show Bug introduced 2018-12-14 02:57 UTC by Report Bug Copy Issue Report Are you sure the usage of `$this->assertNull($form->saveProfile())` targeting `PHPUnit\Framework\Assert::assertNull()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
99			}
100
101			private function getUser() {
102			$user = $this->getmockbuilder('\common\models\User')
103			->disableoriginalconstructor()
104			->setmethods(['getisnewrecord', 'attributes', 'save', 'findbypasswordresettoken', 'removepasswordresettoken', 'getid'])
105			->getmock();
106			$user->method('attributes')->willReturn([
107			'timezone',
108			'partner_email1',
109			'partner_email2',
110			'partner_email3',
111			'send_email',
112			'email_category',
113			'expose_graph',
114			]);
115			$user->timezone = 'America/Los_Angeles';
			0 ignored issues – show Bug introduced 2018-08-15 06:29 UTC by Report Bug Copy Issue Report Accessing `timezone` on the interface `PHPUnit\Framework\MockObject\MockObject` suggest that you code against a concrete implementation. How about adding an `instanceof` check? Loading history...
116			$user->partner_email1 = '[email protected]';
			0 ignored issues – show Bug introduced 2018-08-15 06:29 UTC by Report Bug Copy Issue Report Accessing `partner_email1` on the interface `PHPUnit\Framework\MockObject\MockObject` suggest that you code against a concrete implementation. How about adding an `instanceof` check? Loading history...
117			$user->partner_email2 = '[email protected]';
			0 ignored issues – show Bug introduced 2018-08-15 06:29 UTC by Report Bug Copy Issue Report Accessing `partner_email2` on the interface `PHPUnit\Framework\MockObject\MockObject` suggest that you code against a concrete implementation. How about adding an `instanceof` check? Loading history...
118			$user->partner_email3 = '[email protected]';
			0 ignored issues – show Bug introduced 2018-08-15 06:29 UTC by Report Bug Copy Issue Report Accessing `partner_email3` on the interface `PHPUnit\Framework\MockObject\MockObject` suggest that you code against a concrete implementation. How about adding an `instanceof` check? Loading history...
119			$user->send_email = true;
			0 ignored issues – show Bug introduced 2019-02-02 18:41 UTC by Report Bug Copy Issue Report Accessing `send_email` on the interface `PHPUnit\Framework\MockObject\MockObject` suggest that you code against a concrete implementation. How about adding an `instanceof` check? Loading history...
120			$user->email_category = 4;
			0 ignored issues – show Bug introduced 2019-04-29 01:03 UTC by Report Bug Copy Issue Report Accessing `email_category` on the interface `PHPUnit\Framework\MockObject\MockObject` suggest that you code against a concrete implementation. How about adding an `instanceof` check? Loading history...
121			$user->expose_graph = true;
			0 ignored issues – show Bug introduced 2019-03-01 18:32 UTC by Report Bug Copy Issue Report Accessing `expose_graph` on the interface `PHPUnit\Framework\MockObject\MockObject` suggest that you code against a concrete implementation. How about adding an `instanceof` check? Loading history...
122			return $user;
123			}
124			}
125
126			class FakeUserBehavior {
127			public $scores = true;
128			public function getCheckInBreakdown() { return $this->scores; }
129			}
130

CorWatts / fasterscale

EditProfileFormTest::getUser() A last analyzed 2021-01-22 05:57 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

EditProfileFormTest::getUser() A
last analyzed 2021-01-22 05:57 UTC