Skip to content

Commit f28cfdf

Browse files
committed
Include failure messages in release diagnostics annotations
1 parent af13e23 commit f28cfdf

1 file changed

Lines changed: 9 additions & 4 deletions

File tree

‎.github/workflows/release-diagnostics.yml‎

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ jobs:
2222
PG_DIAG_DOCKER_INTEGRATION: "1"
2323
PYTHONDONTWRITEBYTECODE: "1"
2424
run: python -m pytest -q -p no:cacheprovider --junitxml=release-tests.xml
25-
- name: Report failed test names
25+
- name: Report failed tests
2626
if: always()
2727
run: |
2828
python - <<'PY'
@@ -33,8 +33,13 @@ jobs:
3333
print("::error::pytest did not produce its result document")
3434
else:
3535
for case in ET.parse(path).iter("testcase"):
36-
if case.find("failure") is not None or case.find("error") is not None:
36+
failure = case.find("failure")
37+
if failure is None:
38+
failure = case.find("error")
39+
if failure is not None:
3740
name = case.get("classname", "") + "::" + case.get("name", "")
38-
name = name.replace("%", "%25").replace("\r", "%0D").replace("\n", "%0A")
39-
print("::error title=Failed release test::" + name)
41+
detail = failure.get("message", "")[:2000]
42+
message = name + "\n" + detail
43+
message = message.replace("%", "%25").replace("\r", "%0D").replace("\n", "%0A")
44+
print("::error title=Failed release test::" + message)
4045
PY

0 commit comments

Comments
 (0)