Skip to content

Commit e304ee7

Browse files
committed
minor #8600 Add special Abstract prefix not usage for PHPUnit test cases (Soullivaneuh)
This PR was merged into the 3.3 branch. Discussion ---------- Add special Abstract prefix not usage for PHPUnit test cases As we can see here: ![image](https://user-images.githubusercontent.com/1698357/32507068-bddf178a-c3e6-11e7-8d28-05f3b18ea18c.png) You are not using the `Abstract` prefix at all for the test cases. I assume this is to follow PHPUnit conventions on this part? In this case, the coding standard may be more accurate about this. Commits ------- f4d122a Add special Abstract prefix not usage for PHPUnit test cases
2 parents a7d8d97 + f4d122a commit e304ee7

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

contributing/code/standards.rst

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -190,9 +190,10 @@ Naming Conventions
190190

191191
* Use namespaces for all classes;
192192

193-
* Prefix abstract classes with ``Abstract``. Please note some early Symfony classes
194-
do not follow this convention and have not been renamed for backward compatibility
195-
reasons. However all new abstract classes must follow this naming convention;
193+
* Prefix all abstract classes with ``Abstract`` except PHPUnit ``*TestCase``.
194+
Please note some early Symfony classes do not follow this convention and
195+
have not been renamed for backward compatibility reasons. However all new
196+
abstract classes must follow this naming convention;
196197

197198
* Suffix interfaces with ``Interface``;
198199

0 commit comments

Comments
 (0)