SiteSummaryTest::testSourceRecords() - Code Metrics - Inspection of "Pulls/1.0/refactor" - bringyourownideas/silverstripe-maintenance - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#49)

unknown

created 2018-05-21 23:27 UTC

SiteSummaryTest::testSourceRecords() A

↳ Parent: SiteSummaryTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	8
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	8
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	6
nc	1
nop	0

<?php

namespace BringYourOwnIdeas\Maintenance\Tests\Reports;

use Package;
use SapphireTest;
use SiteSummary;

class SiteSummaryTest extends SapphireTest
{
    protected static $fixture_file = 'Package.yml';

    public function testSourceRecords()
    {
        $summaryReport = new SiteSummary;
        $records = $summaryReport->sourceRecords(null);
        $firstRecord = $records->first();
        $this->assertInstanceOf(Package::class, $firstRecord);

        $this->assertStringStartsWith('pretend/', $firstRecord->Name);

    }

    public function testOnlySilverStripeModulesAreShown()
    {
        $summaryReport = new SiteSummary;
        $records = $summaryReport->sourceRecords(null);
        $this->assertCount(3, $records);

        foreach ($records as $record) {
            $this->assertEquals('silverstripe-module', $record->Type);

        }
    }
}


1			<?php
2
3			namespace BringYourOwnIdeas\Maintenance\Tests\Reports;
4
5			use Package;
6			use SapphireTest;
7			use SiteSummary;
8
9			class SiteSummaryTest extends SapphireTest
10			{
11			protected static $fixture_file = 'Package.yml';
12
13			public function testSourceRecords()
14			{
15			$summaryReport = new SiteSummary;
16			$records = $summaryReport->sourceRecords(null);
17			$firstRecord = $records->first();
18			$this->assertInstanceOf(Package::class, $firstRecord);
			0 ignored issues – show Bug introduced 2018-05-17 04:21 UTC by Report Bug Copy Issue Report The method `assertInstanceOf()` does not seem to exist on `object<BringYourOwnIdeas...eports\SiteSummaryTest>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
19			$this->assertStringStartsWith('pretend/', $firstRecord->Name);
			0 ignored issues – show Bug introduced 2018-05-17 06:43 UTC by Report Bug Copy Issue Report The method `assertStringStartsWith()` does not seem to exist on `object<BringYourOwnIdeas...eports\SiteSummaryTest>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
20			}
21
22			public function testOnlySilverStripeModulesAreShown()
23			{
24			$summaryReport = new SiteSummary;
25			$records = $summaryReport->sourceRecords(null);
26			$this->assertCount(3, $records);
			0 ignored issues – show Bug introduced 2018-05-21 22:27 UTC by Report Bug Copy Issue Report The method `assertCount()` does not seem to exist on `object<BringYourOwnIdeas...eports\SiteSummaryTest>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
27			foreach ($records as $record) {
28			$this->assertEquals('silverstripe-module', $record->Type);
			0 ignored issues – show Bug introduced 2018-05-21 03:05 UTC by Report Bug Copy Issue Report The method `assertEquals()` does not seem to exist on `object<BringYourOwnIdeas...eports\SiteSummaryTest>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
29			}
30			}
31			}
32

bringyourownideas / silverstripe-maintenance

Pull Request — master (#49)

SiteSummaryTest::testSourceRecords() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like