Sonar

JaCoCo : Allow to specify "exclclassloader" via properties for plugin

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 2.12
  • Component/s: Code Coverage
  • Labels:
    None
  • Number of attachments :
    0

Description

This option might be required in case of special frameworks that conflict with JaCoCo code instrumentation.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Evgeny Mandrikov added a comment -

Done in r4107

Show
Evgeny Mandrikov added a comment - Done in r4107
Hide
Permalink
Freddy Mallet added a comment -

Hi Evgeny, in fact I don't know how to test this feature, could you provide a simple example ?
Thanks

Show
Freddy Mallet added a comment - Hi Evgeny, in fact I don't know how to test this feature, could you provide a simple example ? Thanks
Hide
Permalink
Evgeny Mandrikov added a comment -

Hi Freddy,

In fact I have only one example in my hands - this feature is used in a build of JaCoCo, but this will not help you to test this ticket, because build of JaCoCo uses "reuseReports" mode instead of dynamic analysis.

However in fact this ticket is just about ability to pass this configuration from Sonar settings into agent during dynamic analysis. I don't think that we should loose time on tests to ensure that this parameter correctly handled by JaCoCo, because even if doesn't work - this will not be an issue on our side

So this ticket can be simply verified by checking that agent was executed and was configured properly.

Show
Evgeny Mandrikov added a comment - Hi Freddy, In fact I have only one example in my hands - this feature is used in a build of JaCoCo, but this will not help you to test this ticket, because build of JaCoCo uses "reuseReports" mode instead of dynamic analysis. However in fact this ticket is just about ability to pass this configuration from Sonar settings into agent during dynamic analysis. I don't think that we should loose time on tests to ensure that this parameter correctly handled by JaCoCo, because even if doesn't work - this will not be an issue on our side So this ticket can be simply verified by checking that agent was executed and was configured properly.

People

  • Assignee:
    Evgeny Mandrikov
    Reporter:
    Evgeny Mandrikov
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: