Skip to content

ESP-IDF: add certificate_auth to sample tests #715

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

Merged
merged 3 commits into from
Jan 8, 2025

Conversation

szczys
Copy link
Contributor

@szczys szczys commented Dec 31, 2024

Add the certificate_auth sample to the ESP-IDF testing suite.

Copy link

github-actions bot commented Dec 31, 2024

Visit the preview URL for this PR (updated for commit 8a9f3b7):

https://golioth-firmware-sdk-doxygen-dev--pr715-szczys-esp-idf-jlf4e981.web.app

(expires Tue, 14 Jan 2025 15:42:50 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: a9993e61697a3983f3479e468bcb0b616f9a0578

Copy link

github-actions bot commented Dec 31, 2024

Code Coverage

Code Coverage

Package Line Rate Branch Rate Health
include.golioth 75% 50%
port.linux 62% 34%
port.utils 58% 46%
port.zephyr 58% 25%
src 70% 31%
Summary 68% (2695 / 3940) 31% (1111 / 3621)

Copy link

codecov bot commented Dec 31, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

see 5 files with indirect coverage changes

@szczys szczys force-pushed the szczys/esp-idf-incorporate-certs-test branch from e2ed63e to 761bccd Compare January 3, 2025 15:52
Copy link
Contributor

@sam-golioth sam-golioth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🥳


- name: Generate certificates
env:
DEVICE_NAME: ${{ needs.rand_name.outputs.device_name }}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clever!

szczys added 3 commits January 7, 2025 09:35
Add random name generation, certificate generation, certificate staging,
and ENV variables. This prepares the workflow for the addition of pytest
for teh certificate_auth sample.

Signed-off-by: Mike Szczys <[email protected]>
Merge all binaries into build/merge.bin at build time. This file is
necessary for automated testing.

Signed-off-by: Mike Szczys <[email protected]>
Add pytest for automated testing of the certificate_auth sample.

Signed-off-by: Mike Szczys <[email protected]>
@szczys szczys force-pushed the szczys/esp-idf-incorporate-certs-test branch from 761bccd to 8a9f3b7 Compare January 7, 2025 15:35
@szczys szczys merged commit 9ef09fc into main Jan 8, 2025
134 of 140 checks passed
@szczys szczys deleted the szczys/esp-idf-incorporate-certs-test branch January 8, 2025 22:39
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.

2 participants