INFO: Saved 'n' coverage measures for file '...'

[guwirth]

• reduce the noise in V1.2.0 LOG file.
• make it DEBUG
• close INFO: Saved 'n' coverage measures for file '...' #1624

---

This change is 

[ivangalkin]

Reviewed 1 of 1 files at r1.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @guwirth and @Bertk)

---

cxx-sensors/src/main/java/org/sonar/cxx/sensors/coverage/CxxCoverageSensor.java, line 180 at r1 (raw file):

Quoted 10 lines of code…

          try {
            newCoverage.save();
          } catch (RuntimeException ex) {
            LOG.error("Cannot save measure for file '{}' , ignoring measure. ", filePath, ex);
            CxxUtils.validateRecovery(ex, getLanguage());
          }
          if (LOG.isDebugEnabled()) {
            LOG.debug("Saved '{}' coverage measures for file '{}'", measures.size(), filePath);
          }

...should be...

Quoted 10 lines of code…

          try {
            newCoverage.save();
            if (LOG.isDebugEnabled()) {
              LOG.debug("Saved '{}' coverage measures for file '{}'", measures.size(), filePath);
            }
          } catch (RuntimeException ex) {
            LOG.error("Cannot save measure for file '{}' , ignoring measure. ", filePath, ex);
            CxxUtils.validateRecovery(ex, getLanguage());
          }

...,because in case of a RuntimeException AND IsRecoveryEnabled() == true we will still trace an "saved" message (although nothing was saved actually)

[ivangalkin]

thx!