SonarQube Runner
  1. SonarQube Runner
  2. SONARUNNER-41

When the properties "sonar.binaries", "sonar.tests" and "sonar.libraries" doesn't match any existing directory/file the analysis should not fail

    Details

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

      Description

      Unless the property has been specifically forced on a sub-module.

      This behavior is not true for the "sonar.sources", which should always be checked for existence (see SONARPLUGINS-1856).

      Note that in the case of sub-modules that inherit properties from their parent, it should be possible to set a specific property to empty string (''): in this case, no verification should be done.

        Activity

        Hide
        Fabrice Bellingard added a comment -

        Great!
        Manually tested.

        Show
        Fabrice Bellingard added a comment - Great! Manually tested.
        Hide
        Julien HENRY added a comment -

        No need for ITs (covered by UTs)

        Show
        Julien HENRY added a comment - No need for ITs (covered by UTs)
        Hide
        Freddy Mallet added a comment -

        @Julien, I'm reopening this ticket because a warning message should be logged when a property value is incorrect.

        Show
        Freddy Mallet added a comment - @Julien, I'm reopening this ticket because a warning message should be logged when a property value is incorrect.
        Hide
        Julien HENRY added a comment -

        Fixed

        Show
        Julien HENRY added a comment - Fixed
        Hide
        Fabrice Bellingard added a comment -

        Perfect Julien!

        Show
        Fabrice Bellingard added a comment - Perfect Julien!
        Hide
        Julien HENRY added a comment -

        ITs updated / covered by UTs.

        Show
        Julien HENRY added a comment - ITs updated / covered by UTs.

          People

          • Assignee:
            Julien HENRY
            Reporter:
            Fabrice Bellingard
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: