aglinxinyuan opened a new pull request, #6099: URL: https://github.com/apache/texera/pull/6099
### What changes were proposed in this PR? Add unit test coverage for three small value/utility classes, selected from the Codecov report. No production-code changes. | File | Codecov before | New coverage | | --- | --- | --- | | `Attribute.java` | 45.5% | new spec: accessors, null constructor checks, `toString` (lowercase wire name), all reachable `equals` branches, and `hashCode` (incl. the documented case-insensitive-equals / case-sensitive-hashCode quirk) | | `ObjectMapperUtils.scala` | 0% | new spec: `warmupObjectMapperForOperatorsSerde` spawns the named warmup thread, which is joined so its body executes | | `ArrowSourceOpDesc.scala` | 75% | extends the existing spec to cover `inferSchema` — happy path (local temp Arrow file) and the invalid-file `IOException` | ### Any related issues, documentation, discussions? Follow-up to the review feedback on #6043: prioritize tests that fill uncovered code paths. ### How was this PR tested? - `sbt "WorkflowOperator/testOnly *ObjectMapperUtilsSpec *ArrowSourceOpDescSpec"` and `sbt "WorkflowCore/testOnly *AttributeSpec"` — all green - scalafmt + scalafix on both modules — clean ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Code (Opus 4.8 [1M context]) -- 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]
