Java+TestNG+Selenium框架中@AfterMethod方法并行执行错误问题咨询

排查synchronized失效的根因 #

• 锁对象范围不正确：如果你的synchronized修饰的是普通实例方法，TestNG默认执行parallel="methods"策略时会为每个测试方法创建独立的类实例，实例锁只对当前对象生效，不同实例的锁互不干扰，完全起不到跨线程互斥的效果。
• 共享变量作用域问题：如果票据编号存储在静态成员变量、被测系统全局缓存、公共存储介质中，即使加了方法锁，只要校验间隙其他线程修改了公共存储的数值，依旧会出现校验结果异常。
• 校验操作非原子性：如果check_1的读取和check_2的读取拆分到了同步块之外，锁只能保护同步块内的操作，还是会出现两次读取中间数据被修改的问题。

对应解决方案 #

方案1：线程隔离存储（优先推荐，不影响并行效率） #

将每个测试生成的票据编号存入ThreadLocal线程本地存储，不同线程的测试数据完全隔离，不会互相干扰，示例代码如下：

// 定义线程本地存储变量
private static ThreadLocal<String> check1Num = new ThreadLocal<>();
private static ThreadLocal<String> check2Num = new ThreadLocal<>();

// 测试方法生成票据后存入当前线程的存储
@Test
public void testGenerateReceipt() {
    // 原有生成票据、写入系统的逻辑
    check1Num.set(生成的check_1票据编号);
    check2Num.set(生成的check_2票据编号);
}

// AfterMethod中直接读取当前线程的存储值校验
@AfterMethod
public void validateReceiptNum() {
    try {
        String c1 = check1Num.get();
        String c2 = check2Num.get();
        // 原有校验逻辑
    } finally {
        // 校验完成后清除存储，避免内存泄漏
        check1Num.remove();
        check2Num.remove();
    }
}

方案2：全局锁包裹原子校验 #

如果票据是被测系统的全局资源，无法做线程隔离，需要将两次读取+校验的整个操作包在全局同步块中，确保操作原子性：

// 定义全局唯一的锁对象，所有线程共享该锁
private static final Object RECEIPT_VALIDATE_LOCK = new Object();

@AfterMethod
public void validateReceiptNum() {
    synchronized (RECEIPT_VALIDATE_LOCK) {
        // 注意：必须在锁块内重新读取最新的两个票据编号，不能使用之前缓存的值
        String c1 = 从被测系统查询最新check_1编号;
        String c2 = 从被测系统查询最新check_2编号;
        // 原有校验逻辑
    }
}

方案3：调整TestNG并行策略 #

如果业务允许降低并行粒度，可以修改testng.xml配置，将并行策略从parallel="methods"调整为parallel="tests"或parallel="instances"，同一个测试类的方法串行执行，避免类内测试方法互相干扰。

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