From 15852031d4e99b90ab57022c9188d9edfc078137 Mon Sep 17 00:00:00 2001
From: Christopher Lam <christopher.lck@gmail.com>
Date: Mon, 13 Sep 2021 23:43:42 +0800
Subject: [PATCH] [utest-account.cpp] increase test coverage

adds getter/setter tests for

xaccAccountSetLastNum
xaccAccountSetTaxUSCode
xaccAccountSetTaxUSPayerNameSource
---
 libgnucash/engine/test/utest-Account.cpp | 54 ++++++++++++++++++++++++
 1 file changed, 54 insertions(+)

diff --git a/libgnucash/engine/test/utest-Account.cpp b/libgnucash/engine/test/utest-Account.cpp
index 6cad13b4f6..9fca144cd1 100644
--- a/libgnucash/engine/test/utest-Account.cpp
+++ b/libgnucash/engine/test/utest-Account.cpp
@@ -1108,6 +1108,60 @@ test_gnc_account_kvp_setters_getters (Fixture *fixture, gconstpointer pData)
     xaccAccountSetColor (account, nullptr);
     g_assert_cmpstr (xaccAccountGetColor (account), ==, nullptr);
 
+    // last_num getter/setter
+    g_assert_cmpstr (xaccAccountGetLastNum (account), ==, nullptr);
+
+    xaccAccountSetLastNum (account, "red");
+    g_assert_cmpstr (xaccAccountGetLastNum (account), ==, "red");
+
+    xaccAccountSetLastNum (account, "");
+    g_assert_cmpstr (xaccAccountGetLastNum (account), ==, "");
+
+    xaccAccountSetLastNum (account, "  ");
+    g_assert_cmpstr (xaccAccountGetLastNum (account), ==, "  ");
+
+    xaccAccountSetLastNum (account, "unset");
+    g_assert_cmpstr (xaccAccountGetLastNum (account), ==, "unset");
+
+    xaccAccountSetLastNum (account, nullptr);
+    g_assert_cmpstr (xaccAccountGetLastNum (account), ==, nullptr);
+
+    // tax_us_code getter/setter
+    g_assert_cmpstr (xaccAccountGetTaxUSCode (account), ==, nullptr);
+
+    xaccAccountSetTaxUSCode (account, "red");
+    g_assert_cmpstr (xaccAccountGetTaxUSCode (account), ==, "red");
+
+    xaccAccountSetTaxUSCode (account, "");
+    g_assert_cmpstr (xaccAccountGetTaxUSCode (account), ==, "");
+
+    xaccAccountSetTaxUSCode (account, "  ");
+    g_assert_cmpstr (xaccAccountGetTaxUSCode (account), ==, "  ");
+
+    xaccAccountSetTaxUSCode (account, "unset");
+    g_assert_cmpstr (xaccAccountGetTaxUSCode (account), ==, "unset");
+
+    xaccAccountSetTaxUSCode (account, nullptr);
+    g_assert_cmpstr (xaccAccountGetTaxUSCode (account), ==, nullptr);
+
+    // tax_us_pns getter/setter
+    g_assert_cmpstr (xaccAccountGetTaxUSPayerNameSource (account), ==, nullptr);
+
+    xaccAccountSetTaxUSPayerNameSource (account, "red");
+    g_assert_cmpstr (xaccAccountGetTaxUSPayerNameSource (account), ==, "red");
+
+    xaccAccountSetTaxUSPayerNameSource (account, "");
+    g_assert_cmpstr (xaccAccountGetTaxUSPayerNameSource (account), ==, "");
+
+    xaccAccountSetTaxUSPayerNameSource (account, "  ");
+    g_assert_cmpstr (xaccAccountGetTaxUSPayerNameSource (account), ==, "  ");
+
+    xaccAccountSetTaxUSPayerNameSource (account, "unset");
+    g_assert_cmpstr (xaccAccountGetTaxUSPayerNameSource (account), ==, "unset");
+
+    xaccAccountSetTaxUSPayerNameSource (account, nullptr);
+    g_assert_cmpstr (xaccAccountGetTaxUSPayerNameSource (account), ==, nullptr);
+
     // filter getter/setter
     g_assert_cmpstr (xaccAccountGetFilter (account), ==, nullptr);