mirror of
https://github.com/PHPOffice/PhpSpreadsheet.git
synced 2026-09-01 04:59:14 +00:00
oleibman
e637713de9
When the Dynamic Array PR #3962 was introduced, it left the default as Return Array as Value. At some point, the default should be changed to Return Array as Array. This would, of course, be a breaking change, one which will not be part of Release 4. However, it will possibly be part of Release 5. Rather than relying on the default setting, this PR explicitly sets Return Array as Value when tests require that setting. This will make it easier to identify potential breaks when the default is changed. The entire test suite will now succeed with either setting as default. In making these changes, a few minor problems were discovered with how Array as Array is handled. These are fixed with this PR.
52 lines
2.0 KiB
PHP
52 lines
2.0 KiB
PHP
<?php
|
|
|
|
declare(strict_types=1);
|
|
|
|
namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Statistical;
|
|
|
|
use PhpOffice\PhpSpreadsheet\Calculation\Exception as CalcException;
|
|
|
|
class CountBlankTest extends AllSetupTeardown
|
|
{
|
|
#[\PHPUnit\Framework\Attributes\DataProvider('providerCOUNTBLANK')]
|
|
public function testCOUNTBLANK(mixed $expectedResult, mixed ...$args): void
|
|
{
|
|
$this->runTestCaseNoBracket('COUNTBLANK', $expectedResult, ...$args);
|
|
}
|
|
|
|
public static function providerCOUNTBLANK(): array
|
|
{
|
|
return require 'tests/data/Calculation/Statistical/COUNTBLANK.php';
|
|
}
|
|
|
|
public function testOutliers(): void
|
|
{
|
|
$sheet = $this->getSheet();
|
|
$sheet->getCell('C1')->setValue(1);
|
|
$sheet->getCell('C2')->setValue(2);
|
|
$sheet->getCell('C4')->setValue(4);
|
|
$sheet->getCell('A1')->setValue('=COUNTBLANK(5)');
|
|
|
|
try {
|
|
$sheet->getCell('A1')->getCalculatedValue();
|
|
self::fail('Should receive exception for non-array arg');
|
|
} catch (CalcException $e) {
|
|
self::assertStringContainsString('Must specify range of cells', $e->getMessage());
|
|
}
|
|
$sheet->getCell('A2')->setValue('=COUNTBLANK({1;2;4})');
|
|
|
|
try {
|
|
$sheet->getCell('A1')->getCalculatedValue();
|
|
self::fail('Should receive exception for inline array arg');
|
|
} catch (CalcException $e) {
|
|
self::assertStringContainsString('Must specify range of cells', $e->getMessage());
|
|
}
|
|
$sheet->getCell('A3')->setValue('=COUNTBLANK(C1:C1)');
|
|
self::assertSame(0, $sheet->getCell('A3')->getCalculatedValue(), 'arg is single non-blank cell');
|
|
$sheet->getCell('A4')->setValue('=COUNTBLANK(D2:D2)');
|
|
self::assertSame(1, $sheet->getCell('A4')->getCalculatedValue(), 'arg is single null cell');
|
|
$sheet->getCell('A5')->setValue('=COUNTBLANK(D3:D4)');
|
|
self::assertSame(2, $sheet->getCell('A5')->getCalculatedValue(), 'arg is two cells both null');
|
|
}
|
|
}
|