Skip to content

Commit f1bd7b8

Browse files
xaloppmmoll
authored andcommitted
autofix coding standard issues for tests
1 parent e42684a commit f1bd7b8

File tree

7 files changed

+143
-111
lines changed

7 files changed

+143
-111
lines changed

.travis.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,4 +23,4 @@ before_script:
2323

2424
script:
2525
- (cd $PHPCS_DIR ; ../../bin/phpunit --filter MO4)
26-
- $PHPCS_DIR/bin/phpcs $MO4CS_DIR/Sniffs --standard=$PHPCS_DIR/phpcs.xml.dist --report=summary -np
26+
- $PHPCS_DIR/bin/phpcs $MO4CS_DIR --standard=$PHPCS_DIR/phpcs.xml.dist --report=full -np

Tests/Commenting/PropertyCommentUnitTest.php

Lines changed: 19 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,8 @@
3232

3333
class PropertyCommentUnitTest extends AbstractSniffUnitTest
3434
{
35+
36+
3537
/**
3638
* Returns the lines where errors should occur.
3739
*
@@ -42,27 +44,28 @@ class PropertyCommentUnitTest extends AbstractSniffUnitTest
4244
*
4345
* @return array(int => int)
4446
*/
45-
protected function getErrorList($testFile = '')
47+
protected function getErrorList($testFile='')
4648
{
4749
switch ($testFile) {
4850
case 'PropertyCommentUnitTest.pass.inc':
4951
return array();
5052
case 'PropertyCommentUnitTest.fail.inc':
5153
return array(
52-
7 => 1,
53-
10 => 1,
54-
17 => 1,
55-
26 => 2,
56-
29 => 1,
57-
34 => 1,
58-
37 => 1,
59-
41 => 1,
60-
);
54+
7 => 1,
55+
10 => 1,
56+
17 => 1,
57+
26 => 2,
58+
29 => 1,
59+
34 => 1,
60+
37 => 1,
61+
41 => 1,
62+
);
6163
}
6264

6365
return null;
6466

65-
}
67+
}//end getErrorList()
68+
6669

6770
/**
6871
* Returns the lines where warnings should occur.
@@ -75,5 +78,8 @@ protected function getErrorList($testFile = '')
7578
protected function getWarningList()
7679
{
7780
return array();
78-
}
79-
}
81+
82+
}//end getWarningList()
83+
84+
85+
}//end class

Tests/Formatting/AlphabeticalUseStatementsUnitTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ class AlphabeticalUseStatementsUnitTest extends AbstractSniffUnitTest
4545
*
4646
* @return array(int => int)
4747
*/
48-
protected function getErrorList($testFile = '')
48+
protected function getErrorList($testFile='')
4949
{
5050
switch ($testFile) {
5151
case 'AlphabeticalUseStatementsUnitTest.pass.inc':

Tests/Formatting/ArrayAlignmentUnitTest.php

Lines changed: 42 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@
3333
class ArrayAlignmentUnitTest extends AbstractSniffUnitTest
3434
{
3535

36+
3637
/**
3738
* Returns the lines where errors should occur.
3839
*
@@ -43,50 +44,51 @@ class ArrayAlignmentUnitTest extends AbstractSniffUnitTest
4344
*
4445
* @return array(int => int)
4546
*/
46-
protected function getErrorList($testFile = '')
47+
protected function getErrorList($testFile='')
4748
{
4849
switch ($testFile) {
4950
case 'ArrayAlignmentUnitTest.pass.inc':
5051
return array();
5152
case 'ArrayAlignmentUnitTest.fail.inc':
5253
return array(
53-
5 => 1,
54-
10 => 1,
55-
17 => 2,
56-
18 => 2,
57-
22 => 1,
58-
28 => 1,
59-
38 => 1,
60-
43 => 1,
61-
45 => 1,
62-
49 => 1,
63-
51 => 1,
64-
58 => 1,
65-
59 => 1,
66-
61 => 1,
67-
67 => 1,
68-
70 => 1,
69-
71 => 1,
70-
73 => 1,
71-
82 => 1,
72-
83 => 1,
73-
85 => 1,
74-
93 => 1,
75-
94 => 1,
76-
97 => 1,
77-
105 => 1,
78-
130 => 1,
79-
132 => 1,
80-
134 => 1,
81-
136 => 2,
82-
145 => 1,
83-
151 => 2,
84-
);
85-
}
54+
5 => 1,
55+
10 => 1,
56+
17 => 2,
57+
18 => 2,
58+
22 => 1,
59+
28 => 1,
60+
38 => 1,
61+
43 => 1,
62+
45 => 1,
63+
49 => 1,
64+
51 => 1,
65+
58 => 1,
66+
59 => 1,
67+
61 => 1,
68+
67 => 1,
69+
70 => 1,
70+
71 => 1,
71+
73 => 1,
72+
82 => 1,
73+
83 => 1,
74+
85 => 1,
75+
93 => 1,
76+
94 => 1,
77+
97 => 1,
78+
105 => 1,
79+
130 => 1,
80+
132 => 1,
81+
134 => 1,
82+
136 => 2,
83+
145 => 1,
84+
151 => 2,
85+
);
86+
}//end switch
8687

8788
return null;
8889

89-
}
90+
}//end getErrorList()
91+
9092

