MatchesSnapshots::getSnapshotTestName() - Code Metrics - Inspection of "Updated public-ish api" - spatie/phpunit-snapshot-assertions - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 7fd26f...360cbc )

by Sebastian

created 2017-03-27 11:45 UTC

MatchesSnapshots::getSnapshotTestName() A

↳ Parent: MatchesSnapshots

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	4
rs	10
c	0
b	0
f	0
cc	1
eloc	2
nc	1
nop	0

<?php

namespace Spatie\Snapshots;

use PHPUnit\Framework\ExpectationFailedException;
use ReflectionClass;
use Spatie\Snapshots\Drivers\JsonDriver;
use Spatie\Snapshots\Drivers\VarDriver;
use Spatie\Snapshots\Drivers\XmlDriver;

trait MatchesSnapshots
{
    public function assertMatchesSnapshot($actual, Driver $driver = null)
    {
        $snapshot = $this->createSnapshotWithDriver($driver ?? new VarDriver());

        $this->doSnapShotAssertion($snapshot, $actual);
    }

    public function assertMatchesXmlSnapshot($actual)
    {
        $this->assertMatchesSnapshot($actual, new XmlDriver());
    }

    public function assertMatchesJsonSnapshot($actual)
    {
        $this->assertMatchesSnapshot($actual, new JsonDriver());
    }

    /**
     * Determines the snapshot's test, which is the first part of the ID.
     * By default, the test class' name is used.
     *
     * @return string
     */
    protected function getSnapshotTestName(): string
    {
        return (new ReflectionClass($this))->getShortName();
    }

    /**
     * Determines the snapshot's test case, which is the second part of the ID.
     * By default, the test case's method name is used.
     *
     * @return string
     */
    protected function getSnapshotTestCaseName(): string
    {
        return $this->getName();
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }

    /**
     * Determines the directory where snapshots are stored. By default a
     * `__snapshots__` directory is created at the same level as the test
     * class.
     *
     * @return string
     */
    protected function getSnapshotDirectory(): string
    {
        return dirname((new ReflectionClass($this))->getFileName()).
            DIRECTORY_SEPARATOR.
            '__snapshots__';
    }

    /**
     * Determines whether or not the snapshot should be updated instead of
     * matched.
     *
     * Override this method it you want to use a different flag or mechanism
     * than `-d --update-snapshots`.
     *
     * @return bool
     */
    protected function shouldUpdateSnapshot(): bool
    {
        return in_array('--update-snapshots', $_SERVER['argv'], true);
    }

    protected function createSnapshotWithDriver(Driver $driver): Snapshot
    {
        return Snapshot::forTestCase(
            $this->getSnapshotTestName(),
            $this->getSnapshotTestCaseName(),
            $this->getSnapshotDirectory(),
            $driver
        );
    }

    protected function doSnapShotAssertion(Snapshot $snapshot, $actual)
    {
        if (! $snapshot->exists()) {
            $snapshot->create($actual);

            return $this->markTestIncomplete("Snapshot created for {$snapshot->id()}");
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        }

        if ($this->shouldUpdateSnapshot()) {
            try {
                // We only want to update snapshots which need updating. If the snapshot doesn't
                // match the expected output, we'll catch the failure, create a new snapshot and
                // mark the test as incomplete.
                $snapshot->assertMatches($actual);
            } catch (ExpectationFailedException $exception) {
                $snapshot->create($actual);

                return $this->markTestIncomplete("Snapshot updated for {$snapshot->id()}");
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            }
        }

        $snapshot->assertMatches($actual);
    }
}


1			<?php
2
3			namespace Spatie\Snapshots;
4
5			use PHPUnit\Framework\ExpectationFailedException;
6			use ReflectionClass;
7			use Spatie\Snapshots\Drivers\JsonDriver;
8			use Spatie\Snapshots\Drivers\VarDriver;
9			use Spatie\Snapshots\Drivers\XmlDriver;
10
11			trait MatchesSnapshots
12			{
13			public function assertMatchesSnapshot($actual, Driver $driver = null)
14			{
15			$snapshot = $this->createSnapshotWithDriver($driver ?? new VarDriver());
16
17			$this->doSnapShotAssertion($snapshot, $actual);
18			}
19
20			public function assertMatchesXmlSnapshot($actual)
21			{
22			$this->assertMatchesSnapshot($actual, new XmlDriver());
23			}
24
25			public function assertMatchesJsonSnapshot($actual)
26			{
27			$this->assertMatchesSnapshot($actual, new JsonDriver());
28			}
29
30			/**
31			* Determines the snapshot's test, which is the first part of the ID.
32			* By default, the test class' name is used.
33			*
34			* @return string
35			*/
36			protected function getSnapshotTestName(): string
37			{
38			return (new ReflectionClass($this))->getShortName();
39			}
40
41			/**
42			* Determines the snapshot's test case, which is the second part of the ID.
43			* By default, the test case's method name is used.
44			*
45			* @return string
46			*/
47			protected function getSnapshotTestCaseName(): string
48			{
49			return $this->getName();
			0 ignored issues – show Bug introduced 2017-03-27 11:46 UTC by Report Bug Copy Issue Report It seems like `getName()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
50			}
51
52			/**
53			* Determines the directory where snapshots are stored. By default a
54			* `__snapshots__` directory is created at the same level as the test
55			* class.
56			*
57			* @return string
58			*/
59			protected function getSnapshotDirectory(): string
60			{
61			return dirname((new ReflectionClass($this))->getFileName()).
62			DIRECTORY_SEPARATOR.
63			'__snapshots__';
64			}
65
66			/**
67			* Determines whether or not the snapshot should be updated instead of
68			* matched.
69			*
70			* Override this method it you want to use a different flag or mechanism
71			* than `-d --update-snapshots`.
72			*
73			* @return bool
74			*/
75			protected function shouldUpdateSnapshot(): bool
76			{
77			return in_array('--update-snapshots', $_SERVER['argv'], true);
78			}
79
80			protected function createSnapshotWithDriver(Driver $driver): Snapshot
81			{
82			return Snapshot::forTestCase(
83			$this->getSnapshotTestName(),
84			$this->getSnapshotTestCaseName(),
85			$this->getSnapshotDirectory(),
86			$driver
87			);
88			}
89
90			protected function doSnapShotAssertion(Snapshot $snapshot, $actual)
91			{
92			if (! $snapshot->exists()) {
93			$snapshot->create($actual);
94
95			return $this->markTestIncomplete("Snapshot created for {$snapshot->id()}");
			0 ignored issues – show Bug introduced 2017-03-24 18:19 UTC by Report Bug Copy Issue Report It seems like `markTestIncomplete()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
96			}
97
98			if ($this->shouldUpdateSnapshot()) {
99			try {
100			// We only want to update snapshots which need updating. If the snapshot doesn't
101			// match the expected output, we'll catch the failure, create a new snapshot and
102			// mark the test as incomplete.
103			$snapshot->assertMatches($actual);
104			} catch (ExpectationFailedException $exception) {
105			$snapshot->create($actual);
106
107			return $this->markTestIncomplete("Snapshot updated for {$snapshot->id()}");
			0 ignored issues – show Bug introduced 2017-03-24 18:19 UTC by Report Bug Copy Issue Report It seems like `markTestIncomplete()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
108			}
109			}
110
111			$snapshot->assertMatches($actual);
112			}
113			}
114

spatie / phpunit-snapshot-assertions

GitHub Access Token became invalid

Push — master ( 7fd26f...360cbc )

MatchesSnapshots::getSnapshotTestName() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like