Reduce logging level for JCasC YAML loading messages (#975)

* Reduce logging level for JCasC YAML loading messages

* Change logging levels in tests

Author: oleg-nenashev

integrations/src/test/java/io/jenkins/plugins/casc/MailExtTest.java

@@ -29,7 +29,7 @@ public class MailExtTest {
 
     @Rule
     public RuleChain chain= RuleChain
-            .outerRule(logging.record(Logger.getLogger(Attribute.class.getName()), Level.INFO).capture(2048))
+            .outerRule(logging.record(Logger.getLogger(Attribute.class.getName()), Level.FINER).capture(2048))
             .around(j);
 
     private static final String SMTP_PASSWORD = "myPassword";

plugin/src/main/java/io/jenkins/plugins/casc/Attribute.java

@@ -204,7 +204,8 @@ public boolean isSecret(@CheckForNull Owner target) {
     }
 
     public void setValue(Owner target, Type value) throws Exception {
-        LOGGER.info("Setting " + target + '.' + name + " = " + (isSecret(target) ? "****" : value));
+        LOGGER.log(Level.FINE, "Setting {0}. {1} = {2}",
+                new Object[] {target, name, (isSecret(target) ? "****" : value)});
         setter.setValue(target, value);
     }
 

plugin/src/main/java/io/jenkins/plugins/casc/impl/configurators/DataBoundConfigurator.java

@@ -121,11 +121,11 @@ private T tryConstructor(Constructor<T> constructor, Mapping config, Configurati
                     clazz.getPackage().isAnnotationPresent(ParametersAreNonnullByDefault.class))) {
 
                     if (Set.class.isAssignableFrom(t)) {
-                        LOGGER.log(Level.INFO, "The parameter to be set is @Nonnull but is not present; " +
+                        LOGGER.log(Level.FINER, "The parameter to be set is @Nonnull but is not present; " +
                                                            "setting equal to empty set.");
                         args[i] = Collections.emptySet();
                     } else if (List.class.isAssignableFrom(t)) {
-                        LOGGER.log(Level.INFO, "The parameter to be set is @Nonnull but is not present; " +
+                        LOGGER.log(Level.FINER, "The parameter to be set is @Nonnull but is not present; " +
                                                            "setting equal to empty list.");
                         args[i] = Collections.emptyList();
                     } else if (String.class.isAssignableFrom(t)) {
@@ -160,7 +160,8 @@ private T tryConstructor(Constructor<T> constructor, Mapping config, Configurati
                         final Configurator configurator = context.lookupOrFail(k);
                         args[i] = configurator.configure(value, context);
                     }
-                    LOGGER.info("Setting " + target + "." + names[i] + " = " + (t == Secret.class ? "****" : value));
+                    LOGGER.log(Level.FINE, "Setting {0}. {1} = {2}",
+                        new Object[] {target, names[i], t == Secret.class ? "****" : value});
                 } else if (t.isPrimitive()) {
                     args[i] = defaultValue(t);
                 }

plugin/src/test/java/io/jenkins/plugins/casc/core/ProxyConfiguratorTest.java

@@ -33,7 +33,7 @@ public class ProxyConfiguratorTest {
 
     @Rule
     public RuleChain chain = RuleChain
-            .outerRule(logging.record(Logger.getLogger(Attribute.class.getName()), Level.INFO).capture(2048))
+            .outerRule(logging.record(Logger.getLogger(Attribute.class.getName()), Level.FINER).capture(2048))
             .around(new EnvVarsRule())
             .around(j);
 

plugin/src/test/java/io/jenkins/plugins/casc/misc/RoundTripAbstractTest.java

@@ -114,7 +114,7 @@ public void roundTripTest() throws IOException {
 
             // Start recording the logs just before restarting, to avoid capture the previous startup. We're look there
             // if the "magic token" is there
-            logging.record("io.jenkins.plugins.casc", Level.INFO).capture(5);
+            logging.record("io.jenkins.plugins.casc", Level.FINER).capture(2048);
         });
 
         r.then(step -> {