AlenkaF commented on PR #47147: URL: https://github.com/apache/arrow/pull/47147#issuecomment-3112020390
The linter error is related, but before you make any changes, I’d like to revise my earlier suggestion regarding the test location — apologies for pointing you in the wrong direction initially. I wasn’t entirely sure where the tests should go, but I now think `python/pyarrow/tests/parquet/test_pandas.py` would be a more appropriate place, since we're really testing Parquet functionality here. It might be helpful to look through the existing tests in that file to see how metadata is being verified — for example, [`test_pandas_parquet_custom_metadata`](https://github.com/apache/arrow/blob/085baa24614cccfaf2cd99ea548d456a0cfdd76e/python/pyarrow/tests/parquet/test_pandas.py#L68-L72), or similar tests that use `schema.pandas_metadata`. Given that, the second test (`test_attributes_metadata_in_json`) may no longer be necessary, and the two tests could potentially be merged into one. -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
