From ca47ec70e65ead59234bb0a1288fded7969c1577 Mon Sep 17 00:00:00 2001 From: Vincent Bouthinon Date: Thu, 29 Aug 2019 11:37:39 +0200 Subject: [PATCH] [fix] Passage Log.info to Log.error to make visible in jenkins job --- .../com/efluid/tcbc/TestAPIsDependenceJar.java | 4 ++-- .../efluid/tcbc/TestControleClasseEnDoublon.java | 12 ++++++------ .../java/com/efluid/tcbc/TestDependenceJar.java | 8 ++++---- .../tcbc/process/BilanControleByteCode.java | 16 ++++++++-------- .../com/efluid/tcbc/process/ScanneClasspath.java | 14 +++++++------- 5 files changed, 27 insertions(+), 27 deletions(-) diff --git a/src/main/java/com/efluid/tcbc/TestAPIsDependenceJar.java b/src/main/java/com/efluid/tcbc/TestAPIsDependenceJar.java index 76b2396..1694215 100644 --- a/src/main/java/com/efluid/tcbc/TestAPIsDependenceJar.java +++ b/src/main/java/com/efluid/tcbc/TestAPIsDependenceJar.java @@ -75,8 +75,8 @@ protected int logBilan() { super.logBilan(); doLogList(apis, "Apis used of library : " + libraryControl); if (apis.isEmpty()) { - LOG.info("Aucun appel au jar {}. Verifier son nom et s'il est bien présent dans le classpath : ", libraryControl); - LOG.info(System.getProperty("java.class.path")); + LOG.error("Aucun appel au jar {}. Verifier son nom et s'il est bien présent dans le classpath : ", libraryControl); + LOG.error(System.getProperty("java.class.path")); } else if (erreur) { return apis.size(); } diff --git a/src/main/java/com/efluid/tcbc/TestControleClasseEnDoublon.java b/src/main/java/com/efluid/tcbc/TestControleClasseEnDoublon.java index 0d83ce1..f5b18da 100644 --- a/src/main/java/com/efluid/tcbc/TestControleClasseEnDoublon.java +++ b/src/main/java/com/efluid/tcbc/TestControleClasseEnDoublon.java @@ -51,14 +51,14 @@ protected void traitementClasseEnCours() { @Override protected int logBilan() { super.logBilan(); - LOG.info("|=== Classes en doublon ===|"); + LOG.error("|=== Classes en doublon ===|"); classesEnDoublon.forEach((nom, classes) -> { - LOG.info("[{}]", nom); - classes.forEach(classe -> LOG.info("\t{}", classe.getNomJar())); + LOG.error("[{}]", nom); + classes.forEach(classe -> LOG.error("\t{}", classe.getNomJar())); }); - LOG.info("Classes trouvees : {}", classesParcourues.size()); - LOG.info("Classes en doublon : {}", classesEnDoublon.size()); - LOG.info("Classes en doublon exclues : {}", getExclusions().get(ERREUR).size()); + LOG.error("Classes trouvees : {}", classesParcourues.size()); + LOG.error("Classes en doublon : {}", classesEnDoublon.size()); + LOG.error("Classes en doublon exclues : {}", getExclusions().get(ERREUR).size()); return classesEnDoublon.size(); } } diff --git a/src/main/java/com/efluid/tcbc/TestDependenceJar.java b/src/main/java/com/efluid/tcbc/TestDependenceJar.java index d1d1ce3..0adf2c1 100644 --- a/src/main/java/com/efluid/tcbc/TestDependenceJar.java +++ b/src/main/java/com/efluid/tcbc/TestDependenceJar.java @@ -81,11 +81,11 @@ protected void terminate() { private void logDependances(String nom, Map dependances, boolean graphViz) { TreeMap mapTrie; if (!dependances.isEmpty()) { - LOG.info(nom + " :"); + LOG.error(nom + " :"); mapTrie = new TreeMap(new ValueComparator(dependances)); mapTrie.putAll(dependances); for (String cle : mapTrie.keySet()) { - LOG.info("\treference {} type class of {}", mapTrie.get(cle), cle); + LOG.error("\treference {} type class of {}", mapTrie.get(cle), cle); if (graphViz) { addToFichierGraphViz(nom, cle, mapTrie.get(cle).toString()); } @@ -101,8 +101,8 @@ private void addToFichierGraphViz(String jar, String dependance, String nombre) } private void logJarNonReference() { - LOG.info(""); - getJarsTraites().stream().sorted().filter(jar -> !dependances.containsKey(jar.getNom())).forEach(jar -> LOG.info("jar not referenced : {}", jar.getNom())); + LOG.error(""); + getJarsTraites().stream().sorted().filter(jar -> !dependances.containsKey(jar.getNom())).forEach(jar -> LOG.error("jar not referenced : {}", jar.getNom())); } @Override diff --git a/src/main/java/com/efluid/tcbc/process/BilanControleByteCode.java b/src/main/java/com/efluid/tcbc/process/BilanControleByteCode.java index 3e89560..87bc180 100644 --- a/src/main/java/com/efluid/tcbc/process/BilanControleByteCode.java +++ b/src/main/java/com/efluid/tcbc/process/BilanControleByteCode.java @@ -28,20 +28,20 @@ public int execute() { private void loggerSynthese() { TestControleByteCode.doLogList(controle.getClassesReferenceesNonTrouveesOuChargees().values(), "Classes référencées non trouvées :"); - LOG.info("Classes en erreur lors du chargement : {}", controle.getClassesReferenceesNonTrouveesOuChargees().size()); + LOG.error("Classes en erreur lors du chargement : {}", controle.getClassesReferenceesNonTrouveesOuChargees().size()); controle.getClassesReferenceesNonTrouveesOuChargees().values().forEach(classe -> - LOG.info("\t{}", classe)); - LOG.info("Jars en erreur : {}", controle.getJarsTraites().stream().filter(Jar::isErreur).count()); - LOG.info("=== Synthèse classes en erreur ({}) ===", controle.getJarsTraites().stream().mapToLong(jar -> jar.getClassesEnErreur().size()).sum()); + LOG.error("\t{}", classe)); + LOG.error("Jars en erreur : {}", controle.getJarsTraites().stream().filter(Jar::isErreur).count()); + LOG.error("=== Synthèse classes en erreur ({}) ===", controle.getJarsTraites().stream().mapToLong(jar -> jar.getClassesEnErreur().size()).sum()); controle.getJarsTraites().stream().filter(Jar::isErreur).forEach(jar -> - LOG.info("\t{} : {}", jar.getNom(), jar.getClassesEnErreur().size() + lineSeparator())); - LOG.info("Nombre d'erreurs totales : {}", controle.getJarsTraites().stream().flatMap(jar -> jar.getClassesEnErreur().stream()).mapToInt(Classe::getNbErreurs).sum()); + LOG.error("\t{} : {}", jar.getNom(), jar.getClassesEnErreur().size() + lineSeparator())); + LOG.error("Nombre d'erreurs totales : {}", controle.getJarsTraites().stream().flatMap(jar -> jar.getClassesEnErreur().stream()).mapToInt(Classe::getNbErreurs).sum()); } private void loggerJar(Jar jar) { if (!jar.getClassesEnErreur().isEmpty()) { - LOG.info("|=== {} ===|", jar.getNom()); - LOG.info("\t|=== Classes en erreur ===|"); + LOG.error("|=== {} ===|", jar.getNom()); + LOG.error("\t|=== Classes en erreur ===|"); } for (Classe classeEnErreur : jar.getClassesEnErreur()) { LOG.error("\t\t{} : {} erreur(s)", classeEnErreur.getNom(), classeEnErreur.getNbErreurs()); diff --git a/src/main/java/com/efluid/tcbc/process/ScanneClasspath.java b/src/main/java/com/efluid/tcbc/process/ScanneClasspath.java index a9ed6fa..b275c96 100644 --- a/src/main/java/com/efluid/tcbc/process/ScanneClasspath.java +++ b/src/main/java/com/efluid/tcbc/process/ScanneClasspath.java @@ -163,13 +163,13 @@ private void chargerConfiguration() { LOG.error("STACKTRACE", ex); } if (scanByJarExclusion()) { - LOG.info("Jars excluded : {}", jarsExcluded); + LOG.error("Jars excluded : {}", jarsExcluded); } else { - LOG.info("Jars inclus : {}", jarsInclus); + LOG.error("Jars inclus : {}", jarsInclus); } - LOG.info("Exclusion des classes a ne pas traiter : {}", filtreClassesExclues); - LOG.info("Erreurs a ne pas traiter: {}", filtreErreursExclues); + LOG.error("Exclusion des classes a ne pas traiter : {}", filtreClassesExclues); + LOG.error("Erreurs a ne pas traiter: {}", filtreErreursExclues); } /** @@ -255,7 +255,7 @@ public FileVisitResult visitFile(final Path file, final BasicFileAttributes attr * Parcours toutes les classes du jar en cours */ private void controlerJar() { - LOG.info("Scans jar : " + jarEnCours); + LOG.error("Scans jar : " + jarEnCours); try (JarFile jar = new JarFile(jarEnCours.getNom())) { Enumeration enumeration = jar.entries(); JarEntry jarEntry; @@ -310,8 +310,8 @@ public boolean isExclu(Exclusion typeExclusion, final String str) { public static void doLogList(Collection col, String msgEntete) { if (col != null && !col.isEmpty()) { List liste = (List) ((col instanceof List) ? col : new ArrayList<>(col)); - LOG.info("|==== {} ====|", msgEntete); - liste.stream().sorted().forEach(s -> LOG.info("\t{}", s)); + LOG.error("|==== {} ====|", msgEntete); + liste.stream().sorted().forEach(s -> LOG.error("\t{}", s)); } }