more unit tests

Author: adireddy

test/CheckTestCase.hx

@@ -9,22 +9,24 @@ class CheckTestCase extends haxe.unit.TestCase {
 
 	static inline var FILE_NAME = "Test.hx";
 
-	function messageEquals(expected:LintMessage, actual:LintMessage) {
-		assertEquals(expected.fileName, actual.fileName);
-		assertEquals(expected.moduleName, actual.moduleName);
-		assertEquals(expected.line, actual.line);
-		assertEquals(expected.column, actual.column);
-		assertEquals(expected.severity, actual.severity);
-		assertEquals(expected.message, actual.message);
+	var _checker:Checker;
+	var _reporter:TestReporter;
+
+	override public function setup() {
+		_checker = new Checker();
+		_reporter = new TestReporter();
 	}
 
 	function checkMessage(src, check):String {
-		var checker = new Checker();
-		var rep = new TestReporter();
-		checker.addCheck(check);
-		checker.addReporter(rep);
-		checker.process([{name:FILE_NAME, content:src}]);
-		return rep.message;
+		_checker.addCheck(check);
+		_checker.addReporter(_reporter);
+		_checker.process([{name:FILE_NAME, content:src}]);
+		return _reporter.message;
+	}
+
+	override public function tearDown() {
+		_checker = null;
+		_reporter = null;
 	}
 }
 
@@ -37,11 +39,8 @@ class TestReporter implements IReporter {
 	}
 
 	public function start() {}
-
 	public function finish() {}
-
 	public function fileStart(f:LintFile) {}
-
 	public function fileFinish(f:LintFile) {}
 
 	public function addMessage(m:LintMessage) {

test/LineLengthCheckTest.hx

@@ -0,0 +1,54 @@
+package ;
+
+import checkstyle.checks.LineLengthCheck;
+
+class LineLengthCheckTest extends CheckTestCase {
+
+	public function testDefaultLineLength(){
+		var msg = checkMessage(LineLengthTests.TEST1, new LineLengthCheck());
+		assertEquals(msg, 'Too long line (> 80)');
+	}
+
+	public function testCorrectLineLength(){
+		var msg = checkMessage(LineLengthTests.TEST2, new LineLengthCheck());
+		assertEquals(msg, '');
+	}
+
+	public function testConfigurableLineLength(){
+		var check = new LineLengthCheck();
+		check.maxCharacters = 40;
+
+		var msg = checkMessage(LineLengthTests.TEST3, check);
+		assertEquals(msg, 'Too long line (> 40)');
+	}
+}
+
+class LineLengthTests {
+	public static inline var TEST1:String = "
+	class Test {
+		var _a:Int;
+		public function new() {
+			_a = 10;
+			if (_a > 200 && _a < 250 && _a != 240 && _a != 250 && _a != 260 && _a != 270 && _a != 280 && _a != 290) {
+				_a = -1;
+			}
+		}
+	}";
+
+	public static inline var TEST2:String =
+	"class Test {
+		public function new() {
+			var b:Int;
+		}
+	}";
+
+	public static inline var TEST3:String =
+	"class Test {
+		public function new() {
+			_a = 10;
+			if (_a > 200 && _a < 250 && _a != 240 && _a != 250) {
+				_a = -1;
+			}
+		}
+	}";
+}

test/TestMain.hx

@@ -9,6 +9,8 @@ class TestMain {
 		runner.add(new BlockFormatCheckTest());
 		runner.add(new EmptyLinesCheckTest());
 
+		runner.add(new LineLengthCheckTest());
+
 		runner.add(new ListenerNameCheckTest());
 
 		var success = runner.run();