sungwy commented on issue #1051: URL: https://github.com/apache/iceberg-python/issues/1051#issuecomment-2286706578
I think it would be good to assert the coverage failure boundary on all tests instead of enforcing them on the unit and integration tests separately. What I mean is: If we set the `--data-file` argument for the `coverage run` to a specific value for the unit tests (`.coverage.unit`), and then a different one for the integration test (`.coverage.integration`), then we could `coverage combine` these data files and then produce the `coverage report` together. So each commands will just `coverage run` and write the output to the specific coverage data files, and we can run the commands to combine and report the coverage together. -- 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-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
