Merge pull request #87 from antstackio/dev

New changes

Author: sleepy0owl

.github/workflows/test.yml

@@ -24,4 +24,4 @@ jobs:
           databricks-repo-branch: 'main'
           databricks-token: ${{ secrets.DATABRICKS_TOKEN }}
           databricks-host: 'https://dbc-89c9e463-7ff5.cloud.databricks.com'
-          databricks-repo-id: '3704530822795613'
+          databricks-repo-id: '2811515952360794'

__tests__/main.test.ts

@@ -2,19 +2,17 @@ import * as process from 'process'
 import {expect, test} from '@jest/globals'
 import * as utils from '../src/utils'
 
-
 test('test runs', () => {
     process.env['DATABRICKS_HOST'] = 'https://xyz.cloud.databricks.com'
     process.env['DATABRICKS_REPO_ID'] = '3704530822795627'
-  })
+})
 
 test('databricks host', () => {
     const host = utils.get_databricks_host()
-    expect(typeof host).toBe("string")
+    expect(typeof host).toBe('string')
 })
 
-
 test('databricks repo id', () => {
     const host = utils.get_databricks_repo_id()
-    expect(typeof host).toBe("string")
-})
+    expect(typeof host).toBe('string')
+})

dist/index.js

@@ -25,13 +25,21 @@ async function run(): Promise<void> {
         )
 
         const status = response.status
+        core.info(`status code ${status}`)
         if (status === 200) {
             core.info('Deployed the code successfully')
+        } else if (status === 401) {
+            core.info(
+                'Authentication error, please check your databricks token!'
+            )
         } else {
             core.setFailed('Failed to update the repo')
         }
     } catch (error) {
-        if (error instanceof Error) core.setFailed(error.message)
+        if (error instanceof Error) {
+            core.info(error.message)
+            core.setFailed(error.message)
+        }
     }
 }
 

dist/index.js.map

@@ -2,16 +2,14 @@ import * as core from '@actions/core'
 
 export function get_databricks_host(): string {
     const databricks_host_input: string = core.getInput('databricks-host')
-    const databricks_host_env: string = process.env['DATABRICKS_HOST'] || ''
+    const databricks_host_env: string = process.env['DATABRICKS_HOST'] ?? ''
 
     if (!databricks_host_input && !databricks_host_env) {
         throw new Error(
             'databricks-host or DATABRICKS_HOST environment variable must be set.'
         )
     } else {
-        return databricks_host_input
-            ? databricks_host_input
-            : databricks_host_env
+        return databricks_host_input || databricks_host_env
     }
 }
 
@@ -23,9 +21,7 @@ export function get_databricks_repo_id(): string {
             'databricks-repo-id or DATABRICKS_REPO_ID environment variable must be set.'
         )
     } else {
-        return databricks_repo_input
-            ? databricks_repo_input
-            : databricks_repo_env
+        return databricks_repo_input || databricks_repo_env
     }
 }