CI Add unit test workflow #388

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

Jump to bottom

Merged

pennam merged 8 commits into arduino-libraries:master from pennam:ci-unit-test

Apr 9, 2025

Merged

CI Add unit test workflow #388

pennam merged 8 commits into arduino-libraries:master from pennam:ci-unit-test

Apr 9, 2025

Conversation

pennam

Copy link

Contributor

@pennam pennam commented Apr 8, 2025

No description provided.

@codecov Codecov

Copy link

codecov bot commented Apr 9, 2025 •

edited

Loading

Codecov Report

Attention: Patch coverage is 50.00000% with 5 lines in your changes missing coverage. Please review.

Please upload report for BASE (master@7a039d7). Learn more about missing BASE report.

Files with missing lines	Patch %	Lines
src/utility/GATT.cpp	50.00%	5 Missing ⚠️

Additional details and impacted files

@@ Coverage Diff @@
## master #388 +/- ##
========================================
 Coverage ? 9.62% 
========================================
 Files ? 28 
 Lines ? 3656 
 Branches ? 0 
========================================
 Hits ? 352 
 Misses ? 3304 
 Partials ? 0

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions GitHub Actions

Copy link

github-actions bot commented Apr 9, 2025

Memory usage change @ 0f02ff3

Board	flash	%	RAM for global variables	%
`arduino:mbed_nano:nano33ble`	💚 -584 - -448	-0.06 - -0.05	0 - 0	0.0 - 0.0
`arduino:mbed_nano:nanorp2040connect`	💚 -559 - -477	-0.0 - -0.0	0 - 0	0.0 - 0.0
`arduino:megaavr:uno2018:mode=on`	💚 -679 - -577	-1.4 - -1.19	0 - 0	0.0 - 0.0
`arduino:renesas_uno:unor4wifi`	💚 -496 - -480	-0.19 - -0.18	0 - 0	0.0 - 0.0
`arduino:samd:mkrwifi1010`	💚 -576 - -480	-0.22 - -0.18	0 - 0	0.0 - 0.0
`arduino:samd:nano_33_iot`	💚 -576 - -480	-0.22 - -0.18	0 - 0	0.0 - 0.0

Click for full report table

Board	`examples/Central/LedControl` flash	%	`examples/Central/PeripheralExplorer` flash	%	`examples/Central/Scan` flash	%	`examples/Central/ScanCallback` flash	%	`examples/Central/SensorTagButton` flash	%	`examples/Peripheral/Advertising/EnhancedAdvertising` flash	%	`examples/Peripheral/Advertising/RawDataAdvertising` flash	%	`examples/Peripheral/BatteryMonitor` flash	%	`examples/Peripheral/ButtonLED` flash	%	`examples/Peripheral/CallbackLED` flash	%	`examples/Peripheral/EncryptedBatteryMonitor` flash	%	`examples/Peripheral/LED` flash	%
`arduino:mbed_nano:nano33ble`	-584	-0.06	-520	-0.05	-512	-0.05	-448	-0.05	-576	-0.06	-584	-0.06	-576	-0.06	-512	-0.05	-584	-0.06	-576	-0.06	-584	-0.06	-576	-0.06
`arduino:mbed_nano:nanorp2040connect`	-559	-0.0	-477	-0.0	-477	-0.0	-477	-0.0	-559	-0.0	-559	-0.0	-559	-0.0	-477	-0.0	-559	-0.0	-559	-0.0	-557	-0.0	-559	-0.0
`arduino:megaavr:uno2018:mode=on`	-599	-1.23	-577	-1.19	-637	-1.31	-587	-1.21	-599	-1.23	-599	-1.23	-599	-1.23	-649	-1.33	-599	-1.23	-599	-1.23	-679	-1.4	-599	-1.23
`arduino:renesas_uno:unor4wifi`	-488	-0.19	-488	-0.19	-496	-0.19	-480	-0.18	-480	-0.18	-488	-0.19	-480	-0.18	-488	-0.19	-488	-0.19	-480	-0.18	-480	-0.18	-480	-0.18
`arduino:samd:mkrwifi1010`	-568	-0.22	-480	-0.18	-480	-0.18	-480	-0.18	-568	-0.22	-568	-0.22	-568	-0.22	-480	-0.18	-568	-0.22	-568	-0.22	-576	-0.22	-568	-0.22
`arduino:samd:nano_33_iot`	-568	-0.22	-480	-0.18	-480	-0.18	-480	-0.18	-568	-0.22	-568	-0.22	-568	-0.22	-480	-0.18	-568	-0.22	-568	-0.22	-576	-0.22	-568	-0.22

Click for full report CSV

Board,examples/Central/LedControl<br>flash,%,examples/Central/LedControl<br>RAM for global variables,%,examples/Central/PeripheralExplorer<br>flash,%,examples/Central/PeripheralExplorer<br>RAM for global variables,%,examples/Central/Scan<br>flash,%,examples/Central/Scan<br>RAM for global variables,%,examples/Central/ScanCallback<br>flash,%,examples/Central/ScanCallback<br>RAM for global variables,%,examples/Central/SensorTagButton<br>flash,%,examples/Central/SensorTagButton<br>RAM for global variables,%,examples/Peripheral/Advertising/EnhancedAdvertising<br>flash,%,examples/Peripheral/Advertising/EnhancedAdvertising<br>RAM for global variables,%,examples/Peripheral/Advertising/RawDataAdvertising<br>flash,%,examples/Peripheral/Advertising/RawDataAdvertising<br>RAM for global variables,%,examples/Peripheral/BatteryMonitor<br>flash,%,examples/Peripheral/BatteryMonitor<br>RAM for global variables,%,examples/Peripheral/ButtonLED<br>flash,%,examples/Peripheral/ButtonLED<br>RAM for global variables,%,examples/Peripheral/CallbackLED<br>flash,%,examples/Peripheral/CallbackLED<br>RAM for global variables,%,examples/Peripheral/EncryptedBatteryMonitor<br>flash,%,examples/Peripheral/EncryptedBatteryMonitor<br>RAM for global variables,%,examples/Peripheral/LED<br>flash,%,examples/Peripheral/LED<br>RAM for global variables,%
arduino:mbed_nano:nano33ble,-584,-0.06,0,0.0,-520,-0.05,0,0.0,-512,-0.05,0,0.0,-448,-0.05,0,0.0,-576,-0.06,0,0.0,-584,-0.06,0,0.0,-576,-0.06,0,0.0,-512,-0.05,0,0.0,-584,-0.06,0,0.0,-576,-0.06,0,0.0,-584,-0.06,0,0.0,-576,-0.06,0,0.0
arduino:mbed_nano:nanorp2040connect,-559,-0.0,0,0.0,-477,-0.0,0,0.0,-477,-0.0,0,0.0,-477,-0.0,0,0.0,-559,-0.0,0,0.0,-559,-0.0,0,0.0,-559,-0.0,0,0.0,-477,-0.0,0,0.0,-559,-0.0,0,0.0,-559,-0.0,0,0.0,-557,-0.0,0,0.0,-559,-0.0,0,0.0
arduino:megaavr:uno2018:mode=on,-599,-1.23,0,0.0,-577,-1.19,0,0.0,-637,-1.31,0,0.0,-587,-1.21,0,0.0,-599,-1.23,0,0.0,-599,-1.23,0,0.0,-599,-1.23,0,0.0,-649,-1.33,0,0.0,-599,-1.23,0,0.0,-599,-1.23,0,0.0,-679,-1.4,0,0.0,-599,-1.23,0,0.0
arduino:renesas_uno:unor4wifi,-488,-0.19,0,0.0,-488,-0.19,0,0.0,-496,-0.19,0,0.0,-480,-0.18,0,0.0,-480,-0.18,0,0.0,-488,-0.19,0,0.0,-480,-0.18,0,0.0,-488,-0.19,0,0.0,-488,-0.19,0,0.0,-480,-0.18,0,0.0,-480,-0.18,0,0.0,-480,-0.18,0,0.0
arduino:samd:mkrwifi1010,-568,-0.22,0,0.0,-480,-0.18,0,0.0,-480,-0.18,0,0.0,-480,-0.18,0,0.0,-568,-0.22,0,0.0,-568,-0.22,0,0.0,-568,-0.22,0,0.0,-480,-0.18,0,0.0,-568,-0.22,0,0.0,-568,-0.22,0,0.0,-576,-0.22,0,0.0,-568,-0.22,0,0.0
arduino:samd:nano_33_iot,-568,-0.22,0,0.0,-480,-0.18,0,0.0,-480,-0.18,0,0.0,-480,-0.18,0,0.0,-568,-0.22,0,0.0,-568,-0.22,0,0.0,-568,-0.22,0,0.0,-480,-0.18,0,0.0,-568,-0.22,0,0.0,-568,-0.22,0,0.0,-576,-0.22,0,0.0,-568,-0.22,0,0.0

