fix: update the level of unwanted info logs to debug (#5312)

INRIA · Aug 27, 2023 · 95b1bb2 · 95b1bb2
1 parent 155a9b3
commit 95b1bb2
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 10 deletions.
diff --git a/src/main/java/spoon/Launcher.java b/src/main/java/spoon/Launcher.java
@@ -551,7 +551,7 @@ protected void processArguments() {
 
  protected void reportClassPathMode() {
  String cpmode = jsapActualArgs.getString("cpmode").toUpperCase();
- factory.getEnvironment().report(null, Level.INFO, "Running in " + cpmode + " mode (doc: http://spoon.gforge.inria.fr/launcher.html).");
+ factory.getEnvironment().report(null, Level.DEBUG, "Running in " + cpmode + " mode (doc: http://spoon.gforge.inria.fr/launcher.html).");
  }
 
  /**
@@ -723,10 +723,9 @@ public PrettyPrinter createPrettyPrinter() {
  @Override
  public void run() {
  Environment env = modelBuilder.getFactory().getEnvironment();
- env.reportProgressMessage(getVersionMessage());
- env.reportProgressMessage("running Spoon...");
-
- env.reportProgressMessage("start processing...");
+ env.debugMessage(getVersionMessage());
+ env.reportProgressMessage("Running Spoon...");
+ env.debugMessage("Start processing...");
 
  long tstart = System.currentTimeMillis();
 
@@ -743,7 +742,7 @@ public void run() {
 
  long t = System.currentTimeMillis();
 
- env.debugMessage("program spooning done in " + (t - tstart) + " ms");
+ env.debugMessage("Program spooning done in " + (t - tstart) + " ms");
  env.reportEnd();
 
  }

diff --git a/src/main/java/spoon/support/QueueProcessingManager.java b/src/main/java/spoon/support/QueueProcessingManager.java
@@ -109,7 +109,7 @@ public void process(Collection<? extends CtElement> elements) {
  int i = 0;
  while ((p = processors.poll()) != null) {
  try {
- getFactory().getEnvironment().reportProgressMessage(p.getClass().getName());
+ getFactory().getEnvironment().debugMessage(p.getClass().getName());
  current = p;
  p.init(); // load the properties
  p.process();

diff --git a/src/main/java/spoon/support/StandardEnvironment.java b/src/main/java/spoon/support/StandardEnvironment.java
@@ -316,7 +316,7 @@ private void print(String message, Level messageLevel) {
  */
  @Override
  public void reportEnd() {
- print("end of processing: ", Level.INFO);
+ print("End of processing: ", Level.DEBUG);
  if (warningCount > 0) {
  print(warningCount + " warning", Level.INFO);
  if (warningCount > 1) {
@@ -335,7 +335,7 @@ public void reportEnd() {
  if ((errorCount + warningCount) > 0) {
  print("\n", Level.INFO);
  } else {
- print("no errors, no warnings", Level.INFO);
+ print("No errors, no warnings", Level.DEBUG);
  }
  }
 

diff --git a/src/test/java/spoon/test/logging/LogTest.java b/src/test/java/spoon/test/logging/LogTest.java
@@ -74,7 +74,7 @@ public void testAllLevelsForLogs(Pair<Level, Integer> levelAndExpectedCount) {
  private static Stream<Pair<Level, Integer>> getLogLevelsAndExpectedCounts() {
  return Stream.of(
  Pair.of(Level.DEBUG, 6),
- Pair.of(Level.INFO, 2),
+ Pair.of(Level.INFO, 1),
  Pair.of(Level.WARN, 0),
  Pair.of(Level.ERROR, 0),
  Pair.of(Level.OFF, 0)