Run JSCS against eslint/rules/ and Magento2/

Author: loginesta

Magento2/Tests/Eslint/AndSelfTest.js

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

Magento2/Tests/Eslint/BindUnbindTest.js

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

Magento2/Tests/Eslint/ClickEventShorthandTest.js

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

Magento2/Tests/Eslint/DelegateUndelegateTest.js

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

Magento2/Tests/Eslint/EventShorthandTest.js

@@ -1,4 +1,5 @@
-$(document).ready(function(){
+$(document).ready(function () {
     'use strict';
-    $( "#result" ).load( "ajax/test.html" );
-});
+
+    $('#result').load('ajax/test.html');
+});

Magento2/Tests/Eslint/SizeTest.js

@@ -1,4 +1,5 @@
-$(document).ready(function(){
+$(document).ready(function () {
     'use strict';
-    $("div").size();
-});
+
+    $('div').size();
+});

Magento2/Tests/Eslint/TrimTest.js

@@ -1,4 +1,5 @@
-$(document).ready(function(){
+$(document).ready(function () {
     'use strict';
-    $.trim("    hello, how are you?    ");
-});
+
+    $.trim('    hello, how are you?    ');
+});

Magento2/Tests/Legacy/CopyrightAnotherExtensionsFilesUnitTest.2.js

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

Magento2/Tests/Legacy/CopyrightAnotherExtensionsFilesUnitTest.4.js

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

eslint/rules/jquery-no-andSelf.js

@@ -1,34 +1,49 @@
 module.exports = {
-  meta: {
-    type: 'suggestion',
-    docs: {
-      description: 'Disallow the use of the deprecated `andSelf` method',
-      category: 'jQuery deprecated functions',
-      recommended: true,
-      url: 'https://api.jquery.com/andself/'
+    meta: {
+        type: 'suggestion',
+        docs: {
+            description: 'Disallow the use of the deprecated `andSelf` method',
+            category: 'jQuery deprecated functions',
+            recommended: true,
+            url: 'https://api.jquery.com/andself/'
+        },
+        schema: [],
+        messages: {
+            andSelf: 'jQuery.andSelf() removed, use jQuery.addBack()'
+        }
     },
-    schema: [],
-    messages: {
-      andSelf: 'jQuery.andSelf() removed, use jQuery.addBack()'
-    }
-  },
 
-  create: function(context) {
-    'use strict';
-    var utils = require('./utils.js');
+    /**
+     * Executes the function to check if andSelf is used.
+     *
+     * @param {Object} context
+     * @returns {Object}
+     */
+    create: function (context) {
+        'use strict';
 
-    return {
-      CallExpression: function(node) {
-        if (node.callee.type !== 'MemberExpression') return;
-        if (node.callee.property.name !== 'andSelf') return;
+        var utils = require('./utils.js');
 
-        if (utils.isjQuery(node)) {
-          context.report({
-            node: node,
-            messageId: 'andSelf'
-          });
-        }
-      }
-    };
-  }
+        return {
+            /**
+             * Checks if andSelf is used in the node and reports it.
+             *
+             * @param {Object} node - The node to check.
+             */
+            CallExpression: function (node) {
+                // jscs:disable requireCurlyBraces
+                if (node.callee.type !== 'MemberExpression') return;
+
+                if (node.callee.property.name !== 'andSelf') return;
+                // jscs:enable requireCurlyBraces
+
+                if (utils.isjQuery(node)) {
+                    context.report({
+                        node: node,
+                        messageId: 'andSelf'
+                    });
+                }
+            }
+        };
+    }
 };