Skip to content

Commit e16ad9a

Browse files
committed
DEVOPS-306: Give up on getting coverage into Sonar.
1 parent 4ca00d0 commit e16ad9a

File tree

3 files changed

+9
-3
lines changed

3 files changed

+9
-3
lines changed

CI.Jenkinsfile

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,13 @@ def runSonnarForPythonVersion(sourceDir, ver){
2525
sonarExec="echo Skipping Sonar for this version."
2626
}
2727

28+
//
29+
//
30+
// NOTE: The test coverage data is not making it into Sonar.
31+
// I don't think it is worth spending more time on until
32+
// Sonar is upgraded.
33+
//
34+
//
2835
sh "docker run \
2936
--pull always \
3037
--rm --volume ${sourceDir}:/source \

sonar-project.properties

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
11
sonar.projectKey=rosette-api-ruby-binding
22
sonar.ruby.coverage.reportPaths=coverage/.resultset.json
3+
#sonar.ruby.coverage.reportPaths=coverage/coverage.json
34
sonar.exclusions=**/tests/**,**/examples/**

tests/tests_spec.rb

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,7 @@
44
require 'simplecov'
55
SimpleCov.formatter = SimpleCov::Formatter::JSONFormatter
66
SimpleCov.start do
7-
# Going to try disabling this until we upgrade Sonar.
8-
# https://sonarsource.atlassian.net/browse/SONARSLANG-477
9-
# enable_coverage :branch
7+
enable_coverage :branch
108
end
119

1210
require 'rosette_api'

0 commit comments

Comments
 (0)