Merge pull request #85 from magento-commerce/AC-205

AC-1159: Move eslint tests to magento-coding-standard

Author: loginesta

.github/workflows/php.yml

@@ -42,7 +42,7 @@ jobs:
     - name: Install dependencies
       run: npm install
     - name: Run ESLint
-      run: npm run eslint -- eslint/rules
+      run: npm run eslint -- eslint/rules Magento2/ --ignore-pattern 'Magento2/Tests/Eslint/*Test.js'
 
     - name: Validate composer
       run: composer validate

Magento2/Tests/Eslint/ClickEventShorthand.js renamed to Magento2/Tests/Eslint/ClickEventShorthandTest.js

@@ -17,7 +17,7 @@ class ClickEventShorthandTest extends AbstractEslintTestCase
     public function testExecute(): void
     {
         $this->assertFileContainsError(
-            'ClickEventShorthand.js',
+            'ClickEventShorthandTest.js',
             ['Instead of .blur(fn) use .on("blur", fn). Instead of .blur() use .trigger("blur")']
         );
     }

Magento2/Tests/Eslint/ClickEventShorthandTest.php

@@ -6,5 +6,5 @@
 define([
     'jquery'
 ], function (){
-    
+    'use strict';
 });

Magento2/Tests/Legacy/CopyrightAnotherExtensionsFilesUnitTest.2.js

@@ -6,5 +6,5 @@
 define([
     'jquery'
 ], function (){
-    
+    'use strict';
 });