Skip to content

PM-6510 Mark timed-out Marathon Match tests as failed -> dev - #2424

Merged
jmgasper merged 1 commit into
devfrom
fix/PM-6510-marathon-timeout-status
Oct 1, 2026
Merged

jmgasper merged 1 commit into
devfrom
fix/PM-6510-marathon-timeout-status

Conversation

@jmgasper

@jmgasper jmgasper commented Oct 1, 2026

Copy link
Copy Markdown
Collaborator

Related JIRA Ticket:

https://topcoder.atlassian.net/browse/PM-6510

What's in this PR?

A timed-out Marathon Match submission could show Passed because the UI trusted the runner's completion status or treated any numeric score as a success. Resolve settled non-passing or negative results as Failed for both provisional and system testing, including legacy score fields, while preserving pending/cancelled lifecycle states, successful zero scores, and the newest retry result.

The attached timeout artifacts and the dev API both confirm the reported attempt failed: submission 4lwkAWaH6Vzg5f has repeated 10-second timeouts and -1 aggregates with isPassing: false in each scorer phase. The fix is confined to platform-ui; AWS access was unnecessary.

Validation:

  • 177 tests passed across Marathon utilities, challenge detail flows, and submission history, including the provisional/system timeout regressions.
  • yarn lint passed.
  • NODE_OPTIONS=--max-old-space-size=8192 yarn run build passed on Node 22.13.0.

@jmgasper
jmgasper requested a review from kkartunov as a code owner October 1, 2026 05:24
@jmgasper
jmgasper merged commit 98866e3 into dev Oct 1, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant