Use strong-remoting's new TypeRegistry

weflex · Sep 9, 2016 · 6e1defc · 6e1defc
1 parent 92a5a08
commit 6e1defc
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 8 deletions.
diff --git a/lib/application.js b/lib/application.js
@@ -138,6 +138,9 @@ app.model = function(Model, config) {
   this.models().push(Model);
 
   if (isPublic && Model.sharedClass) {
+    this.remotes().defineObjectType(Model.modelName, function(data) {
+      return new Model(data);
+    });
     this.remotes().addClass(Model.sharedClass);
     if (Model.settings.trackChanges && Model.Change) {
       this.remotes().addClass(Model.Change.sharedClass);

diff --git a/lib/model.js b/lib/model.js
@@ -134,11 +134,6 @@ module.exports = function(registry) {
       remotingOptions
     );
 
-    // setup a remoting type converter for this model
-    RemoteObjects.convert(typeName, function(val) {
-      return val ? new ModelCtor(val) : val;
-    });
-
     // support remoting prototype methods
     ModelCtor.sharedCtor = function(data, id, fn) {
       var ModelCtor = this;

diff --git a/package.json b/package.json
@@ -46,7 +46,7 @@
     "ejs": "^2.3.1",
     "express": "^4.14.0",
     "inflection": "^1.6.0",
-    "loopback-connector-remote": "^2.0.0-alpha.1",
+    "loopback-connector-remote": "^2.0.0-alpha.2",
     "loopback-datasource-juggler": "^3.0.0-alpha.7",
     "isemail": "^1.2.0",
     "loopback-phase": "^1.2.0",
@@ -55,7 +55,7 @@
     "serve-favicon": "^2.2.0",
     "stable": "^0.1.5",
     "strong-globalize": "^2.6.2",
-    "strong-remoting": "^3.0.0-alpha.5",
+    "strong-remoting": "^3.0.0-alpha.6",
     "uid2": "0.0.3",
     "underscore.string": "^3.0.3"
   },

diff --git a/test/relations.integration.js b/test/relations.integration.js
@@ -656,6 +656,9 @@ describe('relations - integration', function() {
 
   describe('hasAndBelongsToMany', function() {
     beforeEach(function defineProductAndCategoryModels() {
+      // Disable "Warning: overriding remoting type product"
+      this.app.remotes()._typeRegistry._options.warnWhenOverridingType = false;
+
       var product = app.registry.createModel(
          'product',
          { id: 'string', name: 'string' }

diff --git a/test/remote-connector.test.js b/test/remote-connector.test.js
@@ -15,7 +15,10 @@ describe('RemoteConnector', function() {
     beforeEach: function(done) {
       var test = this;
       remoteApp = loopback();
-      remoteApp.set('remoting', { errorHandler: { debug: true, log: false }});
+      remoteApp.set('remoting', {
+        errorHandler: { debug: true, log: false },
+        types: { warnWhenOverridingType: false },
+      });
       remoteApp.use(loopback.rest());
       remoteApp.listen(0, function() {
         test.dataSource = loopback.createDataSource({
@@ -51,6 +54,9 @@ describe('RemoteConnector', function() {
   beforeEach(function(done) {
     var test = this;
     remoteApp = this.remoteApp = loopback();
+    remoteApp.set('remoting', {
+      types: { warnWhenOverridingType: false },
+    });
     remoteApp.use(loopback.rest());
     var ServerModel = this.ServerModel = loopback.PersistedModel.extend('TestModel');