Skip to content

Commit b09ee5c

Browse files
authored
Added uploading test results to CI and changed BAT and AllTests triggers (#240)
## 📝 Description BAT: * Added automatic run for pushes to release branch * Added upload of test results All Tests: * Added automatic trigger to pushes to main branch * Removed automatic trigger for PRs * Added upload of test results ## ✨ Type of Change Select the type of change your PR introduces: - [ ] 🐞 **Bug fix** – Non-breaking change which fixes an issue - [ ] 🚀 **New feature** – Non-breaking change which adds functionality - [ ] 🔨 **Refactor** – Non-breaking change which refactors the code base - [ ] 💥 **Breaking change** – Changes that break existing functionality - [ ] 📚 **Documentation update** - [ ] 🔒 **Security update** - [x] 🧪 **Tests** - [x] 🚂 **CI** ## 🧪 Testing Scenarios Describe how the changes were tested and how reviewers can test them too: - [ ] ✅ Tested manually - [ ] 🤖 Ran automated end-to-end tests - [x] N/A - no changes in code ## ✅ Checklist Before submitting the PR, ensure the following: - [ ] 🔍 PR title is clear and descriptive - [ ] 📝 For internal contributors: If applicable, include the JIRA ticket number (e.g., ITEP-123456) in the PR **title**. Do **not** include full URLs - [ ] 💬 I have commented my code, especially in hard-to-understand areas - [ ] 📄 I have made corresponding changes to the documentation - [ ] ✅ I have added tests that prove my fix is effective or my feature works
1 parent c55a7e9 commit b09ee5c

File tree

2 files changed

+50
-7
lines changed

2 files changed

+50
-7
lines changed

.github/workflows/tests-all.yml

Lines changed: 25 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -20,15 +20,9 @@ on:
2020
required: false
2121
type: number
2222
default: 70
23-
pull_request:
24-
branches:
25-
- release-*
26-
types:
27-
- opened
28-
- synchronize
29-
- reopened
3023
push:
3124
branches:
25+
- main
3226
- release-*
3327

3428
# Trigger workflow when enqueued to a merge group
@@ -97,6 +91,30 @@ jobs:
9791
pip install -r .github/resources/requirements.txt
9892
make run_tests
9993
94+
- name: Rename logs to remove colons
95+
if: always()
96+
run: |
97+
set +e # Ignore errors if no logs are found
98+
find . -wholename "**/test_data/**/*.log" -exec rename 's/:/-/g' {} +
99+
exit 0
100+
101+
- name: Upload Test Reports
102+
if: always()
103+
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 #v4.6.2
104+
with:
105+
name: tests-reports
106+
path: |
107+
tests/**/test_reports/**/*.html
108+
tests/**/test_reports/**/*.xml
109+
110+
- name: Upload Test Logs
111+
if: always()
112+
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 #v4.6.2
113+
with:
114+
name: tests-logs
115+
path: |
116+
**/test_data/**/*.log
117+
100118
- name: "Stop all running Docker containers"
101119
if: always()
102120
run: |

.github/workflows/tests-bat.yml

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ on:
2929
push:
3030
branches:
3131
- main
32+
- release-*
3233

3334
# Trigger workflow when enqueued to a merge group
3435
# (must be under 'on')
@@ -121,6 +122,30 @@ jobs:
121122
git clean -fdx
122123
make run_basic_acceptance_tests
123124
125+
- name: Rename logs to remove colons
126+
if: always()
127+
run: |
128+
set +e # Ignore errors if no logs are found
129+
find . -wholename "**/test_data/**/*.log" -exec rename 's/:/-/g' {} +
130+
exit 0
131+
132+
- name: Upload Test Reports
133+
if: always()
134+
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 #v4.6.2
135+
with:
136+
name: tests-reports
137+
path: |
138+
tests/**/test_reports/**/*.html
139+
tests/**/test_reports/**/*.xml
140+
141+
- name: Upload Test Logs
142+
if: always()
143+
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 #v4.6.2
144+
with:
145+
name: tests-logs
146+
path: |
147+
**/test_data/**/*.log
148+
124149
- name: "Stop all running Docker containers"
125150
if: always()
126151
run: |

0 commit comments

Comments
 (0)