Re: [PR] Test: Add coverage for Output [iceberg-go]

["alex-kar (via GitHub)" <gi...@apache.org>]

alex-kar commented on PR #149:
URL: https://github.com/apache/iceberg-go/pull/149#issuecomment-2357320736

   @zeroshade Please review tests to cover `output`.
   I added two parameters: one with minimal table metadata and another with all fields set. For comparison, I used `github.com/stretchr/testify/assert`, which is already in the project and provides a nice diff view.


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org