CommitTest::testCommitRegEx() - Code Metrics - Inspection of "Merge branch 'develop'" - matteosister/GitElephant - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( a6409c...f9cf0d )

unknown

created 2018-04-30 15:27 UTC

CommitTest::testCommitRegEx() B

↳ Parent: CommitTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	46
Code Lines	36

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	46
rs	8.9411
c	0
b	0
f	0
cc	1
eloc	36
nc	1
nop	0

<?php

/**
 * This file is part of the GitElephant package.
 *
 * (c) Matteo Giachino <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 *
 * Just for fun...
 */

namespace GitElephant\Objects;

use \GitElephant\TestCase;
use \GitElephant\Objects\Commit;
use \GitElephant\Command\ShowCommand;
use \GitElephant\Command\MainCommand;
use \GitElephant\Command\CommandContainer;

/**
 * CommitTest
 *
 * @author Matteo Giachino <[email protected]>
 */

class CommitTest extends TestCase
{
    /**
     * @var \GitElephant\Objects\Commit;
     */
    private $commit;

    public function setUp()
    {
        $this->initRepository();
        $mainCommand = new MainCommand();
        $this->getCaller()->execute($mainCommand->init());
        $this->addFile('foo');
        $this->getCaller()->execute($mainCommand->add());
        $this->getCaller()->execute($mainCommand->commit('first commit'));
    }

    /**
     * commit tests
     */
    public function testCommit()
    {
        $showCommand = new ShowCommand();
        $this->commit = Commit::pick($this->getRepository());
        $this->assertInstanceOf('\GitElephant\Objects\Commit', $this->commit);
        $this->assertInstanceOf('\GitElephant\Objects\Author', $this->commit->getAuthor());
        $this->assertInstanceOf('\GitElephant\Objects\Author', $this->commit->getCommitter());
        $this->assertInstanceOf('\Datetime', $this->commit->getDatetimeAuthor());
        $this->assertInstanceOf('\Datetime', $this->commit->getDatetimeCommitter());
        $this->assertInstanceOf('\GitElephant\Objects\Commit\Message', $this->commit->getMessage());
        $this->assertEquals('first commit', $this->commit->getMessage()->toString());
        $this->assertRegExp('/^\w{40}$/', $this->commit->getSha());
        $this->assertEquals(array(), $this->commit->getParents());
        $this->addFile('foo2');
        $mainCommand = new MainCommand();
        $this->getCaller()->execute($mainCommand->add());
        $this->getCaller()->execute($mainCommand->commit('second commit'));
        $this->getCaller()->execute($showCommand->showCommit('HEAD'));
        $this->commit = Commit::pick($this->getRepository());
        $parents = $this->commit->getParents();
        $this->assertRegExp('/^\w{40}$/', $parents[0]);
    }

    /**
     * constructor regex test
     */
    public function testCommitRegEx()
    {
        $outputLines = array(
            "commit c277373174aa442af12a8e59de1812f3472c15f5",
            "tree 9d36a2d3c5d5bce9c6779a574ed2ba3d274d8016",
            "author matt <[email protected]> 1326214449 +0100",
            "committer matt <[email protected]> 1326214449 +0100",
            "",
            "    first commit"
        );

        $mockRepo = $this->getMockRepository();
        $this->addOutputToMockRepo($mockRepo, $outputLines);

        $commit = Commit::pick($mockRepo);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $committer = $commit->getCommitter();
        $author = $commit->getAuthor();
        $this->assertEquals('matt', $committer->getName());
        $this->assertEquals('matt', $author->getName());

        $outputLines = array(
            "commit c277373174aa442af12a8e59de1812f3472c15f5",
            "tree 9d36a2d3c5d5bce9c6779a574ed2ba3d274d8016",
            "author matt jack <[email protected]> 1326214449 +0100",
            "committer matt jack <[email protected]> 1326214449 +0100",
            "",
            "    first commit"
        );

        $mockRepo = $this->getMockRepository();
        $this->addOutputToMockRepo($mockRepo, $outputLines);

        $commit = Commit::pick($mockRepo);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->doCommitTest(
            $commit,
            'c277373174aa442af12a8e59de1812f3472c15f5',
            '9d36a2d3c5d5bce9c6779a574ed2ba3d274d8016',
            'matt jack',
            'matt jack',
            '[email protected]',
            '[email protected]',
            '1326214449',
            '1326214449',
            'first commit'
        );
    }

