changed adding children test descriptions of first few

Author: jemzir

src/backend/__tests__/tree.test.ts

@@ -2,9 +2,9 @@ import Tree from '../tree';
 import { networkInterfaces } from 'os';
 
 describe('Tree unit test', () => {
-  describe('Constructor', () => {
-    const newTree = new Tree({});
+  const newTree = new Tree({});
 
+  describe('Constructor', () => {
     it('should be able to create a newTree', () => {
       expect(newTree.state).toEqual({});
     });
@@ -31,29 +31,35 @@ describe('Tree unit test', () => {
 
   /**
    *
-   * the tree should have initial values of state, name, etc to be default as per newly created tree
+   * ^^
+   * the tree should have initial values of state,
+   * name, etc to be default as per newly created tree
+   * update the add child and add sibling tests
    *
+   * update the clean tree copy test to make it test for deep equaltiy? (note:
+   * this test may always fail if we make it so because there is no way to have deep equalituy
+   * with some shit that isn't allowed)
    */
 
   describe('Adding children', () => {
-    let newTree = new Tree({});
-    let returnChild = newTree.addChild('stateful', 'child', {}, null);
-  
-    it('should be able to add a child', () => {
-      expect(typeof newTree.children).toEqual('object');
-      expect(Array.isArray(newTree.children)).toBeTruthy;
-    })
+    const returnChild = newTree.addChild('stateful', 'child', {}, null);
 
-    it(`its parent should be newTree`, () => {
+    it('should have the child be in the children\'s array property', () => {
+      // check if returnChild is in the children array property of tree that invoked addChild
+      expect(newTree.children).toContain(returnChild);
+    });
+
+    it('should have the object that invoked it be it\'s parent', () => {
+      // checking parent to be the tree that invoked addChild
       expect(returnChild.parent).toEqual(newTree);
-    })
+    });
 
     it('parent now contains an array of children and each children is a valid tree', () => {
       expect(newTree.children[0]).toHaveProperty('state');
       expect(newTree.children[0]).toHaveProperty('name');
       expect(newTree.children[0]).toHaveProperty('componentData');
-    })
-  })
+    });
+  });
 
   describe('Adding sibling', () => {
     let newTree = new Tree({});
@@ -75,27 +81,6 @@ describe('Tree unit test', () => {
     })  
   })
 
-
-  describe('Adding sibling', () => {
-    let newTree = new Tree({});
-    let returnChild = newTree.addChild('stateful', 'child', {}, null);
-    let returnSibling = returnChild.addSibling('stateful', 'child', {}, null);
-    it('the tree now has 2 children', () => {
-      expect(newTree.children.length).toBe(2);
-    })
-
-    it('both of the children has the parent as this tree', () => {
-      expect(newTree.children[0]).toEqual(returnChild);
-      expect(newTree.children[1]).toEqual(returnSibling);
-    })
-
-    it('both of the children has the parent as this tree', () => {
-      expect(returnChild.parent).toEqual(newTree);
-      expect(returnSibling.parent).toEqual(newTree);
-    })  
-  })
-
-
   describe('Copy & clean tree', () => {
     let newTree = new Tree({});
     let returnChild = newTree.addChild('stateful', 'child', {}, null);