Print config more reliably

The config printing was only happening when compile() or compileModules() was
called or when executed by AbstractCommandLineRunner::doRun(), but that is
sometimes redundant, and sometimes doesn't happen at all.

I've moved this action to the end of the `initModules()` method, which must
always be executed right before performing a compilation action.

This CL also disables debug logging for CompilerTestCase tests by default.
Without that part of the change I was having test timeouts for some tests.
I believe that was because those tests started wasting time printing out
configuration information to debug log files.

PiperOrigin-RevId: 552964960

Author: brad4d

src/com/google/javascript/jscomp/AbstractCommandLineRunner.java

@@ -1301,10 +1301,6 @@ protected int doRun() throws IOException {
     externs = null;
     sources = null;
 
-    if (options.printConfig) {
-      compiler.printConfig();
-    }
-
     Result result;
     // We won't want to process results for cases where compilation is only partially done.
     boolean shouldProcessResults = true;

src/com/google/javascript/jscomp/Compiler.java

@@ -758,6 +758,14 @@ public void initModules(
 
     initAST();
 
+    if (isDebugLoggingEnabled() || options.printConfig) {
+      // Always print configuration logs when debug logging is enabled.
+      // NOTE: initModules() is called by every execution path through the compiler code that
+      // actually performs a compilation action. That's what makes this a good place to put this
+      // logging.
+      printConfig();
+    }
+
     // If debug logging is enabled, write out the module / chunk graph in GraphViz format.
     // This graph is often too big to reasonably render.
     // Using gvpr(1) is recommended to extract the parts of the graph that are of interest.
@@ -892,9 +900,6 @@ public Result compile(
 
     try {
       init(externs, inputs, options);
-      if (options.printConfig) {
-        printConfig();
-      }
       if (!hasErrors()) {
         parseForCompilation();
       }
@@ -951,9 +956,6 @@ public Result compileModules(
 
     try {
       initModules(externs, modules, options);
-      if (options.printConfig) {
-        printConfig();
-      }
       if (!hasErrors()) {
         parseForCompilation();
       }

test/com/google/javascript/jscomp/CompilerTestCase.java

@@ -726,6 +726,13 @@ protected void tearDown() throws Exception {
    */
   protected abstract CompilerPass getProcessor(Compiler compiler);
 
+  /** Enable debug logging for this test run */
+  private boolean debugLoggingEnabled = false;
+
+  public void enableDebugLogging(boolean debugLoggingEnabled) {
+    this.debugLoggingEnabled = debugLoggingEnabled;
+  }
+
   /**
    * Gets the compiler options to use for this test. Use getProcessor to determine what passes
    * should be run.
@@ -759,7 +766,9 @@ protected CompilerOptions getOptions() {
     }
     options.setCodingConvention(getCodingConvention());
     options.setPolymerVersion(1);
-    CompilerTestCaseUtils.setDebugLogDirectoryOn(options);
+    if (debugLoggingEnabled) {
+      CompilerTestCaseUtils.setDebugLogDirectoryOn(options);
+    }
 
     return options;
   }

test/com/google/javascript/jscomp/disambiguate/DisambiguatePropertiesTest.java

@@ -73,6 +73,7 @@ public void setUp() throws Exception {
     this.enableTypeCheck();
     replaceTypesWithColors();
     disableCompareJsDoc();
+    enableDebugLogging(true);
   }
 
   @Override

test/com/google/javascript/jscomp/serialization/SerializeTypedAstPassTest.java

@@ -69,6 +69,7 @@ public void setUp() throws Exception {
     enableTypeCheck();
     this.typesToForwardDeclare = ImmutableSet.of();
     enableSourceInformationAnnotator();
+    enableDebugLogging(true);
   }
 
   @Override