fix: include var name into tracing error message (#779) (#780)

yury-s · web-flow · commit f19cecac20e6 · 2022-01-19T10:45:12.000-08:00
diff --git a/playwright/src/main/java/com/microsoft/playwright/impl/TracingImpl.java b/playwright/src/main/java/com/microsoft/playwright/impl/TracingImpl.java
@@ -93,7 +93,7 @@ private void startImpl(StartOptions options) {
     includeSources = options.sources != null;
     if (includeSources) {
       if (!context.connection.isCollectingStacks()) {
-        throw new PlaywrightException("Source root directories must be provided to enable source collection");
+        throw new PlaywrightException("Source root directory must be specified via PLAYWRIGHT_JAVA_SRC environment variable when source collection is enabled");
       }
       params.addProperty("sources", true);
     }

Original file line number	Diff line number	Diff line change
`@@ -93,7 +93,7 @@ private void startImpl(StartOptions options) {`
`93`	`93`	`includeSources = options.sources != null;`
`94`	`94`	`if (includeSources) {`
`95`	`95`	`if (!context.connection.isCollectingStacks()) {`
`96`		`- throw new PlaywrightException("Source root directories must be provided to enable source collection");`
	`96`	`+ throw new PlaywrightException("Source root directory must be specified via PLAYWRIGHT_JAVA_SRC environment variable when source collection is enabled");`
`97`	`97`	`}`
`98`	`98`	`params.addProperty("sources", true);`
`99`	`99`	`}`