ApplicationTest\Action\ExcelExportActionTest

Complexity

Total Complexity

1

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	testExportTransactionLines()	0	24	1

<?php

declare(strict_types=1);

namespace ApplicationTest\Action;

use Application\Action\ExportTransactionLinesAction;
use Application\Model\TransactionLine;
use ApplicationTest\Traits\TestWithTransaction;
use PHPUnit\Framework\TestCase;

class ExcelExportActionTest extends TestCase
{
    use TestWithTransaction;

    public function testExportTransactionLines(): void
    {
        // Query to generate the Excel file
        $hostname = 'my-ichtus.lan';
        $qb = _em()->getRepository(TransactionLine::class)->createQueryBuilder('tl');
        $action = new ExportTransactionLinesAction($hostname);
        $url = $action->generate($qb->getQuery());

        $this->assertStringStartsWith('https://' . $hostname . '/export/transactionLines/', $url);

        // Try to download the Excel file
        $ch = curl_init();
        curl_setopt($ch, CURLOPT_URL, $url);

It seems like $ch can also be of type false; however, parameter $ch of curl_setopt() does only seem to accept resource, maybe add an additional type check?

        curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
        curl_setopt($ch, CURLOPT_HEADER, true);
        curl_setopt($ch, CURLOPT_SSL_VERIFYHOST, false);
        curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, false);
        $response = curl_exec($ch);

It seems like $ch can also be of type false; however, parameter $ch of curl_exec() does only seem to accept resource, maybe add an additional type check?

        $headerSize = curl_getinfo($ch, CURLINFO_HEADER_SIZE);

It seems like $ch can also be of type false; however, parameter $ch of curl_getinfo() does only seem to accept resource, maybe add an additional type check?

        $header = mb_substr($response, 0, $headerSize);

        $this->assertStringContainsStringIgnoringCase('content-disposition: attachment; filename=Ichtus_compta_ecritures', $header);
        $this->assertStringContainsStringIgnoringCase('content-type: application/vnd.openxmlformats-officedocument.spreadsheetml.sheet', $header);
        $this->assertRegExp('/content\-length: [1-9][0-9]+/', $header);
    }
}