chore: update to latest linting rules

lib/persisted-model.js

@@ -1901,7 +1901,7 @@ module.exports = function(registry) {
       }
 
       next();
-    };
+    }
 
     function deleteHandler(ctx, next) {
       var change = createChangeObject(ctx, 'delete');
@@ -1910,7 +1910,7 @@ module.exports = function(registry) {
       }
 
       next();
-    };
+    }
 
     function createChangeObject(ctx, type) {
       var where = ctx.where;

package.json

@@ -71,7 +71,7 @@
     "coveralls": "^3.0.2",
     "dirty-chai": "^1.2.2",
     "eslint": "^5.3.0",
-    "eslint-config-loopback": "^11.0.0",
+    "eslint-config-loopback": "^12.0.0",
     "eslint-plugin-mocha": "^5.1.0",
     "express-session": "^1.14.0",
     "grunt": "^1.0.1",

test/acl.test.js

@@ -642,7 +642,7 @@ describe('authorized roles propagation in RemotingContext', function() {
       });
     });
     return Promise.all(acls);
-  };
+  }
 
   function makeAuthorizedHttpRequestOnMyTestModel() {
     return request.get('/MyTestModels')

test/multiple-user-principal-types.test.js

@@ -778,9 +778,9 @@ describe('Multiple users with custom principalType', function() {
     return new Promise(function(resolve, reject) {
       emitter.once(name, resolve);
     });
-  };
+  }
 
   function getIds(array) {
     return array.map(function(it) { return it.id; });
-  };
+  }
 });

test/user.test.js

@@ -2989,7 +2989,7 @@ describe('User', function() {
         expect(actualIds).to.eql(expectedIds);
         done();
       });
-    };
+    }
 
     function assertNoAccessTokens(done) {
       AccessToken.find({where: {userId: user.pk}}, function(err, tokens) {