fix #42 - overwrite body and name as well
This commit is contained in:
		
							parent
							
								
									210500d479
								
							
						
					
					
						commit
						b00963776a
					
				
							
								
								
									
										162
									
								
								.github/workflows/ci.yml
									
									
									
									
										vendored
									
									
								
							
							
						
						
									
										162
									
								
								.github/workflows/ci.yml
									
									
									
									
										vendored
									
									
								
							@ -9,10 +9,10 @@ jobs:
 | 
				
			|||||||
      matrix:
 | 
					      matrix:
 | 
				
			||||||
        os: [ubuntu-latest, windows-latest, macos-latest]
 | 
					        os: [ubuntu-latest, windows-latest, macos-latest]
 | 
				
			||||||
    steps:
 | 
					    steps:
 | 
				
			||||||
      - uses: actions/checkout@v3
 | 
					    - uses: actions/checkout@v3
 | 
				
			||||||
      - run: |
 | 
					    - run: |
 | 
				
			||||||
          npm install
 | 
					        npm install
 | 
				
			||||||
          npm run all
 | 
					        npm run all
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  test: # make sure the action works on a clean machine without building
 | 
					  test: # make sure the action works on a clean machine without building
 | 
				
			||||||
    name: E2E test
 | 
					    name: E2E test
 | 
				
			||||||
