added tests for a bug where results were null and the parser was tryi…

…ng to return json. FIxed the bug as well
dmill-bz · Apr 9, 2016 · c865e90 · c865e90
1 parent 5b0cd7e
commit c865e90
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/src/GraphSONTextParser.js b/src/GraphSONTextParser.js
@@ -31,7 +31,11 @@ class GraphSONTextParser extends Parser {
      * @return {Mixed} raw server results
      */
     getResults() {
-        return this.getRawResults()[0].json;
+        if(this.getRawResults() !== null && typeof this.getRawResults()[0] !== "undefined") {
+            return this.getRawResults()[0].json;
+        } else {
+            return [];
+        }
     }
 
     /**

diff --git a/test/GraphSONTextParserTest.js b/test/GraphSONTextParserTest.js
@@ -43,6 +43,11 @@ describe('GraphSONTextParser', () => {
             const parser = new Parser({message:"error message"}, [{text:["id=lala"], json: [{id:"lala"}]}]);
             expect(parser.getResults()).to.eql([{id:"lala"}]);
         });
+
+        it('should return [] on null results', () => {
+            const parser = new Parser({message:"error message"}, null);
+            expect(parser.getResults()).to.eql([]);
+        });
     });
 
     describe('.getHtmlResults()', () => {