DVDoug\BoxPacker\PackedBoxTest

Complexity

Total Complexity

2

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	5

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	testGetters()	0	15	1
A	testVolumeUtilisation()	0	12	1

<?php
/**
 * Box packing (3D bin packing, knapsack problem)
 * @package BoxPacker
 * @author Doug Wright
 */

namespace DVDoug\BoxPacker;

use DVDoug\BoxPacker\Test\TestBox;
use DVDoug\BoxPacker\Test\TestItem;
use PHPUnit\Framework\TestCase;

class PackedBoxTest extends TestCase
{
    function testGetters()

It is generally recommended to explicitly declare the visibility for methods.

    {
        $box = new TestBox('Box', 370, 375, 60, 140, 364, 374, 40, 3000);
        $item = new TestItem('Item', 230, 330, 6, 320, true);

true is of type boolean, but the function expects a integer.

        $boxItems = new ItemList();
        $boxItems->insert($item);

        $packedBox = new PackedBox($box, $boxItems, 1, 2, 3, 4, 0, 0, 0);

        self::assertEquals(1, $packedBox->getRemainingWidth());
        self::assertEquals(2, $packedBox->getRemainingLength());
        self::assertEquals(3, $packedBox->getRemainingDepth());
        self::assertEquals(4, $packedBox->getRemainingWeight());
    }

    function testVolumeUtilisation()

It is generally recommended to explicitly declare the visibility for methods.

    {
        $box = new TestBox('Box', 10, 10, 10, 10, 10, 10, 10, 10);
        $item = new TestItem('Item', 5, 10, 10, 10, true);

true is of type boolean, but the function expects a integer.

        $boxItems = new ItemList();
        $boxItems->insert($item);

        $packedBox = new PackedBox($box, $boxItems, 1, 2, 3, 4, 0, 0, 0);

        self::assertEquals(50, $packedBox->getVolumeUtilisation());
    }
}