Merge pull request #4718 from canufeel/fix/inverse-null-relationships

[BUGFIX beta] Inverse null relationships should throw if model doesn't exist 3

addon/-private/system/relationships/state/create.js

@@ -2,6 +2,7 @@ import Ember from 'ember';
 import ManyRelationship from "ember-data/-private/system/relationships/state/has-many";
 import BelongsToRelationship from "ember-data/-private/system/relationships/state/belongs-to";
 import EmptyObject from "ember-data/-private/system/empty-object";
+import { runInDebug } from 'ember-data/-private/debug';
 
 const { get } = Ember;
 
@@ -16,6 +17,10 @@ function createRelationshipFor(internalModel, relationshipMeta, store) {
 
   if (shouldFindInverse(relationshipMeta)) {
     inverse = internalModel.type.inverseFor(relationshipMeta.key, store);
+  } else {
+    runInDebug(() => {
+      internalModel.type.typeForRelationship(relationshipMeta.key, store);
+    });
   }
 
   if (inverse) {

tests/integration/relationships/inverse-relationships-test.js

@@ -599,3 +599,17 @@ test("inverseFor is only called when inverse is not null", function(assert) {
     });
   });
 });
+
+testInDebug("Inverse null relationships with models that don't exist throw a nice error", function(assert) {
+  User = DS.Model.extend({
+    post: DS.belongsTo('post', { inverse: null })
+  });
+
+  var env = setupStore({ user: User });
+
+  assert.throws(function() {
+    run(function() {
+      env.store.createRecord('user');
+    });
+  }, /No model was found for 'post'/);
+});

tests/unit/debug-test.js

@@ -66,7 +66,8 @@ test("_debugInfo supports arbitray relationship types", function(assert) {
         options: { inverse: null },
         isRelationship: true,
         kind: 'customRelationship',
-        name: 'Custom Relationship'
+        name: 'Custom Relationship',
+        type: 'post'
       })
   });