Sonar Plugins

When launching the Sonar Runner, a meaningful error message must be throw if one of the mandatory properties has not been defined

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: SONAR-RUNNER-1.2
  • Component/s: Sonar Runner
  • Labels:
    None
  • Number of attachments :
    0

Description

Mandatory properties : sonar.projectKey, sonar.projectName, sonar.projectVersion,sources

Issue Links

is depended upon by

Bug - A problem which impairs or prevents the functions of the product. SONARJNKNS-182 Add field validation to properties, propertiesFile fields

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.
is related to

Improvement - An improvement or enhancement to an existing feature or task. SONARPLUGINS-1609 Log a meaningful message when a mandatory Ant Sonar property is missing

  • Critical - Crashes, loss of data, severe memory leak.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.
supercedes

Bug - A problem which impairs or prevents the functions of the product. SONARPLUGINS-1464 When launching the Sonar Runner without providing the sonar-project.properties and/or the project kee, a NullPointerException is thrown without any additional information

  • Critical - Crashes, loss of data, severe memory leak.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

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

Done in r4838.

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

Manually tested and works well ... but if possible it would be useful to display all missing mandatory properties instead of only displaying the first missing one.

Show
Freddy Mallet added a comment - Manually tested and works well ... but if possible it would be useful to display all missing mandatory properties instead of only displaying the first missing one.
Hide
Permalink
Evgeny Mandrikov added a comment -

Done in r4851.

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

Well done Evgeny !

Show
Freddy Mallet added a comment - Well done Evgeny !
Hide
Permalink
Evgeny Mandrikov added a comment -

Closed without integration test - unit test is enough.

Show
Evgeny Mandrikov added a comment - Closed without integration test - unit test is enough.

People

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

Dates

  • Created:
    Updated:
    Resolved: