Update eslint config to include unused imports and indenting

.eslintrc.json

@@ -1,6 +1,6 @@
 {
   "parser": "@babel/eslint-parser",
-  "plugins": ["prettier"],
+  "plugins": ["prettier", "unused-imports"],
   "extends": ["prettier"],
   "env": {
     "es6": true,
@@ -24,6 +24,17 @@
     "one-var": [0],
     "max-len": [0, { "code": 140, "ignoreUrls": true }],
     "comma-dangle": ["error", "always-multiline"],
-    "arrow-parens": 1
+    "arrow-parens": 1,
+    "indent": ["error", 2, { "SwitchCase": 1 }],
+    "unused-imports/no-unused-imports": "error",
+    "unused-imports/no-unused-vars": [
+      "error",
+      {
+        "vars": "all",
+        "varsIgnorePattern": "^_",
+        "args": "after-used",
+        "argsIgnorePattern": "^_"
+      }
+    ]
   }
 }

README.md

@@ -153,3 +153,8 @@ yarn build
 - [Sass](https://marketplace.visualstudio.com/items?itemName=robinbentley.sass-indented)
 - [Stylelint](https://marketplace.visualstudio.com/items?itemName=stylelint.vscode-stylelint)
 - [vscode-remark-lint](https://marketplace.visualstudio.com/items?itemName=drewbourne.vscode-remark-lint)
+
+## Recommended Visual Studio Code settings for this project
+
+- "render whitespace" set to "all"
+- "insert spaces when pressing tab" checked

package.json

@@ -76,6 +76,7 @@
     "eslint-cli": "^1.1.1",
     "eslint-config-prettier": "^9.0.0",
     "eslint-plugin-prettier": "^5.0.0",
+    "eslint-plugin-unused-imports": "^3.1.0",
     "express": "^4.17.1",
     "front-matter": "^4.0.2",
     "fs-extra": "^11.1.1",

src/components/access-code/_macro.spec.js

@@ -124,11 +124,7 @@ describe('macro: access-code', () => {
       }),
     );
 
-    expect(
-      $('.ons-panel__body')
-        .text()
-        .trim(),
-    ).toBe('Example security message.');
+    expect($('.ons-panel__body').text().trim()).toBe('Example security message.');
   });
 
   it('has provided `postTextBoxLinkText` and `postTextBoxLinkUrl`', () => {
@@ -139,11 +135,7 @@ describe('macro: access-code', () => {
       }),
     );
 
-    expect(
-      $('a[href="#3"]')
-        .text()
-        .trim(),
-    ).toBe('Example link text');
+    expect($('a[href="#3"]').text().trim()).toBe('Example link text');
   });
 
   it('has provided `error` output', () => {

src/components/access-code/access-code.dom.js

@@ -6,6 +6,6 @@ domready(async () => {
   if (accessCodeInputs.length) {
     const accessCode = (await import('./access-code')).default;
 
-    accessCodeInputs.forEach(element => new accessCode(element));
+    accessCodeInputs.forEach((element) => new accessCode(element));
   }
 });

src/components/access-code/access-code.spec.js

@@ -14,12 +14,12 @@ describe('script: access-code', () => {
       await page.focus('#test-access-code');
       await page.keyboard.type('1234');
 
-      const valueSample1 = await page.$eval('#test-access-code', element => element.value);
+      const valueSample1 = await page.$eval('#test-access-code', (element) => element.value);
       expect(valueSample1).toBe('1234');
 
       await page.keyboard.type('5678');
 
-      const valueSample2 = await page.$eval('#test-access-code', element => element.value);
+      const valueSample2 = await page.$eval('#test-access-code', (element) => element.value);
       expect(valueSample2).toBe('12345 678');
     });
   });

src/components/accordion/accordion.js

@@ -5,14 +5,14 @@ export default class Accordion {
     this.button = button;
     this.buttonInner = button.querySelector('.ons-accordion__toggle-all-inner');
     this.group = button.getAttribute('data-group');
-    this.detailsEls = detailsEls.filter(details => details.group === this.group);
+    this.detailsEls = detailsEls.filter((details) => details.group === this.group);
     this.totalDetailsEls = this.detailsEls.length;
     this.buttonOpenEl = this.buttonInner.querySelector('.ons-btn__text');
     this.buttonOpen = this.buttonOpenEl.innerHTML.trim();
     this.closeButton = button.getAttribute('data-close-all');
-    this.open = this.detailsEls.find(details => details.open === true);
+    this.open = this.detailsEls.find((details) => details.open === true);
 
-    this.detailsEls.forEach(details => {
+    this.detailsEls.forEach((details) => {
       details.onOpen = this.onOpen.bind(this);
       details.onClose = this.onClose.bind(this);
     });
@@ -31,7 +31,7 @@ export default class Accordion {
 
     const open = !this.canClose();
 
-    this.detailsEls.forEach(details => {
+    this.detailsEls.forEach((details) => {
       details.setOpen(open);
     });
   }

src/components/address-input/_macro.spec.js

@@ -240,11 +240,7 @@ describe('macro: address-input', () => {
         }),
       );
 
-      expect(
-        $('.ons-js-address-search-btn')
-          .text()
-          .trim(),
-      ).toBe('Search for address');
+      expect($('.ons-js-address-search-btn').text().trim()).toBe('Search for address');
     });
   });
 
