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:
00480e85b49f315a85229874edfff91197c7d7ba / Amer Mukhtar 
<[email protected]>
test(frontend): add unit test coverage for parseLogicalOperatorPortID (#6345)

Description

What changes were proposed in this PR?

This PR adds unit test coverage for parseLogicalOperatorPortID in
frontend/src/app/common/util/logical-operator-port-serde.ts. This is a
pure parser function that converts a port-id string into a structured
object, and it previously had no tests. A new spec file,
logical-operator-port-serde.spec.ts, was added following the style of
the existing size-formatter.util.spec.ts. It covers valid inputs, where
"input-0" parses to { portNumber: 0, portType: "input" } and "output-12"
parses to { portNumber: 12, portType: "output" }, as well as invalid
inputs such as "foo-1", "input-", "input-1-2", and the empty string,
which all return undefined. This is a test-only change and no production
code was modified.

Any related issues, documentation, discussions?

Closes #6257

How was this PR tested?

The new unit tests were run via ng test (vitest runner) and all 6 tests
pass.

Was this PR authored or co-authored using generative AI tooling?

Generated-by: Claude Code

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

With regards,
GitHub Actions via GitBox

Reply via email to