eugenegujing opened a new issue, #6441:
URL: https://github.com/apache/texera/issues/6441
### Task Summary
`common/workflow-core/src/main/scala/org/apache/texera/amber/core/storage/model/DatasetFileDocument.scala`
parses a dataset file URI in its constructor into a repository name, a
URL-decoded version hash, and a URL-decoded multi-segment relative file path,
rejecting URIs with fewer than three path segments with
`IllegalArgumentException("URI format is incorrect")`.
Every dataset file read in the engine goes through this parsing, yet the
class has zero references in the test tree — neither the parsing and its error
path nor the parsed-component getters (`getRepositoryName`, `getVersionHash`,
`getFileRelativePath`, `getURI`) are covered anywhere.
Note this is distinct from the existing Python-side tests (#4701 / #4702
cover `pytexera/storage/dataset_file_document.py`, a separate class). On the
Scala side, #5699 added specs for the storage result models but did not touch
`DatasetFileDocument`, and the existing `FileResolverSpec` covers the URI
**construction** side only — the parsing side tested here is its complement.
### Task Type
- [ ] Refactor / Cleanup
- [ ] DevOps / Deployment / CI
- [x] Testing / QA
- [ ] Documentation
- [ ] Performance
- [ ] Other
--
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]
