Apply review feedback

strongloop · Jun 9, 2016 · 7839b88 · 7839b88
1 parent 18473f9
commit 7839b88
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 11 deletions.
diff --git a/lib/application.js b/lib/application.js
@@ -115,7 +115,7 @@ app.model = function(Model, config) {
   if (typeof Model === 'string') {
     var msg = 'app.model(modelName, settings) is no longer supported. ' +
       'Use app.registry.createModel(modelName, definition) and ' +
-      'app.model(modelCtor, config) instead.';
+      'app.model(ModelCtor, config) instead.';
     throw new Error(msg);
   }
 

diff --git a/test/acl.test.js b/test/acl.test.js
@@ -393,20 +393,16 @@ describe('security ACLs', function() {
 });
 
 describe('access check', function() {
-  var app;
-  before(function() {
-    app = loopback();
+  it('should occur before other remote hooks', function(done) {
+    var app = loopback();
+    var MyTestModel = app.registry.createModel('MyTestModel');
+    var checkAccessCalled = false;
+    var beforeHookCalled = false;
+
     app.use(loopback.rest());
     app.enableAuth();
     app.dataSource('test', { connector: 'memory' });
-    var MyTestModel = app.registry.createModel('MyTestModel');
     app.model(MyTestModel, { dataSource: 'test' });
-  });
-
-  it('should occur before other remote hooks', function(done) {
-    var MyTestModel = app.models.MyTestModel;
-    var checkAccessCalled = false;
-    var beforeHookCalled = false;
 
     // fake / spy on the checkAccess method
     MyTestModel.checkAccess = function() {