Files
oleibman 057572ee90 Change Additional Statistical Tests to Use Spreadsheet Context (#3217)
* Change Additional Statistical Tests to Use Spreadsheet Context

With an earlier change, I made all but 18 Statistical tests run in spreadsheet context. This PR changes 12 of those 18. The remaining 6 usually return array results, so it is a tougher task to handle them. I will continue to think on it.

AVERAGEIF, AVERAGEIFS, and COUNTBLANK are changed to throw an Exception when a range is specified as a literal. They previously accepted array (enclosed in braces) literals, and bumbled along till they threw an error for non-array literals. Throwing an exception appears to be analogous to how Excel operates, rather than something more friendly like a VALUE error. There may be other functions which require similar treatment.

There also remains a TODO for COUNTIFS, and possibly other functions. It appears that PhpSpreadsheet counts booleans for both integer and string compares and probably shouldn't. Again, this is a problem for another day.

* Scrutinizer

Fix one problem.

* Scrutinizer Ignores Its Own Suggested Remedy

Try another approach.
2022-12-02 14:14:43 -08:00

77 lines
2.0 KiB
PHP

<?php
namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Statistical;
class CountTest extends AllSetupTeardown
{
/**
* @dataProvider providerBasicCOUNT
*
* @param mixed $expectedResult
*/
public function testBasicCOUNT($expectedResult, ...$args): void
{
$this->runTestCaseNoBracket('COUNT', $expectedResult, ...$args);
}
public function providerBasicCOUNT(): array
{
return require 'tests/data/Calculation/Statistical/BasicCOUNT.php';
}
/**
* @dataProvider providerExcelCOUNT
*
* @param mixed $expectedResult
*/
public function testExcelCOUNT($expectedResult, ...$args): void
{
if (is_array($args[0])) {
$this->runTestCaseNoBracket('COUNT', $expectedResult, ...$args);
} else {
$this->runTestCaseDirect('COUNT', $expectedResult, ...$args);
}
}
public function providerExcelCOUNT(): array
{
return require 'tests/data/Calculation/Statistical/ExcelCOUNT.php';
}
/**
* @dataProvider providerOpenOfficeCOUNT
*
* @param mixed $expectedResult
*/
public function testOpenOfficeCOUNT($expectedResult, ...$args): void
{
$this->setOpenOffice();
if (is_array($args[0])) {
$this->runTestCaseNoBracket('COUNT', $expectedResult, ...$args);
} else {
$this->runTestCaseDirect('COUNT', $expectedResult, ...$args);
}
}
public function providerOpenOfficeCOUNT(): array
{
return require 'tests/data/Calculation/Statistical/OpenOfficeCOUNT.php';
}
/**
* @dataProvider providerGnumericCOUNT
*
* @param mixed $expectedResult
*/
public function testGnumericCOUNT($expectedResult, ...$args): void
{
$this->setGnumeric();
$this->runTestCaseNoBracket('COUNT', $expectedResult, ...$args);
}
public function providerGnumericCOUNT(): array
{
return require 'tests/data/Calculation/Statistical/GnumericCOUNT.php';
}
}