Vladimir Sitnikov

> If we are to be moving towards getting rid of the suite file, then i believe that the first step towards that should be start re-organizing the tests by...

> I just chose to use the IssueTest as a name with a description attribute on the @Test annotation that explains the github issue Well, the convention in the description...

> The description just had the github issue number, not what it does Then it is better to use annotations like [@Issue](https://github.com/allure-framework/allure-java/blob/7f33882a958f366da4376a29eeecc846138198ec/allure-java-commons/src/main/java/io/qameta/allure/Issue.java#L35-L37) rather than `description`

It might be helpful to use Allure annotations (`@Issue`, `@Feature`) so tests have the appropriate metadata, and would help to get better test reporting by applying Allure report plugin.

> I'm not sure we can remove it without a big effort in the cleanup of the tests We can add new tests to `/testng` module rather than to `/testng-core`,...

> Could you start a document that lists all points we have to clarify? I do not get the question. If you need a suite for some reason, it makes...

As a datapoint, the following change in `testng-core/build.gradle.kts` is more-or-less fine to make `./gradlew test` run without errors: ``` // suites("src/test/resources/testng.xml") exclude("test/**") exclude("test_result/**") exclude("org/testng/internal/listeners/parameterindex/TestWithProviderTest*") exclude("org/testng/BasicSample*") exclude("org/testng/DryRunSample*") exclude("org/testng/internal/conflistener/Failing*") exclude("org/testng/internal/paramhandler/ParameterizedSampleTestClass*") ``` ```...

That's possible as well. My current stack is: "make my tests working the way I like" =it requires=> "make testng recognize scopes" =it requires=> "adding tests or something". In practice,...

@cbeust , sorry for the late mention, however, have you considered using scopes? (e.g. SuiteScope, TestScope) For instance, the current `IModuleFactory` supplies `ITestContext context, Class testClass` via factory arguments. However,...

This seems to be a relevant case: https://groups.google.com/g/testng-users/c/KWqo4l4PeU4