@ -21,82 +21,82 @@ jobs:
 | 
				
			|||||||
      matrix:
 | 
					      matrix:
 | 
				
			||||||
        os: [ubuntu-latest, windows-latest, macos-latest]
 | 
					        os: [ubuntu-latest, windows-latest, macos-latest]
 | 
				
			||||||
    steps:
 | 
					    steps:
 | 
				
			||||||
      - uses: actions/checkout@v3
 | 
					    - uses: actions/checkout@v3
 | 
				
			||||||
      - name: Make test pre-release
 | 
					    - name: Make test pre-release
 | 
				
			||||||
        uses: ./
 | 
					      uses: ./
 | 
				
			||||||
        with:
 | 
					      with:
 | 
				
			||||||
          repo_token: ${{ secrets.GITHUB_TOKEN }}
 | 
					        repo_token: ${{ secrets.GITHUB_TOKEN }}
 | 
				
			||||||
          file: README.md
 | 
					        file: README.md
 | 
				
			||||||
          asset_name: TEST.md
 | 
					        asset_name: TEST.md
 | 
				
			||||||
          tag: ci-test-${{ matrix.os }}-${{ github.run_id }}
 | 
					        tag: ci-test-${{ matrix.os }}-${{ github.run_id }}
 | 
				
			||||||
          overwrite: true
 | 
					        overwrite: true
 | 
				
			||||||
          prerelease: true
 | 
					        prerelease: true
 | 
				
			||||||
          make_latest: true
 | 
					        make_latest: true
 | 
				
			||||||
          body: "rofl lol test%0Aianal %25 fubar"
 | 
					        body: "rofl lol test%0Aianal %25 fubar"
 | 
				
			||||||
      - name: Check that the uploaded asset is readable
 | 
					    - name: Check that the uploaded asset is readable
 | 
				
			||||||
        uses: actions/github-script@v2
 | 
					      uses: actions/github-script@v2
 | 
				
			||||||
        with:
 | 
					      with:
 | 
				
			||||||
          github-token: ${{ secrets.GITHUB_TOKEN }}
 | 
					        github-token: ${{ secrets.GITHUB_TOKEN }}
 | 
				
			||||||
          script: |
 | 
					        script: |
 | 
				
			||||||
            const fs = require('fs')
 | 
					          const fs = require('fs')
 | 
				
			||||||
            const assert = require('assert').strict;
 | 
					          const assert = require('assert').strict;
 | 
				
			||||||
          
 | 
					          
 | 
				
			||||||
            const expected = fs.readFileSync("README.md", "utf-8")
 | 
					          const expected = fs.readFileSync("README.md", "utf-8")
 | 
				
			||||||
            const release = await github.repos.getReleaseByTag({
 | 
					          const release = await github.repos.getReleaseByTag({
 | 
				
			||||||
              ...context.repo,
 | 
					            ...context.repo,
 | 
				
			||||||
              tag: "ci-test-${{ matrix.os }}-${{ github.run_id }}",
 | 
					            tag: "ci-test-${{ matrix.os }}-${{ github.run_id }}",
 | 
				
			||||||
            })
 | 
					          })
 | 
				
			||||||
            assert.deepStrictEqual(release.data.prerelease, true)
 | 
					          assert.deepStrictEqual(release.data.prerelease, true)
 | 
				
			||||||
            assert.deepStrictEqual(release.data.body, "rofl lol test\nianal % fubar")
 | 
					          assert.deepStrictEqual(release.data.body, "rofl lol test\nianal % fubar")
 | 
				
			||||||
            assert.deepStrictEqual(release.data.assets[0].name, "TEST.md")
 | 
					          assert.deepStrictEqual(release.data.assets[0].name, "TEST.md")
 | 
				
			||||||
            const actual = await github.request(release.data.assets[0].browser_download_url)
 | 
					          const actual = await github.request(release.data.assets[0].browser_download_url)
 | 
				
			||||||
            assert.deepStrictEqual(expected, actual.data)
 | 
					          assert.deepStrictEqual(expected, actual.data)
 | 
				
			||||||
      - name: Make test promote
 | 
					    - name: Make test promote
 | 
				
			||||||
        uses: ./
 | 
					      uses: ./
 | 
				
			||||||
        with:
 | 
					      with:
 | 
				
			||||||
          repo_token: ${{ secrets.GITHUB_TOKEN }}
 | 
					        repo_token: ${{ secrets.GITHUB_TOKEN }}
 | 
				
			||||||
          file: README.md
 | 
					        file: README.md
 | 
				
			||||||
          asset_name: TEST.md
 | 
					        asset_name: TEST.md
 | 
				
			||||||
          tag: ci-test-${{ matrix.os }}-${{ github.run_id }}
 | 
					        tag: ci-test-${{ matrix.os }}-${{ github.run_id }}
 | 
				
			||||||
          overwrite: true
 | 
					        overwrite: true
 | 
				
			||||||
          promote: true
 | 
					        promote: true
 | 
				
			||||||
          prerelease: false
 | 
					        prerelease: false
 | 
				
			||||||
          make_latest: true
 | 
					        make_latest: true
 | 
				
			||||||
          body: "new body"
 | 
					        body: "new body"
 | 
				
			||||||
      - name: Check that the release is promoted
 | 
					    - name: Check that the release is promoted
 | 
				
			||||||
        uses: actions/github-script@v2
 | 
					      uses: actions/github-script@v2
 | 
				
			||||||
        with:
 | 
					      with:
 | 
				
			||||||
          github-token: ${{ secrets.GITHUB_TOKEN }}
 | 
					        github-token: ${{ secrets.GITHUB_TOKEN }}
 | 
				
			||||||
          retries: 15
 | 
					        retries: 15
 | 
				
			||||||
          script: |
 | 
					        script: |
 | 
				
			||||||
            const fs = require('fs')
 | 
					          const fs = require('fs')
 | 
				
			||||||
            const assert = require('assert').strict;
 | 
					          const assert = require('assert').strict;
 | 
				
			||||||
          
 | 
					          
 | 
				
			||||||
            const expected = fs.readFileSync("README.md", "utf-8")
 | 
					          const expected = fs.readFileSync("README.md", "utf-8")
 | 
				
			||||||
            const release = await github.repos.getReleaseByTag({
 | 
					          const release = await github.repos.getReleaseByTag({
 | 
				
			||||||
              ...context.repo,
 | 
					            ...context.repo,
 | 
				
			||||||
              tag: "ci-test-${{ matrix.os }}-${{ github.run_id }}",
 | 
					            tag: "ci-test-${{ matrix.os }}-${{ github.run_id }}",
 | 
				
			||||||
            })
 | 
					          })
 | 
				
			||||||
            assert.deepStrictEqual(release.data.prerelease, false)
 | 
					          assert.deepStrictEqual(release.data.prerelease, false)
 | 
				
			||||||
            assert.deepStrictEqual(release.data.body, "new body")
 | 
					          assert.deepStrictEqual(release.data.body, "new body")
 | 
				
			||||||
            assert.deepStrictEqual(release.data.assets[0].name, "TEST.md")
 | 
					          assert.deepStrictEqual(release.data.assets[0].name, "TEST.md")
 | 
				
			||||||
            const actual = await github.request(release.data.assets[0].browser_download_url)
 | 
					          const actual = await github.request(release.data.assets[0].browser_download_url)
 | 
				
			||||||
            assert.deepStrictEqual(expected, actual.data)
 | 
					          assert.deepStrictEqual(expected, actual.data)
 | 
				
			||||||
      - name: Clean up
 | 
					    - name: Clean up
 | 
				
			||||||
        if: ${{ always() }}
 | 
					      if: ${{ always() }}
 | 
				
			||||||
        uses: actions/github-script@v2
 | 
					      uses: actions/github-script@v2
 | 
				
			||||||
        with:
 | 
					      with:
 | 
				
			||||||
          github-token: ${{ secrets.GITHUB_TOKEN }}
 | 
					        github-token: ${{ secrets.GITHUB_TOKEN }}
 | 
				
			||||||
          script: |
 | 
					        script: |
 | 
				
			||||||
            const release = await github.repos.getReleaseByTag({
 | 
					          const release = await github.repos.getReleaseByTag({
 | 
				
			||||||
              ...context.repo,
 | 
					            ...context.repo,
 | 
				
			||||||
              tag: "ci-test-${{ matrix.os }}-${{ github.run_id }}",
 | 
					            tag: "ci-test-${{ matrix.os }}-${{ github.run_id }}",
 | 
				
			||||||
            })
 | 
					          })
 | 
				
			||||||
            await github.repos.deleteRelease({
 | 
					          await github.repos.deleteRelease({
 | 
				
			||||||
              ...context.repo,
 | 
					            ...context.repo,
 | 
				
			||||||
              release_id: release.data.id,
 | 
					            release_id: release.data.id,
 | 
				
			||||||
            })
 | 
					          })
 | 
				
			||||||
            await github.git.deleteRef({
 | 
					          await github.git.deleteRef({
 | 
				
			||||||
              ...context.repo,
 | 
					            ...context.repo,
 | 
				
			||||||
              ref: "tags/ci-test-${{ matrix.os }}-${{ github.run_id }}",
 | 
					            ref: "tags/ci-test-${{ matrix.os }}-${{ github.run_id }}",
 | 
				
			||||||
            })
 | 
					          })
 | 
				
			||||||
 | 
				
			|||||||
