Blob Blame Raw
From 29ab40829f86f6bd32229ededcc09bf2736e67bd Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Cristian=20One=C8=9B?= <onet.cristian@gmail.com>
Date: Thu, 16 Jun 2016 09:06:32 +0200
Subject: [PATCH 03/22] Fix the tests of the 4.8 build.

Do this by backporting the change that is already in frameworks in commit 77085ff468336f26f6bcf5f2a1a08e16409172e6. Did not use cherry pick because the testfile was moved in frameworks.

BUG: 364355
---
 kmymoney/mymoney/storage/mymoneymap.h              |  2 +-
 .../mymoney/storage/mymoneyseqaccessmgrtest.cpp    | 52 +++++++++++-----------
 2 files changed, 27 insertions(+), 27 deletions(-)

diff --git a/kmymoney/mymoney/storage/mymoneymap.h b/kmymoney/mymoney/storage/mymoneymap.h
index 0392d06..4dbdf3b 100644
--- a/kmymoney/mymoney/storage/mymoneymap.h
+++ b/kmymoney/mymoney/storage/mymoneymap.h
@@ -195,7 +195,7 @@ public:
     that = *(dynamic_cast<QMap<Key, T>* >(const_cast<MyMoneyMap<Key, T>* >(this)));
   }
 
-  inline size_t count(void) const {
+  inline int count(void) const {
     return QMap<Key, T>::count();
   }
 
diff --git a/kmymoney/mymoney/storage/mymoneyseqaccessmgrtest.cpp b/kmymoney/mymoney/storage/mymoneyseqaccessmgrtest.cpp
index dcb4b4a..58f387e 100644
--- a/kmymoney/mymoney/storage/mymoneyseqaccessmgrtest.cpp
+++ b/kmymoney/mymoney/storage/mymoneyseqaccessmgrtest.cpp
@@ -58,13 +58,13 @@ void MyMoneySeqAccessMgrTest::testEmptyConstructor()
   QCOMPARE(m->m_nextPayeeID, 0ul);
   QCOMPARE(m->m_nextScheduleID, 0ul);
   QCOMPARE(m->m_nextReportID, 0ul);
-  QCOMPARE(m->m_institutionList.count(), 0ul);
-  QCOMPARE(m->m_accountList.count(), 5ul);
-  QCOMPARE(m->m_transactionList.count(), 0ul);
-  QCOMPARE(m->m_transactionKeys.count(), 0ul);
-  QCOMPARE(m->m_payeeList.count(), 0ul);
-  QCOMPARE(m->m_tagList.count(), 0ul);
-  QCOMPARE(m->m_scheduleList.count(), 0ul);
+  QCOMPARE(m->m_institutionList.count(), 0);
+  QCOMPARE(m->m_accountList.count(), 5);
+  QCOMPARE(m->m_transactionList.count(), 0);
+  QCOMPARE(m->m_transactionKeys.count(), 0);
+  QCOMPARE(m->m_payeeList.count(), 0);
+  QCOMPARE(m->m_tagList.count(), 0);
+  QCOMPARE(m->m_scheduleList.count(), 0);
 
   QCOMPARE(m->m_dirty, false);
   QCOMPARE(m->m_creationDate, QDate::currentDate());
@@ -187,7 +187,7 @@ void MyMoneySeqAccessMgrTest::testNewAccount()
 
   QCOMPARE(m->m_nextAccountID, 1ul);
   QCOMPARE(m->dirty(), true);
-  QCOMPARE(m->m_accountList.count(), static_cast<std::size_t>(6));
+  QCOMPARE(m->m_accountList.count(), 6);
   QCOMPARE(m->m_accountList["A000001"].name(), QLatin1String("AccountName"));
 }
 
@@ -235,7 +235,7 @@ void MyMoneySeqAccessMgrTest::testAddNewAccount()
   m->m_dirty = false;
 
   QCOMPARE(m->m_nextAccountID, 2ul);
