Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[EIM-92] [Test] Update IDF version #81

Merged
merged 3 commits into from
Dec 17, 2024

Conversation

Fabricio-ESP
Copy link
Collaborator

Description

This PR removed the check for IDF version 5.3.1 since this version is now replaced by 5.3.2.
Also update CN runner to use version 5.3.2 during installation.

Replaced throw errors on test after hook to just log them.

Related

Testing


Checklist

Before submitting a Pull Request, please ensure the following:

  • 🚨 This PR does not introduce breaking changes.
  • All CI checks (GH Actions) pass.
  • Documentation is updated as needed.
  • Tests are updated or added as necessary.
  • Code is well-commented, especially in complex areas.
  • Git history is clean — commits are squashed to the minimum necessary.

Copy link

github-actions bot commented Dec 13, 2024

Test Results

10 files   - 3  43 suites   - 6   53m 33s ⏱️ - 32m 39s
17 tests +3  17 ✅ + 7  0 💤 ±0  0 ❌  - 4 
43 runs  +6  43 ✅ +15  0 💤 ±0  0 ❌  - 9 

Results for commit c0e74d8. ± Comparison against base commit 7d1a03d.

This pull request removes 2 and adds 5 tests. Note that renamed tests count towards both.
"after all" hook for "Should install IDF using wizard and default values" ‑ Installation using default settings Check IDF Install Wizard steps "after all" hook for "Should install IDF using wizard and default values"
Should install IDF using specified parameters ‑ Installation using custom settings Run custom installation using given parameters Should install IDF using specified parameters
Should build project for the selected target ‑ Installation using default settings create and build sample project Should build project for the selected target
Should build project for the selected target ‑ Installation using mirror jihulab and dl.espressif.com create and build sample project Should build project for the selected target
Should create a new project based on a template ‑ Installation using default settings create and build sample project Should create a new project based on a template
Should set the target ‑ Installation using default settings create and build sample project Should set the target
Should set the target ‑ Installation using mirror jihulab and dl.espressif.com create and build sample project Should set the target

♻️ This comment has been updated with latest results.

@Fabricio-ESP Fabricio-ESP merged commit 5242fac into master Dec 17, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant