Passed
Pull Request — master (#72)
by Chris
05:31 queued 02:59
created

BasketTest   A

Complexity

Total Complexity 7

Size/Duplication

Total Lines 140
Duplicated Lines 28.57 %

Importance

Changes 0
Metric Value
dl 40
loc 140
rs 10
c 0
b 0
f 0
wmc 7

7 Methods

Rating   Name   Duplication   Size   Complexity  
A testGetTotalNoDiscount() 0 8 1
A provideGetTotalNoDiscount() 17 17 1
A testGetTotalWithDiscount() 0 9 1
A provideGetTotalWithDiscount() 17 17 1
A __construct() 0 13 1
B provideContainingOnly() 0 25 1
A testContainingOnly() 0 10 1

How to fix   Duplicated Code   

Duplicated Code

Duplicate code is one of the most pungent code smells. A rule that is often used is to re-structure code once it is duplicated in three or more places.

Common duplication problems, and corresponding solutions are:

1
<?php
2
3
namespace ConferenceTools\Tickets\Domain\ValueObject;
4
5
use ConferenceTools\Tickets\Domain\Service\Configuration;
6
use ConferenceTools\Tickets\Domain\ValueObject\DiscountType\Percentage;
7
8
class BasketTest extends \PHPUnit\Framework\TestCase
9
{
10
    /**
11
     * @var Configuration
12
     */
13
    private $config;
14
15
    public function __construct($name = null, array $data = array(), $dataName = '')
16
    {
17
        parent::__construct($name, $data, $dataName);
18
        $this->config = Configuration::fromArray([
19
            'tickets' => [
20
                'early' => ['name' => 'Early Bird', 'cost' => 5000, 'available' => 75],
21
                'std' => ['name' => 'Standard', 'cost' => 10000, 'available' => 150],
22
                'free' => ['name' => 'Free', 'cost' => 0, 'available' => 0]
23
            ],
24
            'financial' => [
25
                'taxRate' => 10,
26
                'currency' => 'GBP',
27
                'displayTax' => true
28
            ]
29
        ]);
30
    }
31
32
    /**
33
     * @dataProvider provideGetTotalNoDiscount
34
     * @param array $reservations
35
     */
36
    public function testGetTotalNoDiscount(array $reservations, Price $expected)
37
    {
38
        $sut = Basket::fromReservations(
39
            $this->config,
40
            ... $reservations
41
        );
42
43
        $this->assertTrue($expected->equals($sut->getTotal()), 'Total didn\'t match expected total');
44
    }
45
46 View Code Duplication
    public function provideGetTotalNoDiscount()
0 ignored issues
show
Duplication introduced by
This method seems to be duplicated in your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
47
    {
48
        $stdReservation = new TicketReservation($this->config->getTicketType('std'), 'abc');
49
        $earlyReservation = new TicketReservation($this->config->getTicketType('early'), 'abc');
50
51
        return [
52
            [
53
                [$stdReservation],
54
                Price::fromNetCost(new Money(10000, $this->config->getCurrency()), $this->config->getTaxRate())
55
            ],
56
            [
57
                [$stdReservation, $stdReservation],
58
                Price::fromNetCost(new Money(20000, $this->config->getCurrency()), $this->config->getTaxRate())
59
            ],
60
            [
61
                [$stdReservation, $earlyReservation],
62
                Price::fromNetCost(new Money(15000, $this->config->getCurrency()), $this->config->getTaxRate())
63
            ]
64
        ];
65
    }
66
67
    /**
68
     * @dataProvider provideGetTotalWithDiscount
69
     * @param array $reservations
70
     */
71
    public function testGetTotalWithDiscount(array $reservations, Price $expected)
72
    {
73
        $sut = Basket::fromReservationsWithDiscount(
74
            $this->config,
75
            new DiscountCode('50off', '50% off', new Percentage(50)),
76
            ... $reservations
77
        );
78
79
        $this->assertTrue($expected->equals($sut->getTotal()), 'Total didn\'t match expected total');
80
    }
81
82 View Code Duplication
    public function provideGetTotalWithDiscount()
0 ignored issues
show
Duplication introduced by
This method seems to be duplicated in your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
83
    {
84
        $stdReservation = new TicketReservation($this->config->getTicketType('std'), 'abc');
85
        $earlyReservation = new TicketReservation($this->config->getTicketType('early'), 'abc');
86
87
        return [
88
            [
89
                [$stdReservation],
90
                Price::fromNetCost(new Money(5000, $this->config->getCurrency()), $this->config->getTaxRate())
91
            ],
92
            [
93
                [$stdReservation, $stdReservation],
94
                Price::fromNetCost(new Money(10000, $this->config->getCurrency()), $this->config->getTaxRate())
95
            ],
96
            [
97
                [$stdReservation, $earlyReservation],
98
                Price::fromNetCost(new Money(7500, $this->config->getCurrency()), $this->config->getTaxRate())
99
            ]
100
        ];
101
    }
102
103
    /**
104
     * @dataProvider provideContainingOnly
105
     *
106
     * @param $reservations
107
     * @param $ticketTypes
108
     * @param $expected
109
     */
110
    public function testContainingOnly($reservations, $ticketTypes, $expected)
111
    {
112
        $sut = Basket::fromReservations(
113
            $this->config,
114
            ... $reservations
115
        );
116
117
        $result = $sut->containingOnly(...$ticketTypes);
118
119
        $this->assertEquals($expected, array_values($result->getTickets()));
120
121
    }
122
123
    public function provideContainingOnly()
124
    {
125
        $stdReservation = new TicketReservation($this->config->getTicketType('std'), 'abc');
126
        $earlyReservation = new TicketReservation($this->config->getTicketType('early'), 'abc');
127
128
        return [
129
            [
130
                [$stdReservation],
131
                [$this->config->getTicketType('std')],
132
                [$stdReservation]
133
            ],
134
            [
135
                [$stdReservation, $stdReservation],
136
                [$this->config->getTicketType('std')],
137
                [$stdReservation, $stdReservation],
138
            ],
139
            [
140
                [$stdReservation, $earlyReservation],
141
                [$this->config->getTicketType('std')],
142
                [$stdReservation],
143
            ],
144
            [
145
                [$stdReservation, $earlyReservation],
146
                [$this->config->getTicketType('std'), $this->config->getTicketType('early')],
147
                [$stdReservation, $earlyReservation],
148
            ]
149
        ];
150
    }
151
}