-  QCOMPARE(m->m_accountList.count(), static_cast<std::size_t>(7));
+  QCOMPARE(m->m_accountList.count(), 7);
 
   // try to add account to undefined account
   try {
@@ -271,7 +271,7 @@ void MyMoneySeqAccessMgrTest::testAddInstitution()
   i.setName("Inst Name");
 
   m->addInstitution(i);
-  QCOMPARE(m->m_institutionList.count(), static_cast<std::size_t>(1));
+  QCOMPARE(m->m_institutionList.count(), 1);
   QCOMPARE(m->m_nextInstitutionID, 1ul);
   QCOMPARE(m->m_institutionList["I000001"].name(), QLatin1String("Inst Name"));
 }
@@ -950,11 +950,11 @@ void MyMoneySeqAccessMgrTest::testRemovePayee()
   // check that we can remove an unreferenced payee
   MyMoneyPayee p = m->payee("P000001");
   try {
-    QCOMPARE(m->m_payeeList.count(), static_cast<std::size_t>(1));
+    QCOMPARE(m->m_payeeList.count(), 1);
     m->removePayee(p);
     m->commitTransaction();
     m->startTransaction();
-    QCOMPARE(m->m_payeeList.count(), static_cast<std::size_t>(0));
+    QCOMPARE(m->m_payeeList.count(), 0);
     QCOMPARE(m->dirty(), true);
   } catch (const MyMoneyException &) {
     QFAIL("Unexpected exception");
@@ -996,7 +996,7 @@ void MyMoneySeqAccessMgrTest::testRemovePayee()
     QFAIL("Expected exception");
   } catch (const MyMoneyException &) {
   }
-  QCOMPARE(m->m_payeeList.count(), static_cast<std::size_t>(1));
+  QCOMPARE(m->m_payeeList.count(), 1);
 }
 
 void MyMoneySeqAccessMgrTest::testAddTag()
@@ -1046,11 +1046,11 @@ void MyMoneySeqAccessMgrTest::testRemoveTag()
   // check that we can remove an unreferenced tag
   MyMoneyTag ta = m->tag("G000001");
   try {
-    QCOMPARE(m->m_tagList.count(), static_cast<std::size_t>(1));
+    QCOMPARE(m->m_tagList.count(), 1);
     m->removeTag(ta);
     m->commitTransaction();
     m->startTransaction();
-    QCOMPARE(m->m_tagList.count(), static_cast<std::size_t>(0));
+    QCOMPARE(m->m_tagList.count(), 0);
     QCOMPARE(m->dirty(), true);
   } catch (const MyMoneyException &) {
     QFAIL("Unexpected exception");
@@ -1094,7 +1094,7 @@ void MyMoneySeqAccessMgrTest::testRemoveTag()
     QFAIL("Expected exception");
   } catch (const MyMoneyException &) {
   }
-  QCOMPARE(m->m_tagList.count(), static_cast<std::size_t>(1));
+  QCOMPARE(m->m_tagList.count(), 1);
 }
 
 void MyMoneySeqAccessMgrTest::testRemoveAccountFromTree()