@@ -408,11 +404,7 @@ describe('macro: address-input', () => {
       );
 
       expect($('.ons-js-address-manual-btn').attr('href')).toBe('#0');
-      expect(
-        $('.ons-js-address-manual-btn')
-          .text()
-          .trim(),
-      ).toBe('Manually enter address');
+      expect($('.ons-js-address-manual-btn').text().trim()).toBe('Manually enter address');
     });
 
     it('renders `manualLinkText` with `manualLink` when provided', () => {
@@ -425,11 +417,7 @@ describe('macro: address-input', () => {
       );
 
       expect($('.ons-js-address-manual-btn').attr('href')).toBe('https://example.com/edit-address');
-      expect(
-        $('.ons-js-address-manual-btn')
-          .text()
-          .trim(),
-      ).toBe('Manually enter address');
+      expect($('.ons-js-address-manual-btn').text().trim()).toBe('Manually enter address');
     });
   });
 

src/components/address-input/autosuggest.address.dom.js

@@ -6,7 +6,7 @@ async function initialiseAddressAutosuggests() {
   if (addressAutosuggests.length) {
     const AutosuggestAddress = (await import('./autosuggest.address')).default;
 
-    addressAutosuggests.forEach(addressAutosuggest => new AutosuggestAddress(addressAutosuggest));
+    addressAutosuggests.forEach((addressAutosuggest) => new AutosuggestAddress(addressAutosuggest));
   }
 }
 

src/components/address-input/autosuggest.address.js

@@ -224,7 +224,7 @@ export default class AutosuggestAddress {
     for (const item of items) {
       if (item.addressCount === 1 && item.firstUprn !== 0) {
         let result = await this.createAddressObject(item.firstUprn);
-        const matchingItem = items.findIndex(x => x.firstUprn == result.uprn);
+        const matchingItem = items.findIndex((x) => x.firstUprn == result.uprn);
         items[matchingItem] = result;
       }
     }
@@ -244,7 +244,8 @@ export default class AutosuggestAddress {
   }
 
   testFullPostcodeQuery(input) {
-    const fullPostcodeRegex = /\b((?:(?:gir)|(?:[a-pr-uwyz])(?:(?:[0-9](?:[a-hjkpstuw]|[0-9])?)|(?:[a-hk-y][0-9](?:[0-9]|[abehmnprv-y])?)))) ?([0-9][abd-hjlnp-uw-z]{2})\b/i;
+    const fullPostcodeRegex =
+      /\b((?:(?:gir)|(?:[a-pr-uwyz])(?:(?:[0-9](?:[a-hjkpstuw]|[0-9])?)|(?:[a-hk-y][0-9](?:[0-9]|[abehmnprv-y])?)))) ?([0-9][abd-hjlnp-uw-z]{2})\b/i;
     const testFullPostcode = fullPostcodeRegex.test(input);
     if (testFullPostcode) {
       return true;

src/components/address-input/autosuggest.address.setter.js

@@ -103,7 +103,7 @@ export default class AddressSetter {
   }
 
   clearManualInputs() {
-    this.manualInputs.forEach(input => {
+    this.manualInputs.forEach((input) => {
       if (input) {
         input.value = '';
       }
@@ -112,13 +112,13 @@ export default class AddressSetter {
 
   checkManualInputsValues(onLoad) {
     if (onLoad) {
-      this.originalValues = this.manualInputs.map(input => {
+      this.originalValues = this.manualInputs.map((input) => {
         if (input) {
           return input.value;
         }
       });
     } else if (this.uprn.value !== '' && this.originalValues.length) {
-      this.newValues = this.manualInputs.map(input => {
+      this.newValues = this.manualInputs.map((input) => {
         return input.value;
       });
       if (this.originalValues.toString() !== this.newValues.toString()) {