feat: allow skip-login (#11)

* feat: allow skip-login

* do not return

* use skip-login-check

__tests__/main.test.ts

@@ -61,6 +61,14 @@ describe('action', () => {
   })
 
   it('fails if your are not logged in', async () => {
+    getInputMock.mockImplementation(name => {
+      switch (name) {
+        case 'skip-login-check':
+          return 'false'
+        default:
+          return ''
+      }
+    })
     jest.spyOn(io, 'which').mockResolvedValue(path)
     stdErrMessage = 'Unauthorized'
     mockStatusCode = 1
@@ -85,6 +93,39 @@ describe('action', () => {
     )
   })
 
+  it('do not check login if skip-login-check', async () => {
+    getInputMock.mockImplementation(name => {
+      switch (name) {
+        case 'skip-login-check':
+          return 'true'
+        default:
+          return ''
+      }
+    })
+    jest.spyOn(io, 'which').mockResolvedValue(path)
+    stdOutMessage = ''
+    mockStatusCode = 0
+
+    await run.run()
+    expect(runMock).toHaveReturned()
+
+    expect(mockExecFn).not.toHaveBeenNthCalledWith(
+      1,
+      path,
+      ['org', 'list', '--format', 'json'],
+      expect.any(Object)
+    )
+
+    expect(mockExecFn).toHaveBeenNthCalledWith(
+      1,
+      path,
+      ['project', 'build'],
+      undefined
+    )
+
+    expect(infoMock).toHaveBeenNthCalledWith(1, 'Skipping login check.')
+  })
+
   it('does not push if push-project is false', async () => {
     getInputMock.mockImplementation(name => {
       switch (name) {

action.yml

@@ -7,7 +7,7 @@ inputs:
     description: 'Push Upbound project'
     default: 'false'
   project-file:
-    description: 'Path to project definition file. Default to upbound.yaml'
+    description: 'Path to project definition file. Default to upbound.yaml.'
     default: ''
   repository:
     description:
@@ -20,7 +20,10 @@ inputs:
       generated.'
     default: ''
   public:
-    description: 'Create new repositories with public visibility'
+    description: 'Create new repositories with public visibility.'
+    default: 'false'
+  skip-login-check:
+    description: 'Do not check if you are logged in.'
     default: 'false'
 
 branding:

src/main.ts

@@ -8,9 +8,16 @@ async function run(): Promise<void> {
   try {
     const upPath = await getUpPath()
 
-    const isLoggedIn = await verifyLogin(upPath)
-    if (!isLoggedIn) {
-      core.setFailed('User is not logged in. Please log in to Upbound.')
+    // NOTE (markanderstrocme): allowing skipping login check if people are using their own container registry
+    const skipLoginCheck = core.getInput('skip-login-check', { required: true })
+    if (skipLoginCheck.toLowerCase() === 'true') {
+      core.info('Skipping login check.')
+    } else {
+      const isLoggedIn = await verifyLogin(upPath)
+      if (!isLoggedIn) {
+        core.setFailed('User is not logged in. Please log in to Upbound.')
+        return
+      }
     }
 
     const projectFile = core.getInput('project-file')