uncheckAllItems method not update checked state of parent node

packages/elements/src/tree/__test__/tree.test.js

@@ -384,6 +384,50 @@ describe('tree/Tree', () => {
       expect(el.values).to.deep.equal(['1.1', '1.2', '4']);
     });
 
+    it('Uncheck all items correctly with deep nested data', async () => {
+      const el = await fixture('<ef-tree multiple></ef-tree>');
+      el.data = deepNestedData;
+      await elementUpdated(el);
+      el.uncheckAll();
+      el.expandAll();
+      await elementUpdated(el);
+      const item = el.children[3];
+      const itemChild = el.children[4];
+      itemChild.click();
+      await elementUpdated(el);
+      el.uncheckAll();
+      await elementUpdated(el);
+      expect(item.checkedState).to.equal(0);
+      el.uncheckAll();
+      await elementUpdated(el);
+      el.values = ['1.3.1.1'];
+      await elementUpdated(el);
+      el.uncheckAll();
+      await elementUpdated(el);
+      expect(item.checkedState).to.equal(0);
+      expect(itemChild.checkedState).to.equal(0);
+    });
+
+    it('check/uncheck all items correctly in no-relation with deep nested data', async () => {
+      const el = await fixture('<ef-tree multiple no-relation></ef-tree>');
+      el.data = deepNestedData;
+      await elementUpdated(el);
+      el.uncheckAll();
+      el.expandAll();
+      await elementUpdated(el);
+      const item = el.children[3];
+      const itemChild = el.children[4];
+      itemChild.click();
+      await elementUpdated(el);
+      el.uncheckAll();
+      await elementUpdated(el);
+      expect(item.checkedState).to.equal(0);
+      el.checkAll();
+      await elementUpdated(el);
+      expect(item.checkedState).to.equal(1);
+      expect(itemChild.checkedState).to.equal(1);
+    });
+
     it('Can set values programmatically', async () => {
       const el = await fixture('<ef-tree multiple></ef-tree>');
       el.data = nestedData;

packages/elements/src/tree/managers/tree-manager.ts

@@ -410,6 +410,7 @@ export class TreeManager<T extends TreeDataItem> {
         this.forceUpdateOnPath(item);
         this.getItemDescendants(item).forEach(descendant => this._uncheckItem(descendant, false));
       }
+      this.updateItem(item);
       return true;
     }
     return false;
@@ -437,6 +438,8 @@ export class TreeManager<T extends TreeDataItem> {
    * @returns {void}
    */
   public uncheckAllItems (): void {
-    this.editableItems.forEach(item => this.uncheckItem(item));
+    // uncheck items from top levels when manage relationships to avoid redundant re-renders
+    const items = this.manageRelationships ? this.composer.queryItems(() => true, 0) : this.checkedItems;
+    items.forEach(item => this.uncheckItem(item));
   }
 }