fix: prevent array splice notification on non-mutating splice() calls (…

…#6786)

* always project splices

* Change files

* Update packages/web-components/fast-element/src/observation/arrays.spec.ts

---------

Co-authored-by: nicholasrice <[email protected]>

change/@microsoft-fast-element-44cfe95a-1f60-4e2e-a3f1-1d02aba63bab.json

@@ -0,0 +1,7 @@
+{
+    "type": "prerelease",
+    "comment": "Prevent notification of array splices when operation does not mutate array values",
+    "packageName": "@microsoft/fast-element",
+    "email": "[email protected]",
+    "dependentChangeType": "prerelease"
+}

packages/web-components/fast-element/src/observation/arrays.spec.ts

@@ -433,6 +433,25 @@ describe("The ArrayObserver", () => {
 
         expect(wasCalled).to.be.false;
     })
+
+    it("should not deliver splices for .splice() when .splice() does not change the items in the array", async () => {
+        ArrayObserver.enable();
+        const array = [1,2,3,4,5];
+        const observer = Observable.getNotifier(array);
+        let splices;
+
+        observer.subscribe({
+            handleChange(source, args) {
+                splices = args
+            }
+        });
+
+        array.splice(0, array.length, ...array);
+
+        await Updates.next();
+
+        expect(splices.length).to.equal(0);
+    })
 });
 
 describe("The array length observer", () => {

packages/web-components/fast-element/src/observation/arrays.ts

@@ -611,7 +611,7 @@ let defaultSpliceStrategy: SpliceStrategy = Object.freeze({
             if (changes === void 0) {
                 return emptyArray;
             }
-            return changes.length > 1 ? project(current, changes) : changes;
+            return project(current, changes);
         }
 
         return resetSplices;