Merge pull request #8 from stackql/feat/add-tests

Feat/add tests

Author: yunchengyang515

.github/workflows/stackql-assert.yml

@@ -20,7 +20,7 @@ jobs:
       run: | ## use the base64 encoded secret to create json file
         sudo echo ${{ secrets.GOOGLE_CREDS }} | base64 -d > sa-key.json
 
-    - name: query test statement example with auth_obj_path
+    - name: Use test query string and expected results string with auth object
       uses: ./
       with:
         auth_obj_path: './.github/workflows/workflow_scripts/auth.json'
@@ -31,14 +31,14 @@ jobs:
             WHERE project = 'stackql-demo' AND zone = 'australia-southeast1-a' AND name = 'stackql-demo-001';
         expected_results_str: ' [{"name":"stackql-demo-001","status":"TERMINATED"}]'
 
-    - name: query test file example with auth_obj_path
+    - name: Use test query file and expected result string with auth object
       uses: ./
       with:
         auth_obj_path: './.github/workflows/workflow_scripts/auth.json'
         test_query_file_path: './.github/workflows/workflow_scripts/google-example.iql'
         expected_results_str: ' [{"name":"stackql-demo-001","status":"TERMINATED"}]'
 
-    - name: query test statement example with auth string
+    - name: Use test query string and expected results string, with auth string
       uses: ./
       with:
         auth_str: '{ "google": { "type": "service_account",  "credentialsfilepath": "sa-key.json" }}'
@@ -49,19 +49,19 @@ jobs:
             WHERE project = 'stackql-demo' AND zone = 'australia-southeast1-a' AND name = 'stackql-demo-001';
         expected_results_str: '[{"name":"stackql-demo-001","status":"TERMINATED"}]'
 
-    - name: query test file example with auth string
+    - name: Use test query string and expected results file, with auth object
       uses: ./
       with:
-        auth_str: '{ "google": { "type": "service_account",  "credentialsfilepath": "sa-key.json" }}'
+        auth_obj_path: './.github/workflows/workflow_scripts/auth.json'
         test_query_file_path: './.github/workflows/workflow_scripts/google-example.iql'
         expected_results_str: '[{"name":"stackql-demo-001","status":"TERMINATED"}]'
 
-    - name: query test file while expected result is a file
+    - name: Use test query string and expected rows, with auth object
       uses: ./
       with:
         auth_str: '{ "google": { "type": "service_account",  "credentialsfilepath": "sa-key.json" }}'
         test_query_file_path: './.github/workflows/workflow_scripts/google-example.iql'
-        expected_results_file_path: './.github/workflows/workflow_scripts/expected_results.json'
+        expected_rows: 1
 
 
 

.github/workflows/test.yml

@@ -0,0 +1,18 @@
+name: 'Build and Test'
+on:
+  push:
+  pull_request:
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+      
+    steps:
+      - uses: actions/checkout@v3
+      - name: Use Node.js 16
+        uses: actions/setup-node@v3
+        with:
+          node-version: 16.x
+      - run: npm ci
+      - run: npm test
+        

.gitignore

@@ -0,0 +1 @@
+node_modules

.stackql/readline/readline.tmp

@@ -0,0 +1,7 @@
+REGISTRY PULL github v23.01.00104;
+REGISTRY PULL github;
+SHOW PROVIDERS ;
+SHOW PROVIDERS;
+REGISTRY PULL google v23.01.00116;
+REGISTER PULL github latest;
+REGISTRY PULL github latest;

action.yml

@@ -18,7 +18,7 @@ inputs:
     description: expected number of rows from executing test query
     required: false
   expected_results_str:
-    description: expected result from executing test query, support object string
+    description: expected result from executing test query, support object string, overrides expected_results_file_path
     required: false
   expected_results_file_path:
     description: file that stores expected result, json is support
@@ -57,7 +57,7 @@ runs:
       uses: actions/github-script@v6
       with:
         script: |
-          const assertResult = require('./action_scripts/assert.js')
+          const {assertResult} = require('./stackql-assert.js')
           assertResult(core)
       env:
         RESULT: ${{steps.exec-query.outputs.stdout}}