Separate E2E tests into draft + non-draft - each one has its own cleanup (#146)

* Separate E2E tests into draft + non-draft - each one has its own cleanup

* Clean up draft separately

* Avoid double builds on pr from this repo

* That didn't work

---------

Co-authored-by: Yair Morgenstern <[email protected]>

Author: yairm210

.github/workflows/e2e_test.yml

@@ -81,6 +81,18 @@ jobs:
           
           assert.deepStrictEqual(release.data.assets.map(a => a.name), ["TEST.md", "TEST2.md"])
 
+
+    - name: Clean up draft
+      if: ${{ always() }}
+      uses: actions/github-script@v7
+      with:
+        github-token: ${{ secrets.GITHUB_TOKEN }}
+        script: |
+          await github.rest.repos.deleteRelease({
+            ...context.repo,
+            release_id: ${{ steps.make-draft.outputs.release_id }},
+          })
+
     - name: Make test pre-release
       uses: ./
       with:

action.yml

@@ -38,15 +38,14 @@ inputs:
     description: 'Specifies the commitish value that determines where the Git tag is created from. Can be any branch or commit SHA. Unused if the Git tag already exists. Default: the repository\"s default branch (usually `main`).'
   repo_name:
     description: 'Specify the name of the GitHub repository in which the GitHub release will be created, edited, and deleted. If the repository is other than the current, it is required to create a personal access token with `repo`, `user`, `admin:repo_hook` scopes to the foreign repository and add it as a secret. Defaults to the current repository'
-  draft_id:
+  release_id:
     description: 'Specify the ID of the draft release you want to upload to.'
   check_duplicates:
     description: 'Check for duplicate assets with the same name in the release and skip uploading of those. Defaults to "true".'
-    default: true
 outputs:
   browser_download_url:
     description: 'The publicly available URL of the asset.'
-  draft_id:
+  release_id:
     description: 'The ID of the draft release created.'
 runs:
   using: 'node20'