LeKoala\CmsActions\Test\CmsActionsTest

Complexity

Total Complexity

25

Size/Duplication

Total Lines	212
Duplicated Lines	0 %

Importance

Changes	6
Bugs	2	Features	2

15 Methods

Rating	Name	Duplication	Size	Complexity
A	getTestForm()	0	20	4
A	testCustomDeleteTitle()	0	9	1
A	get_fixture_file()	0	6	2
A	tearDown()	0	3	1
A	testLeftAndMain()	0	43	5
A	testHasDefaultTitle()	0	4	1
A	getTestPage()	0	3	1
A	testHasSaveAndClose()	0	10	2
A	getTestModel()	0	3	1
A	testConfirmationMessage()	0	5	1
A	getMemberForm()	0	15	1
A	getAdminMember()	0	3	1
A	setUp()	0	5	1
A	testGridFieldAction()	0	17	1
A	getExtraDataObjects()	0	7	2

<?php

namespace LeKoala\CmsActions\Test;

use SilverStripe\Forms\Form;
use SilverStripe\Security\Member;
use LeKoala\CmsActions\CustomLink;
use SilverStripe\Dev\SapphireTest;
use SilverStripe\Admin\LeftAndMain;

The type SilverStripe\Admin\LeftAndMain was not found. Maybe you did not declare it correctly or list all dependencies?

use SilverStripe\Control\Controller;
use SilverStripe\Versioned\Versioned;

The type SilverStripe\Versioned\Versioned was not found. Maybe you did not declare it correctly or list all dependencies?

use SilverStripe\Forms\CompositeField;
use SilverStripe\Forms\GridField\GridField;
use SilverStripe\Forms\GridField\GridFieldDetailForm;
use SilverStripe\Versioned\VersionedGridFieldItemRequest;

The type SilverStripe\Versioned\VersionedGridFieldItemRequest was not found. Maybe you did not declare it correctly or list all dependencies?

use SilverStripe\Forms\GridField\GridFieldDetailForm_ItemRequest;
use SilverStripe\CMS\Model\SiteTree;

The type SilverStripe\CMS\Model\SiteTree was not found. Maybe you did not declare it correctly or list all dependencies?

/**
 * Tests for Cms Actions module
 */
class CmsActionsTest extends SapphireTest
{
    /**
     * Defines the fixture file to use for this test class
     * @var string
     */
    protected static $fixture_file = 'CmsActionsTest.yml';
    protected static $fixture_file_simple = 'CmsActionsSimpleTest.yml';

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

    public static function get_fixture_file()
    {
        if (class_exists(SiteTree::class)) {
            return self::$fixture_file;
        }
        return self::$fixture_file_simple;
    }

    public static function getExtraDataObjects()
    {
        $arr = parent::getExtraDataObjects();
        if (class_exists(SiteTree::class)) {
            $arr[] = Test_ActionsPage::class;
        }
        return $arr;
    }


    public function setUp(): void
    {
        parent::setUp();
        $controller = Controller::curr();
        $controller->config()->set('url_segment', 'test_controller');
    }

    public function tearDown(): void
    {
        parent::tearDown();
    }

    /**
     * @return Test_ActionsPage
     */
    public function getTestPage()
    {
        return $this->objFromFixture(Test_ActionsPage::class, 'demo');
    }

    /**
     * @return Test_CmsActionsModel
     */
    public function getTestModel()
    {
        return $this->objFromFixture(Test_CmsActionsModel::class, 'demo');
    }

    /**
     * @return Member
     */
    public function getAdminMember()
    {
        return $this->objFromFixture(Member::class, 'admin');
    }

    /**
     * @return Form
     */
    public function getMemberForm()
    {
        $controller = Controller::curr();
        $form = new Form($controller);

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

        $record = $this->getAdminMember();

        $list = Member::get();
        $gridField = new GridField('testGridfield', null, $list);
        $detailForm = new GridFieldDetailForm('testDetailForm');
        $GridFieldDetailForm = new GridFieldDetailForm_ItemRequest($gridField, $detailForm, $record, $controller, 'testPopup');
        $form = $GridFieldDetailForm->ItemEditForm();
        $form->loadDataFrom($record);

The method loadDataFrom() does not exist on SilverStripe\Control\HTTPResponse.

        return $form;

The expression return $form returns the type SilverStripe\Control\HTTPResponse which is incompatible with the documented return type SilverStripe\Forms\Form.

    }

