💚 add meepo.log-level #1

trydofor · Jan 1, 2024 · adbfd99 · adbfd99
1 parent b1b66d8
commit adbfd99
Show file tree

Hide file tree

Showing 19 changed files with 73 additions and 59 deletions.
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -68,6 +68,7 @@ jobs:
         run: >
           mvn -P ci --quiet
           -DrepoToken=${{ secrets.COVERALLS_REPO_TOKEN }}
+          -Dmeepo.log-level=ERROR
           clean test jacoco:report coveralls:report
         env:
           JAVA_HOME: ${{ steps.java-env.outputs.JAVA_HOME }}

diff --git a/src/test/java/pro/fessional/meepo/TraceTest.java b/src/test/java/pro/fessional/meepo/TraceTest.java
@@ -1,15 +1,23 @@
 package pro.fessional.meepo;
 
 import org.junit.jupiter.api.BeforeAll;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 /**
  * @author trydofor
  * @since 2020-11-09
  */
 public class TraceTest {
 
+    protected final Logger logger = LoggerFactory.getLogger(this.getClass());
+
     @BeforeAll
     public static void setTrace() {
-        System.setProperty("org.slf4j.simpleLogger.log.pro.fessional.meepo", "trace");
+        String lvl = System.getProperty("meepo.log-level");
+        if (lvl == null || lvl.isEmpty()) {
+            lvl = "trace";
+        }
+        System.setProperty("org.slf4j.simpleLogger.log.pro.fessional.meepo", lvl);
     }
 }
