OrderdetailTest::testFindPriceForQty() - Code Metrics - Inspection of "Really fixed .scrutinizer.yml" - Part-DB/Part-DB-symfony - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( d286b7...24e4a5 )

by Jan

created 2019-11-09 13:36 UTC

OrderdetailTest::testFindPriceForQty() A

↳ Parent: OrderdetailTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	15
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	11
nc	1
nop	0
dl	0
loc	15
rs	9.9
c	0
b	0
f	0

<?php
/**
 * This file is part of Part-DB (https://github.com/Part-DB/Part-DB-symfony).
 *
 * Copyright (C) 2019 Jan Böhmer (https://github.com/jbtronics)
 *
 * This program is free software; you can redistribute it and/or
 * modify it under the terms of the GNU General Public License
 * as published by the Free Software Foundation; either version 2
 * of the License, or (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with this program; if not, write to the Free Software
 * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA
 */

namespace App\Tests\Entity\PriceSystem;

use App\Entity\PriceInformations\Orderdetail;
use App\Entity\PriceInformations\Pricedetail;
use Doctrine\Common\Collections\Collection;
use PHPUnit\Framework\TestCase;
filter:
    dependency_paths: ["lib/*"]

class OrderdetailTest extends TestCase
{
    public function testAddRemovePricdetails()
    {
        $orderdetail = new Orderdetail();
        $this->assertInstanceOf(Collection::class, $orderdetail->getPricedetails());
        $this->assertTrue($orderdetail->getPricedetails()->isEmpty());

        $pricedetail = new Pricedetail();
        $orderdetail->addPricedetail($pricedetail);
        $this->assertEquals($orderdetail, $pricedetail->getOrderdetail());
        $this->assertEquals(1, $orderdetail->getPricedetails()->count());

        //After removal of the pricedetail, the orderdetail must be empty again
        $orderdetail->removePricedetail($pricedetail);
        $this->assertTrue($orderdetail->getPricedetails()->isEmpty());
    }

