如何解决testng.xml混合并行串行执行时ThreadLocal.get()返回null问题

两个核心问题导致空指针和浏览器通信异常：

1. TestNG线程池复用导致的初始化逻辑漏触发：配置parallel="classes" thread-count="5"时，TestNG会用固定大小线程池执行并行用例，切换到串行组时会复用池内已有线程。如果监听器的初始化逻辑只覆盖并行场景、或者错误地在同线程的上一个方法执行完成后提前清理了driver，就会出现ThreadLocal.get()返回null的情况。
2. 现有DriverFactory实现存在明显缺陷：
   • 给setDriver/removeDriver加了类级别synchronized，既拖慢并行执行效率，也没有解决ThreadLocal访问的空指针问题：removeDriver直接调用tlDriver.get().quit()，如果get返回null会直接抛异常中断清理流程，残留的driver进程占用端口，是触发UnreachableBrowserException的核心原因。
   • setDriver没有兜底分支，如果传入的浏览器参数不匹配chrome/firefox，根本不会执行tlDriver.set()，后续get必然返回null。
   • 单例没有加私有构造，外部可以随意new新的DriverFactory实例，导致不同实例的ThreadLocal存储不互通，出现取值为空的问题。
3. 监听器生命周期匹配错误：类级别并行场景下，同一个类的所有测试方法跑在同一个线程中，如果每跑完一个方法就调用removeDriver，同线程下一个方法执行时必然拿不到driver实例。

1. 重构DriverFactory逻辑 #

去掉无意义的类级别锁，补全空判断、异常兜底和单例约束，代码如下：

public class DriverFactory {
    private static final DriverFactory instance = new DriverFactory();
    private static final ThreadLocal<WebDriver> tlDriver = new ThreadLocal<>();

    // 私有构造禁止外部new实例，保证单例唯一
    private DriverFactory() {}

    public static DriverFactory getInstance() {
        return instance;
    }

    public void setDriver(String browserName) {
        // 初始化前先清理当前线程残留的旧driver，避免脏数据
        if (tlDriver.get() != null) {
            try {
                tlDriver.get().quit();
            } catch (Exception ignored) {}
            tlDriver.remove();
        }

        WebDriver driver;
        if ("chrome".equalsIgnoreCase(browserName)) {
            ChromeOptions chromeOptions = new ChromeOptions();
            chromeOptions.addArguments("disable-infobars");
            chromeOptions.addArguments("--start-maximized");
            // 旧版--headless稳定性差，替换为新版headless模式减少崩溃
            chromeOptions.addArguments("--headless=new");
            System.setProperty("webdriver.chrome.args", "--disable-logging");
            System.setProperty("webdriver.chrome.silentOutput", "true");
            WebDriverManager.chromedriver().setup();
            driver = new ChromeDriver(chromeOptions);
        } else if ("firefox".equalsIgnoreCase(browserName)) {
            System.setProperty("webdriver.gecko.driver", System.getProperty("user.dir") + "/Drivers/geckodriver.exe");
            FirefoxProfile firefoxProfile = new FirefoxProfile();
            FirefoxOptions firefoxOptions = new FirefoxOptions();
            firefoxOptions.setHeadless(true).setProfile(firefoxProfile);
            System.setProperty("webdriver.firefox.args", "--disable-logging");
            System.setProperty("webdriver.firefox.silentOutput", "true");
            driver = new FirefoxDriver(firefoxOptions);
        } else {
            throw new IllegalArgumentException("不支持的浏览器类型: " + browserName);
        }
        tlDriver.set(driver);
    }

    public WebDriver getDriver() {
        WebDriver driver = tlDriver.get();
        if (driver == null) {
            throw new IllegalStateException("当前线程未绑定WebDriver实例，请检查监听器初始化逻辑");
        }
        return driver;
    }

    public void removeDriver() {
        WebDriver driver = tlDriver.get();
        if (driver != null) {
            try {
                driver.quit();
            } catch (Exception e) {
                System.out.println("关闭driver时出现异常: " + e.getMessage());
            } finally {
                // 无论quit是否成功，都必须移除ThreadLocal绑定，避免内存泄漏和脏数据
                tlDriver.remove();
            }
        }
    }
}

2. 调整监听器触发逻辑 #

• 不要给所有测试方法都绑定初始化/销毁逻辑：类级别并行场景下，在beforeInvocation中判断当前执行的方法是所属测试类的第一个@Test方法时，才调用setDriver初始化；在afterInvocation中判断当前执行的方法是所属测试类的最后一个@Test方法时，才调用removeDriver销毁，避免同线程下方法间错误清理driver。
• 新增IExecutionListener实现，在所有测试执行完成后做全局兜底清理，杀掉所有残留的driver进程，避免端口占用导致的浏览器连接异常。
• 初始化逻辑要覆盖所有测试方法，不要只针对并行组做判断：串行组执行时如果复用了线程池里的线程，也要先检查当前线程是否绑定有效driver，没绑定就重新初始化。

3. testng.xml配置优化 #

将并行执行组和串行执行组拆分为两个独立的<test>标签块，并行块单独配置parallel="classes" thread-count="5"，串行块不配置parallel属性默认单线程执行，避免TestNG在同一个test块内混用线程调度逻辑导致的线程复用异常。

内容的提问来源于stack exchange，提问作者Suvarna Modhale