Skip to content

Commit

Permalink
fix response type
Browse files Browse the repository at this point in the history
  • Loading branch information
shogo82148 committed Jan 29, 2020
1 parent 2e0e5f0 commit 8ba1b3d
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 8 deletions.
6 changes: 3 additions & 3 deletions __tests__/main.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import {upload} from '../src/upload-release-asset'

test('Upload Release Asset', async () => {
const uploadReleaseAsset = jest.fn().mockReturnValue({
data: {value: {browser_download_url: 'http://example.com/download'}}
data: {browser_download_url: 'http://example.com/download'}
})
const github = {
repos: {
Expand Down Expand Up @@ -33,7 +33,7 @@ test('Upload Release Asset', async () => {

test('Upload Multiple Files', async () => {
const uploadReleaseAsset = jest.fn().mockReturnValue({
data: {value: {browser_download_url: 'http://example.com/download'}}
data: {browser_download_url: 'http://example.com/download'}
})
const github = {
repos: {
Expand Down Expand Up @@ -84,7 +84,7 @@ test('Upload Multiple Files', async () => {

test('Guess Content Types', async () => {
const uploadReleaseAsset = jest.fn().mockReturnValue({
data: {value: {browser_download_url: 'http://example.com/download'}}
data: {browser_download_url: 'http://example.com/download'}
})
const github = {
repos: {
Expand Down
5 changes: 4 additions & 1 deletion src/main.ts
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,10 @@ async function run(): Promise<void> {
const asset_name = core.getInput('asset_name')
const asset_content_type = core.getInput('asset_content_type')

const github = new GitHub(github_token)
// bug? ReposUploadReleaseAssetResponse doesn't match actual response.
// so ignore types
const github = new GitHub(github_token) as any

const output = await upload({
github,
upload_url,
Expand Down
5 changes: 1 addition & 4 deletions src/upload-release-asset.ts
Original file line number Diff line number Diff line change
Expand Up @@ -21,9 +21,6 @@ interface Response<T> {
data: T
}
interface ReposUploadReleaseAssetResponse {
value: ReposUploadReleaseAssetResponseValue
}
interface ReposUploadReleaseAssetResponseValue {
browser_download_url: string
}
interface GitHubRepos {
Expand Down Expand Up @@ -67,7 +64,7 @@ export async function upload(opts: Options): Promise<Outputs> {
file: fs.readFileSync(file)
})
core.debug(JSON.stringify(response))
return response.data.value.browser_download_url
return response.data.browser_download_url
})
)
return {
Expand Down

0 comments on commit 8ba1b3d

Please sign in to comment.