From 960eae7a0458f8be3a4487cd18286447ec5d818c Mon Sep 17 00:00:00 2001
From: William Durand <will+git@drnd.me>
Date: Mon, 29 Jun 2020 10:55:00 +0200
Subject: [PATCH] fix tests

---
 tests/unit/scanners/test.javascript.js | 21 +++++++++++----------
 1 file changed, 11 insertions(+), 10 deletions(-)

diff --git a/tests/unit/scanners/test.javascript.js b/tests/unit/scanners/test.javascript.js
index 446944afa7a..06ac0e01ca3 100644
--- a/tests/unit/scanners/test.javascript.js
+++ b/tests/unit/scanners/test.javascript.js
@@ -88,7 +88,7 @@ describe('JavaScript Scanner', () => {
     const jsScanner = new JavaScriptScanner(code, 'code.js');
 
     const { linterMessages } = await jsScanner.scan();
-    expect(linterMessages.length).toEqual(0);
+    expect(linterMessages).toEqual([]);
   });
 
   it('should support object spread syntax', async () => {
@@ -100,7 +100,7 @@ describe('JavaScript Scanner', () => {
     const jsScanner = new JavaScriptScanner(code, 'code.js');
 
     const { linterMessages } = await jsScanner.scan();
-    expect(linterMessages.length).toEqual(0);
+    expect(linterMessages).toEqual([]);
   });
 
   it('should support optional chaining', async () => {
@@ -109,7 +109,7 @@ describe('JavaScript Scanner', () => {
     const jsScanner = new JavaScriptScanner(code, 'code.js');
 
     const { linterMessages } = await jsScanner.scan();
-    expect(linterMessages.length).toEqual(0);
+    expect(linterMessages).toEqual([]);
   });
 
   it('should support nullish coalescing operator', async () => {
@@ -118,9 +118,10 @@ describe('JavaScript Scanner', () => {
     const jsScanner = new JavaScriptScanner(code, 'code.js');
 
     const { linterMessages } = await jsScanner.scan();
-    expect(linterMessages.length).toEqual(0);
+    expect(linterMessages).toEqual([]);
   });
 
+  // See: https://github.com/tc39/proposal-class-fields
   // eslint-disable-next-line jest/no-disabled-tests
   it.skip('should support public class fields', async () => {
     const code = 'class MyClass { a = 1; }';
@@ -128,7 +129,7 @@ describe('JavaScript Scanner', () => {
     const jsScanner = new JavaScriptScanner(code, 'code.js');
 
     const { linterMessages } = await jsScanner.scan();
-    expect(linterMessages.length).toEqual(0);
+    expect(linterMessages).toEqual([]);
   });
 
   it('should support BigInt short-hand notation', async () => {
@@ -137,7 +138,7 @@ describe('JavaScript Scanner', () => {
     const jsScanner = new JavaScriptScanner(code, 'code.js');
 
     const { linterMessages } = await jsScanner.scan();
-    expect(linterMessages.length).toEqual(0);
+    expect(linterMessages).toEqual([]);
   });
 
   it('should support dynamic imports', async () => {
@@ -146,7 +147,7 @@ describe('JavaScript Scanner', () => {
     const jsScanner = new JavaScriptScanner(code, 'code.js');
 
     const { linterMessages } = await jsScanner.scan();
-    expect(linterMessages.length).toEqual(0);
+    expect(linterMessages).toEqual([]);
   });
 
   it('should support es6 modules', async () => {
@@ -168,7 +169,7 @@ describe('JavaScript Scanner', () => {
     const jsScanner = new JavaScriptScanner(code, 'code.js');
 
     const { linterMessages } = await jsScanner.scan();
-    expect(linterMessages.length).toEqual(0);
+    expect(linterMessages).toEqual([]);
   });
 
   it('should scan node modules', async () => {
@@ -260,7 +261,7 @@ describe('JavaScript Scanner', () => {
     const jsScanner = new JavaScriptScanner(code, 'badcode.js');
 
     const { linterMessages } = await jsScanner.scan();
-    expect(linterMessages.length).toEqual(0);
+    expect(linterMessages).toEqual([]);
   });
 
   // This test is pretty much copied from ESLint, to make sure dependencies
@@ -362,7 +363,7 @@ describe('JavaScript Scanner', () => {
       );
 
       const { linterMessages } = await runJsScanner(jsScanner);
-      expect(linterMessages.length).toEqual(0);
+      expect(linterMessages).toEqual([]);
     });
   });