WIP: add query tests

There are some tests that have weird timing issues, and because of the polling nature of the
original implementation, we never caught the issue. These should be resolved when able

Author: jshcrowthe

gulp/tasks/test.js

@@ -73,7 +73,7 @@ function runBrowserUnitTests(dev) {
         // Don't include binary test files
         './tests/**/binary/**/*.test.ts',
       ],
-      browsers: !!dev ? ['Chrome'] : config.karma.browsers
+      browsers: !!dev ? ['ChromeHeadless'] : config.karma.browsers,
     });
     new karma.Server(karmaConfig, done).start();
   };

src/database/core/Repo.ts

@@ -286,8 +286,9 @@ export class Repo {
    * @param {number|string|null} newPriority
    * @param {?function(?Error, *=)} onComplete
    */
-  setWithPriority(path: Path, newVal: any, newPriority: number | string | null,
-                          onComplete: ((status: Error | null, errorReason?: string) => any) | null) {
+  setWithPriority(path: Path, newVal: any, 
+                  newPriority: number | string | null,
+                  onComplete: ((status: Error | null, errorReason?: string) => any) | null) {
     this.log_('set', {path: path.toString(), value: newVal, priority: newPriority});
 
     // TODO: Optimize this behavior to either (a) store flag to skip resolving where possible and / or

tests/database/compound_write.test.ts

@@ -215,7 +215,7 @@ describe('CompoundWrite Tests', function() {
     for (var i = 0; i < completeChildren.length; i++) {
       actual[completeChildren[i].name] = completeChildren[i].node;
     }
-    expect(actual).to.equal(expected);
+    expect(actual).to.deep.equal(expected);
   });
 
   it('complete children return all children for root set', function() {
@@ -232,7 +232,7 @@ describe('CompoundWrite Tests', function() {
     for (var i = 0; i < completeChildren.length; i++) {
       actual[completeChildren[i].name] = completeChildren[i].node;
     }
-    expect(actual).to.equal(expected);
+    expect(actual).to.deep.equal(expected);
   });
 
   it('empty merge has no shadowing write', function() {
@@ -369,14 +369,14 @@ describe('CompoundWrite Tests', function() {
     var compoundWrite = CompoundWrite.Empty;
     var path = new Path('child-1');
     compoundWrite = compoundWrite.addWrite(path, LEAF_NODE);
-    expect(compoundWrite.apply(CHILDREN_NODE)).to.equal(CHILDREN_NODE.updateImmediateChild(path.getFront(), LEAF_NODE));
+    expect(compoundWrite.apply(CHILDREN_NODE)).to.deep.equal(CHILDREN_NODE.updateImmediateChild(path.getFront(), LEAF_NODE));
   });
 
   it('Updates existing child', function() {
     var compoundWrite = CompoundWrite.Empty;
     var path = new Path('child-1/foo');
     compoundWrite = compoundWrite.addWrite(path, LEAF_NODE);
-    expect(compoundWrite.apply(CHILDREN_NODE)).to.equal(CHILDREN_NODE.updateChild(path, LEAF_NODE));
+    expect(compoundWrite.apply(CHILDREN_NODE)).to.deep.equal(CHILDREN_NODE.updateChild(path, LEAF_NODE));
   });
 
   it("Doesn't update priority on empty node.", function() {

tests/database/helpers.ts

@@ -103,7 +103,4 @@ export function pause(milliseconds: number) {
 
 export function getPath(query: Query) {
   return query.toString().replace(TEST_PROJECT.databaseURL, '');
-}
-
-export function runs() {}
-export function waitsFor() {}
+}