Add tests for RequireSingleLineConditionSniff limits

tests/Sniffs/ControlStructures/RequireSingleLineConditionSniffTest.php

@@ -17,7 +17,7 @@ public function testErrors(): void
 	{
 		$report = self::checkFile(__DIR__ . '/data/requireSingleLineConditionErrors.php');
 
-		self::assertSame(5, $report->getErrorCount());
+		self::assertSame(7, $report->getErrorCount());
 
 		self::assertSniffError(
 			$report,
@@ -49,6 +49,18 @@ public function testErrors(): void
 			RequireSingleLineConditionSniff::CODE_REQUIRED_SINGLE_LINE_CONDITION,
 			'Condition of "if" should be placed on a single line.',
 		);
+		self::assertSniffError(
+			$report,
+			37,
+			RequireSingleLineConditionSniff::CODE_REQUIRED_SINGLE_LINE_CONDITION,
+			'Condition of "if" should be placed on a single line.',
+		);
+		self::assertSniffError(
+			$report,
+			43,
+			RequireSingleLineConditionSniff::CODE_REQUIRED_SINGLE_LINE_CONDITION,
+			'Condition of "if" should be placed on a single line.',
+		);
 
 		self::assertAllFixedInFile($report);
 	}
@@ -104,4 +116,33 @@ public function testNoErrorsLiveCoding(): void
 		self::assertNoSniffErrorInFile($report);
 	}
 
+	public function testMaxLineLengthErrors(): void
+	{
+		$report = self::checkFile(__DIR__ . '/data/requireSingleLineConditionMaxLineLengthErrors.php', [
+			'maxLineLength' => 80,
+		]);
+
+		self::assertSame(3, $report->getErrorCount());
+
+		self::assertSniffError($report, 4, RequireSingleLineConditionSniff::CODE_REQUIRED_SINGLE_LINE_CONDITION);
+		self::assertSniffError($report, 18, RequireSingleLineConditionSniff::CODE_REQUIRED_SINGLE_LINE_CONDITION);
+		self::assertSniffError($report, 24, RequireSingleLineConditionSniff::CODE_REQUIRED_SINGLE_LINE_CONDITION);
+
+		self::assertAllFixedInFile($report);
+	}
+
+	public function testMaxLineLengthWithoutSimpleConditionsErrors(): void
+	{
+		$report = self::checkFile(__DIR__ . '/data/requireSingleLineConditionMaxLineLengthWithoutSimpleConditionsErrors.php', [
+			'maxLineLength' => 80,
+			'alwaysForSimpleConditions' => false,
+		]);
+
+		self::assertSame(1, $report->getErrorCount());
+
+		self::assertSniffError($report, 4, RequireSingleLineConditionSniff::CODE_REQUIRED_SINGLE_LINE_CONDITION);
+
+		self::assertAllFixedInFile($report);
+	}
+
 }

tests/Sniffs/ControlStructures/data/requireSingleLineConditionErrors.fixed.php

@@ -7,15 +7,23 @@ function () {
 
 	}
 
-	while (doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long') && true) {
+	while (doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long') && true) {
 
 	}
 
 	do {
 
-	} while (doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long') && true);
+	} while (doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long') && true);
 
 	if (doSomething('')) { // Very looooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooong fucking comment
 
 	}
 };
+
+if (doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long')) {
+
+}
+
+if (doSomething( 'short', [ 'short', 'short' ], [ 'veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy', 'long', ] )) {
+
+}

tests/Sniffs/ControlStructures/data/requireSingleLineConditionErrors.php

@@ -14,7 +14,7 @@ function () {
 	}
 
 	while (
-		doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long')
+		doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long')
 		&& true
 	) {
 
@@ -23,7 +23,7 @@ function () {
 	do {
 
 	} while (
-		doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long')
+		doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long')
 		&& true
 	);
 
@@ -33,3 +33,25 @@ function () {
 
 	}
 };
+
+if (
+	doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long')
+) {
+
+}
+
+if (
+	doSomething(
+		'short',
+		[
+			'short',
+			'short'
+		],
+		[
+			'veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy',
+			'long',
+		]
+	)
+) {
+
+}

tests/Sniffs/ControlStructures/data/requireSingleLineConditionMaxLineLengthErrors.fixed.php

@@ -0,0 +1,22 @@
+<?php
+
+function () {
+	if (doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long') && true) {
+
+	}
+
+	if (
+		doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long')
+		&& true
+	) {
+
+	}
+
+	if (doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long')) {
+
+	}
+
+	if (doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long')) {
+
+	}
+};

tests/Sniffs/ControlStructures/data/requireSingleLineConditionMaxLineLengthErrors.php

@@ -0,0 +1,29 @@
+<?php
+
+function () {
+	if (
+		doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long')
+		&& true
+	) {
+
+	}
+
+	if (
+		doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long')
+		&& true
+	) {
+
+	}
+
+	if (
+		doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long')
+	) {
+
+	}
+
+	if (
+		doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long')
+	) {
+
+	}
+};

tests/Sniffs/ControlStructures/data/requireSingleLineConditionMaxLineLengthWithoutSimpleConditionsErrors.fixed.php

@@ -0,0 +1,13 @@
+<?php
+
+function () {
+	if (doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long')) {
+
+	}
+
+	if (
+		doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long')
+	) {
+
+	}
+};

tests/Sniffs/ControlStructures/data/requireSingleLineConditionMaxLineLengthWithoutSimpleConditionsErrors.php

@@ -0,0 +1,15 @@
+<?php
+
+function () {
+	if (
+		doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long')
+	) {
+
+	}
+
+	if (
+		doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long')
+	) {
+
+	}
+};

tests/Sniffs/ControlStructures/data/requireSingleLineConditionNoErrors.php

@@ -5,6 +5,13 @@ function () {
 
 	}
 
+	if (
+		doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long')
+		&& true
+	) {
+
+	}
+
 	while (doSomething('short') && true) {
 
 	}

tests/Sniffs/ControlStructures/data/requireSingleLineConditionWhenDisabledSimpleConditionsNoErrors.php

@@ -1,7 +1,23 @@
 <?php
 
 if (
-	doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long')
+	doSomething('veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy long')
+) {
+
+}
+
+if (
+	doSomething(
+		'short',
+		[
+			'short',
+			'short'
+		],
+		[
+			'veryyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy',
+			'long',
+		]
+	)
 ) {
 
 }