Fixes #51

Author: pigulla

src/modules/parser.ts

@@ -12,7 +12,7 @@ const info = debug('gotql:info:parser')
  */
 function getQueryVars (variables: QueryType['variables']): string {
   info('Getting query variables')
-  if (!variables) return ''
+  if (!variables || Object.keys(variables).length === 0) return ''
 
   let queryVars = '('
   for (let varName in variables) {
@@ -236,7 +236,7 @@ function parseOperation (query: QueryType, allowEmptyFields: boolean = false): s
 
   try {
     let operationArgs = ''
-    if (operation.args) {
+    if (operation.args && Object.keys(operation.args).length > 0) {
       info('Parsing args')
       operationArgs = parseArgs(query.operation.args, query.variables)
       info('Operation args are: %s', operationArgs)

tests/parser.test.ts

@@ -673,4 +673,34 @@ describe('parser', () => {
         const mutationResult = parse(query, ExecutionType.MUTATION)
         expect(mutationResult).toEqual(mutationReturn)
     })
+
+    it('should not add parenthesis if the args object is empty (#TBD)', () => {
+        const query = {
+            name: 'TestQuery',
+            operation: {
+                name: 'TestOp',
+                args: {},
+                fields: ['field1', 'field2']
+            }
+        }
+
+        const testReturn = 'query TestQuery { TestOp { field1 field2 } }'
+        const queryResult = parse(query, ExecutionType.QUERY)
+        expect(queryResult).toEqual(testReturn)
+    });
+
+    it('should handle the empty object for variables (#TBD)', () => {
+        const query = {
+            name: 'TestQuery',
+            operation: {
+                name: 'TestOp',
+                fields: ['field1', 'field2']
+            },
+            variables: {}
+        }
+
+        const testReturn = 'query TestQuery { TestOp { field1 field2 } }'
+        const queryResult = parse(query, ExecutionType.QUERY)
+        expect(queryResult).toEqual(testReturn)
+    });
 })