@pennam pennam requested review from facchinm and andreagilardoni

April 9, 2025 07:58

@pennam pennam force-pushed the ci-unit-test branch from 7ddd0d2 to d627d14 Compare

April 9, 2025 08:41

polldo and others added 8 commits

April 9, 2025 10:50

@polldo @pennam


 [WIP] Add unit test workflow

e59d4ad

based on arduino/cpp-test-action#2

@polldo @pennam


 [CI] Fix uninitialized checked variable

1340d09

@polldo @pennam


 [CI] Improve unit tests

8035af8

- improve test failures readability
- add const qualifiers to golden data for improving safety

@pennam


 [CI] use codecov token

6fa0191

@pennam


 [CI] modernize cmake build

c5bed55

@pennam


 [CI] import c++11 F replacement

a1f8e3e

@pennam


 Wrap prints with #ifdef _BLE_TRACE_

e389eaf

@pennam


 GATTClass::end(): fix possible leak

ee05fe6

@pennam pennam force-pushed the ci-unit-test branch from d627d14 to ee05fe6 Compare

April 9, 2025 08:51

@github-actions GitHub Actions

Copy link

github-actions bot commented Apr 9, 2025

Memory usage change @ ee05fe6

Board	flash	%	RAM for global variables	%
`arduino:mbed_nano:nano33ble`	💚 -584 - -448	-0.06 - -0.05	0 - 0	0.0 - 0.0
`arduino:mbed_nano:nanorp2040connect`	💚 -559 - -477	-0.0 - -0.0	0 - 0	0.0 - 0.0
`arduino:megaavr:uno2018:mode=on`	💚 -679 - -577	-1.4 - -1.19	0 - 0	0.0 - 0.0
`arduino:renesas_uno:unor4wifi`	💚 -496 - -480	-0.19 - -0.18	0 - 0	0.0 - 0.0
`arduino:samd:mkrwifi1010`	💚 -576 - -480	-0.22 - -0.18	0 - 0	0.0 - 0.0
`arduino:samd:nano_33_iot`	💚 -576 - -480	-0.22 - -0.18	0 - 0	0.0 - 0.0

Click for full report table

Board	`examples/Central/LedControl` flash	%	`examples/Central/PeripheralExplorer` flash	%	`examples/Central/Scan` flash	%	`examples/Central/ScanCallback` flash	%	`examples/Central/SensorTagButton` flash	%	`examples/Peripheral/Advertising/EnhancedAdvertising` flash	%	`examples/Peripheral/Advertising/RawDataAdvertising` flash	%	`examples/Peripheral/BatteryMonitor` flash	%	`examples/Peripheral/ButtonLED` flash	%	`examples/Peripheral/CallbackLED` flash	%	`examples/Peripheral/EncryptedBatteryMonitor` flash	%	`examples/Peripheral/LED` flash	%
`arduino:mbed_nano:nano33ble`	-584	-0.06	-520	-0.05	-512	-0.05	-448	-0.05	-576	-0.06	-584	-0.06	-576	-0.06	-512	-0.05	-584	-0.06	-576	-0.06	-584	-0.06	-576	-0.06
`arduino:mbed_nano:nanorp2040connect`	-559	-0.0	-477	-0.0	-477	-0.0	-477	-0.0	-559	-0.0	-559	-0.0	-559	-0.0	-477	-0.0	-559	-0.0	-559	-0.0	-557	-0.0	-559	-0.0
`arduino:megaavr:uno2018:mode=on`	-599	-1.23	-577	-1.19	-637	-1.31	-587	-1.21	-599	-1.23	-599	-1.23	-599	-1.23	-649	-1.33	-599	-1.23	-599	-1.23	-679	-1.4	-599	-1.23
`arduino:renesas_uno:unor4wifi`	-488	-0.19	-488	-0.19	-496	-0.19	-480	-0.18	-480	-0.18	-488	-0.19	-480	-0.18	-488	-0.19	-488	-0.19	-480	-0.18	-480	-0.18	-480	-0.18
`arduino:samd:mkrwifi1010`	-568	-0.22	-480	-0.18	-480	-0.18	-480	-0.18	-568	-0.22	-568	-0.22	-568	-0.22	-480	-0.18	-568	-0.22	-568	-0.22	-576	-0.22	-568	-0.22
`arduino:samd:nano_33_iot`	-568	-0.22	-480	-0.18	-480	-0.18	-480	-0.18	-568	-0.22	-568	-0.22	-568	-0.22	-480	-0.18	-568	-0.22	-568	-0.22	-576	-0.22	-568	-0.22

