cococo icon indicating copy to clipboard operation
cococo copied to clipboard

Long execution time

Open kubajakowski opened this issue 4 years ago • 3 comments

Hi, converting our app's coverage files takes about 20 minutes - large codebase, a lot of tests. It doubles our standard PR's CI build execution time: build + test + sonar static analysis. Do you see any room for improvement in your lib? Maybe there's something on your roadmap? We'd be more than happy to help you with it. Or maybe there's something that we can do on our's project side to reduce conversion time?

Great job! Cheers

kubajakowski avatar Jun 04 '20 15:06 kubajakowski

Hi, thanks for your feedback. I'm sorry but there are currently no plans for further improvements.

Or maybe there's something that we can do on our's project side to reduce conversion time?

If you use Swift Packages in your project you can try to avoid code coverage conversion at all by using swift test / xcrun and generate code coverage reports in llvm-cov. llvm-cov is directly supported by SonarQube, no conversion needed.

heinzl avatar Jun 15 '20 08:06 heinzl

Hi,

Converting our coverage files is also quite slow. Our project contains around 100k lines of code covered by almost 1K of Unit Tests. Xcode generates 2 xcresults, respectively of 2MB and 63MB which will be converted by your script. We usually see a conversion time on Sonar of around 8min which we think is quite slow. Do you think this conversion duration is normal for the size of our project?

guillaumeLeRoy avatar Aug 17 '20 09:08 guillaumeLeRoy

We are no longer using this script I described our new approach in the readme. The repository will be archived.

heinzl avatar Aug 17 '20 09:08 heinzl