Added another function to Table that will process async callback func… (

#571)

* Added another function to Table that will process async callback functions to resolve issue #570.  Also had to bump the ecmaVersion to 8 in eslint to support async await

Signed-off-by: Autumn Calhoun <autumn.calhoun@crowncastle.com>

* #570 updated per code review comments

Signed-off-by: Autumn Calhoun <autumn.calhoun@crowncastle.com>

---------

Signed-off-by: Autumn Calhoun <autumn.calhoun@crowncastle.com>

.eslintrc.json

@@ -9,7 +9,7 @@
     },
     "extends": "eslint:recommended",
     "parserOptions": {
-        "ecmaVersion": 6
+        "ecmaVersion": 8
     },
     "rules": {
         "curly": "error",

CONTRIBUTING.md

@@ -1,6 +1,6 @@
 ## Issues
 
-If you find any issues or have any feature requests, please file them in the [issue tracker](https://github.com/getgauge-contrib/gauge-js/issues).
+If you find any issues or have any feature requests, please file them in the [issue tracker](https://github.com/getgauge/gauge-js/issues).
 
 If you are filing issues, please provide the version of `gauge` core and `gauge-js` plugin that you have installed. You can find it by doing:
 
@@ -13,7 +13,7 @@ $ gauge -v
 **Download**
 
 ```sh
-$ git clone git://github.com/getgauge-contrib/gauge-js --recursive
+$ git clone git://github.com/getgauge/gauge-js --recursive
 ```
 
 **Setup**:

js.json

@@ -51,7 +51,7 @@
             "--init"
         ]
     },
-    "version": "2.4.0",
+    "version": "2.4.1",
     "gaugeVersionSupport": {
         "minimum": "1.0.7",
         "maximum": ""

package.json

@@ -1,6 +1,6 @@
 {
   "name": "gauge-js",
-  "version": "2.4.0",
+  "version": "2.4.1",
   "description": "JavaScript runner for Gauge",
   "main": "index.js",
   "scripts": {

src/table.js

@@ -1,11 +1,17 @@
 var Table = function (protoTable) {
   Object.assign(this, protoTable);
 
-  this.entries = function (callback) {
-    for (var row of this.rows) {
+  this.entries = async function (callback) {
+    const AsyncFunction = (async () => {}).constructor;
+
+    for (const row of this.rows) {
       let entry = {};
       row.cells.forEach((cell, index) => entry[this.headers.cells[index]] = cell);
-      callback(entry);
+      if(callback instanceof AsyncFunction) {
+        await callback(entry);
+      } else {
+        callback(entry);
+      }
     }
   };
 };

test/table.js

@@ -1,5 +1,7 @@
 var Table = require("../src/table");
 var expect = require("chai").expect;
+const util = require("util");
+const setTimeoutPromise = util.promisify(setTimeout);
 
 describe("ProtoTable parsing", function() {
 
@@ -17,6 +19,14 @@ describe("ProtoTable parsing", function() {
 
   var table = new Table(protoTable);
 
+  let getRowData = function(entry) {
+    const rowData = setTimeoutPromise(500, entry).then((value) => {
+      return {cells: [value["Product"], value["Description"]]};
+    });
+    return rowData;
+
+  };
+
   it("Should get headers", function () {
     expect(table.headers).to.deep.equal(protoTable.headers);
   });
@@ -41,6 +51,15 @@ describe("ProtoTable parsing", function() {
         "Description": "Agile project management"
       });
     });
+
+    it("Should process an asynchronous callback action", async function () {
+      let data = [];
+
+      await table.entries(async (entry) => data.push(await getRowData(entry)));
+
+      expect(data).to.deep.equal(protoTable.rows);
+    }).timeout(10000);
+
   });
 
 });