Added registry auth options

.github/workflows/ci.yml

@@ -57,6 +57,6 @@ jobs:
         id: test-action
         uses: ./
         with:
-          salad-organization: salad
-          salad-project: development
-          salad-container-group: test-actions
+          salad_organization: salad
+          salad_project: development
+          salad_container_group: test-actions

__tests__/main.test.ts

@@ -33,57 +33,57 @@ describe('action', () => {
     setOutputMock = jest.spyOn(core, 'setOutput').mockImplementation()
   })
 
-  it('sets the time output', async () => {
-    // Set the action's inputs as return values from core.getInput()
-    getInputMock.mockImplementation(name => {
-      switch (name) {
-        case 'milliseconds':
-          return '500'
-        default:
-          return ''
-      }
-    })
+  // it('sets the time output', async () => {
+  //   // Set the action's inputs as return values from core.getInput()
+  //   getInputMock.mockImplementation(name => {
+  //     switch (name) {
+  //       case 'milliseconds':
+  //         return '500'
+  //       default:
+  //         return ''
+  //     }
+  //   })
 
-    await main.run()
-    expect(runMock).toHaveReturned()
+  //   await main.run()
+  //   expect(runMock).toHaveReturned()
 
-    // Verify that all of the core library functions were called correctly
-    expect(debugMock).toHaveBeenNthCalledWith(1, 'Waiting 500 milliseconds ...')
-    expect(debugMock).toHaveBeenNthCalledWith(
-      2,
-      expect.stringMatching(timeRegex)
-    )
-    expect(debugMock).toHaveBeenNthCalledWith(
-      3,
-      expect.stringMatching(timeRegex)
-    )
-    expect(setOutputMock).toHaveBeenNthCalledWith(
-      1,
-      'time',
-      expect.stringMatching(timeRegex)
-    )
-    expect(errorMock).not.toHaveBeenCalled()
-  })
+  //   // Verify that all of the core library functions were called correctly
+  //   expect(debugMock).toHaveBeenNthCalledWith(1, 'Waiting 500 milliseconds ...')
+  //   expect(debugMock).toHaveBeenNthCalledWith(
+  //     2,
+  //     expect.stringMatching(timeRegex)
+  //   )
+  //   expect(debugMock).toHaveBeenNthCalledWith(
+  //     3,
+  //     expect.stringMatching(timeRegex)
+  //   )
+  //   expect(setOutputMock).toHaveBeenNthCalledWith(
+  //     1,
+  //     'time',
+  //     expect.stringMatching(timeRegex)
+  //   )
+  //   expect(errorMock).not.toHaveBeenCalled()
+  // })
 
-  it('sets a failed status', async () => {
-    // Set the action's inputs as return values from core.getInput()
-    getInputMock.mockImplementation(name => {
-      switch (name) {
-        case 'milliseconds':
-          return 'this is not a number'
-        default:
-          return ''
-      }
-    })
+  // it('sets a failed status', async () => {
+  //   // Set the action's inputs as return values from core.getInput()
+  //   getInputMock.mockImplementation(name => {
+  //     switch (name) {
+  //       case 'milliseconds':
+  //         return 'this is not a number'
+  //       default:
+  //         return ''
+  //     }
+  //   })
 
-    await main.run()
-    expect(runMock).toHaveReturned()
+  //   await main.run()
+  //   expect(runMock).toHaveReturned()
 
-    // Verify that all of the core library functions were called correctly
-    expect(setFailedMock).toHaveBeenNthCalledWith(
-      1,
-      'milliseconds not a number'
-    )
-    expect(errorMock).not.toHaveBeenCalled()
-  })
+  //   // Verify that all of the core library functions were called correctly
+  //   expect(setFailedMock).toHaveBeenNthCalledWith(
+  //     1,
+  //     'milliseconds not a number'
+  //   )
+  //   expect(errorMock).not.toHaveBeenCalled()
+  // })
 })

action.yml

@@ -9,20 +9,75 @@ branding:
 
 # Define your inputs here.
 inputs:
-  salad-organization:
+  salad_organization:
     description: 'Your Salad organization name.'
     required: true
-  salad-project:
+
+  salad_project:
     description: 'Your Salad project name.'
     required: true
-  salad-container-group:
+
+  salad_container_group:
     description: 'Your Salad container group name.'
     required: true
 
-# # Define your outputs here.
-# outputs:
-#   time:
-#     description: 'Your output description here'
+  salad_api_key:
+    description: 'Your Salad API key'
+    required: true
+    default: ${{secrets.SALAD_API_KEY}}
+
+  image_name:
+    description: 'The name of the container, including the registry to deploy'
+    required: true
+
+  # ghcr
+  github_token:
+    description:
+      'Personal Access Token (PAT) used to authenticate with the GitHub
+      Container Registry.'
+    required: false
+
+  # basic auth
+  basic_username:
+    description:
+      'The username for basic auth - to be used in conjunction with
+      `basic_password`'
+    required: false
+
+  basic_password:
+    description:
+      'The passord for basic auth - to be used in conjunction with
+      `basic_username`'
+    required: false
+
+  # gcp gcr
+  service_key:
+    description:
+      'The username for basic auth - to be used in conjunction with
+      `basic_password`'
+    required: false
+
+  # ecr
+  aws_access_key_id:
+    description:
+      'AWS Access Key ID - to be used in conjunction with
+      `aws_secret_access_key`'
+    required: false
+
+  aws_secret_access_key:
+    description:
+      'AWS Secret Access Key - to be used in conjunction with
+      `aws_access_key_id`'
+    required: false
+
+  # docker hub
+  dockerhub_username:
+    description: 'Docker Hub Username'
+    required: false
+
+  dockerhub_password:
+    description: 'Docker Hub Personal Access Token'
+    required: false
 
 runs:
   using: node20

src/main.ts

@@ -7,9 +7,9 @@ import * as core from '@actions/core'
 export async function run(): Promise<void> {
   try {
     core.warning('Starting action')
-    let org = core.getInput('salad-organization')
-    let proj = core.getInput('salad-project')
-    let containerGroup = core.getInput('salad-container-group')
+    const org = core.getInput('salad_organization')
+    const proj = core.getInput('salad_project')
+    const containerGroup = core.getInput('salad_container_group')
 
     core.warning('Org:' + org)
     core.warning('Project:' + proj)