Skip to content
This repository has been archived by the owner on Aug 23, 2020. It is now read-only.

CI code coverage upload needs fixing #1082

Closed
alon-e opened this issue Oct 18, 2018 · 1 comment · Fixed by #1085
Closed

CI code coverage upload needs fixing #1082

alon-e opened this issue Oct 18, 2018 · 1 comment · Fixed by #1085
Labels
C-Build Component - part of the build process

Comments

@alon-e
Copy link
Contributor

alon-e commented Oct 18, 2018

Bug description

Jacoco results fail to upload.

Hardware Spec

travis :)

Steps To Reproduce

just look at the Travis push builds.

Expected behaviour

coverage uploads

Actual behaviour

premature failure due to download issues.

Errors

https://travis-ci.org/iotaledger/iri/jobs/441689304#L1874

test $TRAVIS_PULL_REQUEST = "false" && test $TRAVIS_JDK_VERSION = "oraclejdk8" && wget -O codacy-coverage-reporter-assembly-latest.jar $(curl https://api.github.com/repos/codacy/codacy-coverage-reporter/releases/latest | jq -r .assets[0].browser_download_url)
--2018-10-15 14:32:11--  http://null/

this is due to bad jq parsing:
working command:

test $TRAVIS_PULL_REQUEST = "false" && test $TRAVIS_JDK_VERSION = "oraclejdk8" && wget -O codacy-coverage-reporter-assembly-latest.jar $(curl https://api.github.com/repos/codacy/codacy-coverage-reporter/releases/latest | jq -r '.assets[0].browser_download_url')

(notice the '...')

@alon-e alon-e added the C-Build Component - part of the build process label Oct 18, 2018
@alon-e
Copy link
Contributor Author

alon-e commented Oct 21, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
C-Build Component - part of the build process
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant