site\tests\unit\models\FakeUserBehavior

Complexity

Total Complexity

1

Size/Duplication

Total Lines	3
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	calculateScoresOfLastMonth()	0	1	1

<?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',
			'email_threshold' => 5,
			'partner_email1' => 'partner1@example.com',
			'partner_email2' => 'partner2@example.com',
			'partner_email3' => 'partner3@example.com',
			'expose_graph' => true,
		];

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

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 \.

			->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));

Are you sure the usage of $this->assertEquals($form->timezone, $user->timezone) targeting PHPUnit\Framework\Assert::assertEquals() seems to always return null.

Accessing timezone on the interface PHPUnit\Framework\MockObject\MockObject suggest that you code against a concrete implementation. How about adding an instanceof check?

    expect('email_threshold should be equal', $this->assertEquals($form->email_threshold, $user->email_threshold));

Are you sure the usage of $this->assertEquals($form->email_threshold, $user->email_threshold) targeting PHPUnit\Framework\Assert::assertEquals() seems to always return null.

Accessing email_threshold on the interface PHPUnit\Framework\MockObject\MockObject suggest that you code against a concrete implementation. How about adding an instanceof check?

    expect('partner_email1 should be equal', $this->assertEquals($form->partner_email1, $user->partner_email1));

Are you sure the usage of $this->assertEquals($form->partner_email1, $user->partner_email1) targeting PHPUnit\Framework\Assert::assertEquals() seems to always return null.

Accessing partner_email1 on the interface PHPUnit\Framework\MockObject\MockObject suggest that you code against a concrete implementation. How about adding an instanceof check?

    expect('partner_email2 should be equal', $this->assertEquals($form->partner_email2, $user->partner_email2));

Accessing partner_email2 on the interface PHPUnit\Framework\MockObject\MockObject suggest that you code against a concrete implementation. How about adding an instanceof check?

Are you sure the usage of $this->assertEquals($form->partner_email2, $user->partner_email2) targeting PHPUnit\Framework\Assert::assertEquals() seems to always return null.

    expect('partner_email3 should be equal', $this->assertEquals($form->partner_email3, $user->partner_email3));

Accessing partner_email3 on the interface PHPUnit\Framework\MockObject\MockObject suggest that you code against a concrete implementation. How about adding an instanceof check?

Are you sure the usage of $this->assertEquals($form->partner_email3, $user->partner_email3) targeting PHPUnit\Framework\Assert::assertEquals() seems to always return null.

    expect('expose_graph should be equal', $this->assertEquals($form->expose_graph, $user->expose_graph));

Are you sure the usage of $this->assertEquals($form->expose_graph, $user->expose_graph) targeting PHPUnit\Framework\Assert::assertEquals() seems to always return null.

Accessing expose_graph on the interface PHPUnit\Framework\MockObject\MockObject suggest that you code against a concrete implementation. How about adding an instanceof check?

    expect('send_email should be true', $this->assertTrue($form->send_email));

Are you sure the usage of $this->assertTrue($form->send_email) targeting PHPUnit\Framework\Assert::assertTrue() seems to always return null.

    $user->email_threshold = null;
    $form->loadUser();
    expect('send_email should be false', $this->assertFalse($form->send_email));

Are you sure the usage of $this->assertFalse($form->send_email) targeting PHPUnit\Framework\Assert::assertFalse() seems to always return null.

    $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));

Are you sure the usage of $this->assertFalse($form->send_email) targeting PHPUnit\Framework\Assert::assertFalse() seems to always return null.

  }

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

    Yii::$container
      ->set(\common\components\Graph::class, function () { return $this->graph; });
    Yii::$container
      ->set(\common\interfaces\UserBehaviorInterface::class, function () { return new FakeUserBehavior(); });

    $form->attributes = $this->values;
    expect('saveProfile should return the user', $this->assertEquals($user, $form->saveProfile()));

Are you sure the usage of $this->assertEquals($user, $form->saveProfile()) targeting PHPUnit\Framework\Assert::assertEquals() seems to always return null.

		expect('saveProfile should set the user\'s attributes to be the form values', $this->assertEquals($this->values, $user->attributes));

Are you sure the usage of $this->assertEquals($this->values, $user->attributes) targeting PHPUnit\Framework\Assert::assertEquals() seems to always return null.

Accessing attributes on the interface PHPUnit\Framework\MockObject\MockObject suggest that you code against a concrete implementation. How about adding an instanceof check?

    $form->send_email = 'not_a_boolean';
		$ret = $form->saveProfile();
    expect('saveProfile should return the user with the partner-related settings equal to null', $this->assertNull($ret));

Are you sure the usage of $this->assertNull($ret) targeting PHPUnit\Framework\Assert::assertNull() seems to always return null.

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

The assignment to $ret is dead and can be removed.

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

Are you sure the usage of $this->assertEquals($null_vals, $user->attributes) targeting PHPUnit\Framework\Assert::assertEquals() seems to always return null.

		$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));

Are you sure the usage of $this->assertFalse($ret->expose_graph) targeting PHPUnit\Framework\Assert::assertFalse() seems to always return null.

		$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));

Are you sure the usage of $this->assertTrue($ret->expose_graph) targeting PHPUnit\Framework\Assert::assertTrue() seems to always return null.

		
    $form->send_email = 'not_a_boolean';
    expect('saveProfile should return null if the form validation fails', $this->assertNull($form->saveProfile()));

Are you sure the usage of $this->assertNull($form->saveProfile()) targeting PHPUnit\Framework\Assert::assertNull() seems to always return null.

  }

  private function getUser() {
    $user = $this->getmockbuilder('\common\models\user')
      ->disableoriginalconstructor()
      ->setmethods(['getisnewrecord', 'attributes', 'save', 'findbypasswordresettoken', 'removepasswordresettoken'])
      ->getmock();
    $user->method('attributes')->willReturn([

The method method() does not exist on PHPUnit\Framework\MockObject\MockObject.

      'timezone',
      'email_threshold',
      'partner_email1',
      'partner_email2',
      'partner_email3',
      'expose_graph',
    ]);
    $user->timezone = 'America/Los_Angeles';

Accessing timezone on the interface PHPUnit\Framework\MockObject\MockObject suggest that you code against a concrete implementation. How about adding an instanceof check?

    $user->email_threshold = 5;

Accessing email_threshold on the interface PHPUnit\Framework\MockObject\MockObject suggest that you code against a concrete implementation. How about adding an instanceof check?

    $user->partner_email1 = 'partner1@example.com';

Accessing partner_email1 on the interface PHPUnit\Framework\MockObject\MockObject suggest that you code against a concrete implementation. How about adding an instanceof check?

    $user->partner_email2 = 'partner2@example.com';

Accessing partner_email2 on the interface PHPUnit\Framework\MockObject\MockObject suggest that you code against a concrete implementation. How about adding an instanceof check?

    $user->partner_email3 = 'partner3@example.com';

Accessing partner_email3 on the interface PHPUnit\Framework\MockObject\MockObject suggest that you code against a concrete implementation. How about adding an instanceof check?

    $user->expose_graph = true;

Accessing expose_graph on the interface PHPUnit\Framework\MockObject\MockObject suggest that you code against a concrete implementation. How about adding an instanceof check?

    return $user;
  }
}

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