Skip to content

Commit c3f9366

Browse files
lucasssvazlucasssvaz
committed
test
fix test test test test Test test test fix fix fix test fix Fix
1 parent cbf1e94 commit c3f9366

File tree

6 files changed

+325
-181
lines changed

6 files changed

+325
-181
lines changed

‎.github/workflows/build_tests.yml

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@ jobs:
2020
id: ${{ github.event.pull_request.number || github.ref }}-${{ github.event.pull_request.head.sha || github.sha }}-${{ inputs.chip }}-${{ inputs.type }}
2121
steps:
2222
- name: Check if already built
23-
if: ${{ github.event.pull_request.number != null }}
2423
id: cache-build-binaries
2524
uses: actions/cache/restore@v4
2625
with:
@@ -46,21 +45,6 @@ jobs:
4645
- name: Checkout user repository
4746
if: ${{ steps.check-build.outputs.enabled == 'true' }}
4847
uses: actions/checkout@v4
49-
with:
50-
ref: ${{ github.event.pull_request.head.sha || github.sha }}
51-
persist-credentials: false
52-
sparse-checkout-cone-mode: false
53-
sparse-checkout: |
54-
/*
55-
!.github
56-
57-
# To avoid giving unknown scripts elevated permissions, download them from the master branch
58-
- name: Get CI scripts from master
59-
if: ${{ steps.check-build.outputs.enabled == 'true' }}
60-
run: |
61-
mkdir -p .github
62-
cd .github
63-
curl https://codeload.github.com/${{ github.repository }}/tar.gz/master | tar -xz --strip=2 arduino-esp32-master/.github
6448

6549
- name: Get libs cache
6650
uses: actions/cache@v4

‎.github/workflows/hw.yml

Lines changed: 2 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,7 @@ jobs:
2222
image: python:3.10.1-bullseye
2323
options: --privileged
2424
steps:
25-
- name: Check if already built
26-
if: ${{ github.event.pull_request.number != null }}
25+
- name: Check if already passed
2726
id: cache-results
2827
uses: actions/cache/restore@v4
2928
with:
@@ -48,21 +47,6 @@ jobs:
4847
- name: Checkout user repository
4948
if: ${{ steps.check-tests.outputs.enabled == 'true' }}
5049
uses: actions/checkout@v4
51-
with:
52-
ref: ${{ github.event.pull_request.head.sha || github.sha }}
53-
persist-credentials: false
54-
sparse-checkout-cone-mode: false
55-
sparse-checkout: |
56-
/*
57-
!.github
58-
59-
# To avoid giving unknown scripts elevated permissions, download them from the master branch
60-
- name: Get CI scripts from master
61-
if: ${{ steps.check-tests.outputs.enabled == 'true' }}
62-
run: |
63-
mkdir -p .github
64-
cd .github
65-
curl https://codeload.github.com/${{ github.repository }}/tar.gz/master | tar -xz --strip=2 arduino-esp32-master/.github
6650

6751
# setup-python currently only works on ubuntu images
6852
# - uses: actions/setup-python@v5
@@ -95,7 +79,7 @@ jobs:
9579
9680
- name: Upload ${{ inputs.chip }} ${{ inputs.type }} hardware results as cache
9781
uses: actions/cache/save@v4
98-
if: ${{ always() && steps.check-tests.outputs.enabled == 'true' }}
82+
if: ${{ steps.check-tests.outputs.enabled == 'true' }}
9983
with:
10084
key: tests-${{ env.id }}-results-hw
10185
path: |

‎.github/workflows/qemu.yml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,7 @@ jobs:
1818
QEMU_INSTALL_PATH: "$HOME"
1919
runs-on: ubuntu-latest
2020
steps:
21-
- name: Check if already run
22-
if: ${{ github.event.pull_request.number != null }}
21+
- name: Check if already passed
2322
id: get-cache-results
2423
uses: actions/cache/restore@v4
2524
with:
@@ -125,7 +124,7 @@ jobs:
125124

126125
- name: Upload ${{ inputs.chip }} ${{ inputs.type }} QEMU results as cache
127126
uses: actions/cache/save@v4
128-
if: ${{ always() && steps.check-tests.outputs.enabled == 'true' }}
127+
if: ${{ steps.check-tests.outputs.enabled == 'true' }}
129128
with:
130129
key: tests-${{ env.id }}-results-qemu
131130
path: |

‎.github/workflows/tests.yml

Lines changed: 67 additions & 103 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,22 @@ name: Runtime Tests
22

33
on:
44
workflow_dispatch:
5-
pull_request_target:
6-
types: [opened, reopened, closed, synchronize, labeled, unlabeled]
5+
pull_request:
6+
types: [opened, reopened, synchronize, labeled, unlabeled]
77
paths:
8+
- '.github/workflows/tests.yml'
9+
- '.github/workflows/build_tests.yml'
10+
- '.github/workflows/hw.yml'
11+
- '.github/workflows/qemu.yml'
12+
- '.github/workflows/wokwi.yml'
13+
- '.github/workflows/tests_publish.yml'
14+
- '.github/scripts/*.sh'
15+
- '!.github/scripts/check-cmakelists.sh'
16+
- '!.github/scripts/find_*'
17+
- '!.github/scripts/on-*.sh'
18+
- '!.github/scripts/set_push_chunks.sh'
19+
- '!.github/scripts/update-version.sh'
20+
- '!.github/scripts/upload_py_tools.sh'
821
- 'tests/**'
922
- 'cores/**'
1023
- 'libraries/**'
@@ -19,14 +32,19 @@ concurrency:
1932
group: tests-${{ github.event.pull_request.number || github.ref }}
2033
cancel-in-progress: true
2134

22-
# To avoid giving elevated permissions to the entire workflow, specify default permissions at the top level
23-
# and then override them for specific jobs.
24-
permissions: { contents: read }
25-
2635
jobs:
36+
push-event-file:
37+
name: Push event file
38+
runs-on: ubuntu-latest
39+
steps:
40+
- name: Upload
41+
uses: actions/upload-artifact@v4
42+
with:
43+
name: event_file
44+
path: ${{ github.event_path }}
45+
2746
gen-matrix:
2847
name: Generate matrix
29-
if: github.event.action != 'closed'
3048
runs-on: ubuntu-latest
3149
outputs:
3250
build-types: ${{ steps.set-matrix.outputs.build-types }}
@@ -59,9 +77,8 @@ jobs:
5977
6078
call-build-tests:
6179
name: Build
62-
uses: espressif/arduino-esp32/.github/workflows/build_tests.yml@master
80+
uses: ./.github/workflows/build_tests.yml
6381
needs: gen-matrix
64-
if: github.event.action != 'closed'
6582
strategy:
6683
matrix:
6784
type: ${{ fromJson(needs.gen-matrix.outputs.build-types) }}
@@ -70,13 +87,15 @@ jobs:
7087
type: ${{ matrix.type }}
7188
chip: ${{ matrix.chip }}
7289

90+
# Wokwi tests are run after this workflow as it needs access to secrets
91+
7392
call-hardware-tests:
7493
name: Hardware
75-
uses: espressif/arduino-esp32/.github/workflows/hw.yml@master
94+
uses: ./.github/workflows/hw.yml
7695
needs: [gen-matrix, call-build-tests]
7796
if: |
7897
github.repository == 'espressif/arduino-esp32' &&
79-
(github.event_name != 'pull_request_target' ||
98+
(github.event_name != 'pull_request' ||
8099
contains(github.event.pull_request.labels.*.name, 'hil_test'))
81100
strategy:
82101
fail-fast: false
@@ -87,26 +106,10 @@ jobs:
87106
type: ${{ matrix.type }}
88107
chip: ${{ matrix.chip }}
89108

90-
call-wokwi-tests:
91-
name: Wokwi
92-
uses: espressif/arduino-esp32/.github/workflows/wokwi.yml@master
93-
needs: [gen-matrix, call-build-tests]
94-
if: github.event.action != 'closed'
95-
strategy:
96-
fail-fast: false
97-
matrix:
98-
type: ${{ fromJson(needs.gen-matrix.outputs.wokwi-types) }}
99-
chip: ['esp32', 'esp32s2', 'esp32s3', 'esp32c3', 'esp32c6', 'esp32h2']
100-
secrets:
101-
WOKWI_CLI_TOKEN: ${{ secrets.WOKWI_CLI_TOKEN }}
102-
with:
103-
type: ${{ matrix.type }}
104-
chip: ${{ matrix.chip }}
105-
106109
# This job is disabled for now
107110
call-qemu-tests:
108111
name: QEMU
109-
uses: espressif/arduino-esp32/.github/workflows/qemu.yml@master
112+
uses: ./.github/workflows/qemu.yml
110113
needs: [gen-matrix, call-build-tests]
111114
if: false
112115
strategy:
@@ -118,78 +121,39 @@ jobs:
118121
type: ${{ matrix.type }}
119122
chip: ${{ matrix.chip }}
120123

121-
unit-test-results:
122-
name: Unit Test Results
123-
needs: [call-hardware-tests, call-wokwi-tests, call-qemu-tests]
124-
if: always() && github.event_name == 'pull_request_target'
125-
runs-on: ubuntu-latest
126-
permissions:
127-
checks: write
128-
pull-requests: write
129-
steps:
130-
- name: Download and Extract HW Artifacts
131-
uses: actions/download-artifact@v4
132-
continue-on-error: true
133-
with:
134-
merge-multiple: true
135-
pattern: tests-results-hw-*
136-
path: ./results/hw
137-
138-
- name: Download and Extract Wokwi Artifacts
139-
uses: actions/download-artifact@v4
140-
continue-on-error: true
141-
with:
142-
merge-multiple: true
143-
pattern: tests-results-wokwi-*
144-
path: ./results/wokwi
145-
146-
- name: Download and Extract QEMU Artifacts
147-
uses: actions/download-artifact@v4
148-
continue-on-error: true
149-
with:
150-
merge-multiple: true
151-
pattern: tests-results-qemu-*
152-
path: ./results/qemu
153-
154-
- name: Publish Unit Test Results
155-
uses: EnricoMi/publish-unit-test-result-action@v2
156-
with:
157-
commit: ${{ github.event.pull_request.head.sha || github.sha }}
158-
files: ./results/**/*.xml
159-
clean:
160-
name: Clean objects
161-
needs: unit-test-results
162-
if: always()
163-
permissions:
164-
actions: write
165-
runs-on: ubuntu-latest
166-
steps:
167-
- name: Clean up caches
168-
uses: actions/github-script@v7
169-
with:
170-
script: |
171-
const ref = '${{ github.event.pull_request.number || github.ref }}';
172-
const key_prefix = 'tests-' + ref + '-';
173-
174-
if ('${{ github.event_name }}' == 'pull_request_target' && '${{ github.event.action }}' != 'closed') {
175-
console.log('Skipping cache cleanup for open PR');
176-
return;
177-
}
178-
179-
await github.paginate(github.rest.actions.getActionsCacheList, {
180-
owner: context.repo.owner,
181-
repo: context.repo.repo,
182-
per_page: 100,
183-
key: key_prefix
184-
}).then(caches => {
185-
if (caches) {
186-
for (const cache of caches) {
187-
console.log(`Deleting cache: ${cache.key}`);
188-
github.rest.actions.deleteActionsCacheById({
189-
owner: context.repo.owner,
190-
repo: context.repo.repo,
191-
cache_id: cache.id
192-
});
193-
}
194-
}
195-
});
124+
# clean:
125+
# name: Clean objects
126+
# if: always()
127+
# permissions:
128+
# actions: write
129+
# runs-on: ubuntu-latest
130+
# steps:
131+
# - name: Clean up caches
132+
# uses: actions/github-script@v7
133+
# with:
134+
# script: |
135+
# const ref = '${{ github.event.pull_request.number || github.ref }}';
136+
# const key_prefix = 'tests-' + ref + '-';
137+
138+
# if ('${{ github.event_name }}' == 'pull_request_target' && '${{ github.event.action }}' != 'closed') {
139+
# console.log('Skipping cache cleanup for open PR');
140+
# return;
141+
# }
142+
143+
# await github.paginate(github.rest.actions.getActionsCacheList, {
144+
# owner: context.repo.owner,
145+
# repo: context.repo.repo,
146+
# per_page: 100,
147+
# key: key_prefix
148+
# }).then(caches => {
149+
# if (caches) {
150+
# for (const cache of caches) {
151+
# console.log(`Deleting cache: ${cache.key}`);
152+
# github.rest.actions.deleteActionsCacheById({
153+
# owner: context.repo.owner,
154+
# repo: context.repo.repo,
155+
# cache_id: cache.id
156+
# });
157+
# }
158+
# }
159+
# });

