@@ -697,55 +697,55 @@ public function testClassMethodImplementationChanged($context, $visibility, $cod
697697 $ this ->assertSame ('tmp::tmpMethod ' , $ report [$ context ][$ expectedLevel ][0 ]->getTarget ());
698698 }
699699
700- public function providerImplementationChanged ()
701- {
702- return [
703- ['class ' , 'public ' , 'V023 ' ],
704- ['class ' , 'protected ' , 'V024 ' ],
705- ['class ' , 'private ' , 'V025 ' ],
706- ['trait ' , 'public ' , 'V052 ' ],
707- ['trait ' , 'protected ' , 'V053 ' ],
708- ['trait ' , 'private ' , 'V054 ' ],
709- ];
710- }
700+ public function providerImplementationChanged ()
701+ {
702+ return [
703+ ['class ' , 'public ' , 'V023 ' ],
704+ ['class ' , 'protected ' , 'V024 ' ],
705+ ['class ' , 'private ' , 'V025 ' ],
706+ ['trait ' , 'public ' , 'V052 ' ],
707+ ['trait ' , 'protected ' , 'V053 ' ],
708+ ['trait ' , 'private ' , 'V054 ' ],
709+ ];
710+ }
711711
712712 /**
713713 * @dataProvider providerCaseChanged
714714 */
715- public function testClassMethodCaseChangeChanged ($ context , $ visibility , $ code )
716- {
717- $ constructor = $ this ->getConstructorForContext ($ context );
718- $ classBefore = new $ constructor ('tmp ' , [
719- 'stmts ' => [
720- new ClassMethod ('tmpMethod ' , [
721- 'type ' => Visibility::getModifier ($ visibility ),
722- 'stmts ' => [
723- new MethodCall (new Variable ('test ' ), 'someMethod ' ),
724- ],
725- ]),
726- ],
727- ]);
728-
729- $ classAfter = new $ constructor ('tmp ' , [
730- 'stmts ' => [
731- new ClassMethod ('tmpmethod ' , [
732- 'type ' => Visibility::getModifier ($ visibility ),
733- 'stmts ' => [
734- new MethodCall (new Variable ('test ' ), 'someMethod ' ),
735- ],
736- ]),
737- ],
738- ]);
739-
740- $ analyzer = new ClassMethodAnalyzer ($ context );
741- $ report = $ analyzer ->analyze ($ classBefore , $ classAfter );
742-
743- Assert::assertDifference ($ report , $ context , Level::PATCH );
715+ public function testClassMethodCaseChangeChanged ($ context , $ visibility , $ code )
716+ {
717+ $ constructor = $ this ->getConstructorForContext ($ context );
718+ $ classBefore = new $ constructor ('tmp ' , [
719+ 'stmts ' => [
720+ new ClassMethod ('tmpMethod ' , [
721+ 'type ' => Visibility::getModifier ($ visibility ),
722+ 'stmts ' => [
723+ new MethodCall (new Variable ('test ' ), 'someMethod ' ),
724+ ],
725+ ]),
726+ ],
727+ ]);
728+
729+ $ classAfter = new $ constructor ('tmp ' , [
730+ 'stmts ' => [
731+ new ClassMethod ('tmpmethod ' , [
732+ 'type ' => Visibility::getModifier ($ visibility ),
733+ 'stmts ' => [
734+ new MethodCall (new Variable ('test ' ), 'someMethod ' ),
735+ ],
736+ ]),
737+ ],
738+ ]);
739+
740+ $ analyzer = new ClassMethodAnalyzer ($ context );
741+ $ report = $ analyzer ->analyze ($ classBefore , $ classAfter );
742+
743+ Assert::assertDifference ($ report , $ context , Level::PATCH );
744744
745745 $ expectedLevel = LevelMapping::getLevelForCode ($ code );
746- $ this ->assertSame ($ code , $ report [$ context ][$ expectedLevel ][0 ]->getCode ());
746+ $ this ->assertSame ($ code , $ report [$ context ][$ expectedLevel ][0 ]->getCode ());
747747 $ this ->assertSame (sprintf ('[%s] Method has been renamed (case only). ' , $ visibility ), $ report [$ context ][$ expectedLevel ][0 ]->getReason ());
748- }
748+ }
749749
750750 public function providerCaseChanged ()
751751 {
0 commit comments