From 5d309e982c32a8cfde37846cf55e8b8dab687d7f Mon Sep 17 00:00:00 2001
From: Mark Baker <mark@lange.demon.co.uk>
Date: Sun, 31 Jan 2021 15:09:56 +0100
Subject: [PATCH] Extract remaining Excel function unit tests into separate
 test classes for each function (#1817)

* Extract remaining Financial function unit tests into separate test classes for each function

This makes it easier to manage unit tests if they are individual files rather than all in a single file

It also provides a stepping stone toward making it easier to test Excel functions when Excel errors no longer return a string, but an actual Excel exception that can be handled more cleanly
---
 src/PhpSpreadsheet/Shared/OLE.php             |   3 +-
 src/PhpSpreadsheet/Worksheet/AutoFilter.php   |   4 +-
 .../Calculation/FinancialTest.php             | 500 ------------------
 .../Functions/Financial/CumIpmtTest.php       |  31 ++
 .../Functions/Financial/CumPrincTest.php      |  31 ++
 .../Functions/Financial/DbTest.php            |  31 ++
 .../Functions/Financial/DdbTest.php           |  31 ++
 .../Functions/Financial/DiscTest.php          |  31 ++
 .../Functions/Financial/DollarDeTest.php      |  31 ++
 .../Functions/Financial/DollarFrTest.php      |  31 ++
 .../Functions/Financial/EffectTest.php        |  31 ++
 .../Functions/Financial/FvScheduleTest.php    |  31 ++
 .../Functions/Financial/FvTest.php            |  31 ++
 .../Functions/Financial/IPmtTest.php          |  31 ++
 .../Functions/Financial/IntRateTest.php       |  31 ++
 .../Functions/Financial/IrrTest.php           |  31 ++
 .../Functions/Financial/IsPmtTest.php         |  31 ++
 .../Functions/Financial/MirrTest.php          |  31 ++
 .../Functions/Financial/NPerTest.php          |  31 ++
 .../Functions/Financial/NominalTest.php       |  31 ++
 .../Functions/Financial/NpvTest.php           |  31 ++
 .../Functions/Financial/PDurationTest.php     |  31 ++
 .../Functions/Financial/PriceDiscTest.php     |  31 ++
 .../Functions/Financial/PriceTest.php         |  50 ++
 .../Functions/Financial/PvTest.php            |  31 ++
 .../Functions/Financial/RateTest.php          |  31 ++
 .../Functions/Financial/RriTest.php           |  31 ++
 .../Functions/Financial/SlnTest.php           |  31 ++
 .../Functions/Financial/SydTest.php           |  31 ++
 .../Functions/Financial/XNpvTest.php          |  40 ++
 .../Functions/Financial/XirrTest.php          |  40 ++
 31 files changed, 910 insertions(+), 502 deletions(-)
 delete mode 100644 tests/PhpSpreadsheetTests/Calculation/FinancialTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CumIpmtTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CumPrincTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/DbTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/DdbTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/DiscTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/DollarDeTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/DollarFrTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/EffectTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/FvScheduleTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/FvTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/IPmtTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/IntRateTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/IrrTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/IsPmtTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/MirrTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/NPerTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/NominalTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/NpvTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/PDurationTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/PriceDiscTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/PriceTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/PvTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/RateTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/RriTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/SlnTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/SydTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/XNpvTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/XirrTest.php

diff --git a/src/PhpSpreadsheet/Shared/OLE.php b/src/PhpSpreadsheet/Shared/OLE.php
index d380995c..1c745bdb 100644
--- a/src/PhpSpreadsheet/Shared/OLE.php
+++ b/src/PhpSpreadsheet/Shared/OLE.php
@@ -227,7 +227,8 @@ class OLE
         // in OLE_ChainedBlockStream::stream_open().
         // Object is removed from self::$instances in OLE_Stream::close().
         $GLOBALS['_OLE_INSTANCES'][] = $this;
-        $instanceId = end(array_keys($GLOBALS['_OLE_INSTANCES']));
+        $keys = array_keys($GLOBALS['_OLE_INSTANCES']);
+        $instanceId = end($keys);
 
         $path = 'ole-chainedblockstream://oleInstanceId=' . $instanceId;
         if ($blockIdOrPps instanceof OLE\PPS) {
diff --git a/src/PhpSpreadsheet/Worksheet/AutoFilter.php b/src/PhpSpreadsheet/Worksheet/AutoFilter.php
index c2ded195..4c33eb37 100644
--- a/src/PhpSpreadsheet/Worksheet/AutoFilter.php
+++ b/src/PhpSpreadsheet/Worksheet/AutoFilter.php
@@ -595,7 +595,9 @@ class AutoFilter
             sort($dataValues);
         }
 
-        return array_pop(array_slice($dataValues, 0, $ruleValue));
+        $slice = array_slice($dataValues, 0, $ruleValue);
+
+        return array_pop($slice);
     }
 
     /**
diff --git a/tests/PhpSpreadsheetTests/Calculation/FinancialTest.php b/tests/PhpSpreadsheetTests/Calculation/FinancialTest.php
deleted file mode 100644
index d6135a46..00000000
--- a/tests/PhpSpreadsheetTests/Calculation/FinancialTest.php
+++ /dev/null
@@ -1,500 +0,0 @@
-<?php
-
-namespace PhpOffice\PhpSpreadsheetTests\Calculation;
-
-use PhpOffice\PhpSpreadsheet\Calculation\Financial;
-use PhpOffice\PhpSpreadsheet\Calculation\Functions;
-use PHPUnit\Framework\TestCase;
-
-class FinancialTest extends TestCase
-{
-    protected function setUp(): void
-    {
-        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
-    }
-
-    /**
-     * @dataProvider providerCUMIPMT
-     *
-     * @param mixed $expectedResult
-     */
-    public function testCUMIPMT($expectedResult, ...$args): void
-    {
-        $result = Financial::CUMIPMT(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerCUMIPMT()
-    {
-        return require 'tests/data/Calculation/Financial/CUMIPMT.php';
-    }
-
-    /**
-     * @dataProvider providerCUMPRINC
-     *
-     * @param mixed $expectedResult
-     */
-    public function testCUMPRINC($expectedResult, ...$args): void
-    {
-        $result = Financial::CUMPRINC(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerCUMPRINC()
-    {
-        return require 'tests/data/Calculation/Financial/CUMPRINC.php';
-    }
-
-    /**
-     * @dataProvider providerDB
-     *
-     * @param mixed $expectedResult
-     */
-    public function testDB($expectedResult, ...$args): void
-    {
-        $result = Financial::DB(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerDB()
-    {
-        return require 'tests/data/Calculation/Financial/DB.php';
-    }
-
-    /**
-     * @dataProvider providerDDB
-     *
-     * @param mixed $expectedResult
-     */
-    public function testDDB($expectedResult, ...$args): void
-    {
-        $result = Financial::DDB(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerDDB()
-    {
-        return require 'tests/data/Calculation/Financial/DDB.php';
-    }
-
-    /**
-     * @dataProvider providerDISC
-     *
-     * @param mixed $expectedResult
-     */
-    public function testDISC($expectedResult, ...$args): void
-    {
-        $result = Financial::DISC(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerDISC()
-    {
-        return require 'tests/data/Calculation/Financial/DISC.php';
-    }
-
-    /**
-     * @dataProvider providerDOLLARDE
-     *
-     * @param mixed $expectedResult
-     */
-    public function testDOLLARDE($expectedResult, ...$args): void
-    {
-        $result = Financial::DOLLARDE(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerDOLLARDE()
-    {
-        return require 'tests/data/Calculation/Financial/DOLLARDE.php';
-    }
-
-    /**
-     * @dataProvider providerDOLLARFR
-     *
-     * @param mixed $expectedResult
-     */
-    public function testDOLLARFR($expectedResult, ...$args): void
-    {
-        $result = Financial::DOLLARFR(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerDOLLARFR()
-    {
-        return require 'tests/data/Calculation/Financial/DOLLARFR.php';
-    }
-
-    /**
-     * @dataProvider providerEFFECT
-     *
-     * @param mixed $expectedResult
-     */
-    public function testEFFECT($expectedResult, ...$args): void
-    {
-        $result = Financial::EFFECT(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerEFFECT()
-    {
-        return require 'tests/data/Calculation/Financial/EFFECT.php';
-    }
-
-    /**
-     * @dataProvider providerFV
-     *
-     * @param mixed $expectedResult
-     */
-    public function testFV($expectedResult, ...$args): void
-    {
-        $result = Financial::FV(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerFV()
-    {
-        return require 'tests/data/Calculation/Financial/FV.php';
-    }
-
-    /**
-     * @dataProvider providerFVSCHEDULE
-     *
-     * @param mixed $expectedResult
-     */
-    public function testFVSCHEDULE($expectedResult, ...$args): void
-    {
-        $result = Financial::FVSCHEDULE(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerFVSCHEDULE()
-    {
-        return require 'tests/data/Calculation/Financial/FVSCHEDULE.php';
-    }
-
-    /**
-     * @dataProvider providerINTRATE
-     *
-     * @param mixed $expectedResult
-     */
-    public function testINTRATE($expectedResult, ...$args): void
-    {
-        $result = Financial::INTRATE(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerINTRATE()
-    {
-        return require 'tests/data/Calculation/Financial/INTRATE.php';
-    }
-
-    /**
-     * @dataProvider providerIPMT
-     *
-     * @param mixed $expectedResult
-     */
-    public function testIPMT($expectedResult, ...$args): void
-    {
-        $result = Financial::IPMT(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerIPMT()
-    {
-        return require 'tests/data/Calculation/Financial/IPMT.php';
-    }
-
-    /**
-     * @dataProvider providerIRR
-     *
-     * @param mixed $expectedResult
-     */
-    public function testIRR($expectedResult, ...$args): void
-    {
-        $result = Financial::IRR(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerIRR()
-    {
-        return require 'tests/data/Calculation/Financial/IRR.php';
-    }
-
-    /**
-     * @dataProvider providerISPMT
-     *
-     * @param mixed $expectedResult
-     */
-    public function testISPMT($expectedResult, ...$args): void
-    {
-        $result = Financial::ISPMT(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerISPMT()
-    {
-        return require 'tests/data/Calculation/Financial/ISPMT.php';
-    }
-
-    /**
-     * @dataProvider providerMIRR
-     *
-     * @param mixed $expectedResult
-     */
-    public function testMIRR($expectedResult, ...$args): void
-    {
-        $result = Financial::MIRR(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerMIRR()
-    {
-        return require 'tests/data/Calculation/Financial/MIRR.php';
-    }
-
-    /**
-     * @dataProvider providerNOMINAL
-     *
-     * @param mixed $expectedResult
-     */
-    public function testNOMINAL($expectedResult, ...$args): void
-    {
-        $result = Financial::NOMINAL(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerNOMINAL()
-    {
-        return require 'tests/data/Calculation/Financial/NOMINAL.php';
-    }
-
-    /**
-     * @dataProvider providerNPER
-     *
-     * @param mixed $expectedResult
-     */
-    public function testNPER($expectedResult, ...$args): void
-    {
-        $result = Financial::NPER(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerNPER()
-    {
-        return require 'tests/data/Calculation/Financial/NPER.php';
-    }
-
-    /**
-     * @dataProvider providerNPV
-     *
-     * @param mixed $expectedResult
-     */
-    public function testNPV($expectedResult, ...$args): void
-    {
-        $result = Financial::NPV(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerNPV()
-    {
-        return require 'tests/data/Calculation/Financial/NPV.php';
-    }
-
-    /**
-     * @dataProvider providerPRICE
-     *
-     * @param mixed $expectedResult
-     */
-    public function testPRICE($expectedResult, ...$args): void
-    {
-        $result = Financial::PRICE(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-7);
-    }
-
-    public function providerPRICE()
-    {
-        return require 'tests/data/Calculation/Financial/PRICE.php';
-    }
-
-    /**
-     * @dataProvider providerPRICE3
-     *
-     * @param mixed $expectedResult
-     */
-    public function testPRICE3($expectedResult, ...$args): void
-    {
-        // These results (PRICE function with basis codes 2 and 3)
-        // agree with published algorithm, LibreOffice, and Gnumeric.
-        // They do not agree with Excel.
-        $result = Financial::PRICE(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-7);
-    }
-
-    public function providerPRICE3()
-    {
-        return require 'tests/data/Calculation/Financial/PRICE3.php';
-    }
-
-    /**
-     * @dataProvider providerPRICEDISC
-     *
-     * @param mixed $expectedResult
-     */
-    public function testPRICEDISC($expectedResult, array $args): void
-    {
-        $result = Financial::PRICEDISC(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerPRICEDISC()
-    {
-        return require 'tests/data/Calculation/Financial/PRICEDISC.php';
-    }
-
-    /**
-     * @dataProvider providerPV
-     *
-     * @param mixed $expectedResult
-     */
-    public function testPV($expectedResult, array $args): void
-    {
-        $result = Financial::PV(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerPV()
-    {
-        return require 'tests/data/Calculation/Financial/PV.php';
-    }
-
-    /**
-     * @dataProvider providerRATE
-     *
-     * @param mixed $expectedResult
-     */
-    public function testRATE($expectedResult, ...$args): void
-    {
-        $result = Financial::RATE(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerRATE()
-    {
-        return require 'tests/data/Calculation/Financial/RATE.php';
-    }
-
-    /**
-     * @dataProvider providerXIRR
-     *
-     * @param mixed $expectedResult
-     * @param mixed $message
-     */
-    public function testXIRR($expectedResult, $message, ...$args): void
-    {
-        $result = Financial::XIRR(...$args);
-        if (is_numeric($result) && is_numeric($expectedResult)) {
-            if ($expectedResult != 0) {
-                $frac = $result / $expectedResult;
-                if ($frac > 0.999999 && $frac < 1.000001) {
-                    $result = $expectedResult;
-                }
-            }
-        }
-        self::assertEquals($expectedResult, $result, $message);
-    }
-
-    public function providerXIRR()
-    {
-        return require 'tests/data/Calculation/Financial/XIRR.php';
-    }
-
-    /**
-     * @dataProvider providerXNPV
-     *
-     * @param mixed $expectedResult
-     * @param mixed $message
-     */
-    public function testXNPV($expectedResult, $message, ...$args): void
-    {
-        $result = Financial::XNPV(...$args);
-        if (is_numeric($result) && is_numeric($expectedResult)) {
-            if ($expectedResult != 0) {
-                $frac = $result / $expectedResult;
-                if ($frac > 0.999999 && $frac < 1.000001) {
-                    $result = $expectedResult;
-                }
-            }
-        }
-        self::assertEquals($expectedResult, $result, $message);
-    }
-
-    public function providerXNPV()
-    {
-        return require 'tests/data/Calculation/Financial/XNPV.php';
-    }
-
-    /**
-     * @dataProvider providerPDURATION
-     *
-     * @param mixed $expectedResult
-     */
-    public function testPDURATION($expectedResult, array $args): void
-    {
-        $result = Financial::PDURATION(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerPDURATION()
-    {
-        return require 'tests/data/Calculation/Financial/PDURATION.php';
-    }
-
-    /**
-     * @dataProvider providerRRI
-     *
-     * @param mixed $expectedResult
-     */
-    public function testRRI($expectedResult, array $args): void
-    {
-        $result = Financial::RRI(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerRRI()
-    {
-        return require 'tests/data/Calculation/Financial/RRI.php';
-    }
-
-    /**
-     * @dataProvider providerSLN
-     *
-     * @param mixed $expectedResult
-     */
-    public function testSLN($expectedResult, array $args): void
-    {
-        $result = Financial::SLN(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerSLN()
-    {
-        return require 'tests/data/Calculation/Financial/SLN.php';
-    }
-
-    /**
-     * @dataProvider providerSYD
-     *
-     * @param mixed $expectedResult
-     */
-    public function testSYD($expectedResult, array $args): void
-    {
-        $result = Financial::SYD(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerSYD()
-    {
-        return require 'tests/data/Calculation/Financial/SYD.php';
-    }
-}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CumIpmtTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CumIpmtTest.php
new file mode 100644
index 00000000..ff82bcbc
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CumIpmtTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class CumIpmtTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerCUMIPMT
+     *
+     * @param mixed $expectedResult
+     */
+    public function testCUMIPMT($expectedResult, ...$args): void
+    {
+        $result = Financial::CUMIPMT(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerCUMIPMT()
+    {
+        return require 'tests/data/Calculation/Financial/CUMIPMT.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CumPrincTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CumPrincTest.php
new file mode 100644
index 00000000..d0c3d69d
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CumPrincTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class CumPrincTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerCUMPRINC
+     *
+     * @param mixed $expectedResult
+     */
+    public function testCUMPRINC($expectedResult, ...$args): void
+    {
+        $result = Financial::CUMPRINC(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerCUMPRINC()
+    {
+        return require 'tests/data/Calculation/Financial/CUMPRINC.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/DbTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/DbTest.php
new file mode 100644
index 00000000..6ea05d15
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/DbTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class DbTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerDB
+     *
+     * @param mixed $expectedResult
+     */
+    public function testDB($expectedResult, ...$args): void
+    {
+        $result = Financial::DB(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerDB()
+    {
+        return require 'tests/data/Calculation/Financial/DB.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/DdbTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/DdbTest.php
new file mode 100644
index 00000000..ba7ed0cc
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/DdbTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class DdbTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerDDB
+     *
+     * @param mixed $expectedResult
+     */
+    public function testDDB($expectedResult, ...$args): void
+    {
+        $result = Financial::DDB(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerDDB()
+    {
+        return require 'tests/data/Calculation/Financial/DDB.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/DiscTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/DiscTest.php
new file mode 100644
index 00000000..5f581028
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/DiscTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class DiscTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerDISC
+     *
+     * @param mixed $expectedResult
+     */
+    public function testDISC($expectedResult, ...$args): void
+    {
+        $result = Financial::DISC(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerDISC()
+    {
+        return require 'tests/data/Calculation/Financial/DISC.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/DollarDeTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/DollarDeTest.php
new file mode 100644
index 00000000..ba9116cd
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/DollarDeTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class DollarDeTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerDOLLARDE
+     *
+     * @param mixed $expectedResult
+     */
+    public function testDOLLARDE($expectedResult, ...$args): void
+    {
+        $result = Financial::DOLLARDE(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerDOLLARDE()
+    {
+        return require 'tests/data/Calculation/Financial/DOLLARDE.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/DollarFrTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/DollarFrTest.php
new file mode 100644
index 00000000..21e0d39e
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/DollarFrTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class DollarFrTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerDOLLARFR
+     *
+     * @param mixed $expectedResult
+     */
+    public function testDOLLARFR($expectedResult, ...$args): void
+    {
+        $result = Financial::DOLLARFR(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerDOLLARFR()
+    {
+        return require 'tests/data/Calculation/Financial/DOLLARFR.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/EffectTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/EffectTest.php
new file mode 100644
index 00000000..7c866ed4
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/EffectTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class EffectTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerEFFECT
+     *
+     * @param mixed $expectedResult
+     */
+    public function testEFFECT($expectedResult, ...$args): void
+    {
+        $result = Financial::EFFECT(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerEFFECT()
+    {
+        return require 'tests/data/Calculation/Financial/EFFECT.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/FvScheduleTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/FvScheduleTest.php
new file mode 100644
index 00000000..3e145325
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/FvScheduleTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class FvScheduleTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerFVSCHEDULE
+     *
+     * @param mixed $expectedResult
+     */
+    public function testFVSCHEDULE($expectedResult, ...$args): void
+    {
+        $result = Financial::FVSCHEDULE(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerFVSCHEDULE()
+    {
+        return require 'tests/data/Calculation/Financial/FVSCHEDULE.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/FvTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/FvTest.php
new file mode 100644
index 00000000..0d24fb4d
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/FvTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class FvTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerFV
+     *
+     * @param mixed $expectedResult
+     */
+    public function testFV($expectedResult, ...$args): void
+    {
+        $result = Financial::FV(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerFV()
+    {
+        return require 'tests/data/Calculation/Financial/FV.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/IPmtTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/IPmtTest.php
new file mode 100644
index 00000000..53d0c81e
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/IPmtTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class IPmtTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerIPMT
+     *
+     * @param mixed $expectedResult
+     */
+    public function testIPMT($expectedResult, ...$args): void
+    {
+        $result = Financial::IPMT(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerIPMT()
+    {
+        return require 'tests/data/Calculation/Financial/IPMT.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/IntRateTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/IntRateTest.php
new file mode 100644
index 00000000..6533e057
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/IntRateTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class IntRateTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerINTRATE
+     *
+     * @param mixed $expectedResult
+     */
+    public function testINTRATE($expectedResult, ...$args): void
+    {
+        $result = Financial::INTRATE(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerINTRATE()
+    {
+        return require 'tests/data/Calculation/Financial/INTRATE.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/IrrTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/IrrTest.php
new file mode 100644
index 00000000..eb27357b
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/IrrTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class IrrTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerIRR
+     *
+     * @param mixed $expectedResult
+     */
+    public function testIRR($expectedResult, ...$args): void
+    {
+        $result = Financial::IRR(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerIRR()
+    {
+        return require 'tests/data/Calculation/Financial/IRR.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/IsPmtTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/IsPmtTest.php
new file mode 100644
index 00000000..f41229cd
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/IsPmtTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class IsPmtTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerISPMT
+     *
+     * @param mixed $expectedResult
+     */
+    public function testISPMT($expectedResult, ...$args): void
+    {
+        $result = Financial::ISPMT(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerISPMT()
+    {
+        return require 'tests/data/Calculation/Financial/ISPMT.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/MirrTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/MirrTest.php
new file mode 100644
index 00000000..3d8208ac
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/MirrTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class MirrTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerMIRR
+     *
+     * @param mixed $expectedResult
+     */
+    public function testMIRR($expectedResult, ...$args): void
+    {
+        $result = Financial::MIRR(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerMIRR()
+    {
+        return require 'tests/data/Calculation/Financial/MIRR.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/NPerTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/NPerTest.php
new file mode 100644
index 00000000..01735a12
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/NPerTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class NPerTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerNPER
+     *
+     * @param mixed $expectedResult
+     */
+    public function testNPER($expectedResult, ...$args): void
+    {
+        $result = Financial::NPER(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerNPER()
+    {
+        return require 'tests/data/Calculation/Financial/NPER.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/NominalTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/NominalTest.php
new file mode 100644
index 00000000..2ccaae24
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/NominalTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class NominalTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerNOMINAL
+     *
+     * @param mixed $expectedResult
+     */
+    public function testNOMINAL($expectedResult, ...$args): void
+    {
+        $result = Financial::NOMINAL(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerNOMINAL()
+    {
+        return require 'tests/data/Calculation/Financial/NOMINAL.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/NpvTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/NpvTest.php
new file mode 100644
index 00000000..b49fc21e
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/NpvTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class NpvTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerNPV
+     *
+     * @param mixed $expectedResult
+     */
+    public function testNPV($expectedResult, ...$args): void
+    {
+        $result = Financial::NPV(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerNPV()
+    {
+        return require 'tests/data/Calculation/Financial/NPV.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/PDurationTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/PDurationTest.php
new file mode 100644
index 00000000..88e56379
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/PDurationTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class PDurationTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerPDURATION
+     *
+     * @param mixed $expectedResult
+     */
+    public function testPDURATION($expectedResult, array $args): void
+    {
+        $result = Financial::PDURATION(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerPDURATION()
+    {
+        return require 'tests/data/Calculation/Financial/PDURATION.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/PriceDiscTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/PriceDiscTest.php
new file mode 100644
index 00000000..503d7a4f
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/PriceDiscTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class PriceDiscTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerPRICEDISC
+     *
+     * @param mixed $expectedResult
+     */
+    public function testPRICEDISC($expectedResult, array $args): void
+    {
+        $result = Financial::PRICEDISC(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerPRICEDISC()
+    {
+        return require 'tests/data/Calculation/Financial/PRICEDISC.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/PriceTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/PriceTest.php
new file mode 100644
index 00000000..d3d5bd8d
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/PriceTest.php
@@ -0,0 +1,50 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class PriceTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerPRICE
+     *
+     * @param mixed $expectedResult
+     */
+    public function testPRICE($expectedResult, ...$args): void
+    {
+        $result = Financial::PRICE(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-7);
+    }
+
+    public function providerPRICE()
+    {
+        return require 'tests/data/Calculation/Financial/PRICE.php';
+    }
+
+    /**
+     * @dataProvider providerPRICE3
+     *
+     * @param mixed $expectedResult
+     */
+    public function testPRICE3($expectedResult, ...$args): void
+    {
+        // These results (PRICE function with basis codes 2 and 3)
+        // agree with published algorithm, LibreOffice, and Gnumeric.
+        // They do not agree with Excel.
+        $result = Financial::PRICE(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-7);
+    }
+
+    public function providerPRICE3()
+    {
+        return require 'tests/data/Calculation/Financial/PRICE3.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/PvTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/PvTest.php
new file mode 100644
index 00000000..fffb01b8
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/PvTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class PvTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerPV
+     *
+     * @param mixed $expectedResult
+     */
+    public function testPV($expectedResult, array $args): void
+    {
+        $result = Financial::PV(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerPV()
+    {
+        return require 'tests/data/Calculation/Financial/PV.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/RateTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/RateTest.php
new file mode 100644
index 00000000..d8a0cba3
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/RateTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class RateTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerRATE
+     *
+     * @param mixed $expectedResult
+     */
+    public function testRATE($expectedResult, ...$args): void
+    {
+        $result = Financial::RATE(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerRATE()
+    {
+        return require 'tests/data/Calculation/Financial/RATE.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/RriTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/RriTest.php
new file mode 100644
index 00000000..b0fe81fd
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/RriTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class RriTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerRRI
+     *
+     * @param mixed $expectedResult
+     */
+    public function testRRI($expectedResult, array $args): void
+    {
+        $result = Financial::RRI(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerRRI()
+    {
+        return require 'tests/data/Calculation/Financial/RRI.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/SlnTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/SlnTest.php
new file mode 100644
index 00000000..81fc0d57
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/SlnTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class SlnTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerSLN
+     *
+     * @param mixed $expectedResult
+     */
+    public function testSLN($expectedResult, array $args): void
+    {
+        $result = Financial::SLN(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerSLN()
+    {
+        return require 'tests/data/Calculation/Financial/SLN.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/SydTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/SydTest.php
new file mode 100644
index 00000000..c6b4c191
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/SydTest.php
@@ -0,0 +1,31 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class SydTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerSYD
+     *
+     * @param mixed $expectedResult
+     */
+    public function testSYD($expectedResult, array $args): void
+    {
+        $result = Financial::SYD(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerSYD()
+    {
+        return require 'tests/data/Calculation/Financial/SYD.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/XNpvTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/XNpvTest.php
new file mode 100644
index 00000000..02dc54f9
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/XNpvTest.php
@@ -0,0 +1,40 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class XNpvTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerXNPV
+     *
+     * @param mixed $expectedResult
+     * @param mixed $message
+     */
+    public function testXNPV($expectedResult, $message, ...$args): void
+    {
+        $result = Financial::XNPV(...$args);
+        if (is_numeric($result) && is_numeric($expectedResult)) {
+            if ($expectedResult != 0) {
+                $frac = $result / $expectedResult;
+                if ($frac > 0.999999 && $frac < 1.000001) {
+                    $result = $expectedResult;
+                }
+            }
+        }
+        self::assertEquals($expectedResult, $result, $message);
+    }
+
+    public function providerXNPV()
+    {
+        return require 'tests/data/Calculation/Financial/XNPV.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/XirrTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/XirrTest.php
new file mode 100644
index 00000000..042ef298
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/XirrTest.php
@@ -0,0 +1,40 @@
+<?php
+
+namespace PhpOffice\PhpSpreadsheetTests\Calculation\Functions\Financial;
+
+use PhpOffice\PhpSpreadsheet\Calculation\Financial;
+use PhpOffice\PhpSpreadsheet\Calculation\Functions;
+use PHPUnit\Framework\TestCase;
+
+class XirrTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerXIRR
+     *
+     * @param mixed $expectedResult
+     * @param mixed $message
+     */
+    public function testXIRR($expectedResult, $message, ...$args): void
+    {
+        $result = Financial::XIRR(...$args);
+        if (is_numeric($result) && is_numeric($expectedResult)) {
+            if ($expectedResult != 0) {
+                $frac = $result / $expectedResult;
+                if ($frac > 0.999999 && $frac < 1.000001) {
+                    $result = $expectedResult;
+                }
+            }
+        }
+        self::assertEquals($expectedResult, $result, $message);
+    }
+
+    public function providerXIRR()
+    {
+        return require 'tests/data/Calculation/Financial/XIRR.php';
+    }
+}