Cucumber TestNG中@After钩子方法执行两次问题求助

我之前在结合Cucumber和TestNG做自动化测试时，也碰到过一模一样的坑——明明只跑了一个Feature文件，测试报告里却冒出来两条类名完全一致的记录，@After钩子还平白无故执行了两次。结合我的踩坑经验，给你几个排查和解决的方向：

1. 先查TestNG配置是否重复触发测试 #

这是最常见的原因！打开你的testng.xml文件，看看是不是同时通过类名和包名引用了同一个测试类，比如：

<!-- 错误示例：同时指定包和类，TestNG会重复执行 -->
<test name="CucumberTests">
    <packages>
        <package name="com.yourproject.tests"/>
    </packages>
    <classes>
        <class name="com.yourproject.tests.YourCucumberTest"/>
    </classes>
</test>

TestNG会分别处理包扫描和类引用逻辑，导致同一个测试类被执行两次。解决办法很简单：只保留其中一种引用方式（要么用包扫描，要么直接指定类）。

2. 检查测试类的注解是否冲突 #

如果你的测试类继承了AbstractTestNGCucumberTests，绝对不要额外加TestNG的@Test注解！因为父类AbstractTestNGCucumberTests已经内置了TestNG的测试入口方法，重复添加@Test会让TestNG把这个类当成两个独立的测试用例执行。

错误写法：

@Test // 多余的注解，会触发重复执行
public class YourCucumberTest extends AbstractTestNGCucumberTests {
    // ... 你的代码
}

正确写法：直接去掉@Test注解，只保留继承关系即可。

3. 排查ExtentReport的日志创建逻辑 #

从你的代码片段看，你在tearDown里调用了reportMgr.getLogger()——如果这个方法每次调用都会新建一个ExtentTest实例，而不是复用已有的记录，就会导致即使钩子只执行一次，报告里也会多出一条重复记录。

建议调整日志初始化逻辑：在Cucumber的@Before钩子中创建ExtentTest并通过ThreadLocal存储，然后在@After钩子中直接复用这个实例，而不是每次都调用getLogger新建。

示例代码：

private ThreadLocal<ExtentTest> testThreadLocal = new ThreadLocal<>();

@Before
public void setUp(Scenario scen) {
    ExtentTest logger = reportMgr.createTest(scen.getName());
    testThreadLocal.set(logger);
}

@After
public void tearDown(Scenario scen) throws IOException {
    ExtentTest logger = testThreadLocal.get();
    // 后续的日志操作...
}

4. 验证测试类是否被多次实例化 #

可以在tearDown方法里打印当前实例的哈希值和线程ID，判断是不是同一个实例执行了两次：

@After
public void tearDown(Scenario scen) throws IOException {
    System.out.println("TearDown执行，实例哈希：" + this.hashCode() + "，线程ID：" + Thread.currentThread().getId());
    // ... 其他代码
}

如果输出了两个不同的哈希值，说明测试类被实例化了两次，大概率是Cucumber的glue路径重复包含了钩子类，或者TestNG的监听器配置有问题。

5. 检查Cucumber和TestNG版本兼容性 #

某些旧版本的Cucumber（比如v4.x之前）和TestNG组合会存在生命周期冲突的bug，建议升级到最新的稳定版（比如Cucumber v7.x + TestNG v7.x），很多此类问题都会被官方修复。

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