Skip to content

Commit ed30b83

Browse files
AlexHaxeAlexHaxe
committed
fixed Java warnings
1 parent 4b5e987 commit ed30b83

File tree

2 files changed

+22
-42
lines changed

2 files changed

+22
-42
lines changed

‎test/checkstyle/config/ConfigParserTest.hx‎

Lines changed: 13 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,7 @@ class ConfigParserTest {
77

88
@Test
99
public function testCheckstyleConfig() {
10-
var configParser:ConfigParser = new ConfigParser(function(message:String) {
11-
Assert.fail(message);
12-
});
10+
var configParser:ConfigParser = new ConfigParser(reportConfigParserFailure);
1311

1412
Assert.isNotNull(configParser.checker.checks);
1513
Assert.isTrue(configParser.checker.checks.length == 0);
@@ -23,9 +21,7 @@ class ConfigParserTest {
2321

2422
@Test
2523
public function testExtendsConfigPath() {
26-
var configParser:ConfigParser = new ConfigParser(function(message:String) {
27-
Assert.fail(message);
28-
});
24+
var configParser:ConfigParser = new ConfigParser(reportConfigParserFailure);
2925

3026
Assert.isNotNull(configParser.checker.checks);
3127
Assert.isTrue(configParser.checker.checks.length == 0);
@@ -40,9 +36,7 @@ class ConfigParserTest {
4036

4137
@Test
4238
public function testValidateParserThread() {
43-
var configParser:ConfigParser = new ConfigParser(function(message:String) {
44-
Assert.fail(message);
45-
});
39+
var configParser:ConfigParser = new ConfigParser(reportConfigParserFailure);
4640
configParser.parseAndValidateConfig({numberOfCheckerThreads: 0}, "");
4741
Assert.areEqual(5, configParser.numberOfCheckerThreads);
4842
configParser.parseAndValidateConfig({numberOfCheckerThreads: 10}, "");
@@ -61,9 +55,7 @@ class ConfigParserTest {
6155

6256
@Test
6357
public function testCheckCount() {
64-
var configParser:ConfigParser = new ConfigParser(function(message:String) {
65-
Assert.fail(message);
66-
});
58+
var configParser:ConfigParser = new ConfigParser(reportConfigParserFailure);
6759

6860
#if (haxe_ver >= 4.0)
6961
Assert.areEqual(74, configParser.getCheckCount());
@@ -74,9 +66,7 @@ class ConfigParserTest {
7466

7567
@Test
7668
public function testUnusedChecks() {
77-
var configParser:ConfigParser = new ConfigParser(function(message:String) {
78-
Assert.fail(message);
79-
});
69+
var configParser:ConfigParser = new ConfigParser(reportConfigParserFailure);
8070

8171
Assert.areEqual(0, configParser.getUsedCheckCount());
8272
configParser.addAllChecks();
@@ -89,27 +79,21 @@ class ConfigParserTest {
8979

9080
@Test
9181
public function testConfig() {
92-
var configParser:ConfigParser = new ConfigParser(function(message:String) {
93-
Assert.fail(message);
94-
});
82+
var configParser:ConfigParser = new ConfigParser(reportConfigParserFailure);
9583

9684
configParser.addAllChecks();
9785
var config:Config = ConfigUtils.makeConfigFromChecker(configParser.checker);
9886
config.numberOfCheckerThreads = 11;
9987

100-
configParser = new ConfigParser(function(message:String) {
101-
Assert.fail(message);
102-
});
88+
configParser = new ConfigParser(reportConfigParserFailure);
10389
configParser.parseAndValidateConfig(config, "");
10490
Assert.areEqual(configParser.getCheckCount(), configParser.getUsedCheckCount());
10591
Assert.areEqual(11, configParser.numberOfCheckerThreads);
10692
}
10793

10894
@Test
10995
public function testConfigVersion1() {
110-
var configParser:ConfigParser = new ConfigParser(function(message:String) {
111-
Assert.fail(message);
112-
});
96+
var configParser:ConfigParser = new ConfigParser(reportConfigParserFailure);
11397

11498
Assert.isNotNull(configParser.checker.checks);
11599
Assert.isTrue(configParser.checker.checks.length == 0);
@@ -143,9 +127,7 @@ class ConfigParserTest {
143127

144128
@Test
145129
public function testExcludeConfigVersion1() {
146-
var configParser:ConfigParser = new ConfigParser(function(message:String) {
147-
Assert.fail(message);
148-
});
130+
var configParser:ConfigParser = new ConfigParser(reportConfigParserFailure);
149131

150132
Assert.isNotNull(configParser.checker.checks);
151133
Assert.isTrue(configParser.checker.checks.length == 0);
@@ -202,4 +184,8 @@ class ConfigParserTest {
202184
configParser.parseAndValidateConfig(config, LOCAL_PATH);
203185
Assert.areEqual("Check Trace has no property named 'non_existing_property'", failMessage);
204186
}
187+
188+
function reportConfigParserFailure(message:String) {
189+
Assert.fail(message);
190+
}
205191
}

‎test/checkstyle/config/ExcludeManagerTest.hx‎

Lines changed: 9 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,7 @@ class ExcludeManagerTest extends CheckTestCase<ExcludeManagerTests> {
1919

2020
@Test
2121
public function testExcludeAllConfig() {
22-
var configParser:ConfigParser = new ConfigParser(function(message:String) {
23-
Assert.fail(message);
24-
});
22+
var configParser:ConfigParser = new ConfigParser(reportConfigParserFailure);
2523

2624
Assert.isNotNull(configParser.checker.checks);
2725
Assert.isTrue(configParser.checker.checks.length == 0);
@@ -39,9 +37,7 @@ class ExcludeManagerTest extends CheckTestCase<ExcludeManagerTests> {
3937

4038
@Test
4139
public function testExcludeAllConfig2() {
42-
var configParser:ConfigParser = new ConfigParser(function(message:String) {
43-
Assert.fail(message);
44-
});
40+
var configParser:ConfigParser = new ConfigParser(reportConfigParserFailure);
4541

4642
Assert.isNotNull(configParser.checker.checks);
4743
Assert.isTrue(configParser.checker.checks.length == 0);
@@ -61,9 +57,7 @@ class ExcludeManagerTest extends CheckTestCase<ExcludeManagerTests> {
6157

6258
@Test
6359
public function testExcludeConfigDynamic() {
64-
var configParser:ConfigParser = new ConfigParser(function(message:String) {
65-
Assert.fail(message);
66-
});
60+
var configParser:ConfigParser = new ConfigParser(reportConfigParserFailure);
6761

6862
Assert.isNotNull(configParser.checker.checks);
6963
Assert.isTrue(configParser.checker.checks.length == 0);
@@ -84,9 +78,7 @@ class ExcludeManagerTest extends CheckTestCase<ExcludeManagerTests> {
8478

8579
@Test
8680
public function testExcludeConfigDynamicRange() {
87-
var configParser:ConfigParser = new ConfigParser(function(message:String) {
88-
Assert.fail(message);
89-
});
81+
var configParser:ConfigParser = new ConfigParser(reportConfigParserFailure);
9082

9183
Assert.isNotNull(configParser.checker.checks);
9284
Assert.isTrue(configParser.checker.checks.length == 0);
@@ -107,9 +99,7 @@ class ExcludeManagerTest extends CheckTestCase<ExcludeManagerTests> {
10799

108100
@Test
109101
public function testExcludeConfigAllRange() {
110-
var configParser:ConfigParser = new ConfigParser(function(message:String) {
111-
Assert.fail(message);
112-
});
102+
var configParser:ConfigParser = new ConfigParser(reportConfigParserFailure);
113103

114104
Assert.isNotNull(configParser.checker.checks);
115105
Assert.isTrue(configParser.checker.checks.length == 0);
@@ -128,6 +118,10 @@ class ExcludeManagerTest extends CheckTestCase<ExcludeManagerTests> {
128118
assertNoMsg(new DynamicCheck(), CHECKSINFO, CHECKSINFO_FILE_NAME);
129119
assertNoMsg(new ReturnCheck(), CHECKSINFO, CHECKSINFO_FILE_NAME);
130120
}
121+
122+
function reportConfigParserFailure(message:String) {
123+
Assert.fail(message);
124+
}
131125
}
132126

133127
@:enum

0 commit comments

Comments
 (0)