fix code for eslint (attempt 2)

Author: WisdomQuest

__test__/diffFiles.test.js

@@ -17,26 +17,23 @@ const expectFormatJson = readFile('expectDiffFormatJson.txt');
 const expectFormatPlain = readFile('expectDiffFormatPlain.txt');
 const expectFormatStylish = readFile('expectDiffFormatStylish.txt');
 
+const file1Json = '__fixtures__/file1.json';
+const file2Json = '__fixtures__/file2.json';
+const file1Yaml = '__fixtures__/file1.yaml';
+const file2Yaml = '__fixtures__/file2.yaml';
+
 test('compareFilesJsonFormatStylish', () => {
-  expect(genDiff('__fixtures__/file1.json', '__fixtures__/file2.json')).toEqual(
-    expectFormatStylish
-  );
+  expect(genDiff(file1Json, file2Json)).toEqual(expectFormatStylish);
 });
 
 test('compareFilesYamlFormatStylish', () => {
-  expect(
-    genDiff('__fixtures__/file1.yaml', '__fixtures__/file2.yaml', 'stylish')
-  ).toEqual(expectFormatStylish);
+  expect(genDiff(file1Yaml, file2Yaml, 'stylish')).toEqual(expectFormatStylish);
 });
 
 test('compareFilesJsonFormatPlain', () => {
-  expect(
-    genDiff('__fixtures__/file1.yaml', '__fixtures__/file2.json', 'plain')
-  ).toEqual(expectFormatPlain);
+  expect(genDiff(file1Yaml, file2Json, 'plain')).toEqual(expectFormatPlain);
 });
 
 test('compareFilesFormatJson', () => {
-  expect(
-    genDiff('__fixtures__/file1.yaml', '__fixtures__/file2.yaml', 'json')
-  ).toEqual(expectFormatJson);
+  expect(genDiff(file1Yaml, file2Yaml, 'json')).toEqual(expectFormatJson);
 });

coverage/lcov-report/block-navigation.js

@@ -1,87 +1,84 @@
- 
 var jumpToCode = (function init() {
-    // Classes of code we would like to highlight in the file view
-    var missingCoverageClasses = ['.cbranch-no', '.cstat-no', '.fstat-no'];
+  // Classes of code we would like to highlight in the file view
+  var missingCoverageClasses = ['.cbranch-no', '.cstat-no', '.fstat-no'];
 
-    // Elements to highlight in the file listing view
-    var fileListingElements = ['td.pct.low'];
+  // Elements to highlight in the file listing view
+  var fileListingElements = ['td.pct.low'];
 
-    // We don't want to select elements that are direct descendants of another match
-    var notSelector = ':not(' + missingCoverageClasses.join('):not(') + ') > '; // becomes `:not(a):not(b) > `
+  // We don't want to select elements that are direct descendants of another match
+  var notSelector = ':not(' + missingCoverageClasses.join('):not(') + ') > '; // becomes `:not(a):not(b) > `
 
-    // Selecter that finds elements on the page to which we can jump
-    var selector =
-        fileListingElements.join(', ') +
-        ', ' +
-        notSelector +
-        missingCoverageClasses.join(', ' + notSelector); // becomes `:not(a):not(b) > a, :not(a):not(b) > b`
+  // Selecter that finds elements on the page to which we can jump
+  var selector =
+    fileListingElements.join(', ') +
+    ', ' +
+    notSelector +
+    missingCoverageClasses.join(', ' + notSelector); // becomes `:not(a):not(b) > a, :not(a):not(b) > b`
 
-    // The NodeList of matching elements
-    var missingCoverageElements = document.querySelectorAll(selector);
+  // The NodeList of matching elements
+  var missingCoverageElements = document.querySelectorAll(selector);
 
-    var currentIndex;
+  var currentIndex;
 
-    function toggleClass(index) {
-        missingCoverageElements
-            .item(currentIndex)
-            .classList.remove('highlighted');
-        missingCoverageElements.item(index).classList.add('highlighted');
-    }
-
-    function makeCurrent(index) {
-        toggleClass(index);
-        currentIndex = index;
-        missingCoverageElements.item(index).scrollIntoView({
-            behavior: 'smooth',
-            block: 'center',
-            inline: 'center'
-        });
-    }
+  function toggleClass(index) {
+    missingCoverageElements.item(currentIndex).classList.remove('highlighted');
+    missingCoverageElements.item(index).classList.add('highlighted');
+  }
 
-    function goToPrevious() {
-        var nextIndex = 0;
-        if (typeof currentIndex !== 'number' || currentIndex === 0) {
-            nextIndex = missingCoverageElements.length - 1;
-        } else if (missingCoverageElements.length > 1) {
-            nextIndex = currentIndex - 1;
-        }
+  function makeCurrent(index) {
+    toggleClass(index);
+    currentIndex = index;
+    missingCoverageElements.item(index).scrollIntoView({
+      behavior: 'smooth',
+      block: 'center',
+      inline: 'center',
+    });
+  }
 
-        makeCurrent(nextIndex);
+  function goToPrevious() {
+    var nextIndex = 0;
+    if (typeof currentIndex !== 'number' || currentIndex === 0) {
+      nextIndex = missingCoverageElements.length - 1;
+    } else if (missingCoverageElements.length > 1) {
+      nextIndex = currentIndex - 1;
     }
 
-    function goToNext() {
-        var nextIndex = 0;
+    makeCurrent(nextIndex);
+  }
 
-        if (
-            typeof currentIndex === 'number' &&
-            currentIndex < missingCoverageElements.length - 1
-        ) {
-            nextIndex = currentIndex + 1;
-        }
+  function goToNext() {
+    var nextIndex = 0;
 
-        makeCurrent(nextIndex);
+    if (
+      typeof currentIndex === 'number' &&
+      currentIndex < missingCoverageElements.length - 1
+    ) {
+      nextIndex = currentIndex + 1;
     }
 
-    return function jump(event) {
-        if (
-            document.getElementById('fileSearch') === document.activeElement &&
-            document.activeElement != null
-        ) {
-            // if we're currently focused on the search input, we don't want to navigate
-            return;
-        }
+    makeCurrent(nextIndex);
+  }
 
-        switch (event.which) {
-            case 78: // n
-            case 74: // j
-                goToNext();
-                break;
-            case 66: // b
-            case 75: // k
-            case 80: // p
-                goToPrevious();
-                break;
-        }
-    };
+  return function jump(event) {
+    if (
+      document.getElementById('fileSearch') === document.activeElement &&
+      document.activeElement != null
+    ) {
+      // if we're currently focused on the search input, we don't want to navigate
+      return;
+    }
+
+    switch (event.which) {
+      case 78: // n
+      case 74: // j
+        goToNext();
+        break;
+      case 66: // b
+      case 75: // k
+      case 80: // p
+        goToPrevious();
+        break;
+    }
+  };
 })();
 window.addEventListener('keydown', jumpToCode);