    /**
     * testCommitDate
     */
    public function testCommitDate()
    {
        $outputLines = array(
            "commit c277373174aa442af12a8e59de1812f3472c15f5",
            "tree c277373174aa442af12a8e59de1812f3472c15f6",
            "author John Doe <[email protected]> 1326214449 +0100",
            "committer Jack Doe <[email protected]> 1326214449 +0100",
            "",
            "    First commit"
        );

        $mockRepo = $this->getMockRepository();
        $this->addOutputToMockRepo($mockRepo, $outputLines);

        $commit = Commit::pick($mockRepo);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->doCommitTest(
            $commit,
            'c277373174aa442af12a8e59de1812f3472c15f5',
            'c277373174aa442af12a8e59de1812f3472c15f6',
            'John Doe',
            'Jack Doe',
            '[email protected]',
            '[email protected]',
            '1326214449',
            '1326214449',
            'First commit'
        );
    }

    /**
     * testCreateFromOutputLines
     */
    public function testCreateFromOutputLines()
    {
        $outputLines = array(
            "commit c277373174aa442af12a8e59de1812f3472c15f5",
            "tree 9d36a2d3c5d5bce9c6779a574ed2ba3d274d8016",
            "author John Doe <[email protected]> 1326214000 +0100",
            "committer Jack Doe <[email protected]> 1326214100 +0100",
            "",
            "    Initial commit"
        );

        $commit = Commit::createFromOutputLines($this->getRepository(), $outputLines);
        $this->doCommitTest(
            $commit,
            'c277373174aa442af12a8e59de1812f3472c15f5',
            '9d36a2d3c5d5bce9c6779a574ed2ba3d274d8016',
            'John Doe',
            'Jack Doe',
            '[email protected]',
            '[email protected]',
            '1326214000',
            '1326214100',
            'Initial commit'
        );
    }

    /**
     * testCreate
     */
    public function testCreate()
    {
        $this->getRepository()->init();
        $this->addFile('test');
        $this->repository->stage();
        $commit = Commit::create($this->repository, 'first commit', true);
        $this->assertEquals($commit, $this->repository->getCommit());
    }

    /**
     * testGetDiff
     */
    public function testGetDiff()
    {
        $this->getRepository()->init();
        $this->addFile('test');
        $this->repository->stage();
        $commit = Commit::create($this->repository, 'first commit', true);
        $diff = $commit->getDiff();
        $this->assertInstanceOf('GitElephant\Objects\Diff\Diff', $diff);
        $this->assertCount(1, $diff);

        $this->addFile('test2');
        $this->addFile('test3');
        $this->repository->stage();
        $commit = Commit::create($this->repository, 'second commit', true);
        $diff = $commit->getDiff();
        $this->assertInstanceOf('GitElephant\Objects\Diff\Diff', $diff);
        $this->assertCount(2, $diff);
    }

    public function testRevParse()
    {
        $this->getRepository()->init();
        $this->addFile('test');
        $this->repository->stage();
        $commit = Commit::create($this->repository, 'first commit', true);

        $revParse = $commit->revParse();
        $this->assertEquals($commit->getSha(), $revParse[0]);
    }

    public function testCommitWithoutTag()
    {
        $this->getRepository()->init();
        $this->addFile('test');
        $this->repository->stage();
        $commit = Commit::create($this->repository, 'first commit', true);
        $this->assertFalse($commit->tagged());
    }

    public function testCommitWithTag()
    {
        $this->getRepository()->init();
        $this->addFile('test');
        $this->repository->stage();
        $commit = Commit::create($this->repository, 'first commit', true);
        Tag::create($this->repository, '1.0.0');
        $this->assertTrue($commit->tagged());
        $this->assertCount(1, $commit->getTags());
        $commitTags = $commit->getTags();
        $this->assertEquals('1.0.0', $commitTags[0]->getName());
    }

}


