@AfterClass config method is executed for EACH @Test method when parallel == methods

[dr29bart]

TestNG Version

7.5 (also reproducible in previous releases: 7.3.0)

Expected behavior

a configuration method annotated with @AfterClass is executed only ONCE after all @Test methods in the class are finished

Actual behavior

a configuration method annotated with @AfterClass is executed multiple times after all @Test methods in the class are finished

Is the issue reproducible on runner?

• Shell
• Maven
• Gradle
• Ant
• Eclipse
• IntelliJ
• NetBeans

Test case sample

import org.testng.annotations.AfterClass;
import org.testng.annotations.BeforeClass;
import org.testng.annotations.Test;

public class ATest {

    @BeforeClass
    public void setup() {
        System.out.println("BEFORE_" + Thread.currentThread().getName());
    }

    @AfterClass
    public void tearDown() {
        System.out.println("AFTER_" + Thread.currentThread().getName());
    }

    @Test
    public void testa() {
        System.out.println("testa_" + Thread.currentThread().getName());
    }

    @Test
    public void testb() {
        System.out.println("testb_" + Thread.currentThread().getName());
    }
}

Output:

BEFORE_TestNG-test=tqa-1
testb_TestNG-test=tqa-2
testa_TestNG-test=tqa-1
AFTER_TestNG-test=tqa-1
AFTER_TestNG-test=tqa-2

[dr29bart]

It looks like inst is never actually added to the m_classMethodMap.getInvokedAfterClassMethods()
https://github.com/cbeust/testng/blob/0ace1d4bdfc4f8c5ccd0849ad4768f4dd08146fd/testng-core/src/main/java/org/testng/internal/invokers/TestMethodWorker.java#L201-L206

 Set<Object> instances = 
     invokedAfterClassMethods.computeIfAbsent(testClass, key -> Sets.newHashSet()); 
 Object inst = mi.getInstance(); 
 if (!instances.contains(inst)) { 
   invokeInstances.add(inst); 
   instances.add(inst);
 }