Skip to content

Commit 1aa54ec

Browse files
bpamiriclaude
andcommitted
Fix JUnit XML int parsing and force Node.js 24 for download-artifact
- Cast count attributes to int() in JUnit XML generation (CFML returns floats for numeric values, causing ValueError in publish action) - Set FORCE_JAVASCRIPT_ACTIONS_TO_NODE24 on publish-results and test-matrix-summary jobs (download-artifact@v6 still ships Node 20) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 29c8870 commit 1aa54ec

File tree

1 file changed

+10
-6
lines changed

1 file changed

+10
-6
lines changed

.github/workflows/tests.yml

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -305,17 +305,17 @@ jobs:
305305
process_suite(parent_el, child)
306306
307307
root = Element('testsuites',
308-
tests=str(d.get('totalSpecs', 0)),
309-
failures=str(d.get('totalFail', 0)),
310-
errors=str(d.get('totalError', 0)),
308+
tests=str(int(d.get('totalSpecs', 0))),
309+
failures=str(int(d.get('totalFail', 0))),
310+
errors=str(int(d.get('totalError', 0))),
311311
time=str(d.get('totalDuration', 0) / 1000))
312312
313313
for b in d.get('bundleStats', []):
314314
ts = SubElement(root, 'testsuite',
315315
name=b.get('name', ''),
316-
tests=str(b.get('totalSpecs', 0)),
317-
failures=str(b.get('totalFail', 0)),
318-
errors=str(b.get('totalError', 0)),
316+
tests=str(int(b.get('totalSpecs', 0))),
317+
failures=str(int(b.get('totalFail', 0))),
318+
errors=str(int(b.get('totalError', 0))),
319319
time=str(b.get('totalDuration', 0) / 1000))
320320
for s in b.get('suiteStats', []):
321321
process_suite(ts, s)
@@ -436,6 +436,8 @@ jobs:
436436
permissions:
437437
checks: write
438438
pull-requests: write
439+
env:
440+
FORCE_JAVASCRIPT_ACTIONS_TO_NODE24: true
439441
steps:
440442
- name: Download JUnit artifacts
441443
uses: actions/download-artifact@v6
@@ -459,6 +461,8 @@ jobs:
459461
needs: tests
460462
if: always()
461463
runs-on: ubuntu-latest
464+
env:
465+
FORCE_JAVASCRIPT_ACTIONS_TO_NODE24: true
462466
steps:
463467
- name: Download all test result artifacts
464468
uses: actions/download-artifact@v6

0 commit comments

Comments
 (0)