Dynamic\Jobs\Test\Page\JobTest

Complexity

Total Complexity

9

Size/Duplication

Total Lines	142
Duplicated Lines	0 %

Importance

Changes

0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	testGetApplicationLink()	0	18	1
A	testGetCMSFields()	0	6	1
A	testGetApplyButton()	0	5	1
A	testPopulateDefaults()	0	6	1
A	testCanView()	0	6	1
A	testCanDelete()	0	12	1
A	testProvidePermissions()	0	8	1
A	testCanEdit()	0	12	1
A	testCanCreate()	0	12	1

<?php

namespace Dynamic\Jobs\Test\Page;

use Dynamic\Jobs\Page\Job;
use Dynamic\Jobs\Page\JobCollection;
use SilverStripe\Assets\File;
use SilverStripe\Core\Injector\Injector;
use SilverStripe\Dev\SapphireTest;
use SilverStripe\Forms\FieldList;
use SilverStripe\Security\Member;

/**
 * Class JobTest
 * @package Dynamic\Jobs\Tests
 */
class JobTest extends SapphireTest
{
    /**
     * @var string
     */
    protected static $fixture_file = '../fixtures.yml';

    /**
     * Tests populateDefaults()
     */
    public function testPopulateDefaults()
    {
        /** @var Job $object */
        $object = Injector::inst()->create(Job::class);
        $object->populateDefaults();
        $this->assertEquals(date('Y-m-d'), $object->PostDate);
    }

    /**
     * Tests getCMSFields()
     */
    public function testGetCMSFields()
    {
        /** @var Job $object */
        $object = $this->objFromFixture(Job::class, 'one');
        $fields = $object->getCMSFields();
        $this->assertInstanceOf(FieldList::class, $fields);
    }

    /**
     * Tests getApplyButton()
     */
    public function testGetApplyButton()
    {
        /** @var Job $object */
        $object = Injector::inst()->create(Job::class);
        $this->assertStringEndsWith('apply', $object->getApplyButton());
    }

    /**
     * Tests getApplicationLink()
     */
    public function testGetApplicationLink()
    {
        /** @var Job $object */
        $object = $this->objFromFixture(Job::class, 'one');
        /** @var JobCollection $parent */
        $parent = $this->objFromFixture(JobCollection::class, 'default');

        $object->ParentID = $parent->ID;

The property ParentID does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $object->write();

        $this->assertFalse($object->getApplicationLink());

        // TODO - fix this part
        /** @var File $file */
        $file = $this->objFromFixture(File::class, 'File');

        $parent->ApplicationID = $file->ID;

The property ApplicationID does not exist on Dynamic\Jobs\Page\JobCollection. Since you implemented __set, consider adding a @property annotation.

        $parent->write();

        // print_r($file->ID);
        // print_r($object->parent()->Application()->ID);

        // $this->assertEquals($file->URL, $object->getApplicationLink());
    }

    /**
     * Tests providePermissions()
     */
    public function testProvidePermissions()
    {
        /** @var Job $object */
        $object = Injector::inst()->create(Job::class);

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

        $perms = [

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

            'Job_EDIT' => 'Edit a Job',
            'Job_DELETE' => 'Delete a Job',
            'Job_CREATE' => 'Create a Job',
        ];
        //$this->assertEquals($perms, $object->providePermissions());
    }

    /**
     * Tests canCreate()
     */
    public function testCanCreate()
    {
        /** @var Job $object */
        $object = Injector::inst()->create(Job::class);

        $admin = $this->objFromFixture(Member::class, 'Admin');
        $manage = $this->objFromFixture(Member::class, 'Manager');
        $visitor = $this->objFromFixture(Member::class, 'Visitor');

        $this->assertTrue($object->canCreate($admin));
        $this->assertTrue($object->canCreate($manage));
        $this->assertFalse($object->canCreate($visitor));
    }

    /**
     * Tests canEdit()
     */
    public function testCanEdit()
    {
        /** @var Job $object */
        $object = Injector::inst()->create(Job::class);

        $admin = $this->objFromFixture(Member::class, 'Admin');
        $manage = $this->objFromFixture(Member::class, 'Manager');
        $visitor = $this->objFromFixture(Member::class, 'Visitor');

        $this->assertTrue($object->canEdit($admin));
        $this->assertTrue($object->canEdit($manage));
        $this->assertFalse($object->canEdit($visitor));
    }

    /**
     * Tests canDelete()
     */
    public function testCanDelete()
    {
        /** @var Job $object */
        $object = Injector::inst()->create(Job::class);

        $admin = $this->objFromFixture(Member::class, 'Admin');
        $manage = $this->objFromFixture(Member::class, 'Manager');
        $visitor = $this->objFromFixture(Member::class, 'Visitor');

        $this->assertTrue($object->canDelete($admin));
        $this->assertTrue($object->canDelete($manage));
        $this->assertFalse($object->canDelete($visitor));
    }

    /**
     * Tests canView()
     */
    public function testCanView()
    {
        /** @var Job $object */
        $object = Injector::inst()->create(Job::class);
        $manage = $this->objFromFixture(Member::class, 'Manager');
        $this->assertTrue($object->canView($manage));
    }
}