Better exception error message in case runtime modules are not recorded for a given loader

ivan-ristovic · ivan-ristovic · commit f086f9d6db7d · 2023-03-07T10:50:22.000+01:00
diff --git a/substratevm/src/com.oracle.svm.hosted/src/com/oracle/svm/hosted/ModuleLayerFeature.java b/substratevm/src/com.oracle.svm.hosted/src/com/oracle/svm/hosted/ModuleLayerFeature.java
@@ -706,7 +706,7 @@ public Module getRuntimeModuleForHostedModule(ClassLoader loader, String hostedM
                 if (optional) {
                     return null;
                 } else {
-                    throw VMError.shouldNotReachHere("Failed to find runtime module for hosted module " + hostedModuleName + ". No runtime modules have been registered for class loader: " + loader);
+                    throw VMError.shouldNotReachHere("Failed to find runtime module for hosted module " + hostedModuleName + ". No runtime modules have been registered for class loader: " + loader.getName());
                 }
             }
             Module runtimeModule = loaderRuntimeModules.get(hostedModuleName);

Original file line number	Diff line number	Diff line change
`@@ -706,7 +706,7 @@ public Module getRuntimeModuleForHostedModule(ClassLoader loader, String hostedM`
`706`	`706`	`if (optional) {`
`707`	`707`	`return null;`
`708`	`708`	`} else {`
`709`		`- throw VMError.shouldNotReachHere("Failed to find runtime module for hosted module " + hostedModuleName + ". No runtime modules have been registered for class loader: " + loader);`
	`709`	`+ throw VMError.shouldNotReachHere("Failed to find runtime module for hosted module " + hostedModuleName + ". No runtime modules have been registered for class loader: " + loader.getName());`
`710`	`710`	`}`
`711`	`711`	`}`
`712`	`712`	`Module runtimeModule = loaderRuntimeModules.get(hostedModuleName);`