FakturaVydanaTest::makeInvoice() - Code Metrics - Inspection of "Doplněno spárování faktury s dokladem a její unit..." - Spoje-NET/FlexiPeeHP - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 5edd48...19a6ea )

by Vítězslav

created 2016-06-14 15:56 UTC

FakturaVydanaTest::makeInvoice() D

↳ Parent: FakturaVydanaTest

Complexity

Conditions	11
Paths	1024

Size

Total Lines	53
Code Lines	32

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
cc	11
eloc	32
c	2
b	0
f	0
nc	1024
nop	1
dl	0
loc	53
rs	4.1538

How to fix Long Method Complexity

<?php

namespace Test\FlexiPeeHP;

use FlexiPeeHP\FakturaVydana;

/**
 * Generated by PHPUnit_SkeletonGenerator on 2016-04-27 at 17:32:11.
 */
class FakturaVydanaTest extends FlexiBeeRWTest
{
    /**
     * @var FakturaVydana
     */
    protected $object;

    /**
     * Sets up the fixture, for example, opens a network connection.
     * This method is called before a test is executed.
     */
    protected function setUp()
    {
        $this->object = new FakturaVydana();
    }

    /**
     * Tears down the fixture, for example, closes a network connection.
     * This method is called after a test is executed.
     */
    protected function tearDown()
    {

    }

    /**
     * @covers FlexiPeeHP\FakturaVydana::hotovostniUhrada
     */
    public function testhotovostniUhrada()
    {
        $this->makeInvoice();
        $this->object->unsetDataValue('kod');
        $this->object->hotovostniUhrada($this->object->getDataValue('sumCelkZakl'));
        $this->assertEquals(201, $this->object->lastResponseCode,
            _('Invoice settle error'));
    }

    /**
     * @covers FlexiPeeHP\FakturaVydana::sparujPlatbu
     */
    public function testsparujPlatbu()
    {
        $this->makeInvoice();
        $doklad     = new \FlexiPeeHP\PokladniPohyb();
        $value      = $this->object->getDataValue('sumCelkZakl');
        $dataPohybu = [
            'kod' => 'FP'.time(),
            'typDokl' => 'code:STANDARD',
            'typPohybuK' => 'typPohybu.prijem',
            'datVyst' => date("Y-m-d", time() - 60 * 60 * 24),
            'jakUhrK' => 'jakUhrazeno.rucne1',
            'pokladna' => 'code:POKLADNA KČ',
            'generovatSkl' => false,
            'zdrojProSkl' => false,
            'firma' => $this->object->getDataValue('firma'),
            'bezPolozek' => true,
            'poznam' => $this->poznam,
            'primUcet' => 'code:013001',
            'sumZklCelkem' => $value
        ];
        $doklad->takeData($dataPohybu);
        $doklad->insertToFlexiBee();
        $doklad->unsetDataValue('kod');
        $this->object->sparujPlatbu($doklad);
        $this->assertEquals(201, $doklad->lastResponseCode,
            _('Invoice match error'));
    }

    /**
     * Crerate testing invoice
     * 
     * @param array $invoiceData
     */
    public function makeInvoice($invoiceData = [])
    {
        if (!isset($invoiceData['kod'])) {
            $invoiceData['kod'] = 'PeeHP'.time();
        }
        if (!isset($invoiceData['varSym'])) {
            $invoiceData['varSym'] = \Ease\Sand::randomNumber(1000, 99999);
        }
        if (!isset($invoiceData['datVyst'])) {
            $invoiceData['datVyst'] = date("Y-m-d", time() - 60 * 60 * 24);
        }
        if (!isset($invoiceData['typDokl'])) {
            $invoiceData['typDokl'] = 'code:FAKTURA';
        }
        if (!isset($invoiceData['zdrojProSkl'])) {
            $invoiceData['zdrojProSkl'] = false;
        }
        if (!isset($invoiceData['dobropisovano'])) {
            $invoiceData['dobropisovano'] = false;
        }

        if (!isset($invoiceData['polozky'])) {
            $invoiceData['bezPolozek'] = true;
        }

        if (!isset($invoiceData['sumCelkZakl'])) {
            $scale                      = pow(1000, 2);
            $invoiceData['sumCelkZakl'] = round(mt_rand(10 * $scale,
                    9000 * $scale) / $scale, 2);
        }

        if (!isset($invoiceData['firma'])) {
            $adresar = new \FlexiPeeHP\Adresar();

            $adresy = $adresar->getFlexiData(null,
                ['typVztahuK' => 'typVztahu.odberatel']);

            $dodavatel = $adresy[array_rand($adresy)];

            $invoiceData['firma'] = 'code:'.$dodavatel['kod'];
        }

        if (!isset($ivoiceData['poznam'])) {

            $invoiceData['poznam'] = $this->poznam;
        }

        $this->object->takeData($invoiceData);
        $result = $this->object->insertToFlexiBee();
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

        $id = $this->object->getLastInsertedId();
        $this->object->setDataValue('id', $id);
        return $id;
    }
}