‎.github/workflows/tests_publish.yml

Lines changed: 71 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,71 @@
1+
name: Publish test results
2+
3+
on:
4+
workflow_run:
5+
workflows: ["Wokwi tests"]
6+
types:
7+
- completed
8+
9+
# No permissions by default
10+
permissions: { contents: read }
11+
12+
jobs:
13+
unit-test-results:
14+
name: Unit Test Results
15+
if: |
16+
github.event.workflow_run.conclusion == 'success' ||
17+
github.event.workflow_run.conclusion == 'failure' ||
18+
github.event.workflow_run.conclusion == 'timed_out'
19+
runs-on: ubuntu-latest
20+
permissions:
21+
actions: read
22+
statuses: write
23+
checks: write
24+
pull-requests: write
25+
steps:
26+
- name: Download and Extract Artifacts
27+
uses: dawidd6/action-download-artifact@v6
28+
with:
29+
run_id: ${{ github.event.workflow_run.id }}
30+
path: ./artifacts
31+
32+
- name: Get original info
33+
run: |
34+
original_event=$(cat ./artifacts/parent-artifacts/event.txt)
35+
original_sha=$(cat ./artifacts/parent-artifacts/sha.txt)
36+
echo "original_event=$original_event" >> $GITHUB_ENV
37+
echo "original_sha=$original_sha" >> $GITHUB_ENV
38+
39+
echo "original_event = $original_event"
40+
echo "original_sha = $original_sha"
41+
42+
- name: Publish Unit Test Results
43+
uses: EnricoMi/publish-unit-test-result-action@v2
44+
with:
45+
commit: ${{ env.original_sha }}
46+
event_file: ./artifacts/parent-artifacts/event_file/event.json
47+
event_name: ${{ env.original_event }}
48+
files: ./artifacts/**/*.xml
49+
action_fail: true
50+
51+
- name: Report conclusion
52+
uses: actions/github-script@v7
53+
if: always()
54+
with:
55+
script: |
56+
const owner = '${{ github.repository_owner }}';
57+
const repo = '${{ github.repository }}'.split('/')[1];
58+
const sha = '${{ env.original_sha }}';
59+
core.debug(`owner: ${owner}`);
60+
core.debug(`repo: ${repo}`);
61+
core.debug(`sha: ${sha}`);
62+
const { context: name, state } = (await github.rest.repos.createCommitStatus({
63+
context: 'Runtime Tests / Report results (${{ env.original_event }} -> workflow_run -> workflow_run)',
64+
owner: owner,
65+
repo: repo,
66+
sha: sha,
67+
state: '${{ job.status }}',
68+
description: '${{ job.status }}' ? 'Runtime tests successful' : 'Runtime tests failed',
69+
target_url: 'https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}'
70+
})).data;
71+
core.info(`${name} is ${state}`);

0 commit comments

Comments
 (0)