Fix query detection

s12v · Nov 19, 2017 · f307fe1 · f307fe1
1 parent cd5830b
commit f307fe1
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 1 deletion.
diff --git a/client/components/Table.jsx b/client/components/Table.jsx
@@ -30,7 +30,7 @@ export default class Table extends React.Component {
     }
 
     handleRequest(har) {
-        if (!HarUtils.getGraphQLQuery(har)) {
+        if (!HarUtils.isGraphQLQuery(har)) {
             return;
         }
 

diff --git a/client/services/HarUtils.js b/client/services/HarUtils.js
@@ -28,6 +28,11 @@ export default class HarUtils {
         return {};
     }
 
+    static isGraphQLQuery(har) {
+        const query = this.getGraphQLQuery(har);
+        return query.hasOwnProperty('query');
+    }
+
     static getGraphQLQuery(har) {
         if (!this.isJson(har)) {
             return {};

diff --git a/client/test/HarUtils.spec.js b/client/test/HarUtils.spec.js
@@ -29,6 +29,17 @@ describe('HarUtils', () => {
         expect(HarUtils.isJson(har)).to.equal(false);
     });
 
+    it('should detect a GraphQL query', () => {
+        let har = harWithHeader({}, 'content-type', 'application/json');
+        har = harWithPostData(har, '{"query": "{}"}');
+        expect(HarUtils.isGraphQLQuery(har)).to.equal(true);
+    });
+
+    it('should detect not a GraphQL query', () => {
+        let har = harWithHeader({}, 'content-type', 'application/json');
+        expect(HarUtils.isGraphQLQuery(har)).to.equal(false);
+    });
+
     it('should return GraphQL query', () => {
         let har = harWithHeader({}, 'content-type', 'application/json');
         har = harWithPostData(har, '{"query": "{}"}');