Merge pull request #1112 from strongloop/fix/failing-test

Fix test broken by recent juggler changes

package.json

@@ -52,7 +52,7 @@
     "underscore.string": "~2.3.3"
   },
   "peerDependencies": {
-    "loopback-datasource-juggler": "^2.8.0"
+    "loopback-datasource-juggler": "^2.17.0"
   },
   "devDependencies": {
     "browserify": "~4.2.3",
@@ -79,7 +79,7 @@
     "karma-phantomjs-launcher": "~0.1.4",
     "karma-script-launcher": "~0.1.0",
     "loopback-boot": "^1.1.0",
-    "loopback-datasource-juggler": "^2.8.0",
+    "loopback-datasource-juggler": "^2.17.0",
     "loopback-testing": "~0.2.0",
     "mocha": "~1.21.4",
     "serve-favicon": "~2.1.3",

test/loopback.test.js

@@ -344,6 +344,8 @@ describe('loopback', function() {
         emailVerificationRequired: false
       });
 
+      var baseName = model.settings.base.name;
+
       loopback.configureModel(model, {
         dataSource: null,
         options: {
@@ -357,7 +359,9 @@ describe('loopback', function() {
       expect(model.settings).to.have.property('emailVerificationRequired',
         false);
       expect(model.settings).to.have.property('realmRequired', true);
-      expect(model.settings).to.not.have.property('base');
+
+      // configureModel MUST NOT change Model's base class
+      expect(model.settings.base.name).to.equal(baseName);
     });
   });