    /**
     * @param Controller $controller
     * @param DataObject $record

The type LeKoala\CmsActions\Test\DataObject was not found. Maybe you did not declare it correctly or list all dependencies?

     * @return Form
     */
    public function getTestForm($controller = null, $record = null)
    {
        if (!$controller) {
            $controller = Controller::curr();
        }
        if (!$record) {
            $record = $this->getTestModel();
        }
        $list = Test_CmsActionsModel::get();
        $gridField = new GridField('testGridfield', null, $list);
        $detailForm = new GridFieldDetailForm('testDetailForm');
        if ($record->hasExtension(Versioned::class)) {
            $GridFieldDetailForm = new VersionedGridFieldItemRequest($gridField, $detailForm, $record, $controller, 'testPopup');
        } else {
            $GridFieldDetailForm = new GridFieldDetailForm_ItemRequest($gridField, $detailForm, $record, $controller, 'testPopup');
        }
        $form = $GridFieldDetailForm->ItemEditForm();
        $form->loadDataFrom($record);

        return $form;

The expression return $form returns the type SilverStripe\Control\HTTPResponse which is incompatible with the documented return type SilverStripe\Forms\Form.

    }

    public function testCustomDeleteTitle()
    {
        $form = $this->getTestForm();

        /** @var Test_CmsActionsModel $record */
        $record = $form->getRecord();

        $delete = $form->Actions()->fieldByName("action_doDelete");

Are you sure the assignment to $delete is correct as $form->Actions()->fieldByName('action_doDelete') targeting SilverStripe\Forms\FieldList::fieldByName() seems to always return null.

        $this->assertEquals($delete->Title(), $record->getDeleteButtonTitle());
    }

    public function testHasSaveAndClose()
    {
        $form = $this->getTestForm();

        $doSaveAndClose = $form->Actions()->fieldByName("action_doSaveAndClose");

Are you sure the assignment to $doSaveAndClose is correct as $form->Actions()->fieldByName('action_doSaveAndClose') targeting SilverStripe\Forms\FieldList::fieldByName() seems to always return null.

        // It can be nested in MajorActions, then we need to use dot notation
        if (!$doSaveAndClose) {

$doSaveAndClose is of type null, thus it always evaluated to false.

            $doSaveAndClose = $form->Actions()->fieldByName("MajorActions.action_doSaveAndClose");

Are you sure the assignment to $doSaveAndClose is correct as $form->Actions()->fieldByName('MajorActions.action_doSaveAndClose') targeting SilverStripe\Forms\FieldList::fieldByName() seems to always return null.

        }
        $this->assertNotEmpty($doSaveAndClose);
    }

    public function testHasDefaultTitle()
    {
        $customLink = new CustomLink('doTest');
        $this->assertEquals('Do test', $customLink->getTitle());
    }

    public function testConfirmationMessage()
    {
        $customLink = new CustomLink('doTest');
        $customLink->setConfirmation(true);
        $this->assertStringContainsString('sure', $customLink->getConfirmation());
    }

    public function testGridFieldAction()
    {
        $form = $this->getTestForm();

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

        $action = new Test_GridFieldAction;

        $record = $this->getTestModel();
        $list = Test_CmsActionsModel::get();
        $gridField = new GridField('testGridfield', null, $list);
        $actionName = 'test';
        $arguments = ['ID' => $record->ID];
        $data = [];

        $result = $action->doHandle($gridField, $actionName, $arguments, $data);

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

Are you sure the assignment to $result is correct as $action->doHandle($gridField, $actionName, $arguments, $data) targeting LeKoala\CmsActions\Test\Test_GridFieldAction::doHandle() seems to always return null.

        $this->assertEquals($actionName, $action->performedActionName);
        $this->assertEquals($arguments, $action->performedArguments);
        $this->assertEquals($data, $action->performedData);
    }

    public function testLeftAndMain()
    {
        if (!class_exists(SiteTree::class)) {
            $this->assertTrue(true); // make phpunit happy
            return;
        }
        $page = $this->getTestPage();
        $leftAndMain = LeftAndMain::create();
        $form = $this->getTestForm($leftAndMain, $page);

        // otherwise getRecord complains
        $leftAndMain->record = $page;
        $result = $leftAndMain->doCustomAction(

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

            [
                'action_doCustomAction' => [
                    'testAction' => 1
                ],
                'ID' => $page->ID,
                'ClassName' => $page->ClassName
            ],
            $form
        );

        $this->assertEquals($page->testAction(), $form->getMessage());

        $list = [];
        $simpleList = [];
        foreach ($form->Actions() as $action) {
            if ($action instanceof CompositeField) {
                $arr = [];
                foreach ($action->getChildren() as $subAction) {
                    $arr[] = $subAction->getName() . ' (' . get_class($subAction) . ')';
                    $simpleList[] = $subAction->getName();
                }
                $list[] = $arr;
            } else {
                $list[] = $action->getName() . ' (' . get_class($action) . ')';
                $simpleList[] = $action->getName();
            }
        }
        $filteredSimpleList = array_unique($simpleList);
        // We should not have duplicated actions
        $this->assertEquals($filteredSimpleList, $simpleList);
    }
}