@@ -1267,7 +1267,7 @@ void MyMoneySeqAccessMgrTest::testAddSchedule()
 
 
   try {
-    QCOMPARE(m->m_scheduleList.count(), static_cast<std::size_t>(0));
+    QCOMPARE(m->m_scheduleList.count(), 0);
     MyMoneyTransaction t1;
     MyMoneySplit s1, s2;
     s1.setAccountId("A000001");
@@ -1287,7 +1287,7 @@ void MyMoneySeqAccessMgrTest::testAddSchedule()
 
     m->addSchedule(schedule);
 
-    QCOMPARE(m->m_scheduleList.count(), static_cast<std::size_t>(1));
+    QCOMPARE(m->m_scheduleList.count(), 1);
     QCOMPARE(schedule.id(), QLatin1String("SCH000001"));
     QCOMPARE(m->m_scheduleList["SCH000001"].id(), QLatin1String("SCH000001"));
   } catch (const MyMoneyException &) {
@@ -1342,7 +1342,7 @@ void MyMoneySeqAccessMgrTest::testModifySchedule()
   sched.setName("New Sched-Name");
   try {
     m->modifySchedule(sched);
-    QCOMPARE(m->m_scheduleList.count(), static_cast<std::size_t>(1));
+    QCOMPARE(m->m_scheduleList.count(), 1);
     QCOMPARE(m->m_scheduleList["SCH000001"].name(), QLatin1String("New Sched-Name"));
 
   } catch (const MyMoneyException &) {
@@ -1373,7 +1373,7 @@ void MyMoneySeqAccessMgrTest::testRemoveSchedule()
   try {
     m->removeSchedule(sched);
     m->commitTransaction();
-    QCOMPARE(m->m_scheduleList.count(), static_cast<std::size_t>(0));
+    QCOMPARE(m->m_scheduleList.count(), 0);
 
   } catch (const MyMoneyException &) {
     m->rollbackTransaction();
@@ -1548,13 +1548,13 @@ void MyMoneySeqAccessMgrTest::testScheduleList()
 void MyMoneySeqAccessMgrTest::testAddCurrency()
 {
   MyMoneySecurity curr("EUR", "Euro", "?", 100, 100);
-  QCOMPARE(m->m_currencyList.count(), static_cast<std::size_t>(0));
+  QCOMPARE(m->m_currencyList.count(), 0);
   m->m_dirty = false;
   try {
     m->addCurrency(curr);
     m->commitTransaction();
     m->startTransaction();
-    QCOMPARE(m->m_currencyList.count(), static_cast<std::size_t>(1));
+    QCOMPARE(m->m_currencyList.count(), 1);
     QCOMPARE(m->m_currencyList["EUR"].name(), QLatin1String("Euro"));
     QCOMPARE(m->dirty(), true);
   } catch (const MyMoneyException &) {
@@ -1582,7 +1582,7 @@ void MyMoneySeqAccessMgrTest::testModifyCurrency()
     m->modifyCurrency(curr);
     m->commitTransaction();
     m->startTransaction();
-    QCOMPARE(m->m_currencyList.count(), static_cast<std::size_t>(1));
+    QCOMPARE(m->m_currencyList.count(), 1);
     QCOMPARE(m->m_currencyList["EUR"].name(), QLatin1String("EURO"));
     QCOMPARE(m->dirty(), true);
   } catch (const MyMoneyException &) {
@@ -1611,7 +1611,7 @@ void MyMoneySeqAccessMgrTest::testRemoveCurrency()
     m->removeCurrency(curr);
     m->commitTransaction();
     m->startTransaction();
-    QCOMPARE(m->m_currencyList.count(), static_cast<std::size_t>(0));
+    QCOMPARE(m->m_currencyList.count(), 0);
     QCOMPARE(m->dirty(), true);
   } catch (const MyMoneyException &) {
     QFAIL("Unexpected exception");
@@ -1668,7 +1668,7 @@ void MyMoneySeqAccessMgrTest::testCurrencyList()
   try {
     m->addCurrency(unknownCurr);
     m->m_dirty = false;
-    QCOMPARE(m->m_currencyList.count(), static_cast<std::size_t>(2));
+    QCOMPARE(m->m_currencyList.count(), 2);
     QCOMPARE(m->currencyList().count(), 2);
     QCOMPARE(m->dirty(), false);
   } catch (const MyMoneyException &) {
@@ -1797,7 +1797,7 @@ void MyMoneySeqAccessMgrTest::testAddOnlineJob()
 
   QCOMPARE(m->m_nextOnlineJobID, 1ul);
   QCOMPARE(m->dirty(), true);
-  QCOMPARE(m->m_onlineJobList.count(), static_cast<std::size_t>(1));
+  QCOMPARE(m->m_onlineJobList.count(), 1);
   QVERIFY(! m->m_onlineJobList["O000001"].isNull());
 
 }
-- 
2.7.4