Skip to content
This repository was archived by the owner on Nov 21, 2019. It is now read-only.

Commit 4213479

Browse files
Merge branch '0.3'
2 parents 0bba285 + 5565299 commit 4213479

File tree

7 files changed

+15
-2
lines changed

7 files changed

+15
-2
lines changed

composer.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
},
1616
"require": {
1717
"php": "^7.2",
18-
"dealerdirect/phpcodesniffer-composer-installer": "^0.4",
18+
"dealerdirect/phpcodesniffer-composer-installer": "^0.4 || ^0.5",
1919
"slevomat/coding-standard": "^4.8",
2020
"squizlabs/php_codesniffer": "^3.3.2"
2121
},

tests/RulesetTests.php

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
use PHPUnit\Framework\TestCase;
1414
use Symfony\Component\Finder\Finder;
1515
use function array_combine;
16+
use function array_diff;
1617
use function array_filter;
1718
use function array_map;
1819
use function explode;
@@ -53,11 +54,12 @@ public function it_finds_and_fixes_violations(
5354
string $contents,
5455
string $fixed,
5556
array $messages,
57+
array $ignoreMessages,
5658
?string $description,
5759
?string $fixedEncoding
5860
) : void {
5961
$file = $this->createFile($filename, $contents);
60-
$actual = flatten($this->getMessages($file));
62+
$actual = array_diff(flatten($this->getMessages($file)), $ignoreMessages);
6163

6264
sort($actual);
6365
sort($messages);
@@ -84,6 +86,10 @@ public function cases() : iterable
8486
$parts['messages'] = array_filter(explode("\n", $parts['messages']));
8587
}
8688

89+
if (isset($parts['ignore-messages'])) {
90+
$parts['ignore-messages'] = array_filter(explode("\n", $parts['ignore-messages']));
91+
}
92+
8793
$keys = ['fixed', 'fixed-encoding', 'fixed-line-endings', 'messages'];
8894
if (empty($parts['contents'])) {
8995
throw new LogicException("Couldn't find contents in {$file->getRelativePathname()}");
@@ -134,6 +140,7 @@ public function cases() : iterable
134140
$parts['contents'],
135141
$parts['fixed'],
136142
$parts['messages'] ?? [],
143+
$parts['ignore-messages'] ?? [],
137144
$parts['description'] ?? null,
138145
$parts['fixed-encoding'] ?? null,
139146
];

tests/cases/classes/one-per-file

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,5 +19,6 @@ class Bar
1919

2020
---MESSAGES---
2121
11:1 PSR1.Classes.ClassDeclaration.MultipleClasses
22+
---IGNORE-MESSAGES---
2223
11:1 Squiz.Classes.ClassFileName.NoMatch
2324
---

tests/cases/classes/property-visibility

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,5 +16,6 @@ class Foo
1616

1717
---MESSAGES---
1818
9:9 PSR2.Classes.PropertyDeclaration.ScopeMissing
19+
---IGNORE-MESSAGES---
1920
9:9 PSR2.Classes.PropertyDeclaration.VarUsed
2021
---

tests/cases/interfaces/one-per-file

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,5 +19,6 @@ interface Bar
1919

2020
---MESSAGES---
2121
11:1 PSR1.Classes.ClassDeclaration.MultipleClasses
22+
---IGNORE-MESSAGES---
2223
11:1 Squiz.Classes.ClassFileName.NoMatch
2324
---

tests/cases/trait/one-per-file

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,4 +19,6 @@ trait Bar
1919

2020
---MESSAGES---
2121
11:1 PSR1.Classes.ClassDeclaration.MultipleClasses
22+
---IGNORE-MESSAGES---
23+
11:1 Squiz.Classes.ClassFileName.NoMatch
2224
---

tests/cases/trait/property-visibility

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,5 +16,6 @@ trait Foo
1616

1717
---MESSAGES---
1818
9:9 PSR2.Classes.PropertyDeclaration.ScopeMissing
19+
---IGNORE-MESSAGES---
1920
9:9 PSR2.Classes.PropertyDeclaration.VarUsed
2021
---

0 commit comments

Comments
 (0)