ArafatKhan2198 commented on PR #6983: URL: https://github.com/apache/ozone/pull/6983#issuecomment-2255264364
> Thanks @devabhishekpal for the patch and @ArafatKhan2198 for the review request. This is a good step to enable automated testing on UI and something I personally look forward to. Once UI unit tests is done, we can consider to enable UI e2e testing (using things like puppeteer / cypress / playwright). > > I'm still trying to read the relevant documentations regarding the unit test frameworks since I'm not familiar with UI unit tests before, so I might need some more time. > > @ArafatKhan2198 @dombizita @smitajoshi12 @devmadhuu Meanwhile, I think it's good to start familiarizing ourselves with the UI unit tests (and UI in general) so that we can review the related tests meaningful. That's a great point @ivandika3. It is absolutely necessary to go through them. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
