The GitHub Actions job "Required Checks" on 
texera.git/test-global-port-identity-serde has succeeded.
Run started by GitHub user eugenegujing (triggered by eugenegujing).

Head commit for run:
d1285843a5261b9f54d99b17a729bf03f5ec8587 / Eugene <[email protected]>
test(workflow-core): add more GlobalPortIdentitySerde edge cases

Pin three more behaviors of GlobalPortIdentitySerde that were previously
uncovered:

- a value containing '=' round-trips (only ',' is a sensitive separator; the
  deserializer anchors on the literal `field=` prefix and captures up to the
  next comma), the counterpart to the existing comma test
- an empty logicalOpId serializes into `(logicalOpId=,...)` but the result can
  no longer be deserialized, since the deserializer's `[^,]+` requires at least
  one character — the serialize side of the empty-field asymmetry
- the serialize-side require guards name the offending field in their message
  (logicalOpId / layerName / portId), so a regression that wires a check to the
  wrong field is caught instead of silently still throwing

Verified with:
  sbt "WorkflowCore/testOnly 
org.apache.texera.amber.util.serde.GlobalPortIdentitySerdeSpec 
org.apache.texera.amber.util.serde.PortIdentitySerdeSpec"
-> 31 tests, 0 failures (1 pendingUntilFixed, pre-existing).

Co-Authored-By: Claude Opus 4.8 (1M context)

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

With regards,
GitHub Actions via GitBox

Reply via email to