mirror of
https://github.com/PHPOffice/PhpSpreadsheet.git
synced 2026-09-02 13:41:21 +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.
53 lines
1.4 KiB
PHP
53 lines
1.4 KiB
PHP
<?php
|
|
|
|
declare(strict_types=1);
|
|
|
|
namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Information;
|
|
|
|
use PhpOffice\PhpSpreadsheet\Calculation\Calculation;
|
|
use PhpOffice\PhpSpreadsheet\Calculation\Information\ExcelError;
|
|
use PhpOffice\PhpSpreadsheet\Calculation\Information\Value;
|
|
use PHPUnit\Framework\Attributes\DataProvider;
|
|
use PHPUnit\Framework\TestCase;
|
|
|
|
class IsOddTest extends TestCase
|
|
{
|
|
public function testIsOddNoArgument(): void
|
|
{
|
|
$result = Value::isOdd();
|
|
self::assertSame(ExcelError::NAME(), $result);
|
|
}
|
|
|
|
#[DataProvider('providerIsOdd')]
|
|
public function testIsOdd(bool|string $expectedResult, mixed $value): void
|
|
{
|
|
$result = Value::isOdd($value);
|
|
self::assertEquals($expectedResult, $result);
|
|
}
|
|
|
|
public static function providerIsOdd(): array
|
|
{
|
|
return require 'tests/data/Calculation/Information/IS_ODD.php';
|
|
}
|
|
|
|
#[DataProvider('providerIsOddArray')]
|
|
public function testIsOddArray(array $expectedResult, string $values): void
|
|
{
|
|
$calculation = Calculation::getInstance();
|
|
|
|
$formula = "=ISODD({$values})";
|
|
$result = $calculation->calculateFormula($formula);
|
|
self::assertSame($expectedResult, $result);
|
|
}
|
|
|
|
public static function providerIsOddArray(): array
|
|
{
|
|
return [
|
|
'vector' => [
|
|
[[false, true, false, true, false]],
|
|
'{-2, -1, 0, 1, 2}',
|
|
],
|
|
];
|
|
}
|
|
}
|