site\tests\unit\models\ChangeEmailFormTest::testChangeEmail()

Complexity

Conditions	1
Paths	1

Size

Total Lines	41
Code Lines	32

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
  public function testChangeEmail() {

    $this->specify('changeEmail() should return true and do nothing when the email is already taken', function() {
      $user = $this->getUser();
      $user
        ->expects($this->never())
        ->method('save');
      $desired_email = 'new_email@email.com';
      $user
        ->method('findByEmail')

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

        ->willReturn(true); // true, as a stand-in for some valid user object

      $form = new ChangeEmailForm($user);
      $form->desired_email = $desired_email;
      $this->assertTrue($form->changeEmail(), 'should always return true');
      $this->assertNull($user->desired_email, 'should be null because we didn\'t set the new email');

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

      $this->assertNotEquals($user->desired_email, $desired_email);
    });

    $this->specify('changeEmail() should return true, set values, and send an email when the email is available', function() {
      $user = $this->getUser();
      $desired_email = 'new_email@email.com';
      $user
        ->method('findByEmail')
        ->willReturn(null);
      $user->email = 'email@email.com';

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

      $form = new ChangeEmailForm($user);
      $form->desired_email = $desired_email;

      expect('changeEmail should return true', $this->assertTrue($form->changeEmail()));

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

      expect('the user\'s desired email should now match what they entered in the form', $this->assertEquals($user->desired_email, $desired_email));

Accessing desired_email 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($user->desired_email, $desired_email) targeting PHPUnit\Framework\Assert::assertEquals() seems to always return null.

      $this->tester->seeEmailIsSent();

The property tester does not exist on site\tests\unit\models\ChangeEmailFormTest. Did you maybe forget to declare it?

      $emailMessage = $this->tester->grabLastSentEmail();
      expect('valid email is sent', $emailMessage)->isInstanceOf('yii\mail\MessageInterface');
      expect($emailMessage->getTo())->hasKey($desired_email);
      expect($emailMessage->getFrom())->hasKey(Yii::$app->params['supportEmail']);
      expect($emailMessage->getSubject())->equals(Yii::$app->name . ' -- your requested email change');
      expect($emailMessage->toString())->contains($user->email);
    });