fix(async-children): create observable for aysnc children only once (f…

…ixes #80)

src/tree.ts

@@ -82,20 +82,25 @@ export class Tree {
    * @returns {Observable<Tree[]>} An observable which emits children once they are loaded.
    */
   public get childrenAsync(): Observable<Tree[]> {
-    if(this.canLoadChildren()) {
-      setTimeout(() => this._childrenLoadingState = ChildrenLoadingState.Loading);
-      return new Observable((observer: Observer<Tree[]>) => {
+    if (this.canLoadChildren()) {
+      return this.childrenAsyncOnce();
+    }
+    return Observable.of(this.children);
+  }
+
+  private childrenAsyncOnce: () => Observable<Tree[]> = _.once(() => {
+    return new Observable((observer: Observer<Tree[]>) => {
+      setTimeout(() => {
+        this._childrenLoadingState = ChildrenLoadingState.Loading;
         this._loadChildren((children: TreeModel[]) => {
           this._children = _.map(children, (child: TreeModel) => new Tree(child, this));
           this._childrenLoadingState = ChildrenLoadingState.Completed;
           observer.next(this.children);
           observer.complete();
         });
       });
-    }
-
-    return Observable.of(this.children);
-  }
+    });
+  });
 
   /**
    * Create a new node in the current tree.

test/tree.spec.ts

@@ -744,6 +744,7 @@ describe('Tree', () => {
     });
 
     tree.switchFoldingType();
+    expect(tree.childrenAsync === tree.childrenAsync).toEqual(true, 'observable for children loading gets created just once');
     tree.childrenAsync.subscribe(() => {
       tree.childrenAsync.subscribe((children: Tree[]) => {
         expect(loadCount).toEqual(1, 'children should be loaded only once');