AC-1100: Catch badly put newlines in LESS files

Author: svera

Magento2/Sniffs/Less/ColonSpacingSniff.php

@@ -40,6 +40,12 @@ public function process(File $phpcsFile, $stackPtr)
     {
         $tokens = $phpcsFile->getTokens();
 
+        $nextSemicolon = $phpcsFile->findNext(T_SEMICOLON, $stackPtr);
+        if (false !== $nextSemicolon && ($tokens[$nextSemicolon]['line'] !== $tokens[$stackPtr]['line'])) {
+            $error = 'Expected 1 space after colon in style definition; newline found';
+            $phpcsFile->addError($error, $stackPtr, 'AfterNewline');
+        }
+        
         if ($this->needValidateSpaces($phpcsFile, $stackPtr, $tokens)) {
             $this->validateSpaces($phpcsFile, $stackPtr, $tokens);
         }
@@ -56,12 +62,7 @@ public function process(File $phpcsFile, $stackPtr)
      */
     private function needValidateSpaces(File $phpcsFile, $stackPtr, $tokens)
     {
-        $nextSemicolon = $phpcsFile->findNext(T_SEMICOLON, $stackPtr);
-
-        if (false === $nextSemicolon
-            || ($tokens[$nextSemicolon]['line'] !== $tokens[$stackPtr]['line'])
-            || TokenizerSymbolsInterface::BITWISE_AND === $tokens[$stackPtr - 1]['content']
-        ) {
+        if (TokenizerSymbolsInterface::BITWISE_AND === $tokens[$stackPtr - 1]['content']) {
             return false;
         }
 
@@ -98,15 +99,12 @@ private function validateSpaces(File $phpcsFile, $stackPtr, array $tokens)
             $phpcsFile->addError('Expected 1 space after colon in style definition; 0 found', $stackPtr, 'NoneAfter');
         } else {
             $content = $tokens[($stackPtr + 1)]['content'];
-            if (false === strpos($content, $phpcsFile->eolChar)) {
+            if (false !== strpos($content, ' ')) {
                 $length  = strlen($content);
                 if ($length !== 1) {
                     $error = sprintf('Expected 1 space after colon in style definition; %s found', $length);
                     $phpcsFile->addError($error, $stackPtr, 'After');
                 }
-            } else {
-                $error = 'Expected 1 space after colon in style definition; newline found';
-                $phpcsFile->addError($error, $stackPtr, 'AfterNewline');
             }
         }
     }

Magento2/Tests/Less/ColonSpacingUnitTest.less

@@ -13,7 +13,7 @@ div#foo {
 }
 
 .blah {
-  foo :'xyz';
+  foo :'jkl';
 }
 
 .foo {

Magento2/Tests/Less/ColonSpacingUnitTest.php

@@ -16,6 +16,7 @@ public function getErrorList()
             8 => 1,
             12 => 1,
             16 => 2,
+            20 => 1,
         ];
     }