MessageOptionsTest::testDefaultValues() - Code Metrics - Inspection of "Fixed documentation mentioning false is the value..." - acelaya/ZF2-AcMailer - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( f3c189...e92436 )

by Alejandro

created 2016-06-04 17:25 UTC

MessageOptionsTest::testDefaultValues() A

↳ Parent: MessageOptionsTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	13
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	1

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

<?php
namespace AcMailerTest\Options;

use AcMailer\Options\AttachmentsOptions;
use AcMailer\Options\BodyOptions;
use AcMailer\Options\MessageOptions;
use PHPUnit_Framework_TestCase as TestCase;

/**
 * Class MessageOptionsTest
 * @author Alejandro Celaya Alastrué
 * @link http://www.alejandrocelaya.com
 */
class MessageOptionsTest extends TestCase
{
    /**
     * @var MessageOptions
     */
    protected $messageOptions;

    public function setUp()
    {
        $this->messageOptions = new MessageOptions();
    }

    public function testDefaultValues()
    {
        $this->assertEquals('', $this->messageOptions->getFrom());
        $this->assertEquals('', $this->messageOptions->getFromName());
        $this->assertEquals('', $this->messageOptions->getReplyTo());
        $this->assertEquals('', $this->messageOptions->getReplyToName());
        $this->assertEquals([], $this->messageOptions->getTo());
        $this->assertEquals([], $this->messageOptions->getCc());
        $this->assertEquals([], $this->messageOptions->getBcc());
        $this->assertEquals('', $this->messageOptions->getSubject());
        $this->assertInstanceOf('AcMailer\Options\BodyOptions', $this->messageOptions->getBody());
        $this->assertInstanceOf('AcMailer\Options\AttachmentsOptions', $this->messageOptions->getAttachments());
    }

    public function testSetBody()
    {
        $expected = new BodyOptions();
        $this->assertSame($this->messageOptions, $this->messageOptions->setBody($expected));
        $this->assertSame($expected, $this->messageOptions->getBody());

        $this->messageOptions->setBody([]);
        $this->assertInstanceOf('AcMailer\Options\BodyOptions', $this->messageOptions->getBody());
    }

    public function testSetAttachments()
    {
        $expected = new AttachmentsOptions();
        $this->assertSame($this->messageOptions, $this->messageOptions->setAttachments($expected));
        $this->assertSame($expected, $this->messageOptions->getAttachments());

        $this->messageOptions->setAttachments([]);
        $this->assertInstanceOf('AcMailer\Options\AttachmentsOptions', $this->messageOptions->getAttachments());
    }

    /**
     * @expectedException \AcMailer\Exception\InvalidArgumentException
     */
    public function testInvalidBodyThrowsException()
    {
        $this->messageOptions->setBody(new \stdClass());
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    /**
     * @expectedException \AcMailer\Exception\InvalidArgumentException
     */
    public function testInvalidAttachmentsThrowException()
    {
        $this->messageOptions->setAttachments('foo');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }
}


1			<?php
2			namespace AcMailerTest\Options;
3
4			use AcMailer\Options\AttachmentsOptions;
5			use AcMailer\Options\BodyOptions;
6			use AcMailer\Options\MessageOptions;
7			use PHPUnit_Framework_TestCase as TestCase;
8
9			/**
10			* Class MessageOptionsTest
11			* @author Alejandro Celaya Alastrué
12			* @link http://www.alejandrocelaya.com
13			*/
14			class MessageOptionsTest extends TestCase
15			{
16			/**
17			* @var MessageOptions
18			*/
19			protected $messageOptions;
20
21			public function setUp()
22			{
23			$this->messageOptions = new MessageOptions();
24			}
25
26			public function testDefaultValues()
27			{
28			$this->assertEquals('', $this->messageOptions->getFrom());
29			$this->assertEquals('', $this->messageOptions->getFromName());
30			$this->assertEquals('', $this->messageOptions->getReplyTo());
31			$this->assertEquals('', $this->messageOptions->getReplyToName());
32			$this->assertEquals([], $this->messageOptions->getTo());
33			$this->assertEquals([], $this->messageOptions->getCc());
34			$this->assertEquals([], $this->messageOptions->getBcc());
35			$this->assertEquals('', $this->messageOptions->getSubject());
36			$this->assertInstanceOf('AcMailer\Options\BodyOptions', $this->messageOptions->getBody());
37			$this->assertInstanceOf('AcMailer\Options\AttachmentsOptions', $this->messageOptions->getAttachments());
38			}
39
40			public function testSetBody()
41			{
42			$expected = new BodyOptions();
43			$this->assertSame($this->messageOptions, $this->messageOptions->setBody($expected));
44			$this->assertSame($expected, $this->messageOptions->getBody());
45
46			$this->messageOptions->setBody([]);
47			$this->assertInstanceOf('AcMailer\Options\BodyOptions', $this->messageOptions->getBody());
48			}
49
50			public function testSetAttachments()
51			{
52			$expected = new AttachmentsOptions();
53			$this->assertSame($this->messageOptions, $this->messageOptions->setAttachments($expected));
54			$this->assertSame($expected, $this->messageOptions->getAttachments());
55
56			$this->messageOptions->setAttachments([]);
57			$this->assertInstanceOf('AcMailer\Options\AttachmentsOptions', $this->messageOptions->getAttachments());
58			}
59
60			/**
61			* @expectedException \AcMailer\Exception\InvalidArgumentException
62			*/
63			public function testInvalidBodyThrowsException()
64			{
65			$this->messageOptions->setBody(new \stdClass());
			0 ignored issues – show Documentation introduced 2015-11-30 18:16 UTC by Report Bug Copy Issue Report `new \stdClass()` is of type `object<stdClass>`, but the function expects a `object<AcMailer\Options\BodyOptions>\|array`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
66			}
67
68			/**
69			* @expectedException \AcMailer\Exception\InvalidArgumentException
70			*/
71			public function testInvalidAttachmentsThrowException()
72			{
73			$this->messageOptions->setAttachments('foo');
			0 ignored issues – show Documentation introduced 2015-11-30 18:16 UTC by Report Bug Copy Issue Report `'foo'` is of type `string`, but the function expects a `object<AcMailer\Options\AttachmentsOptions>\|array`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
74			}
75			}
76

acelaya / ZF2-AcMailer

Push — develop ( f3c189...e92436 )

MessageOptionsTest::testDefaultValues() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like