Ignore methods in code coverage that are not implemented.

2025-02-25 18:45:27 -06:00 · 2015-01-25 11:36:34 +01:00 · 2015-01-25 11:36:34 +01:00 · f3460cca49
commit f3460cca49
parent 64ce53ac30
12 changed files with 42 additions and 1 deletions
--- a/app/lib/FireflyIII/Database/Account/Account.php
+++ b/app/lib/FireflyIII/Database/Account/Account.php
@ -402,6 +402,7 @@ class Account implements CUDInterface, CommonDatabaseCallsInterface, AccountInte

    /**
     * @SuppressWarnings(PHPMD.UnusedFormalParameter)
+     * @codeCoverageIgnore
     *
     * @param $what
     *
@ -418,6 +419,7 @@ class Account implements CUDInterface, CommonDatabaseCallsInterface, AccountInte
     *
     * @return Collection
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function get()
    {
--- a/app/lib/FireflyIII/Database/AccountType/AccountType.php
+++ b/app/lib/FireflyIII/Database/AccountType/AccountType.php
@ -23,6 +23,7 @@ class AccountType implements CUDInterface, CommonDatabaseCallsInterface
     *
     * @return bool
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function destroy(Eloquent $model)
    {
@ -35,6 +36,7 @@ class AccountType implements CUDInterface, CommonDatabaseCallsInterface
     *
     * @return \Eloquent
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function store(array $data)
    {
@ -48,6 +50,7 @@ class AccountType implements CUDInterface, CommonDatabaseCallsInterface
     *
     * @return bool
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function update(Eloquent $model, array $data)
    {
@ -64,6 +67,7 @@ class AccountType implements CUDInterface, CommonDatabaseCallsInterface
     *
     * @return array
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function validate(array $model)
    {
@ -79,6 +83,7 @@ class AccountType implements CUDInterface, CommonDatabaseCallsInterface
     *
     * @return \Eloquent
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function find($objectId)
    {
@ -115,6 +120,7 @@ class AccountType implements CUDInterface, CommonDatabaseCallsInterface
     *
     * @return Collection
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function get()
    {
@ -128,6 +134,7 @@ class AccountType implements CUDInterface, CommonDatabaseCallsInterface
     *
     * @return Collection
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function getByIds(array $ids)
    {
--- a/app/lib/FireflyIII/Database/Bill/Bill.php
+++ b/app/lib/FireflyIII/Database/Bill/Bill.php
@ -141,6 +141,7 @@ class Bill implements CUDInterface, CommonDatabaseCallsInterface, BillInterface
     *
     * @return \Eloquent
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function find($objectId)
    {
@ -156,6 +157,7 @@ class Bill implements CUDInterface, CommonDatabaseCallsInterface, BillInterface
     *
     * @return \AccountType|null
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function findByWhat($what)
    {
@ -178,6 +180,7 @@ class Bill implements CUDInterface, CommonDatabaseCallsInterface, BillInterface
     *
     * @return Collection
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function getByIds(array $ids)
    {
--- a/app/lib/FireflyIII/Database/Budget/Budget.php
+++ b/app/lib/FireflyIII/Database/Budget/Budget.php
@ -183,6 +183,7 @@ class Budget implements CUDInterface, CommonDatabaseCallsInterface, BudgetInterf
     *
     * @return \AccountType|null
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function findByWhat($what)
    {
@ -208,6 +209,7 @@ class Budget implements CUDInterface, CommonDatabaseCallsInterface, BudgetInterf
     *
     * @return Collection
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function getByIds(array $ids)
    {
--- a/app/lib/FireflyIII/Database/Category/Category.php
+++ b/app/lib/FireflyIII/Database/Category/Category.php
@ -107,6 +107,7 @@ class Category implements CUDInterface, CommonDatabaseCallsInterface
     *
     * @return \Eloquent
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function find($objectId)
    {
@ -122,6 +123,7 @@ class Category implements CUDInterface, CommonDatabaseCallsInterface
     *
     * @return \AccountType|null
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function findByWhat($what)
    {
@ -147,6 +149,7 @@ class Category implements CUDInterface, CommonDatabaseCallsInterface
     *
     * @return Collection
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function getByIds(array $ids)
    {
--- a/app/lib/FireflyIII/Database/PiggyBank/PiggyBank.php
+++ b/app/lib/FireflyIII/Database/PiggyBank/PiggyBank.php
@ -21,7 +21,6 @@ class PiggyBank extends PiggyBankShared implements CUDInterface, CommonDatabaseC
     *
     * @return mixed
     * @throws FireflyException
-     * @throws NotImplementedException
     */
    public function findRepetitionByDate(\PiggyBank $piggyBank, Carbon $date)
    {
--- a/app/lib/FireflyIII/Database/PiggyBank/PiggyBankShared.php
+++ b/app/lib/FireflyIII/Database/PiggyBank/PiggyBankShared.php
@ -66,6 +66,7 @@ class PiggyBankShared
     *
     * @return \AccountType|null
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function findByWhat($what)
    {
--- a/app/lib/FireflyIII/Database/Transaction/Transaction.php
+++ b/app/lib/FireflyIII/Database/Transaction/Transaction.php
@ -26,6 +26,7 @@ class Transaction implements CUDInterface, CommonDatabaseCallsInterface
     *
     * @return bool
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function destroy(Eloquent $model)
    {
@ -67,6 +68,7 @@ class Transaction implements CUDInterface, CommonDatabaseCallsInterface
     *
     * @return bool
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function update(Eloquent $model, array $data)
    {
@ -103,6 +105,7 @@ class Transaction implements CUDInterface, CommonDatabaseCallsInterface
     *
     * @return \Eloquent
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function find($objectId)
    {
@ -118,6 +121,7 @@ class Transaction implements CUDInterface, CommonDatabaseCallsInterface
     *
     * @return \AccountType|null
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function findByWhat($what)
    {
@ -129,6 +133,7 @@ class Transaction implements CUDInterface, CommonDatabaseCallsInterface
     *
     * @return Collection
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function get()
    {
@ -142,6 +147,7 @@ class Transaction implements CUDInterface, CommonDatabaseCallsInterface
     *
     * @return Collection
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function getByIds(array $ids)
    {
--- a/app/lib/FireflyIII/Database/TransactionCurrency/TransactionCurrency.php
+++ b/app/lib/FireflyIII/Database/TransactionCurrency/TransactionCurrency.php
@ -95,6 +95,7 @@ class TransactionCurrency implements TransactionCurrencyInterface, CommonDatabas
     *
     * @param int $objectId
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     *
     * @return \Eloquent
     */
@ -110,6 +111,7 @@ class TransactionCurrency implements TransactionCurrencyInterface, CommonDatabas
     *
     * @param $what
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     *
     * @return \AccountType|null
     */
@ -133,6 +135,7 @@ class TransactionCurrency implements TransactionCurrencyInterface, CommonDatabas
     *
     * @param array $objectIds
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     *
     * @return Collection
     */
--- a/app/lib/FireflyIII/Database/TransactionJournal/TransactionJournal.php
+++ b/app/lib/FireflyIII/Database/TransactionJournal/TransactionJournal.php
@ -413,6 +413,7 @@ class TransactionJournal implements TransactionJournalInterface, CUDInterface, C
     *
     * @return \AccountType|null
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function findByWhat($what)
    {
--- a/app/lib/FireflyIII/Database/TransactionType/TransactionType.php
+++ b/app/lib/FireflyIII/Database/TransactionType/TransactionType.php
@ -26,6 +26,7 @@ class TransactionType implements CUDInterface, CommonDatabaseCallsInterface
     *
     * @return bool
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function destroy(Eloquent $model)
    {
@ -39,6 +40,7 @@ class TransactionType implements CUDInterface, CommonDatabaseCallsInterface
     *
     * @return \Eloquent
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function store(array $data)
    {
@ -53,6 +55,7 @@ class TransactionType implements CUDInterface, CommonDatabaseCallsInterface
     *
     * @return bool
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function update(Eloquent $model, array $data)
    {
@ -69,6 +72,7 @@ class TransactionType implements CUDInterface, CommonDatabaseCallsInterface
     *
     * @return array
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function validate(array $model)
    {
@ -84,6 +88,7 @@ class TransactionType implements CUDInterface, CommonDatabaseCallsInterface
     *
     * @return \Eloquent
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function find($objectId)
    {
@ -120,6 +125,7 @@ class TransactionType implements CUDInterface, CommonDatabaseCallsInterface
     *
     * @return Collection
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function get()
    {
@ -133,6 +139,7 @@ class TransactionType implements CUDInterface, CommonDatabaseCallsInterface
     *
     * @return Collection
     * @throws NotImplementedException
+     * @codeCoverageIgnore
     */
    public function getByIds(array $ids)
    {
--- a/app/lib/FireflyIII/Shared/Toolkit/Steam.php
+++ b/app/lib/FireflyIII/Shared/Toolkit/Steam.php
@ -35,6 +35,7 @@ class Steam
    }

    /**
+     * @codeCoverageIgnore
     * @param $boolean
     *
     * @return string
@ -61,12 +62,18 @@ class Steam
    {
        $pct = $repetition->currentamount / $piggyBank->targetamount * 100;
        if ($pct > 100) {
+            // @codeCoverageIgnoreStart
            return 100;
+            // @codeCoverageIgnoreEnd
        } else {
            return floor($pct);
        }
    }

+    /**
+     * @codeCoverageIgnore
+     * @throws \Exception
+     */
    public function removeEmptyBudgetLimits()
    {
        $user = \Auth::user();