Skip to content
This repository has been archived by the owner on Dec 3, 2023. It is now read-only.

Commit

Permalink
[PHPStanRules] Drop trait requires test, not a good idea
Browse files Browse the repository at this point in the history
  • Loading branch information
TomasVotruba committed Feb 22, 2021
1 parent c6bd65a commit 127bfe9
Show file tree
Hide file tree
Showing 6 changed files with 2 additions and 65 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -21,21 +21,6 @@ public function testRule(array $filePaths, array $expectedErrorMessagesWithLines

public function provideData(): Iterator
{
$errorMessage = sprintf(CheckRequiredMethodNamingRule::ERROR_MESSAGE, 'autowireRequiredByTrait');
yield [
[__DIR__ . '/Fixture/ClassUsingRequiredByTrait.php', __DIR__ . '/Fixture/RequiredByTrait.php'],
[[$errorMessage, 12]],
];

$errorMessage = sprintf(CheckRequiredMethodNamingRule::ERROR_MESSAGE, 'autowireRequiredByTraitCorrect');
yield [
[
__DIR__ . '/Fixture/ClassUsingRequiredByTraitCorrect.php',
__DIR__ . '/Fixture/RequiredByTraitCorrect.php',
],
[[$errorMessage, 12]],
];

yield [[__DIR__ . '/Fixture/SkipWithoutRequired.php'], []];
yield [[__DIR__ . '/Fixture/SkipCorretName.php'], []];

Expand Down

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

2 changes: 2 additions & 0 deletions phpstan.neon
Original file line number Diff line number Diff line change
Expand Up @@ -474,3 +474,5 @@ parameters:
message: '#Do not inherit from abstract class, better use composition#'
paths:
- packages/git-wrapper/src
- packages/set-config-resolver/src/ConfigResolver.php
- packages/set-config-resolver/src/SetAwareConfigResolver.php

0 comments on commit 127bfe9

Please sign in to comment.