Skip to content

Conversation

euripedesrocha
Copy link
Collaborator

Description

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.

david-cermak and others added 30 commits June 9, 2025 12:42
Apply the pre-commit hook whitespace fixes to all files in the repo.

(Line endings, blank lines at end of file, trailing whitespace)
Also removes unused references to http-request app size in idf_performance.h
This is needed for testing different connection modes in the mqtt
library, specifically:
* test case CONFIG_EXAMPLE_CONNECT_CASE_NO_CERT: default server - expect to connect normally
Enables building C3 examples in CI.

Fixes related warnings/errors and disables examples that cannot run.
Update ROM API. Port changes from bringup branch.
Adds qemu configuration
Generalize the get_dut() to enable choosing DUT class per configuration
- Mqtt client will return error on wrong parameters. This fix the test
  code to setup the client for each test case.
* Fix thread safe issue in paho-mqtt library
* Move the weekend test to ethernet runner
Also increase the default sizes and repeat-counts to send more data and
exercise the library more intensly
euripedesrocha and others added 18 commits June 9, 2025 12:42
Publish test cases are prone to fail depending on Ci load.
- Add random client id on each iteration
- Make timeout configuration dependent on more scenario parameters
- Makes each case to subscribe/publish to unique topics
- Makes test to wait for the successful subscription on the runner
4kB stack size for mosquitto runner task wasn't enough and with
recent updates of sock_utils it caused stack overflows
Certificates expired, so need to update for testing to pass.
- Previous fix ommited one of the client certificates by mistaque.
- This regenerates all certificates to clean that up.
- Example CA certificates that are used for self-signed client certificates
need to include the Key Usage parameter.
- Python3.13 changed the default context of the SSL context that is
generated using ssl.create_default_context() by enabling the VERIFY_X509_STRICT
flag by default
This test case fails mostly for network related issues, with retrys we
might avoid the failure of the whole job.
@euripedesrocha euripedesrocha force-pushed the adds_publish_connect branch 10 times, most recently from 0fbdabd to 6b23f40 Compare June 20, 2025 13:29
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.