<fix>: add testcases

Author: xiaoma20082008

velocity-engine-core/src/main/java/org/apache/velocity/spi/compilers/BaseCompiler.java

@@ -35,7 +35,7 @@
 public abstract class BaseCompiler implements Compiler {
 
   private static final Pattern PACKAGE_PATTERN = Pattern.compile("package\\s+([_a-zA-Z][_a-zA-Z0-9\\.]*);");
-  private static final Pattern CLASS_PATTERN = Pattern.compile("public\\s+class\\s+([_a-zA-Z][_a-zA-Z0-9]*)\\s+");
+  private static final Pattern CLASS_PATTERN = Pattern.compile("class\\s+([_a-zA-Z][_a-zA-Z0-9]*)\\s+");
 
   protected final Logger logger = LoggerFactory.getLogger(getClass());
   protected Cache<String, Object> cache;

velocity-engine-core/src/main/java/org/apache/velocity/spi/translators/templates/CompiledVisitor.java

@@ -98,12 +98,13 @@ public Object visit(ASTAssignment node, Object data) {
 
   @Override
   public Object visit(ASTComment node, Object data) {
-    return null;
+    return "";
   }
 
   @Override
   public Object visit(ASTDirective node, Object data) {
-    return null;
+    // TODO: 2021/11/23
+    return "";
   }
 
   @Override
@@ -404,7 +405,7 @@ public String visit(SimpleNode node, Object data) {
     sourceCode.append("public final class ").append(clsName).append(" extends CompiledTemplate {\r\n\r\n");
     sourceCode.append("  @Override\r\n");
     sourceCode.append("  protected void doRender(Context $context, Writer $output) throws Exception {\r\n");
-    sourceCode.append("    ").append(methodCode).append("\r\n");
+    sourceCode.append("    ").append("").append("\r\n");
     sourceCode.append("  }\r\n");
     sourceCode.append("}\r\n");
     String clazz = sourceCode.toString();

velocity-engine-core/src/test/java/org/apache/velocity/spi/translators/CompiledTranslatorTest.java

@@ -1,16 +1,62 @@
 package org.apache.velocity.spi.translators;
 
+import java.io.IOException;
+import java.io.Reader;
+import java.util.Locale;
+import org.apache.velocity.api.Resource;
+import org.apache.velocity.runtime.RuntimeInstance;
+import org.apache.velocity.runtime.parser.ParseException;
+import org.apache.velocity.spi.Loader;
+import org.apache.velocity.spi.Parser;
 import org.apache.velocity.spi.compilers.JavassistCompiler;
-import org.apache.velocity.spi.loaders.resources.StringResource;
+import org.apache.velocity.spi.loaders.ClasspathLoader;
+import org.apache.velocity.spi.parsers.TemplateParser;
+import org.junit.Assert;
 import org.junit.Test;
 
 public class CompiledTranslatorTest {
 
   @Test
   public void test_translate() {
+    RuntimeInstance runtimeInstance = new RuntimeInstance();
+    runtimeInstance.init();
+    Loader loader = creatLoader(runtimeInstance);
+    Parser parser = createParser(runtimeInstance);
+
     CompiledTranslator translator = new CompiledTranslator();
     translator.setCompiler(new JavassistCompiler());
-    translator.translate(new StringResource(null, "", null, "utf-8", 0L, null), null);
+    Resource resource;
+    String res;
+    try {
+      resource = loader.load("eval1.vm", null, null);
+      try (Reader reader = resource.openReader()) {
+        StringBuilder out = new StringBuilder();
+        int ch;
+        while ((ch = reader.read()) != -1) {
+          out.append((char) ch);
+        }
+        res = out.toString();
+      }
+      translator.translate(resource, parser.parse("eval1.vm", res));
+    } catch (IOException | ParseException e) {
+      Assert.fail(e.getMessage());
+    }
+  }
+
+  private Parser createParser(RuntimeInstance runtimeInstance) {
+    TemplateParser parser = new TemplateParser();
+    parser.setInnerParser(runtimeInstance.createNewParser());
+    return parser;
   }
 
+  private Loader creatLoader(RuntimeInstance runtimeInstance) {
+    ClasspathLoader loader = new ClasspathLoader();
+    loader.setEngine(runtimeInstance);
+    loader.setEncoding("utf-8");
+    loader.setLocale(Locale.getDefault());
+    loader.setReloadable(true);
+    loader.setTemplateSuffix(new String[]{".vm"});
+    loader.setTemplateDirectory(new String[]{"evaluate/"});
+    return loader;
+  }
 }