1			<?php
2
3			namespace Test\FlexiPeeHP;
4
5			use FlexiPeeHP\FakturaVydana;
6
7			/**
8			* Generated by PHPUnit_SkeletonGenerator on 2016-04-27 at 17:32:11.
9			*/
10			class FakturaVydanaTest extends FlexiBeeRWTest
11			{
12			/**
13			* @var FakturaVydana
14			*/
15			protected $object;
16
17			/**
18			* Sets up the fixture, for example, opens a network connection.
19			* This method is called before a test is executed.
20			*/
21			protected function setUp()
22			{
23			$this->object = new FakturaVydana();
24			}
25
26			/**
27			* Tears down the fixture, for example, closes a network connection.
28			* This method is called after a test is executed.
29			*/
30			protected function tearDown()
31			{
32
33			}
34
35			/**
36			* @covers FlexiPeeHP\FakturaVydana::hotovostniUhrada
37			*/
38			public function testhotovostniUhrada()
39			{
40			$this->makeInvoice();
41			$this->object->unsetDataValue('kod');
42			$this->object->hotovostniUhrada($this->object->getDataValue('sumCelkZakl'));
43			$this->assertEquals(201, $this->object->lastResponseCode,
44			_('Invoice settle error'));
45			}
46
47			/**
48			* @covers FlexiPeeHP\FakturaVydana::sparujPlatbu
49			*/
50			public function testsparujPlatbu()
51			{
52			$this->makeInvoice();
53			$doklad = new \FlexiPeeHP\PokladniPohyb();
54			$value = $this->object->getDataValue('sumCelkZakl');
55			$dataPohybu = [
56			'kod' => 'FP'.time(),
57			'typDokl' => 'code:STANDARD',
58			'typPohybuK' => 'typPohybu.prijem',
59			'datVyst' => date("Y-m-d", time() - 60 * 60 * 24),
60			'jakUhrK' => 'jakUhrazeno.rucne1',
61			'pokladna' => 'code:POKLADNA KČ',
62			'generovatSkl' => false,
63			'zdrojProSkl' => false,
64			'firma' => $this->object->getDataValue('firma'),
65			'bezPolozek' => true,
66			'poznam' => $this->poznam,
67			'primUcet' => 'code:013001',
68			'sumZklCelkem' => $value
69			];
70			$doklad->takeData($dataPohybu);
71			$doklad->insertToFlexiBee();
72			$doklad->unsetDataValue('kod');
73			$this->object->sparujPlatbu($doklad);
74			$this->assertEquals(201, $doklad->lastResponseCode,
75			_('Invoice match error'));
76			}
77
78			/**
79			* Crerate testing invoice
80			*
81			* @param array $invoiceData
82			*/
83			public function makeInvoice($invoiceData = [])
84			{
85			if (!isset($invoiceData['kod'])) {
86			$invoiceData['kod'] = 'PeeHP'.time();
87			}
88			if (!isset($invoiceData['varSym'])) {
89			$invoiceData['varSym'] = \Ease\Sand::randomNumber(1000, 99999);
90			}
91			if (!isset($invoiceData['datVyst'])) {
92			$invoiceData['datVyst'] = date("Y-m-d", time() - 60 * 60 * 24);
93			}
94			if (!isset($invoiceData['typDokl'])) {
95			$invoiceData['typDokl'] = 'code:FAKTURA';
96			}
97			if (!isset($invoiceData['zdrojProSkl'])) {
98			$invoiceData['zdrojProSkl'] = false;
99			}
100			if (!isset($invoiceData['dobropisovano'])) {
101			$invoiceData['dobropisovano'] = false;
102			}
103
104			if (!isset($invoiceData['polozky'])) {
105			$invoiceData['bezPolozek'] = true;
106			}
107
108			if (!isset($invoiceData['sumCelkZakl'])) {
109			$scale = pow(1000, 2);
110			$invoiceData['sumCelkZakl'] = round(mt_rand(10 * $scale,
111			9000 * $scale) / $scale, 2);
112			}
113
114			if (!isset($invoiceData['firma'])) {
115			$adresar = new \FlexiPeeHP\Adresar();
116
117			$adresy = $adresar->getFlexiData(null,
118			['typVztahuK' => 'typVztahu.odberatel']);
119
120			$dodavatel = $adresy[array_rand($adresy)];
121
122			$invoiceData['firma'] = 'code:'.$dodavatel['kod'];
123			}
124
125			if (!isset($ivoiceData['poznam'])) {
			0 ignored issues – show Bug introduced 2016-06-14 15:59 UTC by Report Bug Copy Issue Report The variable `$ivoiceData` does not exist. Did you mean `$invoiceData`? This check looks for variables that are accessed but have not been defined. It raises an issue if it finds another variable that has a similar name. The variable may have been renamed without also renaming all references. Loading history...
126			$invoiceData['poznam'] = $this->poznam;
127			}
128
129			$this->object->takeData($invoiceData);
130			$result = $this->object->insertToFlexiBee();
			0 ignored issues – show Unused Code introduced 2016-06-14 15:59 UTC by Report Bug Copy Issue Report `$result` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
131
132			$id = $this->object->getLastInsertedId();
133			$this->object->setDataValue('id', $id);
134			return $id;
135			}
136			}

Spoje-NET / FlexiPeeHP

Push — master ( 5edd48...19a6ea )

FakturaVydanaTest::makeInvoice() D

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like