Refactor PlaywrightIT to use environment variable for headless mode and add null checks for resource cleanup

Author: fredpena

src/test/java/org/javadominicano/PlaywrightIT.java

@@ -22,16 +22,24 @@ public static void beforeAll() {
         playwright = Playwright.create();
         BrowserType browserType = playwright.chromium();
 
-        BrowserType.LaunchOptions launchOptions = new BrowserType.LaunchOptions().setHeadless(true);
-        launchOptions.headless = false;
+        boolean headless = Boolean.parseBoolean(
+                System.getenv().getOrDefault("HEADLESS", "true")
+        );
+
+        BrowserType.LaunchOptions launchOptions = new BrowserType.LaunchOptions()
+                .setHeadless(headless);
 
         browser = browserType.launch(launchOptions);
     }
 
     @AfterAll
     public static void afterAll() {
-        browser.close();
-        playwright.close();
+        if (browser != null) {
+            browser.close();
+        }
+        if (playwright != null) {
+            playwright.close();
+        }
     }
 
     @BeforeEach
@@ -50,8 +58,8 @@ void beforeEach() {
 
     @AfterEach
     void afterEach() {
-        page.close();
-        browserContext.close();
+        if (page != null) page.close();
+        if (browserContext != null) browserContext.close();
     }
 
 }