Skip to content

Commit a8746f4

Browse files
authored
[Test] Fix CLI test (#284)
Fix CLI test typo Enable extended test on release branch PR Fix test result artifact name and workflow cleaning step Fix offline test workflow call. Pin python 3.13.8
1 parent 1a4747d commit a8746f4

File tree

5 files changed

+10
-13
lines changed

5 files changed

+10
-13
lines changed

.github/workflows/build.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -861,7 +861,7 @@ jobs:
861861
with:
862862
run_id: ${{ github.run_id }}
863863
ref: ${{ github.event.pull_request.head.ref || github.ref }}
864-
run_extended: ${{(github.event_name == 'release' && github.event.action == 'created') || github.event_name == 'schedule' || (github.event_name == 'pull_request' && contains(github.event.pull_request.labels.*.name, 'EXTENDEDTEST'))}}
864+
run_extended: ${{(github.event_name == 'release' && github.event.action == 'created') || github.event_name == 'schedule' || (github.event_name == 'pull_request' && contains(github.event.pull_request.labels.*.name, 'EXTENDEDTEST')) || startsWith(github.event.pull_request.head.ref, 'release')}}
865865
run_self_hosted: ${{(github.event_name == 'release' && github.event.action == 'created') || github.event_name == 'schedule' || (github.event_name == 'pull_request' && contains(github.event.pull_request.labels.*.name, 'CNRUNNER'))}}
866866

867867
Autotest-GUI:
@@ -872,7 +872,7 @@ jobs:
872872
with:
873873
run_id: ${{ github.run_id }}
874874
ref: ${{ github.event.pull_request.head.ref || github.ref }}
875-
run_extended: ${{(github.event_name == 'release' && github.event.action == 'created') || github.event_name == 'schedule' || (github.event_name == 'pull_request' && contains(github.event.pull_request.labels.*.name, 'EXTENDEDTEST'))}}
875+
run_extended: ${{(github.event_name == 'release' && github.event.action == 'created') || github.event_name == 'schedule' || (github.event_name == 'pull_request' && contains(github.event.pull_request.labels.*.name, 'EXTENDEDTEST')) || startsWith(github.event.pull_request.head.ref, 'release')}}
876876

877877
update-release-info:
878878
name: Update Release Information

.github/workflows/build_offline_installer_archives.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -583,3 +583,4 @@ jobs:
583583
with:
584584
ref: ${{ inputs.ref || github.ref }}
585585
run_id: ${{ github.run_id }}
586+
eim_cli_run_id: ${{ (github.event_name == 'workflow_call' && inputs.run_id) || 'release' }}

.github/workflows/test_cli.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -248,7 +248,7 @@ jobs:
248248
uses: actions/upload-artifact@v4
249249
if: matrix.os != 'windows-latest'
250250
with:
251-
name: autotest-CLI-results-${{ matrix.package_name }}${{ matrix.run_on == 'MirrorRunner' && '-AlternateMirrors' || '' }}${{inputs.run_self_hosted == 'true' && '-CN' || '' }}
251+
name: autotest-CLI-results-${{ matrix.package_name }}${{ matrix.run_on == 'MirrorRunner' && '-AlternateMirrors' || '' }}$${{ matrix.os == 'self-hosted' && '-CN' || '' }}
252252
path: |
253253
./tests/results-*.json
254254
./tests/*.log
@@ -257,13 +257,13 @@ jobs:
257257
uses: actions/upload-artifact@v4
258258
if: matrix.os == 'windows-latest'
259259
with:
260-
name: autotest-CLI-results-${{ matrix.package_name }}${{ matrix.run_on == 'MirrorRunner' && '-AlternateMirrors' || '' }}${inputs.run_self_hosted == 'true' && '-CN' || '' }}
260+
name: autotest-CLI-results-${{ matrix.package_name }}
261261
path: |
262262
./tests/results-*.json
263263
./tests/*.log
264264
265265
- name: Clean Up runner (Self-hosted MirrorRunner)
266-
if: matrix.run_on == 'MirrorRunner' && inputs.run_self_hosted == 'true'
266+
if: matrix.run_on == 'MirrorRunner' && matrix.os == 'self-hosted'
267267
run: |
268268
rm -rf ~/.local/share/eim
269269
rm -rf ~/runner/_work/idf-im-ui

.github/workflows/test_offline.yml

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ jobs:
6464
- name: Set up Python
6565
uses: actions/setup-python@v6
6666
with:
67-
python-version: "3.13"
67+
python-version: "3.13.8"
6868

6969
- name: Download Build Info files
7070
uses: actions/download-artifact@v5
@@ -122,6 +122,7 @@ jobs:
122122
- name: Run IDF offline installation (non-Windows)
123123
if: matrix.os != 'windows-latest'
124124
run: |
125+
python3 --version
125126
export LOG_TO_FILE="true"
126127
export EIM_CLI_PATH="../artifacts/eim"
127128
export BUILD_INFO_PATH="../artifacts"
@@ -140,15 +141,10 @@ jobs:
140141
141142
ls ./artifacts/
142143
143-
- name: Update Python (Windows)
144-
if: matrix.os == 'windows-latest'
145-
uses: actions/setup-python@v6
146-
with:
147-
python-version: "3.11"
148-
149144
- name: Run IDF offline installation (Windows)
150145
if: matrix.os == 'windows-latest'
151146
run: |
147+
python3 --version
152148
$env:LOG_TO_FILE="true"
153149
$env:EIM_CLI_PATH = "..\artifacts\eim.exe"
154150
$env:BUILD_INFO_PATH = "..\artifacts"

tests/scripts/CLIWizardInstall.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -180,7 +180,7 @@ export function runCLIWizardInstallTest({
180180
await new Promise((resolve) => setTimeout(resolve, 500));
181181

182182
const selectPyPIMirror = await testRunner.waitForOutput(
183-
"Select a PyPI mirror for download Python packages"
183+
"Select a PyPI mirror to download Python packages"
184184
);
185185
expect(selectPyPIMirror, "Failed to ask for PyPI download mirror").to.be
186186
.true;

0 commit comments

Comments
 (0)