* Add debug console output to watch response from query for top-level output.

* Better handling for no specification of outputs in query call.
* Add tests for top-level query output.

The top-level output may be being mangled through the OpenAPI stack. Need to investigate.

Author: robbear

client/src/sdk/LocalConnection.js

@@ -234,7 +234,9 @@ class LocalConnection extends Connection {
             action.source.type = 'Source'
 
             action.outputs = []
-            action.outputs.push(params.out)
+            if (params.out != null) {
+                action.outputs.push(params.out)
+            }
 
             action.source.name = params.name || 'query'
             action.source.path = params.path || ''
@@ -249,6 +251,14 @@ class LocalConnection extends Connection {
 
             this.runAction(action, txnParams)
                 .then(res => {
+                    // TODO(PR): remove
+                    // Compare the raw response.text with the OpenAPI-deserialized
+                    // TransactionResult to look at the top-level output. The expected
+                    // values appear correct in the response, but we run into AnyType
+                    // types in the TransactionResult top-level output.
+                    console.dir(JSON.parse(res.response.text));
+                    console.dir(res.transactionResult);
+                    
                     const tr = res.transactionResult;
                     const txn_output = tr.output;
                     const query_output = tr.actions[0].result.output;

client/test/sdk/LocalConnection.test.js

@@ -199,6 +199,48 @@ describe('LocalConnection', () => {
         })
     })
 
+    describe('#top-level query output', () => {
+        it(`def output = 2`, () => {
+            return defaultConnection.query({
+                query: 'def output = 2',
+            }).then(res => {
+                assert(res.txn_output[0].columns[0][0] === 2);
+            });
+        })
+        it(`def output = {(1,); (2,); (3,)}`, () => {
+            return defaultConnection.query({
+                query: 'def output = {(1,); (2,); (3,)}',
+            }).then(res => {
+                assert(setsEqual(new Set(res.txn_output[0].columns[0]), new Set([1,2,3])));
+            });
+        })
+        it(`def output = {(1.1,); (2.2,); (3.4,)}`, () => {
+            return defaultConnection.query({
+                query: 'def output = {(1.1,); (2.2,); (3.4,)}',
+            }).then(res => {
+                assert(setsEqual(new Set(res.txn_output[0].columns[0]), new Set([1.1,2.2,3.4])));
+            });
+        })
+        it(`def output = {(parse_decimal[64, 2, \"1.1\"],); (parse_decimal[64, 2, \"2.2\"],); (parse_decimal[64, 2, \"3.4\"],)}`, () => {
+            return defaultConnection.query({
+                query: 'def output = {(parse_decimal[64, 2, \"1.1\"],); (parse_decimal[64, 2, \"2.2\"],); (parse_decimal[64, 2, \"3.4\"],)}',
+            }).then(res => {
+                assert(setsEqual(new Set(res.txn_output[0].columns[0]), new Set([1.1,2.2,3.4])));
+            });
+        })
+        it(`def output = {(1, 5); (2, 7); (3, 9)}`, () => {
+            return defaultConnection.query({
+                query: 'def output = {(1, 5); (2, 7); (3, 9)}',
+            }).then(res => {
+                let success = [];
+                success = [[1,2,3],[5,7,9]].map((X,i) => {
+                    return setsEqual(new Set(X), new Set(res.txn_output[0].columns[i]));
+                });
+                assert.deepStrictEqual(success, [true, true]);
+            });
+        })
+    })
+
     describe('#list_edb', () => {
         it(`create_database() should execute without error`, () => {
             defaultConnection.defaultOpenMode = 'CREATE_OVERWRITE'