fix(@embark/test-runner): fix describe in describe tests

dapps/tests/app/test/token_spec.js

@@ -50,46 +50,52 @@ config({
   }
 });
 
-describe("Token", function () {
+describe("Token", function() {
   this.timeout(0);
 
-  it("not deploy Token", function () {
+  describe('Token Contract', function() {
-    assert.strictEqual(Token.address, undefined);
+    it("not deploy Token", function() {
+      assert.strictEqual(Token.address, undefined);
+    });
   });
 
-  it("should deploy MyToken and MyToken2", function () {
+  describe('MyToken Contracts', function() {
-    assert.ok(MyToken.options.address);
+    it("should deploy MyToken and MyToken2", function() {
-    assert.ok(MyToken2.options.address);
+      assert.ok(MyToken.options.address);
+      assert.ok(MyToken2.options.address);
+    });
+
+    it("set MyToken Balance correctly", async function() {
+      let result = await MyToken.methods._supply().call();
+      assert.strictEqual(parseInt(result, 10), 1000);
+    });
+
+    it("set MyToken2 Balance correctly", async function() {
+      let result = await MyToken2.methods._supply().call();
+      assert.strictEqual(parseInt(result, 10), 2000);
+    });
   });
 
-  it("set MyToken Balance correctly", async function () {
+  describe('Other Contarcts', function() {
-    let result = await MyToken.methods._supply().call();
+    it("get right address", function() {
-    assert.strictEqual(parseInt(result, 10), 1000);
+      assert.strictEqual(AlreadyDeployedToken.options.address.toLowerCase(),
-  });
+        "0xCAFECAFECAFECAFECAFECAFECAFECAFECAFECAFE".toLowerCase());
+    });
 
-  it("set MyToken2 Balance correctly", async function () {
+    it("should use onDeploy", async function() {
-    let result = await MyToken2.methods._supply().call();
+      let result = await Test.methods.addr().call();
-    assert.strictEqual(parseInt(result, 10), 2000);
+      assert.strictEqual(result, MyToken.options.address);
-  });
+    });
 
-  it("get right address", function () {
+    it("should not deploy if deployIf returns false", function() {
-    assert.strictEqual(AlreadyDeployedToken.options.address.toLowerCase(),
+      assert.ok(!SomeContract.options.address);
-      "0xCAFECAFECAFECAFECAFECAFECAFECAFECAFECAFE".toLowerCase());
+    });
-  });
 
-  it("should use onDeploy", async function () {
+    it("should set the ens attr to the address of embark.eth", async function() {
-    let result = await Test.methods.addr().call();
+      let result = await Test.methods.ens().call();
-    assert.strictEqual(result, MyToken.options.address);
+      // Testing that it is an address as we don't really know the address
-  });
+      assert.strictEqual(web3.utils.isAddress(result), true);
-
+      assert.notStrictEqual(result, '0x0000000000000000000000000000000000000000');
-  it("should not deploy if deployIf returns false", function() {
+    });
-    assert.ok(!SomeContract.options.address);
-  });
-
-  it("should set the ens attr to the address of embark.eth", async function() {
-    let result = await Test.methods.ens().call();
-    // Testing that it is an address as we don't really know the address
-    assert.strictEqual(web3.utils.isAddress(result), true);
-    assert.notStrictEqual(result, '0x0000000000000000000000000000000000000000');
   });
 });

packages/embark-test-runner/src/index.js

@@ -235,9 +235,20 @@ class TestRunner {
                 if (global.embark.needConfig) {
                   global.config({});
                 }
-                global.embark.onReady((_err, accounts) => {
+
+                function runDescribe(accounts) {
                   self.ogMochaDescribe(describeName, callback.bind(mocha, accounts));
+                  setImmediate(() => {
+                    global.run(); // Call `run` in setImmediate to make sure tests with no `config()` don't hang
+                  });
                   global.run(); // This tells mocha that it can run the test (used in conjunction with `delay()`
+                }
+                if (global.embark.ready) {
+                  // Call describe straightaway otherwise `describe` inside `describe` end with 0 Passing
+                  return runDescribe(global.embark.accounts);
+                }
+                global.embark.onReady((_err, accounts) => {
+                  runDescribe(accounts);
                 });
               }