Appease the linter!

Author: rossjones

lib/importer/attribute-types.test.js

@@ -53,7 +53,7 @@ test('optional basic string', () => {
 test('required basic number', () => {
   const t = attributeTypes.requiredType(attributeTypes.basicNumberType);
 
-  r = t("foo");
+  let r = t("foo");
   expect(r.valid).toBeFalsy();
   expect(r.warnings).toMatchObject([]);
   expect(r.errors).toMatchObject(["This is not a valid number"]);
@@ -68,7 +68,7 @@ test('optional basic number', () => {
   // Also tests that optionalType converts errors into warnings
   const t = attributeTypes.optionalType(attributeTypes.basicNumberType);
 
-  r = t("foo");
+  let r = t("foo");
   expect(r.valid).toBeTruthy();
   expect(r.value).toBeUndefined();
   expect(r.warnings).toMatchObject(["This is not a valid number"]);

lib/importer/backend.js

@@ -460,6 +460,7 @@ function validateMapping(range, mapping) {
 
   // Output types are optional, we fall back to basicStringType if not specified
   if(mapping.attributeTypes) {
+    // eslint-disable-next-line no-unused-vars
     for(const [attribute, attrType] of Object.entries(mapping.attributeTypes)) {
       assert(attrType instanceof Function);
     }
@@ -515,7 +516,7 @@ exports.SessionPerformMappingJob = (sid, range, mapping) => {
 
     if (foundSomeValues) {
       // Only if we found something do we validate and map the types
-      mappedRecord = {};
+      let mappedRecord = {};
 
       Object.entries(record).forEach((element) => {
         const [attr, inputVal] = element;

lib/importer/backend.test.js

@@ -468,7 +468,7 @@ describe("Backend tests", () => {
 
 test('validation', () => {
   const fixtures = testFiles.get("test-validation").map(prefix_fixture)
-  for( [filename, sheetname] of fixtures) {
+  for( const [filename, sheetname] of fixtures) {
     const sid = backend.CreateSession();
     backend.SessionSetFile(sid, filename);