@ -60,7 +60,6 @@ jobs:
 | 
				
			|||||||
        asset_name: mything
 | 
					        asset_name: mything
 | 
				
			||||||
        tag: ${{ github.ref }}
 | 
					        tag: ${{ github.ref }}
 | 
				
			||||||
        overwrite: true
 | 
					        overwrite: true
 | 
				
			||||||
        promote: true
 | 
					 | 
				
			||||||
        body: "This is my release text"
 | 
					        body: "This is my release text"
 | 
				
			||||||
```
 | 
					```
 | 
				
			||||||
 | 
					
 | 
				
			||||||
@ -128,7 +127,6 @@ jobs:
 | 
				
			|||||||
        file: target/release/my*
 | 
					        file: target/release/my*
 | 
				
			||||||
        tag: ${{ github.ref }}
 | 
					        tag: ${{ github.ref }}
 | 
				
			||||||
        overwrite: true
 | 
					        overwrite: true
 | 
				
			||||||
        promote: true
 | 
					 | 
				
			||||||
        file_glob: true
 | 
					        file_glob: true
 | 
				
			||||||
```
 | 
					```
 | 
				
			||||||
 | 
					
 | 
				
			||||||
@ -162,7 +160,6 @@ jobs:
 | 
				
			|||||||
        asset_name: mything
 | 
					        asset_name: mything
 | 
				
			||||||
        tag: ${{ github.ref }}
 | 
					        tag: ${{ github.ref }}
 | 
				
			||||||
        overwrite: true
 | 
					        overwrite: true
 | 
				
			||||||
        promote: true
 | 
					 | 
				
			||||||
        body: "This is my release text"
 | 
					        body: "This is my release text"
 | 
				
			||||||
```
 | 
					```
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
				
			|||||||
		Loading…
	
		Reference in New Issue
	
	Block a user