App\Tests\Functional\Controller\AbstractAdminController

Complexity

Total Complexity

3

Size/Duplication

Total Lines	59
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	assertAdminListPageHasColumns()	0	22	2
A	processDeleteAction()	0	33	1

<?php

namespace App\Tests\Functional\Controller;

use Symfony\Component\DomCrawler\Crawler;

class AbstractAdminController extends AbstractController
{
    protected function assertAdminListPageHasColumns(array $columns)
    {
        $expectedColumns = $columns;
        $page = $this->getSessionClient()->getCrawler();

        $listPageColumns = $page->filter('.sonata-ba-list-field-header')->children()->each(function(Crawler $column, $i) {

The parameter $i is not used and could be removed.

            if (false == strpos($column->attr('class'), 'sonata-ba-list-field-header-batch')) {

It seems like you are loosely comparing strpos($column->attr('class'), 'sonata-ba-list-field-header-batch') of type integer to the boolean false. If you are specifically checking for 0, consider using something more explicit like === 0 instead.

                return trim($column->text());
            }
        });
        $listPageColumns = array_filter($listPageColumns);

        $notHaveColumns = array_diff($expectedColumns, $listPageColumns);
        $this->assertEquals([], $notHaveColumns, sprintf(
            'Not have columns "%s" in "%s"',
            implode(', ', $notHaveColumns),
            implode(', ', $listPageColumns)
        ));

        $extraColumns = array_diff($listPageColumns, $expectedColumns);
        $this->assertEquals([], $extraColumns, sprintf('Found extra columns "%s"', implode(', ', $extraColumns)));
    }

    protected function processDeleteAction($object)
    {
        $this->logIn();

        $objectNS = get_class($object);
        $namespaceParts = explode('\\', $objectNS);
        $entityName = array_pop($namespaceParts);

        $page = $this->request(sprintf('/admin/%s/%s/delete?tl=en', $entityName, $object->getId()), 'GET', 200);
//        file_put_contents('/tmp/test.html', $this->getSessionClient()->getResponse()->getContent());
        $form = $this->getConfirmDeleteFormObject($page);

        $this->getSessionClient()->followRedirects(true);
        $listPage = $this->getSessionClient()->submit($form);

        $this->assertContains(
            'has been deleted successfully.',
            trim($listPage->filter('.alert-success')->text())
        );

        // Tested softdeleteable and blameable
        $id = $object->getId();
        $this->getEm()->detach($object);

The method Doctrine\ORM\EntityManager::detach() has been deprecated with message: 2.7 This method is being removed from the ORM and won't have any replacement

        $object = $this->getEm()->find($objectNS, $id);
        $this->assertNull($object);

        $this->getEm()->getFilters()->disable('softdeleteable');
        $object = $this->getEm()->find($objectNS, $id);
        $this->assertNotNull($object, sprintf('SoftDeleteable filter is not active for "%s" entity', $entityName));
        $this->assertEquals('admin', $object->getDeletedBy());

        $this->getEm()->getFilters()->enable('softdeleteable');
    }
}