Add goog.loadModule CompilerInput tests & make DepsFinder closer to regex parser

DepsFinder now doesn't mark inputs containing a goog.loadModule as an actual "goog.module", to match the regex parser. It also gives a better error message when encountering a goog.loadModule with a string literal argument.

PiperOrigin-RevId: 505822642

Author: lauraharker

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

@@ -435,7 +435,11 @@ void visitSubtree(Node n, @Nullable Node parent) {
             Node argument = n.getLastChild();
             switch (callee.getString()) {
               case "module":
-                loadFlags.put("module", "goog");
+                // only mark as a goog.module if this is not bundled, e.g.
+                // goog.loadModule(function(exports) {"use strict";goog.module('Foo');
+                if (parent.isExprResult() && parent.getParent().isModuleBody()) {
+                  loadFlags.put("module", "goog");
+                }
                 // Fall-through
               case "provide":
                 if (!argument.isStringLit()) {
@@ -459,7 +463,12 @@ void visitSubtree(Node n, @Nullable Node parent) {
                 return;
 
               case "loadModule":
-                // Process the block of the loadModule argument
+                // Process the block of the loadModule argument if it's a function, as opposed to
+                // a string.
+                if (argument.isStringLit()) {
+                  throw new IllegalArgumentException(
+                      "Unsupported parse of goog.loadModule with string literal");
+                }
                 n = argument.getLastChild();
                 break;