fix: prevent You attempted to update _tracking on Tag issue on Ember-data 4.5+ (#482)

* fix: prevent You attempted to update _tracking on Tag issue ibn Ember-data 4.6

* fix: support correctly ember data below 4.5

* ci: include the new scenario into the CI

---------

Co-authored-by: Vincent Molinié <[email protected]>

Author: VincentMolinie

.github/workflows/ci.yml

@@ -178,7 +178,7 @@ jobs:
       fail-fast: true
       matrix:
         ember-try-scenario:
-          [ember-lts-3.28, ember-lts-4.4, ember-classic]
+          [ember-lts-3.28, ember-lts-4.4, ember-4.6, ember-4.8, ember-classic]
         allow-failure: [false]
         include:
           - ember-try-scenario: ember-release

addon/record-data.js

@@ -967,12 +967,21 @@ export default class FragmentRecordData extends RecordData {
   }
 
   notifyStateChange(key) {
-    this.storeWrapper.notifyStateChange(
-      this.modelName,
-      this.id,
-      this.clientId,
-      key
-    );
+    if (key && gte('ember-data', '4.5.0')) {
+      this.storeWrapper.notifyPropertyChange(
+        this.modelName,
+        this.id,
+        this.clientId,
+        key
+      );
+    } else {
+      this.storeWrapper.notifyStateChange(
+        this.modelName,
+        this.id,
+        this.clientId,
+        key
+      );
+    }
   }
 
   /*

tests/dummy/config/ember-try.js

@@ -25,6 +25,24 @@ module.exports = async function () {
           },
         },
       },
+      {
+        name: 'ember-4.6',
+        npm: {
+          devDependencies: {
+            'ember-source': '~4.6.0',
+            'ember-data': '~4.6.0',
+          },
+        },
+      },
+      {
+        name: 'ember-4.8',
+        npm: {
+          devDependencies: {
+            'ember-source': '~4.8.0',
+            'ember-data': '~4.8.0',
+          },
+        },
+      },
       {
         name: 'ember-release',
         npm: {