data-structure-typed 1.52.9 → 1.53.1

package/test/unit/data-structures/binary-tree/avl-tree-multi-map.test.ts

@@ -75,7 +75,7 @@ describe('AVLTreeMultiMap operations test1', () => {
     const nodeId10 = treeMultimap.getNode(10);
     expect(nodeId10?.key).toBe(10);
 
-    const nodeVal9 = treeMultimap.getNode(node => node.value === 9);
+    const nodeVal9 = treeMultimap.getNode(node => node.key === 9);
     expect(nodeVal9?.key).toBe(9);
 
     const nodesByCount1 = treeMultimap.getNodes(node => node.count === 1);
@@ -331,7 +331,7 @@ describe('AVLTreeMultiMap operations test recursively1', () => {
     const nodeId10 = treeMultimap.getNode(10);
     expect(nodeId10?.key).toBe(10);
 
-    const nodeVal9 = treeMultimap.getNode(node => node.value === 9);
+    const nodeVal9 = treeMultimap.getNode(node => node.key === 9);
     expect(nodeVal9?.key).toBe(9);
 
     const nodesByCount1 = treeMultimap.getNodes(node => node.count === 1);
@@ -678,7 +678,7 @@ describe('AVLTreeMultiMap iterative methods test', () => {
     expect(treeMM.count).toBe(21);
     const cloned = treeMM.clone();
     expect(cloned.root?.left?.key).toBe(1);
-    expect(cloned.root?.right?.value).toBe('c');
+    expect(cloned.get(cloned.root?.right)).toBe('c');
   });
 
   it('should keys', () => {
@@ -761,3 +761,115 @@ describe('AVLTree toEntryFn', () => {
     expect(tree.dfs(node => node.key, 'IN', tree.root, 'RECURSIVE')).toEqual(expected);
   });
 });
+
+describe('AVLTreeMultiMap not map mode count', () => {
+  let tm: AVLTreeMultiMap<number>;
+  beforeEach(() => {
+    tm = new AVLTreeMultiMap<number>([], { isMapMode: false });
+  });
+  it('Should added isolated node count ', () => {
+    tm.addMany([
+      [1, 1],
+      [2, 2],
+      [3, 3],
+      [4, 4],
+      [5, 5]
+    ]);
+    const newNode = new AVLTreeMultiMapNode(3, undefined, 10);
+    tm.add(newNode, 33);
+    expect(tm.count).toBe(15);
+  });
+});
+
+describe('AVLTreeMultiMap not map mode operations test1', () => {
+  it('should perform various operations on a Binary Search Tree with numeric values1', () => {
+    const treeMultimap = new AVLTreeMultiMap<number>([], { isMapMode: false });
+
+    expect(treeMultimap instanceof AVLTreeMultiMap);
+    treeMultimap.add([11, 11]);
+    treeMultimap.add([3, 3]);
+    const idAndValues: [number, number][] = [
+      [11, 11],
+      [3, 3],
+      [15, 15],
+      [1, 1],
+      [8, 8],
+      [13, 13],
+      [16, 16],
+      [2, 2],
+      [6, 6],
+      [9, 9],
+      [12, 12],
+      [14, 14],
+      [4, 4],
+      [7, 7],
+      [10, 10],
+      [5, 5]
+    ];
+    treeMultimap.addMany(idAndValues);
+    expect(treeMultimap.root instanceof AVLTreeMultiMapNode);
+
+    if (treeMultimap.root) expect(treeMultimap.root.key == 11);
+
+    expect(treeMultimap.size).toBe(16);
+    expect(treeMultimap.count).toBe(18);
+
+    expect(treeMultimap.has(6));
+
+    expect(treeMultimap.getHeight(6)).toBe(4);
+    expect(treeMultimap.getDepth(6)).toBe(0);
+    const nodeId10 = treeMultimap.getNode(10);
+    expect(nodeId10?.key).toBe(10);
+
+    const nodeVal9 = treeMultimap.getNode(node => node.key === 9);
+    expect(nodeVal9?.key).toBe(9);
+  });
+});
+
+describe('AVLTreeMultiMap not map mode operations test recursively1', () => {
+  it('should perform various operations on a Binary Search Tree with numeric values1', () => {
+    const treeMultimap = new AVLTreeMultiMap<number>([], {
+      iterationType: 'RECURSIVE',
+      isMapMode: false
+    });
+
+    expect(treeMultimap instanceof AVLTreeMultiMap);
+    treeMultimap.add([11, 11]);
+    treeMultimap.add([3, 3]);
+    const idAndValues: [number, number][] = [
+      [11, 11],
+      [3, 3],
+      [15, 15],
+      [1, 1],
+      [8, 8],
+      [13, 13],
+      [16, 16],
+      [2, 2],
+      [6, 6],
+      [9, 9],
+      [12, 12],
+      [14, 14],
+      [4, 4],
+      [7, 7],
+      [10, 10],
+      [5, 5]
+    ];
+    treeMultimap.addMany(idAndValues);
+    expect(treeMultimap.root).toBeInstanceOf(AVLTreeMultiMapNode);
+
+    if (treeMultimap.root) expect(treeMultimap.root.key).toBe(6);
+
+    expect(treeMultimap.size).toBe(16);
+    expect(treeMultimap.count).toBe(18);
+
+    expect(treeMultimap.has(6));
+
+    expect(treeMultimap.getHeight(6)).toBe(4);
+    expect(treeMultimap.getDepth(6)).toBe(0);
+    const nodeId10 = treeMultimap.getNode(10);
+    expect(nodeId10?.key).toBe(10);
+
+    const nodeVal9 = treeMultimap.getNode(node => node.key === 9);
+    expect(nodeVal9?.key).toBe(9);
+  });
+});

package/test/unit/data-structures/binary-tree/avl-tree.test.ts

@@ -32,8 +32,7 @@ describe('AVL Tree Test', () => {
     expect(lesserSum).toBe(45);
 
     // node15 has type problem. After the uniform design, the generics of containers (DirectedGraph, BST) are based on the type of value. However, this design has a drawback: when I attempt to inherit from the Vertex or BSTNode classes, the types of the results obtained by all methods are those of the parent class.
-    expect(node15?.value).toBe(15);
-
+    expect(node15?.value).toBe(undefined);
     const dfs = tree.dfs(node => node, 'IN');
     expect(dfs[0].key).toBe(1);
     expect(dfs[dfs.length - 1].key).toBe(16);
@@ -107,6 +106,25 @@ describe('AVL Tree Test', () => {
     expect(lastBFSNodes[1].key).toBe(2);
     expect(lastBFSNodes[2].key).toBe(16);
   });
+
+  it('should replace value', () => {
+    const tree = new AVLTree<number, string>([4, 5, [1, '1'], 2, 3], { isMapMode: false });
+    expect(tree.get(1)).toBe('1');
+    expect(tree.getNode(1)?.value).toBe('1');
+    tree.add(1, 'a');
+    expect(tree.get(1)).toBe('a');
+    tree.add([1, 'b']);
+    expect(tree.getNode(1)?.value).toBe('b');
+    expect(tree.get(1)).toBe('b');
+    const treeMap = new AVLTree<number>([4, 5, [1, '1'], 2, 3]);
+    expect(treeMap.get(1)).toBe('1');
+    expect(treeMap.getNode(1)?.value).toBe(undefined);
+    treeMap.add(1, 'a');
+    expect(treeMap.get(1)).toBe('a');
+    treeMap.add([1, 'b']);
+    expect(treeMap.getNode(1)?.value).toBe(undefined);
+    expect(treeMap.get(1)).toBe('b');
+  });
 });
 
 describe('AVL Tree Test recursively', () => {
@@ -140,7 +158,7 @@ describe('AVL Tree Test recursively', () => {
     expect(lesserSum).toBe(45);
 
     // node15 has type problem. After the uniform design, the generics of containers (DirectedGraph, BST) are based on the type of value. However, this design has a drawback: when I attempt to inherit from the Vertex or BSTNode classes, the types of the results obtained by all methods are those of the parent class.
-    expect(node15?.value).toBe(15);
+    expect(node15?.value).toBe(undefined);
 
     const dfs = tree.dfs(node => node, 'IN');
     expect(dfs[0].key).toBe(1);
@@ -419,7 +437,7 @@ describe('AVLTree iterative methods test', () => {
   it('should clone work well', () => {
     const cloned = avl.clone();
     expect(cloned.root?.left?.key).toBe(1);
-    expect(cloned.root?.right?.value).toBe('c');
+    expect(cloned.root?.right?.value).toBe(undefined);
   });
 
   it('should keys', () => {
@@ -437,3 +455,90 @@ describe('AVLTree iterative methods test', () => {
     expect(leaves).toEqual([1, 3]);
   });
 });
+
+describe('AVL Tree not map mode', () => {
+  it('should perform various operations on a AVL Tree', () => {
+    const arr = [11, 3, 15, 1, 8, 13, 16, 2, 6, 9, 12, 14, 4, 7, 10, 5];
+    const tree = new AVLTree<number>([], { isMapMode: false });
+
+    for (const i of arr) tree.add([i, i]);
+
+    tree.add(null);
+    const node6 = tree.getNode(6);
+
+    expect(node6 && tree.getHeight(node6)).toBe(3);
+    expect(node6 && tree.getDepth(node6)).toBe(1);
+
+    const getNodeById = tree.getNode(10);
+    expect(getNodeById?.key).toBe(10);
+
+    const getMinNodeByRoot = tree.getLeftMost();
+    expect(getMinNodeByRoot).toBe(1);
+
+    const node15 = tree.getNode(15);
+    const getMinNodeBySpecificNode = node15 && tree.getLeftMost(node => node, node15);
+    expect(getMinNodeBySpecificNode?.key).toBe(12);
+
+    let subTreeSum = 0;
+    if (node15) tree.dfs(node => (subTreeSum += node.key), 'PRE', node15);
+    expect(subTreeSum).toBe(70);
+
+    let lesserSum = 0;
+    tree.lesserOrGreaterTraverse(node => (lesserSum += node.key), -1, 10);
+    expect(lesserSum).toBe(45);
+
+    // node15 has type problem. After the uniform design, the generics of containers (DirectedGraph, BST) are based on the type of value. However, this design has a drawback: when I attempt to inherit from the Vertex or BSTNode classes, the types of the results obtained by all methods are those of the parent class.
+    expect(tree.get(node15)).toBe(15);
+  });
+});
+
+describe('AVL Tree not map mode test recursively', () => {
+  it('should perform various operations on a AVL Tree', () => {
+    const arr = [11, 3, 15, 1, 8, 13, 16, 2, 6, 9, 12, 14, 4, 7, 10, 5];
+    const tree = new AVLTree<number>([], { iterationType: 'RECURSIVE', isMapMode: false });
+
+    for (const i of arr) tree.add([i, i]);
+
+    const node6 = tree.getNode(6);
+
+    expect(node6 && tree.getHeight(node6)).toBe(3);
+    expect(node6 && tree.getDepth(node6)).toBe(1);
+
+    const getNodeById = tree.getNode(10);
+    expect(getNodeById?.key).toBe(10);
+
+    const getMinNodeByRoot = tree.getLeftMost();
+    expect(getMinNodeByRoot).toBe(1);
+
+    const node15 = tree.getNode(15);
+    const getMinNodeBySpecificNode = node15 && tree.getLeftMost(node => node, node15);
+    expect(getMinNodeBySpecificNode?.key).toBe(12);
+
+    let subTreeSum = 0;
+    if (node15) tree.dfs(node => (subTreeSum += node.key), 'PRE', node15);
+    expect(subTreeSum).toBe(70);
+
+    let lesserSum = 0;
+    tree.lesserOrGreaterTraverse(node => (lesserSum += node.key), -1, 10);
+    expect(lesserSum).toBe(45);
+
+    // node15 has type problem. After the uniform design, the generics of containers (DirectedGraph, BST) are based on the type of value. However, this design has a drawback: when I attempt to inherit from the Vertex or BSTNode classes, the types of the results obtained by all methods are those of the parent class.
+    expect(tree.get(node15)).toBe(15);
+  });
+});
+
+describe('AVLTree iterative methods not map mode', () => {
+  let avl: AVLTree<number, string>;
+  beforeEach(() => {
+    avl = new AVLTree<number, string>([], { isMapMode: false });
+    avl.add([1, 'a']);
+    avl.add([2, 'b']);
+    avl.add([3, 'c']);
+  });
+
+  it('should clone work well', () => {
+    const cloned = avl.clone();
+    expect(cloned.root?.left?.key).toBe(1);
+    expect(cloned.get(cloned.root?.right?.key)).toBe('c');
+  });
+});

package/test/unit/data-structures/binary-tree/binary-tree.test.ts

@@ -20,6 +20,8 @@ describe('BinaryTreeNode', () => {
 
   it('should set and get the value correctly', () => {
     const node: BinaryTreeNode<number> = new BinaryTreeNode<number>(1, 42);
+    expect(node.key).toBe(1);
+
     expect(node.value).toBe(42);
 
     node.value = 55;
@@ -100,10 +102,10 @@ describe('BinaryTree addMany', () => {
       ],
       [undefined, 22, 44, 33]
     );
-    expect(tree.getNodeByKey(2)?.value).toBe(22);
-    expect(tree.getNodeByKey(3)?.value).toBe(33);
-    expect(tree.getNodeByKey(4)?.value).toBe(44);
-    expect(tree.getNodeByKey(1)?.value).toBe(1);
+    expect(tree.get(2)).toBe(22);
+    expect(tree.get(tree.getNodeByKey(3))).toBe(33);
+    expect(tree.get(tree.getNodeByKey(4))).toBe(44);
+    expect(tree.get(tree.getNodeByKey(1))).toBe(1);
   });
 
   it('should addMany undefined and null', () => {
@@ -191,7 +193,7 @@ describe('BinaryTree', () => {
     const node4 = tree.getNode(4);
     expect(tree.has(node4)).toBe(false);
     expect(tree.has(node => node === node4)).toBe(false);
-    expect(tree.has(node => node.value?.toString() === '3')).toBe(true);
+    expect(tree.has(node => node.key?.toString() === '3')).toBe(true);
   });
 
   it('should the clone method work fine', () => {
@@ -319,26 +321,28 @@ describe('BinaryTree', () => {
   it('should isSubtreeBST', () => {
     expect(tree.toVisual()).toBe('');
     tree.addMany([4, 2, 6, 1, 3, 5, 7, 4]);
-    expect(tree.toVisual()).toBe('N for null\n' +
-      '    ___4___    \n' +
-      '   /       \\   \n' +
-      '  _2_     _6_  \n' +
-      ' /   \\   /   \\ \n' +
-      ' 1   3   5   7 \n' +
-      '               \n');
+    expect(tree.toVisual()).toBe(
+      'N for null\n' +
+        '    ___4___    \n' +
+        '   /       \\   \n' +
+        '  _2_     _6_  \n' +
+        ' /   \\   /   \\ \n' +
+        ' 1   3   5   7 \n' +
+        '               \n'
+    );
     const visualized = tree.toVisual(undefined, { isShowUndefined: true, isShowNull: true, isShowRedBlackNIL: true });
     expect(visualized).toBe(
       'U for undefined\n' +
-      'N for null\n' +
-      'S for Sentinel Node(NIL)\n' +
-      '        _______4_______        \n' +
-      '       /               \\       \n' +
-      '    ___2___         ___6___    \n' +
-      '   /       \\       /       \\   \n' +
-      '  _1_     _3_     _5_     _7_  \n' +
-      ' /   \\   /   \\   /   \\   /   \\ \n' +
-      ' U   U   U   U   U   U   U   U \n' +
-      '                               \n'
+        'N for null\n' +
+        'S for Sentinel Node(NIL)\n' +
+        '        _______4_______        \n' +
+        '       /               \\       \n' +
+        '    ___2___         ___6___    \n' +
+        '   /       \\       /       \\   \n' +
+        '  _1_     _3_     _5_     _7_  \n' +
+        ' /   \\   /   \\   /   \\   /   \\ \n' +
+        ' U   U   U   U   U   U   U   U \n' +
+        '                               \n'
     );
 
     expect(tree.isBST(tree.getNode(4), 'RECURSIVE')).toBe(true);
@@ -729,34 +733,59 @@ describe('BinaryTree', () => {
     ]);
   });
 
-  it('should keyValueOrEntryOrRawElementToNode', () => {
+  it('should keyValueNodeEntryRawToNodeAndValue', () => {
     const tree = new BinaryTree<number>();
-    const node0 = tree.keyValueOrEntryOrRawElementToNode(0);
-    expect(node0).toEqual({
-      _left: undefined,
-      _right: undefined,
-      key: 0,
-      parent: undefined,
-      value: undefined
-    });
+    const node0 = tree.keyValueNodeEntryRawToNodeAndValue(0);
+    expect(node0).toEqual([
+      {
+        _left: undefined,
+        _right: undefined,
+        key: 0,
+        parent: undefined,
+        value: undefined
+      },
+      undefined
+    ]);
 
-    const nodeUndefined = tree.keyValueOrEntryOrRawElementToNode(undefined);
-    expect(nodeUndefined).toBe(undefined);
+    const nodeUndefined = tree.keyValueNodeEntryRawToNodeAndValue(undefined);
+    expect(nodeUndefined).toEqual([undefined, undefined]);
 
-    const nodeNull = tree.keyValueOrEntryOrRawElementToNode(null);
-    expect(nodeNull).toBe(null);
+    const nodeNull = tree.keyValueNodeEntryRawToNodeAndValue(null);
+    expect(nodeNull).toEqual([null, undefined]);
 
-    const nodeWithSeparateValue = tree.keyValueOrEntryOrRawElementToNode(7, 77);
-    expect(nodeWithSeparateValue?.value).toBe(77);
+    const [, nodeWithSeparateValue] = tree.keyValueNodeEntryRawToNodeAndValue(7, 77);
+    expect(nodeWithSeparateValue).toBe(77);
 
-    expect(tree.keyValueOrEntryOrRawElementToNode([undefined, 2])).toBe(undefined);
+    expect(tree.keyValueNodeEntryRawToNodeAndValue([undefined, 2])).toEqual([undefined, undefined]);
 
-    expect(tree.keyValueOrEntryOrRawElementToNode(Symbol('test') as unknown as number)).toBe(undefined);
+    expect(tree.keyValueNodeEntryRawToNodeAndValue(Symbol('test') as unknown as number)).toEqual([
+      undefined,
+      undefined
+    ]);
 
     const bTree = new BinaryTree<number, number, { obj: { id: number } }>([], {
       toEntryFn: (ele: { obj: { id: number } }) => [Symbol('test') as unknown as number, ele.obj.id]
     });
-    expect(bTree.keyValueOrEntryOrRawElementToNode({ obj: { id: 1 } })).toBe(undefined);
+    expect(bTree.keyValueNodeEntryRawToNodeAndValue({ obj: { id: 1 } })).toEqual([undefined, undefined]);
+  });
+
+  it('should replace value', () => {
+    const tree = new BinaryTree<number, string>([4, 5, [1, '1'], 2, 3], { isMapMode: false });
+    expect(tree.get(1)).toBe('1');
+    expect(tree.getNode(1)?.value).toBe('1');
+    tree.add(1, 'a');
+    expect(tree.get(1)).toBe('a');
+    tree.add([1, 'b']);
+    expect(tree.getNode(1)?.value).toBe('b');
+    expect(tree.get(1)).toBe('b');
+    const treeMap = new BinaryTree<number>([4, 5, [1, '1'], 2, 3]);
+    expect(treeMap.get(1)).toBe('1');
+    expect(treeMap.getNode(1)?.value).toBe(undefined);
+    treeMap.add(1, 'a');
+    expect(treeMap.get(1)).toBe('a');
+    treeMap.add([1, 'b']);
+    expect(treeMap.getNode(1)?.value).toBe(undefined);
+    expect(treeMap.get(1)).toBe('b');
   });
 });
 
@@ -1039,9 +1068,9 @@ describe('BinaryTree', () => {
     const nodeB = tree.getNode(3);
 
     expect(nodeA?.key).toBe(5);
-    expect(nodeA?.value).toBe('A');
+    expect(nodeA?.value).toBe(undefined);
     expect(nodeB?.key).toBe(3);
-    expect(nodeB?.value).toBe('B');
+    expect(tree.get(nodeB)).toBe('B');
   });
 
   it('should return null when getting a non-existent node', () => {
@@ -1150,17 +1179,17 @@ describe('BinaryTree', () => {
     tree.add([2, 'B']);
     tree.add([null, 'null']);
 
-    const nodes = tree.getNodes(node => node.value === 'B');
+    const nodes = tree.getNodes(node => node.key === 2);
 
     expect(nodes.length).toBe(1);
     expect(nodes[0].key).toBe(2);
 
-    const nodesRec = tree.getNodes(node => node.value === 'B', false, tree.root, 'RECURSIVE');
+    const nodesRec = tree.getNodes(node => node.key === 2, false, tree.root, 'RECURSIVE');
 
     expect(nodesRec.length).toBe(1);
     expect(nodesRec[0].key).toBe(2);
 
-    const nodesItr = tree.getNodes(node => node.value === 'B', false, tree.root, 'ITERATIVE');
+    const nodesItr = tree.getNodes(node => node.key === 2, false, tree.root, 'ITERATIVE');
 
     expect(nodesItr.length).toBe(1);
     expect(nodesItr[0].key).toBe(2);
@@ -1206,6 +1235,97 @@ describe('BinaryTree', () => {
   });
 });
 
+describe('BinaryTree not map mode', () => {
+  let tree: BinaryTree<number, string>;
+
+  beforeEach(() => {
+    tree = new BinaryTree<number, string>([], {
+      iterationType: 'RECURSIVE',
+      isMapMode: false
+    });
+  });
+
+  afterEach(() => {
+    tree.clear();
+  });
+
+  it('should add and find nodes', () => {
+    tree.add([1, 1]);
+    tree.add(undefined);
+    tree.add([2, 2]);
+    tree.add([3, 3]);
+
+    expect(tree.has(1)).toBe(true);
+    expect(tree.has(2)).toBe(true);
+    expect(tree.has(3)).toBe(true);
+    expect(tree.has(4)).toBe(false);
+    const node4 = tree.getNode(4);
+    expect(tree.has(node4)).toBe(false);
+    expect(tree.has(node => node === node4)).toBe(false);
+    expect(tree.has(node => node.value?.toString() === '3')).toBe(true);
+  });
+
+  it('should isSubtreeBST', () => {
+    tree.addMany([
+      new BinaryTreeNode(4),
+      new BinaryTreeNode(2),
+      new BinaryTreeNode(6),
+      new BinaryTreeNode(1),
+      new BinaryTreeNode(3),
+      new BinaryTreeNode(5),
+      new BinaryTreeNode(7),
+      new BinaryTreeNode(4)
+    ]);
+
+    expect(tree.isBST(tree.getNode(4), 'RECURSIVE')).toBe(true);
+    expect(tree.isBST(tree.getNode(4), 'ITERATIVE')).toBe(true);
+  });
+
+  it('should get nodes by key', () => {
+    tree.add([5, 'A']);
+    tree.add([3, 'B']);
+    tree.add([7, 'C']);
+
+    const nodeA = tree.getNode(5);
+    const nodeB = tree.getNode(3);
+
+    expect(nodeA?.key).toBe(5);
+    expect(tree.get(nodeA)).toBe('A');
+    expect(nodeB?.key).toBe(3);
+    expect(tree.get(nodeB)).toBe('B');
+  });
+
+  it('should get nodes by a custom callback', () => {
+    tree.add([5, 'E']);
+    tree.add([4, 'D']);
+    tree.add([3, 'C']);
+    tree.add([7, 'G']);
+    tree.add([null, 'null']);
+    tree.add([1, 'A']);
+    tree.add([6, 'F']);
+    tree.add([null, 'null']);
+    tree.add([2, 'B']);
+    tree.add([null, 'null']);
+
+    const nodes = tree.getNodes(node => node.key === 2);
+
+    expect(nodes.length).toBe(1);
+    expect(nodes[0].key).toBe(2);
+
+    const nodesRec = tree.getNodes(node => node.key === 2, false, tree.root, 'RECURSIVE');
+
+    expect(nodesRec.length).toBe(1);
+    expect(nodesRec[0].key).toBe(2);
+
+    const nodesItr = tree.getNodes(node => node.key === 2, false, tree.root, 'ITERATIVE');
+
+    expect(nodesItr.length).toBe(1);
+    expect(nodesItr[0].key).toBe(2);
+
+    expect(nodesItr).toEqual(nodesRec);
+  });
+});
+
 describe('BinaryTree iterative methods test', () => {
   let binaryTree: BinaryTree<number, string>;
   beforeEach(() => {
@@ -1273,7 +1393,8 @@ describe('BinaryTree iterative methods test', () => {
   it('should clone work well', () => {
     const cloned = binaryTree.clone();
     expect(cloned.root?.left?.key).toBe(2);
-    expect(cloned.root?.right?.value).toBe('c');
+    expect(cloned.root?.right?.value).toBe(undefined);
+    expect(cloned.get(cloned.root?.right)).toBe('c');
   });
 
   it('should keys', () => {
@@ -1315,3 +1436,19 @@ describe('BinaryTree iterative methods test', () => {
     ]);
   });
 });
+
+describe('BinaryTree not map mode iterative methods test', () => {
+  let binaryTree: BinaryTree<number, string>;
+  beforeEach(() => {
+    binaryTree = new BinaryTree<number, string>([], { isMapMode: false });
+    binaryTree.add([1, 'a']);
+    binaryTree.add(2, 'b');
+    binaryTree.add([3, 'c']);
+  });
+
+  it('should clone work well', () => {
+    const cloned = binaryTree.clone();
+    expect(cloned.root?.left?.key).toBe(2);
+    expect(cloned.get(cloned.root?.right)).toBe('c');
+  });
+});