Yicong-Huang opened a new pull request, #4945:
URL: https://github.com/apache/texera/pull/4945

   ### What changes were proposed in this PR?
   
   Move Python tests from `amber/src/main/python/` (co-located with production) 
to `amber/src/test/python/` (Maven source-set sibling). Brings the Python tree 
in line with the Scala layout already used by amber (`src/main/scala` + 
`src/test/scala`) — `amber/build.sbt:59` already had a forward-looking comment 
anticipating this move. `pyproject.toml` moves up to `amber/` so a single 
Python config sits at the module root.
   
   ### Any related issues, documentation, discussions?
   
   Closes #4943.
   
   ### How was this PR tested?
   
   Local `pytest` from `amber/` collects 462 tests — same set as before the 
move. Ruff `check` and `format --check` both pass on `src/main/python` and 
`src/test/python`. Five `test_large_binary_manager` failures observed are 
pre-existing on `upstream/main` (S3 / boto setup, unrelated).
   
   ### Was this PR authored or co-authored using generative AI tooling?
   
   Generated-by: Claude Opus 4.7 (Claude Code)


-- 
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]

Reply via email to