Click for full report CSV

Board,examples/Central/LedControl<br>flash,%,examples/Central/LedControl<br>RAM for global variables,%,examples/Central/PeripheralExplorer<br>flash,%,examples/Central/PeripheralExplorer<br>RAM for global variables,%,examples/Central/Scan<br>flash,%,examples/Central/Scan<br>RAM for global variables,%,examples/Central/ScanCallback<br>flash,%,examples/Central/ScanCallback<br>RAM for global variables,%,examples/Central/SensorTagButton<br>flash,%,examples/Central/SensorTagButton<br>RAM for global variables,%,examples/Peripheral/Advertising/EnhancedAdvertising<br>flash,%,examples/Peripheral/Advertising/EnhancedAdvertising<br>RAM for global variables,%,examples/Peripheral/Advertising/RawDataAdvertising<br>flash,%,examples/Peripheral/Advertising/RawDataAdvertising<br>RAM for global variables,%,examples/Peripheral/BatteryMonitor<br>flash,%,examples/Peripheral/BatteryMonitor<br>RAM for global variables,%,examples/Peripheral/ButtonLED<br>flash,%,examples/Peripheral/ButtonLED<br>RAM for global variables,%,examples/Peripheral/CallbackLED<br>flash,%,examples/Peripheral/CallbackLED<br>RAM for global variables,%,examples/Peripheral/EncryptedBatteryMonitor<br>flash,%,examples/Peripheral/EncryptedBatteryMonitor<br>RAM for global variables,%,examples/Peripheral/LED<br>flash,%,examples/Peripheral/LED<br>RAM for global variables,%
arduino:mbed_nano:nano33ble,-584,-0.06,0,0.0,-520,-0.05,0,0.0,-512,-0.05,0,0.0,-448,-0.05,0,0.0,-576,-0.06,0,0.0,-584,-0.06,0,0.0,-576,-0.06,0,0.0,-512,-0.05,0,0.0,-584,-0.06,0,0.0,-576,-0.06,0,0.0,-584,-0.06,0,0.0,-576,-0.06,0,0.0
arduino:mbed_nano:nanorp2040connect,-559,-0.0,0,0.0,-477,-0.0,0,0.0,-477,-0.0,0,0.0,-477,-0.0,0,0.0,-559,-0.0,0,0.0,-559,-0.0,0,0.0,-559,-0.0,0,0.0,-477,-0.0,0,0.0,-559,-0.0,0,0.0,-559,-0.0,0,0.0,-557,-0.0,0,0.0,-559,-0.0,0,0.0
arduino:megaavr:uno2018:mode=on,-599,-1.23,0,0.0,-577,-1.19,0,0.0,-637,-1.31,0,0.0,-587,-1.21,0,0.0,-599,-1.23,0,0.0,-599,-1.23,0,0.0,-599,-1.23,0,0.0,-649,-1.33,0,0.0,-599,-1.23,0,0.0,-599,-1.23,0,0.0,-679,-1.4,0,0.0,-599,-1.23,0,0.0
arduino:renesas_uno:unor4wifi,-488,-0.19,0,0.0,-488,-0.19,0,0.0,-496,-0.19,0,0.0,-480,-0.18,0,0.0,-480,-0.18,0,0.0,-488,-0.19,0,0.0,-480,-0.18,0,0.0,-488,-0.19,0,0.0,-488,-0.19,0,0.0,-480,-0.18,0,0.0,-480,-0.18,0,0.0,-480,-0.18,0,0.0
arduino:samd:mkrwifi1010,-568,-0.22,0,0.0,-480,-0.18,0,0.0,-480,-0.18,0,0.0,-480,-0.18,0,0.0,-568,-0.22,0,0.0,-568,-0.22,0,0.0,-568,-0.22,0,0.0,-480,-0.18,0,0.0,-568,-0.22,0,0.0,-568,-0.22,0,0.0,-576,-0.22,0,0.0,-568,-0.22,0,0.0
arduino:samd:nano_33_iot,-568,-0.22,0,0.0,-480,-0.18,0,0.0,-480,-0.18,0,0.0,-480,-0.18,0,0.0,-568,-0.22,0,0.0,-568,-0.22,0,0.0,-568,-0.22,0,0.0,-480,-0.18,0,0.0,-568,-0.22,0,0.0,-568,-0.22,0,0.0,-576,-0.22,0,0.0,-568,-0.22,0,0.0