Merge pull request #68 from magento-commerce/imported-loginesta-magento-coding-standard-262

[Imported] AC-1156: Move custom eslint tests to magento-coding-standard repo

Author: loginesta

.github/workflows/php.yml

@@ -25,6 +25,9 @@ jobs:
     name: Tests with PHP ${{ matrix.php-version }} and ${{ matrix.dependencies }} dependencies
 
     steps:
+    - name: Setup node
+      uses: actions/setup-node@v2
+
     - uses: actions/checkout@v2
 
     - name: Setup PHP
@@ -34,6 +37,11 @@ jobs:
       env:
         COMPOSER_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
+    - name: Install dependencies
+      run: npm install
+    - name: Run ESLint
+      run: npm run eslint -- eslint/rules
+
     - name: Validate composer
       run: composer validate
 

.gitignore

@@ -1,5 +1,6 @@
 /vendor/*
 /bin/*
+/node_modules
 
 # IDE files
 .idea/

Magento2/Tests/Eslint/AbstractEslintTestCase.php

@@ -0,0 +1,34 @@
+<?php
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+declare(strict_types=1);
+
+namespace Magento2\Tests\Eslint;
+
+use PHPUnit\Framework\TestCase;
+
+/**
+ * Abstract class AbstractEslintTestCase
+ *
+ * Test Eslint Rules (magento-coding-standard/eslint/rules)
+ */
+abstract class AbstractEslintTestCase extends TestCase
+{
+    /**
+     * @param string $testFile
+     * @param array $expectedMessages
+     */
+    protected function assertFileContainsError(string $testFile, array $expectedMessages): void
+    {
+        exec(
+            'npm run eslint -- Magento2/Tests/Eslint/' . $testFile,
+            $output
+        );
+
+        foreach ($expectedMessages as $message) {
+            $this->assertStringContainsString($message, implode(' ',$output));
+        }
+    }
+}

Magento2/Tests/Eslint/AndSelfTest.js

@@ -0,0 +1,4 @@
+$(document).ready(function() {
+    'use strict';
+    $("div").find("p").andSelf().addClass("border");
+});

Magento2/Tests/Eslint/AndSelfTest.php

@@ -0,0 +1,24 @@
+<?php
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+declare(strict_types=1);
+
+namespace Magento2\Tests\Eslint;
+
+/**
+ * Class AndSelfTest
+ *
+ * Test Eslint Rule: jquery-no-andSelf.js
+ */
+class AndSelfTest extends AbstractEslintTestCase
+{
+    public function testExecute(): void
+    {
+        $this->assertFileContainsError(
+            'AndSelfTest.js',
+            ['jQuery.andSelf() removed, use jQuery.addBack()']
+        );
+    }
+}

Magento2/Tests/Eslint/BindUnbindTest.js

@@ -0,0 +1,4 @@
+$(document).ready(function(){
+    'use strict';
+    $(".btn1").bind("click");
+});

Magento2/Tests/Eslint/BindUnbindTest.php

@@ -0,0 +1,24 @@
+<?php
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+declare(strict_types=1);
+
+namespace Magento2\Tests\Eslint;
+
+/**
+ * Class BindUnbindTest
+ *
+ * Test Eslint Rule: jquery-no-bind-unbind.js
+ */
+class BindUnbindTest extends AbstractEslintTestCase
+{
+    public function testExecute(): void
+    {
+        $this->assertFileContainsError(
+            'BindUnbindTest.js',
+            ['jQuery $.bind and $.unbind are deprecated, use $.on and $.off instead']
+        );
+    }
+}

Magento2/Tests/Eslint/ClickEventShorthand.js

@@ -0,0 +1,4 @@
+$(document).ready(function(){
+    'use strict';
+    $("input").blur();
+});

Magento2/Tests/Eslint/ClickEventShorthandTest.php

@@ -0,0 +1,24 @@
+<?php
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+declare(strict_types=1);
+
+namespace Magento2\Tests\Eslint;
+
+/**
+ * Class ClickEventShorthandTest
+ *
+ * Test Eslint Rule: jquery-no-click-event-shorthand.js
+ */
+class ClickEventShorthandTest extends AbstractEslintTestCase
+{
+    public function testExecute(): void
+    {
+        $this->assertFileContainsError(
+            'ClickEventShorthand.js',
+            ['Instead of .blur(fn) use .on("blur", fn). Instead of .blur() use .trigger("blur")']
+        );
+    }
+}

Magento2/Tests/Eslint/DelegateUndelegateTest.js

@@ -0,0 +1,6 @@
+$(document).ready(function(){
+    'use strict';
+    $( "table" ).delegate( "td", "click", function() {
+        $( this ).toggleClass( "chosen" );
+    });
+});