MigrationTest::testGetFileContent_1() - Code Metrics - Inspection of "Next Revision" - byjg/migration - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#31)

by Joao

created 2021-01-17 00:40 UTC

MigrationTest::testGetFileContent_1() A

↳ Parent: MigrationTest

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	13
Ratio	100 %

Importance

Changes

Metric	Value
dl	13
loc	13
rs	9.8333
c	0
b	0
f	0
cc	1
nc	1
nop	0

<?php
namespace Test;

use ByJG\DbMigration\Migration;
use ByJG\Util\Uri;

class MigrationTest extends \PHPUnit\Framework\TestCase
{
    /**
     * @var Migration
     */
    protected $object;

    public function setUp()
    {
        $this->object = new Migration(new Uri('mysql://localhost'), __DIR__ . '/dirstructure');
    }

    public function tearDown()
    {
        $this->object = null;
    }

    public function testGetBaseSql()
    {
        $base = $this->object->getBaseSql();
        $this->assertEquals(__DIR__ . '/dirstructure/base.sql', $base);
    }

    /**
     * @expectedException \ByJG\DbMigration\Exception\InvalidMigrationFile
     */
    public function testGetBaseSqlNotFound()
    {
        $this->object = new Migration(new Uri('mysql://localhost'), __DIR__ . '/invalid');
        $this->object->getBaseSql();
class User
{
    private $email;

    public function getEmail()
    {
        return $this->email;
    }

    public function setEmail($email)
    {
        $this->email = $email;
    }
}
    }

    public function testGetBaseSqlNotFoundAndNotRequired()
    {
        $this->object = new Migration(new Uri('mysql://localhost'), __DIR__ . '/invalid', false);
        $this->object->getBaseSql();
class User
{
    private $email;

    public function getEmail()
    {
        return $this->email;
    }

    public function setEmail($email)
    {
        $this->email = $email;
    }
}
        $this->assertTrue(true);
    }

    public function testGetMigrationSql1()
    {
        $version = $this->object->getMigrationSql(1, 1);
        $this->assertEquals(__DIR__ . '/dirstructure/migrations/up/00001.sql', $version);
    }

    public function testGetMigrationSql2()
    {
        $version = $this->object->getMigrationSql(2, 1);
        $this->assertEquals(__DIR__ . '/dirstructure/migrations/up/00002.sql', $version);
    }

    public function testGetMigrationSql3()
    {
        $version = $this->object->getMigrationSql(12, 1);
        $this->assertEquals(__DIR__ . '/dirstructure/migrations/up/00012-dev.sql', $version);
    }

    /**
     * @expectedException \ByJG\DbMigration\Exception\InvalidMigrationFile
     * @expectedExceptionMessage version number '13'
     */
    public function testGetMigrationSql4()
    {
        $this->object->getMigrationSql(13, 1);
    }

    public function testGetMigrationSqlDown1()
    {
        $version = $this->object->getMigrationSql(1, -1);
        $this->assertEquals(__DIR__ . '/dirstructure/migrations/down/00001.sql', $version);
    }

    public function testGetMigrationSqlDown2()
    {
        $version = $this->object->getMigrationSql(2, -1);
        $this->assertEquals(__DIR__ . '/dirstructure/migrations/down/00002.sql', $version);
    }

    public function testGetMigrationSqlDown3()
    {
        $version = $this->object->getMigrationSql(12, -1);
        $this->assertEquals(__DIR__ . '/dirstructure/migrations/down/00012-dev.sql', $version);
    }

    /**
     * @expectedException \ByJG\DbMigration\Exception\InvalidMigrationFile
     * @expectedExceptionMessage version number '13'
     */
    public function testGetMigrationSqlDown4()
    {
        $this->object->getMigrationSql(13, -1);
    }

    public function testGetFileContent_NonExists()
    {
        $this->assertEquals(
            [
                "file" => "non-existent",
                "description" => "no description provided. Pro tip: use `-- @description:` to define one.",
                "exists" => false,
                "checksum" => null,
                "content" => null,
            ],
            $this->object->getFileContent("non-existent")
        );
    }

    public function testGetFileContent_1()

    {
        $this->assertEquals(
            [
                "file" => __DIR__ . '/dirstructure/migrations/up/00001.sql',
                "description" => "this is a test",
                "exists" => true,
                "checksum" => "b937afa57e363c9244fa30844dd11d312694f697",
                "content" => "-- @description: this is a test\n\nselect * from mysql.users;\n",
            ],
            $this->object->getFileContent(__DIR__ . '/dirstructure/migrations/up/00001.sql')
        );
    }

