amrishlal commented on issue #5688: URL: https://github.com/apache/incubator-pinot/issues/5688#issuecomment-730530039
I looked at the test classes under `org.apache.pinot.controller.api` package. For test classes that do `startZk(); startController()`, one option may be to create a component level `testng.xml` configuration file with `<suite>...</suite>` that would contain these test classes. This will allow for specifying a `@BeforeSuite` method to run `startZk(); startController()` once before all the suite test cases and `@AfterSuite` to run `stopController();stopZk();` after all the suite test cases. There are also some test classes that invoke `startController()` with properties (`AccessControlTest` for example) and some test classes that don't invoke either `startZk()` or `startController()` (`ControllerFilePathProviderTest` for example), so these would not be listed in the suite. What about other packages under `org.apache.pinot.controller` (for example `...helix.ControllerInstanceToggleTest`)? Do we want to add them to the test suite as well since these classes are also doing `startZk()` or `startController()`? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
