Fix: requireDollarBeforejQueryAssignment - Ignore array destructuring

Fixes #1860
Closes gh-1861

Author: SimenB

lib/rules/require-dollar-before-jquery-assignment.js

@@ -76,7 +76,7 @@ module.exports.prototype = {
             var right;
 
             if (type === 'VariableDeclarator') {
-                if (token.id.type === 'ObjectPattern') {
+                if (token.id.type === 'ObjectPattern' || token.id.type === 'ArrayPattern') {
                     return;
                 }
 

test/specs/rules/require-dollar-before-jquery-assignment.js

@@ -121,6 +121,10 @@ describe('rules/require-dollar-before-jquery-assignment', function() {
             expect(checker.checkString('const {beep, boop} = meep;\nvar $s = $("#id")')).to.have.no.errors();
         });
 
+        it('should not report on array destructuring', function() {
+            expect(checker.checkString('const [beep, boop] = meep;\nvar $s = $("#id")')).to.have.no.errors();
+        });
+
         it('should report with assignment on right hand side of object destructuring', function() {
             expect(checker.checkString('var {foo} = {foo: $(".foo")}'))
               .to.have.one.validation.error.from('requireDollarBeforejQueryAssignment');