    public function testFindPriceForQty()
    {
        $price0 = (new Pricedetail())->setMinDiscountQuantity(0.23);
        $price1 = (new Pricedetail())->setMinDiscountQuantity(1);
        $price5 = (new Pricedetail())->setMinDiscountQuantity(5.3);
        $orderdetail = (new Orderdetail())->addPricedetail($price0)->addPricedetail($price1)->addPricedetail($price5);

        $this->assertNull($orderdetail->findPriceForQty(0));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
        $this->assertNull($orderdetail->findPriceForQty(0.1));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {

        $this->assertEquals($price0, $orderdetail->findPriceForQty(0.5));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
        $this->assertEquals($price1, $orderdetail->findPriceForQty(1));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
        $this->assertEquals($price1, $orderdetail->findPriceForQty(1.5));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
        $this->assertEquals($price5, $orderdetail->findPriceForQty(5.3));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
        $this->assertEquals($price5, $orderdetail->findPriceForQty(10000));
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
    }
}


1			<?php
2			/**
3			* This file is part of Part-DB (https://github.com/Part-DB/Part-DB-symfony).
4			*
5			* Copyright (C) 2019 Jan Böhmer (https://github.com/jbtronics)
6			*
7			* This program is free software; you can redistribute it and/or
8			* modify it under the terms of the GNU General Public License
9			* as published by the Free Software Foundation; either version 2
10			* of the License, or (at your option) any later version.
11			*
12			* This program is distributed in the hope that it will be useful,
13			* but WITHOUT ANY WARRANTY; without even the implied warranty of
14			* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
15			* GNU General Public License for more details.
16			*
17			* You should have received a copy of the GNU General Public License
18			* along with this program; if not, write to the Free Software
19			* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA
20			*/
21
22			namespace App\Tests\Entity\PriceSystem;
23
24			use App\Entity\PriceInformations\Orderdetail;
25			use App\Entity\PriceInformations\Pricedetail;
26			use Doctrine\Common\Collections\Collection;
27			use PHPUnit\Framework\TestCase;
			0 ignored issues – show Bug introduced 2019-11-09 13:40 UTC by Report Bug Copy Issue Report The type `PHPUnit\Framework\TestCase` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
28
29			class OrderdetailTest extends TestCase
30			{
31			public function testAddRemovePricdetails()
32			{
33			$orderdetail = new Orderdetail();
34			$this->assertInstanceOf(Collection::class, $orderdetail->getPricedetails());
35			$this->assertTrue($orderdetail->getPricedetails()->isEmpty());
36
37			$pricedetail = new Pricedetail();
38			$orderdetail->addPricedetail($pricedetail);
39			$this->assertEquals($orderdetail, $pricedetail->getOrderdetail());
40			$this->assertEquals(1, $orderdetail->getPricedetails()->count());
41
42			//After removal of the pricedetail, the orderdetail must be empty again
43			$orderdetail->removePricedetail($pricedetail);
44			$this->assertTrue($orderdetail->getPricedetails()->isEmpty());
45			}
46
47			public function testFindPriceForQty()
48			{
49			$price0 = (new Pricedetail())->setMinDiscountQuantity(0.23);
50			$price1 = (new Pricedetail())->setMinDiscountQuantity(1);
51			$price5 = (new Pricedetail())->setMinDiscountQuantity(5.3);
52			$orderdetail = (new Orderdetail())->addPricedetail($price0)->addPricedetail($price1)->addPricedetail($price5);
53
54			$this->assertNull($orderdetail->findPriceForQty(0));
			0 ignored issues – show Bug introduced 2019-11-09 13:40 UTC by Report Bug Copy Issue Report Are you sure the usage of `$orderdetail->findPriceForQty(0)` targeting `App\Entity\PriceInformat...tail::findPriceForQty()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
55			$this->assertNull($orderdetail->findPriceForQty(0.1));
			0 ignored issues – show Bug introduced 2019-11-09 13:40 UTC by Report Bug Copy Issue Report Are you sure the usage of `$orderdetail->findPriceForQty(0.1)` targeting `App\Entity\PriceInformat...tail::findPriceForQty()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
56
57			$this->assertEquals($price0, $orderdetail->findPriceForQty(0.5));
			0 ignored issues – show Bug introduced 2019-11-09 13:40 UTC by Report Bug Copy Issue Report Are you sure the usage of `$orderdetail->findPriceForQty(0.5)` targeting `App\Entity\PriceInformat...tail::findPriceForQty()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
58			$this->assertEquals($price1, $orderdetail->findPriceForQty(1));
			0 ignored issues – show Bug introduced 2019-11-09 13:40 UTC by Report Bug Copy Issue Report Are you sure the usage of `$orderdetail->findPriceForQty(1)` targeting `App\Entity\PriceInformat...tail::findPriceForQty()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
59			$this->assertEquals($price1, $orderdetail->findPriceForQty(1.5));
			0 ignored issues – show Bug introduced 2019-11-09 13:40 UTC by Report Bug Copy Issue Report Are you sure the usage of `$orderdetail->findPriceForQty(1.5)` targeting `App\Entity\PriceInformat...tail::findPriceForQty()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
60			$this->assertEquals($price5, $orderdetail->findPriceForQty(5.3));
			0 ignored issues – show Bug introduced 2019-11-09 13:40 UTC by Report Bug Copy Issue Report Are you sure the usage of `$orderdetail->findPriceForQty(5.3)` targeting `App\Entity\PriceInformat...tail::findPriceForQty()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
61			$this->assertEquals($price5, $orderdetail->findPriceForQty(10000));
			0 ignored issues – show Bug introduced 2019-11-09 13:40 UTC by Report Bug Copy Issue Report Are you sure the usage of `$orderdetail->findPriceForQty(10000)` targeting `App\Entity\PriceInformat...tail::findPriceForQty()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
62			}
63			}
64

Part-DB / Part-DB-symfony

Push — master ( d286b7...24e4a5 )

OrderdetailTest::testFindPriceForQty() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like