    public function testGetFileContent_2()

    {
        $this->assertEquals(
            [
                "file" => __DIR__ . '/dirstructure/migrations/up/00002.sql',
                "description" => "another test",
                "exists" => true,
                "checksum" => "fd8ab8176291c2dcbf0d91564405e0f98f0cd77e",
                "content" => "--   @description:  another test\n\nselect * from dual;",
            ],
            $this->object->getFileContent(__DIR__ . '/dirstructure/migrations/up/00002.sql')
        );
    }

    public function testGetFileContent_3()

    {
        $this->assertEquals(
            [
                "file" => __DIR__ . '/dirstructure/migrations/up/00003.sql',
                "description" => "no description provided. Pro tip: use `-- @description:` to define one.",
                "exists" => true,
                "checksum" => "73faaa68e2f60c11e75a9ccc18528e0ffa15127a",
                "content" => "select something from sometable;",
            ],
            $this->object->getFileContent(__DIR__ . '/dirstructure/migrations/up/00003.sql')
        );
    }
}


1		<?php
2		namespace Test;
3
4		use ByJG\DbMigration\Migration;
5		use ByJG\Util\Uri;
6
7		class MigrationTest extends \PHPUnit\Framework\TestCase
8		{
9		/**
10		* @var Migration
11		*/
12		protected $object;
13
14		public function setUp()
15		{
16		$this->object = new Migration(new Uri('mysql://localhost'), __DIR__ . '/dirstructure');
17		}
18
19		public function tearDown()
20		{
21		$this->object = null;
22		}
23
24		public function testGetBaseSql()
25		{
26		$base = $this->object->getBaseSql();
27		$this->assertEquals(__DIR__ . '/dirstructure/base.sql', $base);
28		}
29
30		/**
31		* @expectedException \ByJG\DbMigration\Exception\InvalidMigrationFile
32		*/
33		public function testGetBaseSqlNotFound()
34		{
35		$this->object = new Migration(new Uri('mysql://localhost'), __DIR__ . '/invalid');
36		$this->object->getBaseSql();
		0 ignored issues – show Unused Code introduced 2018-03-19 16:22 UTC by Report Bug Copy Issue Report The call to the method `ByJG\DbMigration\Migration::getBaseSql()` seems un-needed as the method has no side-effects. PHP Analyzer performs a side-effects analysis of your code. A side-effect is basically anything that might be visible after the scope of the method is left. Let’s take a look at an example: class User { private $email; public function getEmail() { return $this->email; } public function setEmail($email) { $this->email = $email; } } If we look at the `getEmail()` method, we can see that it has no side-effect. Whether you call this method or not, no future calls to other methods are affected by this. As such code as the following is useless: $user = new User(); $user->getEmail(); // This line could safely be removed as it has no effect. On the hand, if we look at the `setEmail()`, this method _has_ side-effects. In the following case, we could not remove the method call: $user = new User(); $user->setEmail('email@domain'); // This line has a side-effect (it changes an // instance variable). Loading history...
37		}
38
39		public function testGetBaseSqlNotFoundAndNotRequired()
40		{
41		$this->object = new Migration(new Uri('mysql://localhost'), __DIR__ . '/invalid', false);
42		$this->object->getBaseSql();
		0 ignored issues – show Unused Code introduced 2017-11-10 16:26 UTC by Report Bug Copy Issue Report The call to the method `ByJG\DbMigration\Migration::getBaseSql()` seems un-needed as the method has no side-effects. PHP Analyzer performs a side-effects analysis of your code. A side-effect is basically anything that might be visible after the scope of the method is left. Let’s take a look at an example: class User { private $email; public function getEmail() { return $this->email; } public function setEmail($email) { $this->email = $email; } } If we look at the `getEmail()` method, we can see that it has no side-effect. Whether you call this method or not, no future calls to other methods are affected by this. As such code as the following is useless: $user = new User(); $user->getEmail(); // This line could safely be removed as it has no effect. On the hand, if we look at the `setEmail()`, this method _has_ side-effects. In the following case, we could not remove the method call: $user = new User(); $user->setEmail('email@domain'); // This line has a side-effect (it changes an // instance variable). Loading history...
43		$this->assertTrue(true);
44		}
45
46		public function testGetMigrationSql1()
47		{
48		$version = $this->object->getMigrationSql(1, 1);
49		$this->assertEquals(__DIR__ . '/dirstructure/migrations/up/00001.sql', $version);
50		}
51
52		public function testGetMigrationSql2()
53		{
54		$version = $this->object->getMigrationSql(2, 1);
55		$this->assertEquals(__DIR__ . '/dirstructure/migrations/up/00002.sql', $version);
56		}
57
58		public function testGetMigrationSql3()
59		{
60		$version = $this->object->getMigrationSql(12, 1);
61		$this->assertEquals(__DIR__ . '/dirstructure/migrations/up/00012-dev.sql', $version);
62		}
63
64		/**
65		* @expectedException \ByJG\DbMigration\Exception\InvalidMigrationFile
66		* @expectedExceptionMessage version number '13'
67		*/
68		public function testGetMigrationSql4()
69		{
70		$this->object->getMigrationSql(13, 1);
71		}
72
73		public function testGetMigrationSqlDown1()
74		{
75		$version = $this->object->getMigrationSql(1, -1);
76		$this->assertEquals(__DIR__ . '/dirstructure/migrations/down/00001.sql', $version);
77		}
78
79		public function testGetMigrationSqlDown2()
80		{
81		$version = $this->object->getMigrationSql(2, -1);
82		$this->assertEquals(__DIR__ . '/dirstructure/migrations/down/00002.sql', $version);
83		}
84
85		public function testGetMigrationSqlDown3()
86		{
87		$version = $this->object->getMigrationSql(12, -1);
88		$this->assertEquals(__DIR__ . '/dirstructure/migrations/down/00012-dev.sql', $version);
89		}
90
91		/**
92		* @expectedException \ByJG\DbMigration\Exception\InvalidMigrationFile
93		* @expectedExceptionMessage version number '13'
94		*/
95		public function testGetMigrationSqlDown4()
96		{
97		$this->object->getMigrationSql(13, -1);
98		}
99
100		public function testGetFileContent_NonExists()
101		{
102		$this->assertEquals(
103		[
104		"file" => "non-existent",
105		"description" => "no description provided. Pro tip: use `-- @description:` to define one.",
106		"exists" => false,
107		"checksum" => null,
108		"content" => null,
109		],
110		$this->object->getFileContent("non-existent")
111		);
112		}
113
114	View Code Duplication	public function testGetFileContent_1()
		0 ignored issues – show Duplication introduced 2020-09-29 07:11 UTC by Report Bug Copy Issue Report 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...
115		{
116		$this->assertEquals(
117		[
118		"file" => __DIR__ . '/dirstructure/migrations/up/00001.sql',
119		"description" => "this is a test",
120		"exists" => true,
121		"checksum" => "b937afa57e363c9244fa30844dd11d312694f697",
122		"content" => "-- @description: this is a test\n\nselect * from mysql.users;\n",
123		],
124		$this->object->getFileContent(__DIR__ . '/dirstructure/migrations/up/00001.sql')
125		);
126		}
127
128	View Code Duplication	public function testGetFileContent_2()
		0 ignored issues – show Duplication introduced 2020-09-29 07:11 UTC by Report Bug Copy Issue Report 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...
129		{
130		$this->assertEquals(
131		[
132		"file" => __DIR__ . '/dirstructure/migrations/up/00002.sql',
133		"description" => "another test",
134		"exists" => true,
135		"checksum" => "fd8ab8176291c2dcbf0d91564405e0f98f0cd77e",
136		"content" => "-- @description: another test\n\nselect * from dual;",
137		],
138		$this->object->getFileContent(__DIR__ . '/dirstructure/migrations/up/00002.sql')
139		);
140		}
141
142	View Code Duplication	public function testGetFileContent_3()
		0 ignored issues – show Duplication introduced 2020-09-29 07:11 UTC by Report Bug Copy Issue Report 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...
143		{
144		$this->assertEquals(
145		[
146		"file" => __DIR__ . '/dirstructure/migrations/up/00003.sql',
147		"description" => "no description provided. Pro tip: use `-- @description:` to define one.",
148		"exists" => true,
149		"checksum" => "73faaa68e2f60c11e75a9ccc18528e0ffa15127a",
150		"content" => "select something from sometable;",
151		],
152		$this->object->getFileContent(__DIR__ . '/dirstructure/migrations/up/00003.sql')
153		);
154		}
155		}
156

byjg / migration

Pull Request — master (#31)

MigrationTest::testGetFileContent_1() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like