9193
/**
9294
* Returns the lines where warnings should occur.
@@ -99,5 +101,8 @@ protected function getErrorList($testFile = '')
99101
protected function getWarningList()
100102
{
101103
return array();
102-
}
103-
}
104+
105+
}//end getWarningList()
106+
107+
108+
}//end class

Tests/Formatting/UnnecessaryNamespaceUsageUnitTest.php

Lines changed: 40 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,8 @@
3434

3535
class UnnecessaryNamespaceUsageUnitTest extends AbstractSniffUnitTest
3636
{
37+
38+
3739
/**
3840
* Returns the lines where errors should occur.
3941
*
@@ -44,10 +46,12 @@ class UnnecessaryNamespaceUsageUnitTest extends AbstractSniffUnitTest
4446
*
4547
* @return array(int => int)
4648
*/
47-
protected function getErrorList($testFile = '')
49+
protected function getErrorList($testFile='')
4850
{
4951
return array();
50-
}
52+
53+
}//end getErrorList()
54+
5155

5256
/**
5357
* Returns the lines where warnings should occur.
@@ -59,7 +63,7 @@ protected function getErrorList($testFile = '')
5963
*
6064
* @return array(int => int)
6165
*/
62-
protected function getWarningList($testFile = '')
66+
protected function getWarningList($testFile='')
6367
{
6468
switch ($testFile) {
6569
case 'UnnecessaryNamespaceUsageUnitTest.pass.1.inc':
@@ -69,39 +73,42 @@ protected function getWarningList($testFile = '')
6973
return array();
7074
case 'UnnecessaryNamespaceUsageUnitTest.fail.1.inc':
7175
return array(
72-
17 => 1,
73-
19 => 1,
74-
24 => 1,
75-
25 => 1,
76-
26 => 2,
77-
28 => 1,
78-
30 => 2,
79-
32 => 1,
80-
33 => 1,
81-
40 => 1,
82-
44 => 1,
83-
45 => 1,
84-
46 => 1,
85-
52 => 1,
86-
56 => 1,
87-
);
76+
17 => 1,
77+
19 => 1,
78+
24 => 1,
79+
25 => 1,
80+
26 => 2,
81+
28 => 1,
82+
30 => 2,
83+
32 => 1,
84+
33 => 1,
85+
40 => 1,
86+
44 => 1,
87+
45 => 1,
88+
46 => 1,
89+
52 => 1,
90+
56 => 1,
91+
);
8892
case 'UnnecessaryNamespaceUsageUnitTest.fail.2.inc':
8993
return array(
90-
10 => 1,
91-
11 => 1,
92-
);
94+
10 => 1,
95+
11 => 1,
96+
);
9397
case 'UnnecessaryNamespaceUsageUnitTest.fail.3.inc':
9498
return array(
95-
15 => 1,
96-
16 => 1,
97-
17 => 1,
98-
18 => 1,
99-
22 => 1,
100-
23 => 1,
101-
25 => 3,
102-
);
103-
}
99+
15 => 1,
100+
16 => 1,
101+
17 => 1,
102+
18 => 1,
103+
22 => 1,
104+
23 => 1,
105+
25 => 3,
106+
);
107+
}//end switch
104108

105109
return null;
106-
}
107-
}
110+
111+
}//end getWarningList()
112+
113+
114+
}//end class

Tests/Formatting/UseArrayShortTagUnitTest.php

Lines changed: 19 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,8 @@
3333

3434
class UseArrayShortTagUnitTest extends AbstractSniffUnitTest
3535
{
36+
37+
3638
/**
3739
* Returns the lines where errors should occur.
3840
*
@@ -43,25 +45,27 @@ class UseArrayShortTagUnitTest extends AbstractSniffUnitTest
4345
*
4446
* @return array(int => int)
4547
*/
46-
protected function getErrorList($testFile = '')
48+
protected function getErrorList($testFile='')
4749
{
4850
switch ($testFile) {
4951
case 'UseArrayShortTagUnitTest.pass.inc':
5052
return array();
5153
case 'UseArrayShortTagUnitTest.fail.inc':
5254
return array(
53-
3 => 1,
54-
4 => 1,
55-
8 => 1,
56-
10 => 1,
57-
13 => 1,
58-
15 => 1,
59-
16 => 2,
60-
);
55+
3 => 1,
56+
4 => 1,
57+
8 => 1,
58+
10 => 1,
59+
13 => 1,
60+
15 => 1,
61+
16 => 2,
62+
);
6163
}
6264

6365
return null;
64-
}
66+
67+
}//end getErrorList()
68+
6569

6670
/**
6771
* Returns the lines where warnings should occur.
@@ -74,5 +78,8 @@ protected function getErrorList($testFile = '')
7478
protected function getWarningList()
7579
{
7680
return array();
77-
}
78-
}
81+
82+
}//end getWarningList()
83+
84+
85+
}//end class

0 commit comments

Comments
 (0)