Skip to content

Commit 3e87f4c

Browse files
committed
added testcase for single line block
1 parent 1fe051b commit 3e87f4c

File tree

1 file changed

+16
-0
lines changed

1 file changed

+16
-0
lines changed

test/NeedBracesCheckTest.hx

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ class NeedBracesCheckTest extends CheckTestCase {
1313
assertMsg(check, NeedBracesTests.TEST9, '');
1414
assertMsg(check, NeedBracesTests.TEST10, '');
1515
assertMsg(check, NeedBracesTests.TEST12, '');
16+
assertMsg(check, NeedBracesTests.TEST13, '');
1617
}
1718

1819
public function testWrongBraces() {
@@ -42,6 +43,7 @@ class NeedBracesCheckTest extends CheckTestCase {
4243
assertMsg(check, NeedBracesTests.TEST10, 'Body of if branch on same line');
4344
assertMsg(check, NeedBracesTests.TEST11, 'Body of else branch on same line');
4445
assertMsg(check, NeedBracesTests.TEST12, 'Body of else branch on same line');
46+
assertMsg(check, NeedBracesTests.TEST13, 'Single line Block detected');
4547
}
4648

4749
public function testTokenFOR() {
@@ -61,6 +63,7 @@ class NeedBracesCheckTest extends CheckTestCase {
6163
assertMsg(check, NeedBracesTests.TEST10, '');
6264
assertMsg(check, NeedBracesTests.TEST11, '');
6365
assertMsg(check, NeedBracesTests.TEST12, '');
66+
assertMsg(check, NeedBracesTests.TEST13, '');
6467

6568
check.allowSingleLineStatement = false;
6669
assertMsg(check, NeedBracesTests.TEST, 'Body of for loop on same line');
@@ -84,11 +87,13 @@ class NeedBracesCheckTest extends CheckTestCase {
8487
assertMsg(check, NeedBracesTests.TEST10, '');
8588
assertMsg(check, NeedBracesTests.TEST11, 'No braces used for body of if branch');
8689
assertMsg(check, NeedBracesTests.TEST12, 'No braces used for body of else branch');
90+
assertMsg(check, NeedBracesTests.TEST13, '');
8791

8892
check.allowSingleLineStatement = false;
8993
assertMsg(check, NeedBracesTests.TEST, 'Body of if branch on same line');
9094
assertMsg(check, NeedBracesTests.TEST10, 'Body of if branch on same line');
9195
assertMsg(check, NeedBracesTests.TEST11, 'Body of else branch on same line');
96+
assertMsg(check, NeedBracesTests.TEST13, 'Single line Block detected');
9297
}
9398

9499
public function testTokenELSE_IF() {
@@ -108,12 +113,14 @@ class NeedBracesCheckTest extends CheckTestCase {
108113
assertMsg(check, NeedBracesTests.TEST10, '');
109114
assertMsg(check, NeedBracesTests.TEST11, 'No braces used for body of if branch');
110115
assertMsg(check, NeedBracesTests.TEST12, '');
116+
assertMsg(check, NeedBracesTests.TEST13, '');
111117

112118
check.allowSingleLineStatement = false;
113119
assertMsg(check, NeedBracesTests.TEST, 'Body of if branch on same line');
114120
assertMsg(check, NeedBracesTests.TEST10, 'Body of if branch on same line');
115121
assertMsg(check, NeedBracesTests.TEST11, 'Body of else branch on same line');
116122
assertMsg(check, NeedBracesTests.TEST12, 'Body of else branch on same line');
123+
assertMsg(check, NeedBracesTests.TEST13, 'Single line Block detected');
117124
}
118125

119126
public function testTokenWHILE() {
@@ -133,6 +140,7 @@ class NeedBracesCheckTest extends CheckTestCase {
133140
assertMsg(check, NeedBracesTests.TEST10, '');
134141
assertMsg(check, NeedBracesTests.TEST11, '');
135142
assertMsg(check, NeedBracesTests.TEST12, '');
143+
assertMsg(check, NeedBracesTests.TEST13, '');
136144

137145
check.allowSingleLineStatement = false;
138146
assertMsg(check, NeedBracesTests.TEST, 'Body of while loop on same line');
@@ -278,4 +286,12 @@ class NeedBracesTests {
278286
}
279287
}";
280288

289+
public static inline var TEST13:String = "
290+
class Test {
291+
function test() {
292+
if (true) return;
293+
else if (false) { return; }
294+
}
295+
}";
296+
281297
}

0 commit comments

Comments
 (0)