From 0e9014b6ffc54f8960a75794808314e9791ca238 Mon Sep 17 00:00:00 2001
From: Drew Chandler <andrewschandler@gmail.com>
Date: Mon, 27 Mar 2017 11:29:23 -0700
Subject: [PATCH] Remove unnecessary relationship guard

---
 addon/-private/system/relationships/state/has-many.js     | 4 ++--
 addon/-private/system/relationships/state/relationship.js | 7 +------
 2 files changed, 3 insertions(+), 8 deletions(-)

diff --git a/addon/-private/system/relationships/state/has-many.js b/addon/-private/system/relationships/state/has-many.js
index ddfefc219a3..f961954d960 100644
--- a/addon/-private/system/relationships/state/has-many.js
+++ b/addon/-private/system/relationships/state/has-many.js
@@ -89,6 +89,8 @@ export default class ManyRelationship extends Relationship {
     if (this.members.has(record)) {
       return;
     }
+
+    assertPolymorphicType(this.record, this.relationshipMeta, record);
     super.addRecord(record, idx);
     // make lazy later
     this.manyArray.internalAddInternalModels([record], idx);
@@ -130,8 +132,6 @@ export default class ManyRelationship extends Relationship {
   }
 
   notifyRecordRelationshipAdded(record, idx) {
-    assertPolymorphicType(this.record, this.relationshipMeta, record);
-
     this.record.notifyHasManyAdded(this.key, record, idx);
   }
 
diff --git a/addon/-private/system/relationships/state/relationship.js b/addon/-private/system/relationships/state/relationship.js
index 53ad44c8485..0c506170b8b 100644
--- a/addon/-private/system/relationships/state/relationship.js
+++ b/addon/-private/system/relationships/state/relationship.js
@@ -95,12 +95,7 @@ export default class Relationship {
 
     allMembers.forEach(inverseInternalModel => {
       let relationship = inverseInternalModel._relationships.get(this.inverseKey);
-      // TODO: there is always a relationship in this case; this guard exists
-      // because there are tests that fail in teardown after putting things in
-      // invalid state
-      if (relationship) {
-        relationship.inverseDidDematerialize();
-      }
+      relationship.inverseDidDematerialize();
     });
   }