PhpOffice\PhpSpreadsheetTests\Worksheet\Issue4112Test

Complexity

Total Complexity

2

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	providerSheetNumber()	0	8	1
A	testIssue4112()	0	13	1

<?php

declare(strict_types=1);

namespace PhpOffice\PhpSpreadsheetTests\Worksheet;

use PhpOffice\PhpSpreadsheet\Spreadsheet;
use PhpOffice\PhpSpreadsheet\Worksheet\Worksheet;
use PhpOffice\PhpSpreadsheetTests\Functional\AbstractFunctional;

class Issue4112Test extends AbstractFunctional
{
    /**
     * Problem deleting all sheets then adding one.
     *
     * @dataProvider providerSheetNumber
     */
    public function testIssue4112(?int $sheetNumber): void

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

    {
        $mySpreadsheet = new Spreadsheet();
        $mySpreadsheet->removeSheetByIndex(0);
        $worksheet = new Worksheet($mySpreadsheet, 'addedsheet');
        self::assertSame(-1, $mySpreadsheet->getActiveSheetIndex());
        $mySpreadsheet->addSheet($worksheet, 0);
        self::assertSame('addedsheet', $mySpreadsheet->getActiveSheet()->getTitle());
        $row = 1;
        $col = 1;
        $worksheet->getCell([$col, $row])->setValue('id_uti');
        self::assertSame('id_uti', $worksheet->getCell([$col, $row])->getValue());
        $mySpreadsheet->disconnectWorksheets();
    }

    public static function providerSheetNumber(): array
    {
        return [
            'problem case' => [0],
            'normal case' => [null],
            'negative 1 (as if there were no sheets)' => [-1],
            'diffeent negative number' => [-4],
            'positive number' => [4],
        ];
    }
}