added method length check

Author: adireddy

test/LineLengthCheckTest.hx

@@ -4,17 +4,17 @@ import checkstyle.checks.LineLengthCheck;
 
 class LineLengthCheckTest extends CheckTestCase {
 
-	public function testDefaultLineLength(){
+	public function testDefaultLineLength() {
 		var msg = checkMessage(LineLengthTests.TEST1, new LineLengthCheck());
 		assertEquals(msg, 'Too long line (> 80)');
 	}
 
-	public function testCorrectLineLength(){
+	public function testCorrectLineLength() {
 		var msg = checkMessage(LineLengthTests.TEST2, new LineLengthCheck());
 		assertEquals(msg, '');
 	}
 
-	public function testConfigurableLineLength(){
+	public function testConfigurableLineLength() {
 		var check = new LineLengthCheck();
 		check.maxCharacters = 40;
 

test/MethodLengthCheckTest.hx

@@ -0,0 +1,111 @@
+package ;
+
+import checkstyle.checks.MethodLengthCheck;
+
+class MethodLengthCheckTest extends CheckTestCase {
+
+	public function testWrongMethodLength() {
+		var msg = checkMessage(MethodLengthTests.TEST1, new MethodLengthCheck());
+		assertEquals(msg, 'Function is too long: test (> 50 lines, try splitting into multiple functions)');
+	}
+
+	public function testCorrectMethodLength() {
+		var msg = checkMessage(MethodLengthTests.TEST2, new MethodLengthCheck());
+		assertEquals(msg, '');
+	}
+
+	public function testConfigurableMethodLength() {
+		var check = new MethodLengthCheck();
+		check.maxFunctionLines = 10;
+
+		var msg = checkMessage(MethodLengthTests.TEST3, check);
+		assertEquals(msg, 'Function is too long: test (> 10 lines, try splitting into multiple functions)');
+	}
+}
+
+class MethodLengthTests {
+	public static inline var TEST1:String = "
+	class Test {
+		public function test() {
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+		}
+	}";
+
+	public static inline var TEST2:String =
+	"class Test {
+		public function test() {
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+		}
+	}";
+
+	public static inline var TEST3:String =
+	"class Test {
+		public function test() {
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+
+			tarce('TEST');
+		}
+	}";
+}