allow tests to override array observer detection

emberjs · May 8, 2021 · 96adf26 · 96adf26
1 parent 744a9b9
commit 96adf26
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/packages/-ember-data/tests/unit/many-array-test.js b/packages/-ember-data/tests/unit/many-array-test.js
@@ -105,6 +105,7 @@ module('unit/many_array - DS.ManyArray', function (hooks) {
     let willChangeRemoveAmt;
     let willChangeAddAmt;
 
+    DS.ManyArray.proto().__hasArrayObservers = true;
     let originalArrayContentWillChange = DS.ManyArray.proto().arrayContentWillChange;
     let originalArrayContentDidChange = DS.ManyArray.proto().arrayContentDidChange;
 

diff --git a/packages/model/addon/-private/system/many-array.js b/packages/model/addon/-private/system/many-array.js
@@ -141,9 +141,14 @@ export default EmberObject.extend(MutableArray, DeprecatedEvented, {
     this.retrieveLatest();
   },
 
+  // TODO refactor away _hasArrayObservers for tests
+  get _hasArrayObservers() {
+    return this.hasArrayObservers || this.__hasArrayObservers;
+  },
+
   notify() {
     this._isDirty = true;
-    if (this.hasArrayObservers) {
+    if (this._hasArrayObservers) {
       this.retrieveLatest();
     } else {
       this.notifyPropertyChange('[]');
@@ -255,7 +260,7 @@ export default EmberObject.extend(MutableArray, DeprecatedEvented, {
       this._links = jsonApi.links;
     }
 
-    if (this.hasArrayObservers) {
+    if (this._hasArrayObservers) {
       // diff to find changes
       let diff = diffArray(this.currentState, internalModels);
       // it's null if no change found