Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Fix coverage paths in Jenkinsfile - Closes #3051 #3052

Merged
merged 1 commit into from
Mar 12, 2019

Conversation

yatki
Copy link
Contributor

@yatki yatki commented Mar 12, 2019

What was the problem?

Coverage report file paths were wrong in the jenkinsfile which was causing Jenkins jobs to fail.

How did I fix it?

I fixed the file paths.

How to test it?

Review checklist

Copy link
Contributor

@2snEM6 2snEM6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@@ -55,9 +55,9 @@ def teardown(test_name) {
sh """
rm -rf coverage_${test_name}; mkdir -p coverage_${test_name}
npx istanbul report --root framework/test/mocha/.coverage-unit/ --dir framework/test/mocha/.coverage-unit/
cp test/.coverage-unit/lcov.info coverage_${test_name}/ || true
cp framework/test/mocha/.coverage-unit/lcov.info coverage_${test_name}/ || true
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure if this is 100% related but the integration lcov.info seems to be empty:

[Pipeline] sh
+ find coverage/ -name lcov.info
+ sed s/^/-a /
+ xargs lcov -o coverage/merged.lcov
lcov: ERROR: no valid records found in tracefile coverage/coverage_integration/functional/lcov.info
Combining tracefiles.
Reading tracefile coverage/coverage_integration/functional/lcov.info
xargs: lcov: exited with status 255; aborting
[Pipeline] }

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yatki please address this comment in a separate PR if necessary. I will merge this one as other PRs are relying on it.

@MaciejBaj MaciejBaj merged commit 0d6fc9c into development Mar 12, 2019
@MaciejBaj MaciejBaj deleted the 3051-fix-jenkinsfile-for-code-coverage branch March 12, 2019 11:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix coverage paths in Jenkinsfile
4 participants