diff --git a/src/test/java/pro/fessional/meepo/bind/dna/DnaBkbTest.java b/src/test/java/pro/fessional/meepo/bind/dna/DnaBkbTest.java
@@ -11,7 +11,6 @@
  * @since 2020-10-22
  */
 public class DnaBkbTest extends TraceTest {
-
     final DnaBkb d1 = new DnaBkb("// DNA:BKB BlackKingBar", new Clop(0, 14, 1, 1), "BlackKingBar");
     final DnaBkb d2 = new DnaBkb(" // DNA:BKB BlackKingBar", new Clop(1, 15, 1, 1), "BlackKingBar");
 
@@ -27,7 +26,7 @@ public void testHashCode() {
 
     @Test
     public void testToString() {
-        System.out.println(d1);
-        System.out.println(d2);
+        logger.debug("d1={}", d1);
+        logger.debug("d2={}", d2);
     }
 }
diff --git a/src/test/java/pro/fessional/meepo/bind/dna/DnaEndTest.java b/src/test/java/pro/fessional/meepo/bind/dna/DnaEndTest.java
@@ -29,7 +29,7 @@ public void testHashCode() {
 
     @Test
     public void testToString() {
-        System.out.println(d1);
-        System.out.println(d2);
+        logger.debug("d1={}", d1);
+        logger.debug("d2={}", d2);
     }
 }
diff --git a/src/test/java/pro/fessional/meepo/bind/dna/DnaRawTest.java b/src/test/java/pro/fessional/meepo/bind/dna/DnaRawTest.java
@@ -27,8 +27,8 @@ public void testHashCode() {
 
     @Test
     public void testToString() {
-        System.out.println(d1.toString());
-        System.out.println(d2.toString());
+        logger.debug("d1={}", d1);
+        logger.debug("d2={}", d2);
     }
 
 }
diff --git a/src/test/java/pro/fessional/meepo/bind/dna/DnaSetTest.java b/src/test/java/pro/fessional/meepo/bind/dna/DnaSetTest.java
@@ -30,7 +30,7 @@ public void testHashCode() {
 
     @Test
     public void testToString() {
-        System.out.println(d1.toString());
-        System.out.println(d2.toString());
+        logger.debug("d1={}", d1);
+        logger.debug("d2={}", d2);
     }
 }
diff --git a/src/test/java/pro/fessional/meepo/bind/rna/RnaPutTest.java b/src/test/java/pro/fessional/meepo/bind/rna/RnaPutTest.java
@@ -27,7 +27,7 @@ public void testHashCode() {
 
     @Test
     public void testToString() {
-        System.out.println(d1.toString());
-        System.out.println(d2.toString());
+        logger.debug("d1={}", d1);
+        logger.debug("d2={}", d2);
     }
 }
diff --git a/src/test/java/pro/fessional/meepo/bind/rna/RnaRunTest.java b/src/test/java/pro/fessional/meepo/bind/rna/RnaRunTest.java
@@ -30,7 +30,7 @@ public void testHashCode() {
 
     @Test
     public void testToString() {
-        System.out.println(d1.toString());
-        System.out.println(d2.toString());
+        logger.debug("d1={}", d1);
+        logger.debug("d2={}", d2);
     }
 }
diff --git a/src/test/java/pro/fessional/meepo/bind/rna/RnaUseTest.java b/src/test/java/pro/fessional/meepo/bind/rna/RnaUseTest.java
@@ -30,7 +30,7 @@ public void testHashCode() {
 
     @Test
     public void testToString() {
-        System.out.println(d1.toString());
-        System.out.println(d2.toString());
+        logger.debug("d1={}", d1);
+        logger.debug("d2={}", d2);
     }
 }
diff --git a/src/test/java/pro/fessional/meepo/bind/txt/HiMeepoTest.java b/src/test/java/pro/fessional/meepo/bind/txt/HiMeepoTest.java
@@ -27,7 +27,7 @@ public void testHashCode() {
 
     @Test
     public void testToString() {
-        System.out.println(d1.toString());
-        System.out.println(d2.toString());
+        logger.debug("d1={}", d1);
+        logger.debug("d2={}", d2);
     }
 }
diff --git a/src/test/java/pro/fessional/meepo/bind/txt/TxtSimpleTest.java b/src/test/java/pro/fessional/meepo/bind/txt/TxtSimpleTest.java
@@ -27,7 +27,7 @@ public void testHashCode() {
 
     @Test
     public void testToString() {
-        System.out.println(d1.toString());
-        System.out.println(d2.toString());
+        logger.debug("d1={}", d1);
+        logger.debug("d2={}", d2);
     }
 }
diff --git a/src/test/java/pro/fessional/meepo/eval/time/NowTest.java b/src/test/java/pro/fessional/meepo/eval/time/NowTest.java
@@ -1,6 +1,7 @@
 package pro.fessional.meepo.eval.time;
 
 import org.junit.jupiter.api.Test;
+import pro.fessional.meepo.TraceTest;
 
 import java.time.LocalDateTime;
 import java.util.Date;
@@ -9,15 +10,15 @@
  * @author trydofor
  * @since 2021-01-07
  */
-class NowTest {
+class NowTest extends TraceTest {
 
     @Test
     void now() {
         final String s1 = Now.now(new Date(), null);
         final String s2 = Now.now(LocalDateTime.now(), null);
         final String s3 = Now.now(null, null);
-        System.out.println(s1);
-        System.out.println(s2);
-        System.out.println(s3);
+        logger.debug("s1={}", s1);
+        logger.debug("s2={}", s2);
+        logger.debug("s2={}", s3);
     }
 }
diff --git a/src/test/java/pro/fessional/meepo/fail/ParseTest.java b/src/test/java/pro/fessional/meepo/fail/ParseTest.java
@@ -2,14 +2,16 @@
 
 import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.Test;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import pro.fessional.meepo.sack.Parser;
 
 /**
  * @author trydofor
  * @since 2022-09-15
  */
 public class ParseTest {
-
+    protected static final Logger logger = LoggerFactory.getLogger(ParseTest.class);
     @Test
     public void testCross() {
         String tmpl = "// HI-MEEPO\n"
@@ -25,7 +27,7 @@ public void testCross() {
             Assertions.fail();
         }
         catch (Exception e) {
-            e.printStackTrace();
+            logger.info("test exception", e);
         }
     }
 }
diff --git a/src/test/java/pro/fessional/meepo/poof/RnaManagerTest.java b/src/test/java/pro/fessional/meepo/poof/RnaManagerTest.java
@@ -1,6 +1,7 @@
 package pro.fessional.meepo.poof;
 
 import org.junit.jupiter.api.Test;
+import pro.fessional.meepo.TraceTest;
 import pro.fessional.meepo.eval.JavaEval;
 import pro.fessional.meepo.sack.Gene;
 import pro.fessional.meepo.sack.Parser;
@@ -20,15 +21,15 @@
  * @author trydofor
  * @since 2020-11-14
  */
-public class RnaManagerTest {
+public class RnaManagerTest extends TraceTest {
 
     @Test
     public void testFunNow1() {
         String meepo = "/* HI-MEEPO */\n/* RNA:USE /now/fun:now/*/now";
         Gene gene = Parser.parse(meepo);
         Map<String, Object> ctx = new HashMap<>();
         String out = gene.merge(ctx);
-        System.out.println(out);
+        logger.debug(out);
         Pattern ptn = Pattern.compile("\\d{4}-\\d{2}-\\d{2} \\d{2}:\\d{2}:\\d{2}");
         assertTrue(ptn.matcher(out).find());
     }
@@ -39,7 +40,7 @@ public void testFunNow2() {
         Gene gene = Parser.parse(meepo);
         Map<String, Object> ctx = new HashMap<>();
         String out = gene.merge(ctx);
-        System.out.println(out);
+        logger.debug(out);
         Pattern ptn = Pattern.compile("\\d{4}-\\d{2}-\\d{2}");
         assertTrue(ptn.matcher(out).find());
     }
@@ -51,7 +52,7 @@ public void testFunNow() {
         Map<String, Object> ctx = new HashMap<>();
         {
             Object obj = fun.eval(ctx, null);
-            System.out.println(obj);
+            logger.debug("obj={}", obj);
         }
         {
             Object obj = fun.eval(ctx, new java.util.Date(0));
@@ -96,7 +97,7 @@ public void testFunAbs() {
         ctx.put("number", -1);
         Gene gene = Parser.parse(meepo);
         String out = gene.merge(ctx);
-        System.out.println(out);
+        logger.debug(out);
         assertEquals("1", out);
     }
 
@@ -111,7 +112,7 @@ public void testFunCtx() {
 
         Gene gene = Parser.parse(meepo);
         String out = gene.merge(ctx);
-        System.out.println(out);
+        logger.debug(out);
         assertEquals("1", out);
     }
 }
diff --git a/src/test/java/pro/fessional/meepo/poof/impl/map/MapHelperTest.java b/src/test/java/pro/fessional/meepo/poof/impl/map/MapHelperTest.java
@@ -1,6 +1,7 @@
 package pro.fessional.meepo.poof.impl.map;
 
 import org.junit.jupiter.api.Test;
+import pro.fessional.meepo.TraceTest;
 
 import java.lang.reflect.Method;
 import java.util.Enumeration;
@@ -15,7 +16,7 @@
  * @author trydofor
  * @since 2020-11-10
  */
-public class MapHelperTest {
+public class MapHelperTest extends TraceTest {
 
     /**
      * java.runtime.name
@@ -31,16 +32,16 @@ public class MapHelperTest {
      */
     @Test
     public void env() throws NoSuchMethodException {
-        System.out.println(Bean.class.getName());
+        logger.debug(Bean.class.getName());
         Object a = new Object() {};
-        System.out.println(a.getClass().getName());
+        logger.debug(a.getClass().getName());
         Method md = Bean.class.getMethod(getName("code"));
-        System.out.println(md.getName());
-        System.out.println(md.getParameterCount());
+        logger.debug(md.getName());
+        logger.debug("{}", md.getParameterCount());
 
         Enumeration<?> en = System.getProperties().propertyNames();
         while (en.hasMoreElements()) {
-            System.out.println(en.nextElement());
+            logger.debug("{}", en.nextElement());
         }
     }
 
@@ -53,26 +54,26 @@ public void getterSpeed() throws Exception {
         Bean bean = new Bean();
 
         int tm = 100_0000;
-        System.out.println("loop " + tm + " times");
+        logger.debug("loop {} times", tm);
         long s1 = System.currentTimeMillis();
         for (int i = 0; i < tm; i++) {
             map.get("code");
         }
-        System.out.println("map.get cost " + (System.currentTimeMillis() - s1) + "ms");
+        logger.debug("map.get cost {}ms",System.currentTimeMillis() - s1);
         //
         long s2 = System.currentTimeMillis();
         for (int i = 0; i < tm; i++) {
             bean.getCode();
         }
-        System.out.println("bean.get cost " + (System.currentTimeMillis() - s2) + "ms");
+        logger.debug("bean.get cost {}ms", System.currentTimeMillis() - s2);
 
         //
         long s3 = System.currentTimeMillis();
         for (int i = 0; i < tm; i++) {
             Method md = bean.getClass().getMethod(getName("code"));
             md.invoke(bean);
         }
-        System.out.println("bean.each-reflect cost " + (System.currentTimeMillis() - s3) + "ms");
+        logger.debug("bean.each-reflect cost {}ms", System.currentTimeMillis() - s3);
 
         /*
          loop 1000000 times
@@ -95,13 +96,13 @@ public void getterSpeed() throws Exception {
             });
             md.invoke(bean);
         }
-        System.out.println("bean.cached-reflect cost " + (System.currentTimeMillis() - s4) + "ms");
+        logger.debug("bean.cached-reflect cost {}ms", System.currentTimeMillis() - s4);
 
         long s5 = System.currentTimeMillis();
         for (int i = 0; i < tm; i++) {
             MapHelper.get(bean, "code", ARR$EMPTY_STRING); // bean.naviget cost 417ms
         }
-        System.out.println("bean.naviget cost " + (System.currentTimeMillis() - s5) + "ms");
+        logger.debug("bean.naviget cost {}ms", System.currentTimeMillis() - s5);
     }
 
     private String getName(String prop) {

diff --git a/src/test/java/pro/fessional/meepo/sack/HolderTest.java b/src/test/java/pro/fessional/meepo/sack/HolderTest.java
@@ -20,8 +20,8 @@ class HolderTest extends TraceTest {
     void parseSimple() {
         String hd = "this is {{var | PascalCase}} here";
         final Gene gene = Holder.parse(hd);
-        System.out.println(gene.build());
-        System.out.println(gene.graph());
+        logger.debug(gene.build());
+        logger.debug(gene.graph());
         String str = gene.merge(ctx);
         assertEquals("this is TryDoFor here", str);
     }
@@ -30,9 +30,9 @@ void parseSimple() {
     void parseEsc1() {
         String hd = "this is /{{ //{{ var | PascalCase | BIG_SNAKE }} /}} here";
         final Gene gene = Holder.parse(true, hd, "{{", "}}", "/");
-        System.out.println(hd);
-        System.out.println(gene.build());
-        System.out.println(gene.graph());
+        logger.debug(hd);
+        logger.debug(gene.build());
+        logger.debug(gene.graph());
         String str = gene.merge(ctx);
         assertEquals("this is {{ /TRY_DO_FOR }} here", str);
     }
@@ -41,9 +41,9 @@ void parseEsc1() {
     void parseEsc2() {
         String hd = "} this is /${ //${ var | PascalCase | BIG_SNAKE } /} here ${";
         final Gene gene = Holder.parse(true, hd, "${", "}", "/");
-        System.out.println(hd);
-        System.out.println(gene.build());
-        System.out.println(gene.graph());
+        logger.debug(hd);
+        logger.debug(gene.build());
+        logger.debug(gene.graph());
         String str = gene.merge(ctx);
         assertEquals("} this is ${ /TRY_DO_FOR } here ${", str);
     }
@@ -52,9 +52,9 @@ void parseEsc2() {
     void parseEsc3() {
         String hd = "}}}} this is /${ //${ var | PascalCase | BIG_SNAKE }}} /}}}} here ${";
         final Gene gene = Holder.parse(true, hd, "${", "}}}", "/");
-        System.out.println(hd);
-        System.out.println(gene.build());
-        System.out.println(gene.graph());
+        logger.debug(hd);
+        logger.debug(gene.build());
+        logger.debug(gene.graph());
         String str = gene.merge(ctx);
         assertEquals("}}}} this is ${ /TRY_DO_FOR }}}} here ${", str);
     }

diff --git a/src/test/java/pro/fessional/meepo/sack/ParserTest.java b/src/test/java/pro/fessional/meepo/sack/ParserTest.java
@@ -179,7 +179,7 @@ private void checkDnaBkb(HiMeepo meepo, String txt, String name, String build) {
 
         Exon exon = Parser.dealDnaBkb(ctx);
         assertNotNull(exon);
-        System.out.println(exon);
+        logger.debug("{}", exon);
         if (name == null) {
             assertEquals(0, exon.edge.until);
         }