TestWithSpreadsheet::readExport() - Code Metrics - Inspection of "Refactor existing exports #7908" - Ecodev/my-ichtus - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Push — master ( d2dc84...8e498d )

by Adrien

created 2021-02-15 05:44 UTC

TestWithSpreadsheet::readExport() A

↳ Parent: TestWithSpreadsheet

Complexity

Conditions	1
Paths	1

Size

Total Lines	11
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	5
dl	0
loc	11
rs	10
c	1
b	0
f	0
cc	1
nc	1
nop	2

<?php

declare(strict_types=1);

namespace ApplicationTest\Traits;

use PhpOffice\PhpSpreadsheet\Reader\Xlsx;
use PhpOffice\PhpSpreadsheet\Spreadsheet;
use ZipArchive;

trait TestWithSpreadsheet
{
    abstract public static function assertTrue($condition, string $message = ''): void;

    abstract public static function assertStringStartsWith(string $prefix, string $string, string $message = ''): void;

    abstract public static function assertFileExists(string $filename, string $message = ''): void;

    private function readExport(string $hostname, string $url): Spreadsheet
    {
        $baseUrl = 'https://' . $hostname . '/data/export/';

        self::assertStringStartsWith($baseUrl, $url, 'must be absolute URL');

        // Make sure the XLSX file was generated on disk
        $path = 'htdocs/data/export/' . str_replace($baseUrl, '', $url);
        self::assertFileExists($path, 'exported file must exist on disk');

        return $this->readSpreadsheet($path);
    }

    private function readSpreadsheet(string $filename): Spreadsheet
    {
        // Assert that it is a valid ZIP file to prevent PhpSpreadsheet from hanging
        $zip = new ZipArchive();
        $res = $zip->open($filename, ZipArchive::CHECKCONS);
        self::assertTrue($res, 'exported Excel should be a valid ZIP file');
        $zip->close();

        // Re-read it
        $reader = new Xlsx();
        $spreadsheet = $reader->load($filename);
        unlink($filename);

        return $spreadsheet;
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace ApplicationTest\Traits;
6
7			use PhpOffice\PhpSpreadsheet\Reader\Xlsx;
8			use PhpOffice\PhpSpreadsheet\Spreadsheet;
9			use ZipArchive;
10
11			trait TestWithSpreadsheet
12			{
13			abstract public static function assertTrue($condition, string $message = ''): void;
14
15			abstract public static function assertStringStartsWith(string $prefix, string $string, string $message = ''): void;
16
17			abstract public static function assertFileExists(string $filename, string $message = ''): void;
18
19			private function readExport(string $hostname, string $url): Spreadsheet
20			{
21			$baseUrl = 'https://' . $hostname . '/data/export/';
22
23			self::assertStringStartsWith($baseUrl, $url, 'must be absolute URL');
24
25			// Make sure the XLSX file was generated on disk
26			$path = 'htdocs/data/export/' . str_replace($baseUrl, '', $url);
27			self::assertFileExists($path, 'exported file must exist on disk');
28
29			return $this->readSpreadsheet($path);
30			}
31
32			private function readSpreadsheet(string $filename): Spreadsheet
33			{
34			// Assert that it is a valid ZIP file to prevent PhpSpreadsheet from hanging
35			$zip = new ZipArchive();
36			$res = $zip->open($filename, ZipArchive::CHECKCONS);
37			self::assertTrue($res, 'exported Excel should be a valid ZIP file');
38			$zip->close();
39
40			// Re-read it
41			$reader = new Xlsx();
42			$spreadsheet = $reader->load($filename);
43			unlink($filename);
44
45			return $spreadsheet;
46			}
47			}
48

Ecodev / my-ichtus

Push — master ( d2dc84...8e498d )

TestWithSpreadsheet::readExport() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like