The GitHub Actions job "Required Checks" on 
texera.git/test-operator-utility-classes has succeeded.
Run started by GitHub user aglinxinyuan (triggered by aglinxinyuan).

Head commit for run:
c1b3d299c68965f30b32cc7eda9fcc70cd00e66e / Xinyuan Lin <[email protected]>
test(workflow-operator): assert exact retry-attempt counts in URLFetchUtilSpec; 
fix misleading comment

Address Copilot review feedback on #5798:

URLFetchUtilSpec — the failure/default tests only checked the final None,
which would still pass if the retry loop ran the wrong number of times, and
the default-retries test relied on Scala's synthetic 
getInputStreamFromURL$default$2
accessor name (compiler/version-specific). Replace both with a counting
URLStreamHandler that records every openConnection call, then assert the EXACT
number of attempts:
  - retries = 0 -> 0 connections opened (loop body never runs; proves no 
attempt)
  - retries = 1 -> exactly 1
  - retries = 2 -> exactly 2
  - default (no arg) -> exactly 5 (validates the default value behaviorally)
  - first-attempt success -> exactly 1 (proves early return, no extra 
connections)
The realistic file:-URL success path is retained.

OperatorDescriptorUtilsSpec — fix a comment that read '1 worker should get
everything' on a call with totalNumWorkers = 4; it now reads
'1 = 4*0 + 1 -> only the first slot gets the single unit'.

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

With regards,
GitHub Actions via GitBox

Reply via email to