The GitHub Actions job "Required Checks" on texera.git/main has succeeded.
Run started by GitHub user github-merge-queue[bot] (triggered by 
github-merge-queue[bot]).

Head commit for run:
65ce1e52d10b657338b41f220f2326d3fad86a3e / Matthew B. <[email protected]>
test(amber): expand ExecutionResultService spec coverage (#5390)

### What changes were proposed in this PR?
- Pin the exact binary-preview formatting in `convertTuplesToJson`: the
existing spec asserted only byte size and the presence of an ellipsis,
so this adds the 13-bits-or-fewer branch (a 1-byte field renders its
full bit string with no ellipsis) and the exact leading-10 + ellipsis +
trailing-3 format for larger fields.
- Add serialization-contract tests for the `WebOutputMode` /
`WebResultUpdate` ADTs the frontend dispatches on: the `type`
discriminator for `PaginationMode`/`SetSnapshotMode`/`SetDeltaMode`,
plus the shapes of `WebPaginationUpdate` and `WebDataUpdate`.
- Pin `TIMESTAMP` field handling in `convertTuplesToJson`: timestamps
fall through the default arm and are serialized by the shared
`objectMapper`, asserted against the mapper's own output so the check is
time-zone independent.
### Any related issues, documentation, or discussions?
Closes: #5389
### How was this PR tested?
- Run `sbt "WorkflowExecutionService/testOnly
*ExecutionResultServiceSpec"` and expect 17 examples to pass (11
original, 6 added).
- Local sbt runs are blocked on JDK 25 (JaCoCo 0.8.11 cannot instrument
Java 25 bytecode, plus a project plugin load error), so the earlier
additions were exercised directly via ScalaTest off the dependency
classpath; the latest `TIMESTAMP` case was not run locally and relies on
CI, which runs the suite on a supported JDK.
### Was this PR authored or co-authored using generative AI tooling?
Co-authored with Claude Opus 4.8 in compliance with ASF

Report URL: https://github.com/apache/texera/actions/runs/27087987391

With regards,
GitHub Actions via GitBox

Reply via email to