mirror of
https://github.com/PHPOffice/PhpSpreadsheet.git
synced 2026-08-31 04:28:51 +00:00
70b4ecd4d3
They aren't quite interchangeable. Both are used in the test suite, with no indication of why one or the other. I think we'd be best off being consistent. Based on the names, I think `_calculateFormulaValue` was intended as a private, or at least internal, method, so favor `calculateFormula`. I do not intend to rename or re-categorize `_calculateFormulaValue`, just remove its usage when it isn't clearly warranted.
57 lines
1.6 KiB
PHP
57 lines
1.6 KiB
PHP
<?php
|
|
|
|
declare(strict_types=1);
|
|
|
|
namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Logical;
|
|
|
|
use PhpOffice\PhpSpreadsheet\Calculation\Calculation;
|
|
use PHPUnit\Framework\Attributes\DataProvider;
|
|
|
|
class SwitchTest extends AllSetupTeardown
|
|
{
|
|
#[DataProvider('providerSwitch')]
|
|
public function testSWITCH(mixed $expectedResult, mixed ...$args): void
|
|
{
|
|
$this->runTestCase('SWITCH', $expectedResult, ...$args);
|
|
}
|
|
|
|
public static function providerSwitch(): array
|
|
{
|
|
return require 'tests/data/Calculation/Logical/SWITCH.php';
|
|
}
|
|
|
|
#[DataProvider('providerSwitchArray')]
|
|
public function testIfsArray(array $expectedResult, int $expression, int $value1, string $result1, int $value2, string $result2, string $default): void
|
|
{
|
|
$calculation = Calculation::getInstance();
|
|
|
|
$formula = "=SWITCH($expression, $value1, {" . "$result1}, $value2, {" . "$result2}, {" . "$default})";
|
|
$result = $calculation->calculateFormula($formula);
|
|
self::assertSame($expectedResult, $result);
|
|
}
|
|
|
|
public static function providerSwitchArray(): array
|
|
{
|
|
return [
|
|
'Array return' => [
|
|
[[4, 5, 6]],
|
|
2,
|
|
1,
|
|
'1, 2, 3',
|
|
2,
|
|
'4, 5, 6',
|
|
'7, 8, 9',
|
|
],
|
|
'Array return default' => [
|
|
[[7, 8, 9]],
|
|
3,
|
|
1,
|
|
'1, 2, 3',
|
|
2,
|
|
'4, 5, 6',
|
|
'7, 8, 9',
|
|
],
|
|
];
|
|
}
|
|
}
|