xuang7 commented on PR #5220: URL: https://github.com/apache/texera/pull/5220#issuecomment-4550392230
> Good change! But I would kindly ask you to provide tests for all those format utils. The coverage is quite low currently. I added tests for the format utils. For the component call sites, adding tests would require quite a bit of setup, so it felt beyond the scope of this PR and I didn’t include those tests. Thanks! -- 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]
