Fixes #219 Add argument to allow only updating drafts and prereleases

__tests__/Action.test.ts

@@ -37,6 +37,7 @@ const updateBody = 'updateBody'
 const updateDraft = false
 const updateName = 'updateName'
 const updatePrerelease = false
+const updateOnlyUnreleased = false
 const url = 'http://api.example.com'
 
 describe("Action", () => {
@@ -317,7 +318,9 @@ describe("Action", () => {
         expect(applyReleaseDataMock).toBeCalledWith({id: releaseId, upload_url: url})
     }
 
-    function createAction(allowUpdates: boolean, hasArtifact: boolean, removeArtifacts: boolean = false): Action {
+    function createAction(allowUpdates: boolean,
+                          hasArtifact: boolean,
+                          removeArtifacts: boolean = false): Action {
         let inputArtifact: Artifact[]
         if (hasArtifact) {
             inputArtifact = artifacts
@@ -379,7 +382,8 @@ describe("Action", () => {
                 updatedDraft: updateDraft,
                 updatedReleaseBody: updateBody,
                 updatedReleaseName: updateName,
-                updatedPrerelease: updatePrerelease
+                updatedPrerelease: updatePrerelease,
+                updateOnlyUnreleased: updateOnlyUnreleased
             }
         })
         const MockOutputs = jest.fn<Outputs, any>(() => {

__tests__/Inputs.test.ts

@@ -430,6 +430,17 @@ describe('Inputs', () => {
         })
     })
 
+    describe('updateOnlyUnreleased', () => {
+        it('returns false', () => {
+            expect(inputs.updateOnlyUnreleased).toBe(false)
+        })
+
+        it('returns true', () => {
+            mockGetInput.mockReturnValueOnce('true')
+            expect(inputs.updateOnlyUnreleased).toBe(true)
+        })
+    })
+
     function createGlobber(): ArtifactGlobber {
         const MockGlobber = jest.fn<ArtifactGlobber, any>(() => {
             return {

__tests__/Integration.test.ts

@@ -57,7 +57,8 @@ describe.skip('Integration Test', () => {
                 updatedDraft: false,
                 updatedReleaseBody: "This release was generated by release-action's integration test",
                 updatedReleaseName: "Releases Action Integration Test",
-                updatedPrerelease: false
+                updatedPrerelease: false,
+                updateOnlyUnreleased: false
             }
         })
         return new MockInputs();

__tests__/ReleaseValidator.test.ts

@@ -0,0 +1,74 @@
+import {ReleaseValidator} from "../src/ReleaseValidator";
+
+describe("validateReleaseUpdate", () => {
+    describe("updateOnlyUnreleased is disabled", () => {
+        const validator = new ReleaseValidator(false)
+        it('should not throw', () => {
+            const releaseResponse = {
+                draft: false,
+                prerelease: false,
+                name: "Name"
+            }
+            expect(() => {
+                validator.validateReleaseUpdate(releaseResponse)
+            }).not.toThrow()
+        })
+    })
+    describe("updateOnlyUnreleased is enabled", () => {
+        const validator = new ReleaseValidator(true)
+        it('should throw if neither draft or prerelease are enabled', () => {
+            const releaseResponse = {
+                draft: false,
+                prerelease: false,
+                name: "Name"
+            }
+            expect(() => {
+                validator.validateReleaseUpdate(releaseResponse)
+            }).toThrow()
+        })
+        
+        it('should not throw if draft is enabled', () => {
+            const releaseResponse = {
+                draft: true,
+                prerelease: false,
+                name: "Name"
+            }
+            expect(() => {
+                validator.validateReleaseUpdate(releaseResponse)
+            }).not.toThrow()
+        })
+
+        it('should not throw if prerelease is enabled', () => {
+            const releaseResponse = {
+                draft: false,
+                prerelease: true,
+                name: "Name"
+            }
+            expect(() => {
+                validator.validateReleaseUpdate(releaseResponse)
+            }).not.toThrow()
+        })
+
+        it('should not throw if draft & prerelease is enabled', () => {
+            const releaseResponse = {
+                draft: true,
+                prerelease: true,
+                name: "Name"
+            }
+            expect(() => {
+                validator.validateReleaseUpdate(releaseResponse)
+            }).not.toThrow()
+        })
+
+        it('should default error message release name to release', () => {
+            const releaseResponse = {
+                draft: false,
+                prerelease: false,
+                name: null
+            }
+            expect(() => {
+                validator.validateReleaseUpdate(releaseResponse)
+            }).toThrow(`Tried to update "release" which is neither a draft or prerelease. (updateOnlyUnreleased is on)`)
+        })
+    })
+})