From 41c8a4f1b419c3812db5f9da86bb5c702d7be892 Mon Sep 17 00:00:00 2001
From: Mark Baker <mark@lange.demon.co.uk>
Date: Sat, 30 Jan 2021 18:45:31 +0100
Subject: [PATCH] Additional unit tests for previously untested financial
 functions (#1815)

* Additional unit tests for previously untested financial functions, and some additions to follow untested paths
* Start splitting Financial function tests out from the large FinancialTests class into individual test classes for each function
---
 src/PhpSpreadsheet/Calculation/Financial.php  |   6 +-
 .../Calculation/FinancialTest.php             | 128 ------------------
 .../Functions/Financial/AmorDegRcTest.php     |  31 +++++
 .../Functions/Financial/AmorLincTest.php      |  31 +++++
 .../Functions/Financial/CoupDayBsTest.php     |  31 +++++
 .../Functions/Financial/CoupDaysNcTest.php    |  31 +++++
 .../Functions/Financial/CoupDaysTest.php      |  31 +++++
 .../Functions/Financial/CoupNcdTest.php       |  31 +++++
 .../Functions/Financial/CoupNumTest.php       |  31 +++++
 .../Functions/Financial/CoupPcdTest.php       |  31 +++++
 .../Functions/Financial/PriceMatTest.php      |  31 +++++
 .../Functions/Financial/ReceivedTest.php      |  31 +++++
 .../Functions/Financial/TBillEqTest.php       |  31 +++++
 .../Functions/Financial/TBillPriceTest.php    |  31 +++++
 .../Functions/Financial/TBillYieldTest.php    |  31 +++++
 .../Functions/Financial/YieldDiscTest.php     |  31 +++++
 .../Functions/Financial/YieldMatTest.php      |  31 +++++
 .../data/Calculation/Financial/AMORDEGRC.php  |  28 ++--
 tests/data/Calculation/Financial/AMORLINC.php |  28 ++--
 tests/data/Calculation/Financial/PRICEMAT.php |  16 +++
 tests/data/Calculation/Financial/RECEIVED.php |  16 +++
 tests/data/Calculation/Financial/TBILLEQ.php  |  20 +++
 .../data/Calculation/Financial/TBILLPRICE.php |  20 +++
 .../data/Calculation/Financial/TBILLYIELD.php |  12 ++
 .../data/Calculation/Financial/YIELDDISC.php  |  12 ++
 tests/data/Calculation/Financial/YIELDMAT.php |  12 ++
 26 files changed, 604 insertions(+), 159 deletions(-)
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/AmorDegRcTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/AmorLincTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CoupDayBsTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CoupDaysNcTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CoupDaysTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CoupNcdTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CoupNumTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CoupPcdTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/PriceMatTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/ReceivedTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/TBillEqTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/TBillPriceTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/TBillYieldTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/YieldDiscTest.php
 create mode 100644 tests/PhpSpreadsheetTests/Calculation/Functions/Financial/YieldMatTest.php
 create mode 100644 tests/data/Calculation/Financial/PRICEMAT.php
 create mode 100644 tests/data/Calculation/Financial/RECEIVED.php
 create mode 100644 tests/data/Calculation/Financial/TBILLEQ.php
 create mode 100644 tests/data/Calculation/Financial/TBILLPRICE.php
 create mode 100644 tests/data/Calculation/Financial/TBILLYIELD.php
 create mode 100644 tests/data/Calculation/Financial/YIELDDISC.php
 create mode 100644 tests/data/Calculation/Financial/YIELDMAT.php

diff --git a/src/PhpSpreadsheet/Calculation/Financial.php b/src/PhpSpreadsheet/Calculation/Financial.php
index 5a908aa513..728167f426 100644
--- a/src/PhpSpreadsheet/Calculation/Financial.php
+++ b/src/PhpSpreadsheet/Calculation/Financial.php
@@ -2037,7 +2037,7 @@ public static function TBILLEQ($settlement, $maturity, $discount)
             return Functions::VALUE();
         }
 