1			<?php
2
3			/**
4			* This file is part of the GitElephant package.
5			*
6			* (c) Matteo Giachino <[email protected]>
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*
11			* Just for fun...
12			*/
13
14			namespace GitElephant\Objects;
15
16			use \GitElephant\TestCase;
17			use \GitElephant\Objects\Commit;
18			use \GitElephant\Command\ShowCommand;
19			use \GitElephant\Command\MainCommand;
20			use \GitElephant\Command\CommandContainer;
21
22			/**
23			* CommitTest
24			*
25			* @author Matteo Giachino <[email protected]>
26			*/
27
28			class CommitTest extends TestCase
29			{
30			/**
31			* @var \GitElephant\Objects\Commit;
32			*/
33			private $commit;
34
35			public function setUp()
36			{
37			$this->initRepository();
38			$mainCommand = new MainCommand();
39			$this->getCaller()->execute($mainCommand->init());
40			$this->addFile('foo');
41			$this->getCaller()->execute($mainCommand->add());
42			$this->getCaller()->execute($mainCommand->commit('first commit'));
43			}
44
45			/**
46			* commit tests
47			*/
48			public function testCommit()
49			{
50			$showCommand = new ShowCommand();
51			$this->commit = Commit::pick($this->getRepository());
52			$this->assertInstanceOf('\GitElephant\Objects\Commit', $this->commit);
53			$this->assertInstanceOf('\GitElephant\Objects\Author', $this->commit->getAuthor());
54			$this->assertInstanceOf('\GitElephant\Objects\Author', $this->commit->getCommitter());
55			$this->assertInstanceOf('\Datetime', $this->commit->getDatetimeAuthor());
56			$this->assertInstanceOf('\Datetime', $this->commit->getDatetimeCommitter());
57			$this->assertInstanceOf('\GitElephant\Objects\Commit\Message', $this->commit->getMessage());
58			$this->assertEquals('first commit', $this->commit->getMessage()->toString());
59			$this->assertRegExp('/^\w{40}$/', $this->commit->getSha());
60			$this->assertEquals(array(), $this->commit->getParents());
61			$this->addFile('foo2');
62			$mainCommand = new MainCommand();
63			$this->getCaller()->execute($mainCommand->add());
64			$this->getCaller()->execute($mainCommand->commit('second commit'));
65			$this->getCaller()->execute($showCommand->showCommit('HEAD'));
66			$this->commit = Commit::pick($this->getRepository());
67			$parents = $this->commit->getParents();
68			$this->assertRegExp('/^\w{40}$/', $parents[0]);
69			}
70
71			/**
72			* constructor regex test
73			*/
74			public function testCommitRegEx()
75			{
76			$outputLines = array(
77			"commit c277373174aa442af12a8e59de1812f3472c15f5",
78			"tree 9d36a2d3c5d5bce9c6779a574ed2ba3d274d8016",
79			"author matt <[email protected]> 1326214449 +0100",
80			"committer matt <[email protected]> 1326214449 +0100",
81			"",
82			" first commit"
83			);
84
85			$mockRepo = $this->getMockRepository();
86			$this->addOutputToMockRepo($mockRepo, $outputLines);
87
88			$commit = Commit::pick($mockRepo);
			0 ignored issues – show Documentation introduced 2018-03-16 16:26 UTC by Report Bug Copy Issue Report `$mockRepo` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<GitElephant\Repository>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
89			$committer = $commit->getCommitter();
90			$author = $commit->getAuthor();
91			$this->assertEquals('matt', $committer->getName());
92			$this->assertEquals('matt', $author->getName());
93
94			$outputLines = array(
95			"commit c277373174aa442af12a8e59de1812f3472c15f5",
96			"tree 9d36a2d3c5d5bce9c6779a574ed2ba3d274d8016",
97			"author matt jack <[email protected]> 1326214449 +0100",
98			"committer matt jack <[email protected]> 1326214449 +0100",
99			"",
100			" first commit"
101			);
102
103			$mockRepo = $this->getMockRepository();
104			$this->addOutputToMockRepo($mockRepo, $outputLines);
105
106			$commit = Commit::pick($mockRepo);
			0 ignored issues – show Documentation introduced 2018-03-16 16:26 UTC by Report Bug Copy Issue Report `$mockRepo` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<GitElephant\Repository>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
107			$this->doCommitTest(
108			$commit,
109			'c277373174aa442af12a8e59de1812f3472c15f5',
110			'9d36a2d3c5d5bce9c6779a574ed2ba3d274d8016',
111			'matt jack',
112			'matt jack',
113			'[email protected]',
114			'[email protected]',
115			'1326214449',
116			'1326214449',
117			'first commit'
118			);
119			}
120
121			/**
122			* testCommitDate
123			*/
124			public function testCommitDate()
125			{
126			$outputLines = array(
127			"commit c277373174aa442af12a8e59de1812f3472c15f5",
128			"tree c277373174aa442af12a8e59de1812f3472c15f6",
129			"author John Doe <[email protected]> 1326214449 +0100",
130			"committer Jack Doe <[email protected]> 1326214449 +0100",
131			"",
132			" First commit"
133			);
134
135			$mockRepo = $this->getMockRepository();
136			$this->addOutputToMockRepo($mockRepo, $outputLines);
137
138			$commit = Commit::pick($mockRepo);
			0 ignored issues – show Documentation introduced 2018-03-16 16:26 UTC by Report Bug Copy Issue Report `$mockRepo` is of type `object<PHPUnit\Framework\MockObject\MockObject>`, but the function expects a `object<GitElephant\Repository>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
139			$this->doCommitTest(
140			$commit,
141			'c277373174aa442af12a8e59de1812f3472c15f5',
142			'c277373174aa442af12a8e59de1812f3472c15f6',
143			'John Doe',
144			'Jack Doe',
145			'[email protected]',
146			'[email protected]',
147			'1326214449',
148			'1326214449',
149			'First commit'
150			);
151			}
152
153			/**
154			* testCreateFromOutputLines
155			*/
156			public function testCreateFromOutputLines()
157			{
158			$outputLines = array(
159			"commit c277373174aa442af12a8e59de1812f3472c15f5",
160			"tree 9d36a2d3c5d5bce9c6779a574ed2ba3d274d8016",
161			"author John Doe <[email protected]> 1326214000 +0100",
162			"committer Jack Doe <[email protected]> 1326214100 +0100",
163			"",
164			" Initial commit"
165			);
166
167			$commit = Commit::createFromOutputLines($this->getRepository(), $outputLines);
168			$this->doCommitTest(
169			$commit,
170			'c277373174aa442af12a8e59de1812f3472c15f5',
171			'9d36a2d3c5d5bce9c6779a574ed2ba3d274d8016',
172			'John Doe',
173			'Jack Doe',
174			'[email protected]',
175			'[email protected]',
176			'1326214000',
177			'1326214100',
178			'Initial commit'
179			);
180			}
181
182			/**
183			* testCreate
184			*/
185			public function testCreate()
186			{
187			$this->getRepository()->init();
188			$this->addFile('test');
189			$this->repository->stage();
190			$commit = Commit::create($this->repository, 'first commit', true);
191			$this->assertEquals($commit, $this->repository->getCommit());
192			}
193
194			/**
195			* testGetDiff
196			*/
197			public function testGetDiff()
198			{
199			$this->getRepository()->init();
200			$this->addFile('test');
201			$this->repository->stage();
202			$commit = Commit::create($this->repository, 'first commit', true);
203			$diff = $commit->getDiff();
204			$this->assertInstanceOf('GitElephant\Objects\Diff\Diff', $diff);
205			$this->assertCount(1, $diff);
206
207			$this->addFile('test2');
208			$this->addFile('test3');
209			$this->repository->stage();
210			$commit = Commit::create($this->repository, 'second commit', true);
211			$diff = $commit->getDiff();
212			$this->assertInstanceOf('GitElephant\Objects\Diff\Diff', $diff);
213			$this->assertCount(2, $diff);
214			}
215
216			public function testRevParse()
217			{
218			$this->getRepository()->init();
219			$this->addFile('test');
220			$this->repository->stage();
221			$commit = Commit::create($this->repository, 'first commit', true);
222
223			$revParse = $commit->revParse();
224			$this->assertEquals($commit->getSha(), $revParse[0]);
225			}
226
227			public function testCommitWithoutTag()
228			{
229			$this->getRepository()->init();
230			$this->addFile('test');
231			$this->repository->stage();
232			$commit = Commit::create($this->repository, 'first commit', true);
233			$this->assertFalse($commit->tagged());
234			}
235
236			public function testCommitWithTag()
237			{
238			$this->getRepository()->init();
239			$this->addFile('test');
240			$this->repository->stage();
241			$commit = Commit::create($this->repository, 'first commit', true);
242			Tag::create($this->repository, '1.0.0');
243			$this->assertTrue($commit->tagged());
244			$this->assertCount(1, $commit->getTags());
245			$commitTags = $commit->getTags();
246			$this->assertEquals('1.0.0', $commitTags[0]->getName());
247			}
248
249			}
250

matteosister / GitElephant

Push — master ( a6409c...f9cf0d )

CommitTest::testCommitRegEx() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like