LeKoala\CommonExtensions\Test\ExtensionsTest

Complexity

Total Complexity

3

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Importance

Changes	2
Bugs	1	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	testIPExtension()	0	13	2
A	testHasExtensions()	0	5	1

<?php

namespace LeKoala\CommonExtensions\Test;

use SilverStripe\Dev\SapphireTest;
use LeKoala\CommonExtensions\IPExtension;
use SilverStripe\Control\Controller;

class ExtensionsTest extends SapphireTest
{
    /**
     * Defines the fixture file to use for this test class
     * @var string
     */
    protected static $fixture_file = 'Test_CommonExtensions.yml';

    protected static $extra_dataobjects = array(
        Test_CommonExtensions::class,
    );

    public function testHasExtensions()
    {
        $model = new Test_CommonExtensions();

        $this->assertTrue($model->hasExtension(IPExtension::class));
    }

    public function testIPExtension()
    {
        $controller = Controller::curr();

        $model = new Test_CommonExtensions();
        $model->write();
        if ($controller) {

$controller is of type SilverStripe\Control\Controller, thus it always evaluated to true.

            $this->assertNotEmpty($model->IP);

The property IP does not exist on LeKoala\CommonExtensions\Test\Test_CommonExtensions. Since you implemented __get, consider adding a @property annotation.

        }

        $ip = '127.0.0.1';
        $model->IP = $ip;

The property IP does not exist on LeKoala\CommonExtensions\Test\Test_CommonExtensions. Since you implemented __set, consider adding a @property annotation.

        $this->assertEquals($ip, $model->IP);
    }
}