-        if (Functions::getCompatibilityMode() == Functions::COMPATIBILITY_OPENOFFICE) {
+        if (Functions::getCompatibilityMode() === Functions::COMPATIBILITY_OPENOFFICE) {
             ++$maturity;
             $daysBetweenSettlementAndMaturity = DateTime::YEARFRAC($settlement, $maturity) * 360;
         } else {
@@ -2076,7 +2076,7 @@ public static function TBILLPRICE($settlement, $maturity, $discount)
                 return Functions::NAN();
             }
 
-            if (Functions::getCompatibilityMode() == Functions::COMPATIBILITY_OPENOFFICE) {
+            if (Functions::getCompatibilityMode() === Functions::COMPATIBILITY_OPENOFFICE) {
                 ++$maturity;
                 $daysBetweenSettlementAndMaturity = DateTime::YEARFRAC($settlement, $maturity) * 360;
                 if (!is_numeric($daysBetweenSettlementAndMaturity)) {
@@ -2087,7 +2087,7 @@ public static function TBILLPRICE($settlement, $maturity, $discount)
                 $daysBetweenSettlementAndMaturity = (DateTime::getDateValue($maturity) - DateTime::getDateValue($settlement));
             }
 
-            if ($daysBetweenSettlementAndMaturity > 360) {
+            if ($daysBetweenSettlementAndMaturity > self::daysPerYear(DateTime::YEAR($maturity), 1)) {
                 return Functions::NAN();
             }
 
diff --git a/tests/PhpSpreadsheetTests/Calculation/FinancialTest.php b/tests/PhpSpreadsheetTests/Calculation/FinancialTest.php
index e80ef35b66..d6135a466d 100644
--- a/tests/PhpSpreadsheetTests/Calculation/FinancialTest.php
+++ b/tests/PhpSpreadsheetTests/Calculation/FinancialTest.php
@@ -13,134 +13,6 @@ protected function setUp(): void
         Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
     }
 
-    /**
-     * @dataProvider providerAMORDEGRC
-     *
-     * @param mixed $expectedResult
-     */
-    public function testAMORDEGRC($expectedResult, ...$args): void
-    {
-        $result = Financial::AMORDEGRC(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerAMORDEGRC()
-    {
-        return require 'tests/data/Calculation/Financial/AMORDEGRC.php';
-    }
-
-    /**
-     * @dataProvider providerAMORLINC
-     *
-     * @param mixed $expectedResult
-     */
-    public function testAMORLINC($expectedResult, ...$args): void
-    {
-        $result = Financial::AMORLINC(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerAMORLINC()
-    {
-        return require 'tests/data/Calculation/Financial/AMORLINC.php';
-    }
-
-    /**
-     * @dataProvider providerCOUPDAYBS
-     *
-     * @param mixed $expectedResult
-     */
-    public function testCOUPDAYBS($expectedResult, ...$args): void
-    {
-        $result = Financial::COUPDAYBS(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerCOUPDAYBS()
-    {
-        return require 'tests/data/Calculation/Financial/COUPDAYBS.php';
-    }
-
-    /**
-     * @dataProvider providerCOUPDAYS
-     *
-     * @param mixed $expectedResult
-     */
-    public function testCOUPDAYS($expectedResult, ...$args): void
-    {
-        $result = Financial::COUPDAYS(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerCOUPDAYS()
-    {
-        return require 'tests/data/Calculation/Financial/COUPDAYS.php';
-    }
-
-    /**
-     * @dataProvider providerCOUPDAYSNC
-     *
-     * @param mixed $expectedResult
-     */
-    public function testCOUPDAYSNC($expectedResult, ...$args): void
-    {
-        $result = Financial::COUPDAYSNC(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerCOUPDAYSNC()
-    {
-        return require 'tests/data/Calculation/Financial/COUPDAYSNC.php';
-    }
-
-    /**
-     * @dataProvider providerCOUPNCD
-     *
-     * @param mixed $expectedResult
-     */
-    public function testCOUPNCD($expectedResult, ...$args): void
-    {
-        $result = Financial::COUPNCD(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerCOUPNCD()
-    {
-        return require 'tests/data/Calculation/Financial/COUPNCD.php';
-    }
-
-    /**
-     * @dataProvider providerCOUPNUM
-     *
-     * @param mixed $expectedResult
-     */
-    public function testCOUPNUM($expectedResult, ...$args): void
-    {
-        $result = Financial::COUPNUM(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerCOUPNUM()
-    {
-        return require 'tests/data/Calculation/Financial/COUPNUM.php';
-    }
-
-    /**
-     * @dataProvider providerCOUPPCD
-     *
-     * @param mixed $expectedResult
-     */
-    public function testCOUPPCD($expectedResult, ...$args): void
-    {
-        $result = Financial::COUPPCD(...$args);
-        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
-    }
-
-    public function providerCOUPPCD()
-    {
-        return require 'tests/data/Calculation/Financial/COUPPCD.php';
-    }
-
     /**
      * @dataProvider providerCUMIPMT
      *
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/AmorDegRcTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/AmorDegRcTest.php
new file mode 100644
index 0000000000..5d0cb8efba
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/AmorDegRcTest.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 AmorDegRcTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerAMORDEGRC
+     *
+     * @param mixed $expectedResult
+     */
+    public function testAMORDEGRC($expectedResult, ...$args): void
+    {
+        $result = Financial::AMORDEGRC(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerAMORDEGRC()
+    {
+        return require 'tests/data/Calculation/Financial/AMORDEGRC.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/AmorLincTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/AmorLincTest.php
new file mode 100644
index 0000000000..9f6b108022
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/AmorLincTest.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 AmorLincTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerAMORLINC
+     *
+     * @param mixed $expectedResult
+     */
+    public function testAMORLINC($expectedResult, ...$args): void
+    {
+        $result = Financial::AMORLINC(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerAMORLINC()
+    {
+        return require 'tests/data/Calculation/Financial/AMORLINC.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CoupDayBsTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CoupDayBsTest.php
new file mode 100644
index 0000000000..670d4dc2c0
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CoupDayBsTest.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 CoupDayBsTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerCOUPDAYBS
+     *
+     * @param mixed $expectedResult
+     */
+    public function testCOUPDAYBS($expectedResult, ...$args): void
+    {
+        $result = Financial::COUPDAYBS(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerCOUPDAYBS()
+    {
+        return require 'tests/data/Calculation/Financial/COUPDAYBS.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CoupDaysNcTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CoupDaysNcTest.php
new file mode 100644
index 0000000000..da9fc0ea35
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CoupDaysNcTest.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 CoupDaysNcTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerCOUPDAYSNC
+     *
+     * @param mixed $expectedResult
+     */
+    public function testCOUPDAYSNC($expectedResult, ...$args): void
+    {
+        $result = Financial::COUPDAYSNC(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerCOUPDAYSNC()
+    {
+        return require 'tests/data/Calculation/Financial/COUPDAYSNC.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CoupDaysTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CoupDaysTest.php
new file mode 100644
index 0000000000..9e642077ff
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CoupDaysTest.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 CoupDaysTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerCOUPDAYS
+     *
+     * @param mixed $expectedResult
+     */
+    public function testCOUPDAYS($expectedResult, ...$args): void
+    {
+        $result = Financial::COUPDAYS(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerCOUPDAYS()
+    {
+        return require 'tests/data/Calculation/Financial/COUPDAYS.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CoupNcdTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CoupNcdTest.php
new file mode 100644
index 0000000000..75680eafd4
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CoupNcdTest.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 CoupNcdTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerCOUPNCD
+     *
+     * @param mixed $expectedResult
+     */
+    public function testCOUPNCD($expectedResult, ...$args): void
+    {
+        $result = Financial::COUPNCD(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerCOUPNCD()
+    {
+        return require 'tests/data/Calculation/Financial/COUPNCD.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CoupNumTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CoupNumTest.php
new file mode 100644
index 0000000000..bff1b097f6
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CoupNumTest.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 CoupNumTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerCOUPNUM
+     *
+     * @param mixed $expectedResult
+     */
+    public function testCOUPNUM($expectedResult, ...$args): void
+    {
+        $result = Financial::COUPNUM(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerCOUPNUM()
+    {
+        return require 'tests/data/Calculation/Financial/COUPNUM.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CoupPcdTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CoupPcdTest.php
new file mode 100644
index 0000000000..465f005e85
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/CoupPcdTest.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 CoupPcdTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerCOUPPCD
+     *
+     * @param mixed $expectedResult
+     */
+    public function testCOUPPCD($expectedResult, ...$args): void
+    {
+        $result = Financial::COUPPCD(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerCOUPPCD()
+    {
+        return require 'tests/data/Calculation/Financial/COUPPCD.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/PriceMatTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/PriceMatTest.php
new file mode 100644
index 0000000000..0eb8c1040b
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/PriceMatTest.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 PriceMatTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerPRICEMAT
+     *
+     * @param mixed $expectedResult
+     */
+    public function testPRICEMAT($expectedResult, ...$args): void
+    {
+        $result = Financial::PRICEMAT(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerPRICEMAT()
+    {
+        return require 'tests/data/Calculation/Financial/PRICEMAT.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/ReceivedTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/ReceivedTest.php
new file mode 100644
index 0000000000..55cb393a8e
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/ReceivedTest.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 ReceivedTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerRECEIVED
+     *
+     * @param mixed $expectedResult
+     */
+    public function testRECEIVED($expectedResult, ...$args): void
+    {
+        $result = Financial::RECEIVED(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerRECEIVED()
+    {
+        return require 'tests/data/Calculation/Financial/RECEIVED.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/TBillEqTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/TBillEqTest.php
new file mode 100644
index 0000000000..756a505edc
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/TBillEqTest.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 TBillEqTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerTBILLEQ
+     *
+     * @param mixed $expectedResult
+     */
+    public function testTBILLEQ($expectedResult, ...$args): void
+    {
+        $result = Financial::TBILLEQ(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerTBILLEQ()
+    {
+        return require 'tests/data/Calculation/Financial/TBILLEQ.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/TBillPriceTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/TBillPriceTest.php
new file mode 100644
index 0000000000..458f6a83a3
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/TBillPriceTest.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 TBillPriceTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerTBILLPRICE
+     *
+     * @param mixed $expectedResult
+     */
+    public function testTBILLPRICE($expectedResult, ...$args): void
+    {
+        $result = Financial::TBILLPRICE(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerTBILLPRICE()
+    {
+        return require 'tests/data/Calculation/Financial/TBILLPRICE.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/TBillYieldTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/TBillYieldTest.php
new file mode 100644
index 0000000000..3c3057febb
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/TBillYieldTest.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 TBillYieldTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerTBILLYIELD
+     *
+     * @param mixed $expectedResult
+     */
+    public function testTBILLYIELD($expectedResult, ...$args): void
+    {
+        $result = Financial::TBILLYIELD(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerTBILLYIELD()
+    {
+        return require 'tests/data/Calculation/Financial/TBILLYIELD.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/YieldDiscTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/YieldDiscTest.php
new file mode 100644
index 0000000000..a8d5a7e092
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/YieldDiscTest.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 YieldDiscTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerYIELDDISC
+     *
+     * @param mixed $expectedResult
+     */
+    public function testYIELDDISC($expectedResult, ...$args): void
+    {
+        $result = Financial::YIELDDISC(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerYIELDDISC()
+    {
+        return require 'tests/data/Calculation/Financial/YIELDDISC.php';
+    }
+}
diff --git a/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/YieldMatTest.php b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/YieldMatTest.php
new file mode 100644
index 0000000000..9a1eba286a
--- /dev/null
+++ b/tests/PhpSpreadsheetTests/Calculation/Functions/Financial/YieldMatTest.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 YieldMatTest extends TestCase
+{
+    protected function setUp(): void
+    {
+        Functions::setCompatibilityMode(Functions::COMPATIBILITY_EXCEL);
+    }
+
+    /**
+     * @dataProvider providerYIELDMAT
+     *
+     * @param mixed $expectedResult
+     */
+    public function testYIELDMAT($expectedResult, ...$args): void
+    {
+        $result = Financial::YIELDMAT(...$args);
+        self::assertEqualsWithDelta($expectedResult, $result, 1E-8);
+    }
+
+    public function providerYIELDMAT()
+    {
+        return require 'tests/data/Calculation/Financial/YIELDMAT.php';
+    }
+}
diff --git a/tests/data/Calculation/Financial/AMORDEGRC.php b/tests/data/Calculation/Financial/AMORDEGRC.php
index 1e95914120..6d75feaff5 100644
--- a/tests/data/Calculation/Financial/AMORDEGRC.php
+++ b/tests/data/Calculation/Financial/AMORDEGRC.php
@@ -5,22 +5,22 @@
 return [
     [
         776,
-        2400,
-        '2008-08-19',
-        '2008-12-31',
-        300,
-        1,
-        0.14999999999999999,
-        1,
+        2400, '2008-08-19', '2008-12-31', 300, 1, 0.15, 1,
     ],
     [
         42,
-        150,
-        '2011-01-01',
-        '2011-09-30',
-        20,
-        1,
-        0.20000000000000001,
-        4,
+        150, '2011-01-01', '2011-09-30', 20, 1, 0.2, 4,
+    ],
+    [
+        25,
+        150, '2011-01-01', '2011-09-30', 20, 2, 0.2, 4,
+    ],
+    [
+        16,
+        150, '2011-01-01', '2011-09-30', 20, 3, 0.15, 4,
+    ],
+    [
+        42,
+        150, '2011-01-01', '2011-09-30', 20, 1, 0.4, 4,
     ],
 ];
diff --git a/tests/data/Calculation/Financial/AMORLINC.php b/tests/data/Calculation/Financial/AMORLINC.php
index 92adbe13f5..00ab2ae447 100644
--- a/tests/data/Calculation/Financial/AMORLINC.php
+++ b/tests/data/Calculation/Financial/AMORLINC.php
@@ -5,22 +5,22 @@
 return [
     [
         360,
-        2400,
-        '2008-08-19',
-        '2008-12-31',
-        300,
-        1,
-        0.14999999999999999,
-        1,
+        2400, '2008-08-19', '2008-12-31', 300, 1, 0.14999999999999999, 1,
     ],
     [
         30,
-        150,
-        '2011-01-01',
-        '2011-09-30',
-        20,
-        1,
-        0.20000000000000001,
-        4,
+        150, '2011-01-01', '2011-09-30', 20, 1, 0.20000000000000001, 4,
+    ],
+    [
+        22.41666667,
+        150, '2011-01-01', '2011-09-30', 20, 0, 0.20000000000000001, 4,
+    ],
+    [
+        17.58333333,
+        150, '2011-01-01', '2011-09-30', 20, 4, 0.20000000000000001, 4,
+    ],
+    [
+        0.0,
+        150, '2011-01-01', '2011-09-30', 20, 5, 0.20000000000000001, 4,
     ],
 ];
diff --git a/tests/data/Calculation/Financial/PRICEMAT.php b/tests/data/Calculation/Financial/PRICEMAT.php
new file mode 100644
index 0000000000..ac5c242efa
--- /dev/null
+++ b/tests/data/Calculation/Financial/PRICEMAT.php
@@ -0,0 +1,16 @@
+<?php
+
+return [
+    [
+        99.98449887555694,
+        '15-Feb-2008', '13-Apr-2008', '11-Nov-2007', 0.061, 0.061, 0,
+    ],
+    [
+        106.8486842105263,
+        '1-Apr-2017', '31-Mar-2021', '1-Jan-2017', 0.055, 0.035,
+    ],
+    [
+        93.0909090909091,
+        '1-Jul-2017', '1-Jan-2027', '1-Jan-2017', 0.07, 0.08,
+    ],
+];
diff --git a/tests/data/Calculation/Financial/RECEIVED.php b/tests/data/Calculation/Financial/RECEIVED.php
new file mode 100644
index 0000000000..1084c25500
--- /dev/null
+++ b/tests/data/Calculation/Financial/RECEIVED.php
@@ -0,0 +1,16 @@
+<?php
+
+return [
+    [
+        1014584.6544071021,
+        '15-Feb-2008', '15-May-2008', 1000000, 0.0575, 2,
+    ],
+    [
+        1120.2128404396835,
+        '6-Jul-2017', '15-Jan-2020', 1000, 0.0425,
+    ],
+    [
+        273037.54266211606,
+        '15-Feb-2008', '15-Nov-2017', 100000, 0.065,
+    ],
+];
diff --git a/tests/data/Calculation/Financial/TBILLEQ.php b/tests/data/Calculation/Financial/TBILLEQ.php
new file mode 100644
index 0000000000..606e36bdf4
--- /dev/null
+++ b/tests/data/Calculation/Financial/TBILLEQ.php
@@ -0,0 +1,20 @@
+<?php
+
+return [
+    [
+        0.09415149356594302,
+        '31-Mar-2008', '1-Jun-2008', 0.0914,
+    ],
+    [
+        0.025465926064094112,
+        '5-Feb-2019', '1-Feb-2020', 0.0245,
+    ],
+    [
+        0.036787997465875716,
+        '1-Feb-2016', '30-Jan-2017', 0.035,
+    ],
+    [
+        0.025612238,
+        '1-Feb-2017', '30-Jun-2017', 0.025,
+    ],
+];
diff --git a/tests/data/Calculation/Financial/TBILLPRICE.php b/tests/data/Calculation/Financial/TBILLPRICE.php
new file mode 100644
index 0000000000..83c4b5263d
--- /dev/null
+++ b/tests/data/Calculation/Financial/TBILLPRICE.php
@@ -0,0 +1,20 @@
+<?php
+
+return [
+    [
+        98.45,
+        '31-Mar-2008', '1-Jun-2008', 0.09,
+    ],
+    [
+        97.75,
+        '1-Apr-2017', '30-Jun-2017', 0.09,
+    ],
+    [
+        97.54319444444445,
+        '5-Feb-2019', '1-Feb-2020', 0.0245,
+    ],
+    [
+        98.86180556,
+        '1-Feb-2017', '30-Jun-2017', 0.0275,
+    ],
+];
diff --git a/tests/data/Calculation/Financial/TBILLYIELD.php b/tests/data/Calculation/Financial/TBILLYIELD.php
new file mode 100644
index 0000000000..83ddff423c
--- /dev/null
+++ b/tests/data/Calculation/Financial/TBILLYIELD.php
@@ -0,0 +1,12 @@
+<?php
+
+return [
+    [
+        0.09141696292534264,
+        '31-Mar-2008', '1-Jun-2008', 98.45,
+    ],
+    [
+        0.024405125076266018,
+        '1-Feb-2017', '30-Jun-2017', 99,
+    ],
+];
diff --git a/tests/data/Calculation/Financial/YIELDDISC.php b/tests/data/Calculation/Financial/YIELDDISC.php
new file mode 100644
index 0000000000..e6260a8645
--- /dev/null
+++ b/tests/data/Calculation/Financial/YIELDDISC.php
@@ -0,0 +1,12 @@
+<?php
+
+return [
+    [
+        0.05282257198685834,
+        '16-Feb-2008', '1-Mar-2008', 99.795, 100, 2,
+    ],
+    [
+        0.06220123250590336,
+        '1-Jan-2017', '30-Jun-2017', 97, 100,
+    ],
+];
diff --git a/tests/data/Calculation/Financial/YIELDMAT.php b/tests/data/Calculation/Financial/YIELDMAT.php
new file mode 100644
index 0000000000..6f26b15c4f
--- /dev/null
+++ b/tests/data/Calculation/Financial/YIELDMAT.php
@@ -0,0 +1,12 @@
+<?php
+
+return [
+    [
+        0.06095433369153867,
+        '15-Mar-2008', '3-Nov-2008', '8-Nov-2007', 0.0625, 100.0123, 0,
+    ],
+    [
+        0.04210977320221025,
+        '1-Jan-2017', '30-Jun-2018', '01-Jul-2014', 0.055, 101,
+    ],
+];