mmyster/feature/lk unit testing framework boost ut #993

MMyster · 2022-09-02T15:54:46Z

✅ (spike): Add lk_boost_ut

Need:

✨ (include): add header file boost::ut for unit testing #989

codecov · 2022-09-02T15:58:14Z

Codecov Report

Merging #993 (24a9b5a) into develop (59051ec) will decrease coverage by 0.00%.
The diff coverage is n/a.

❗ Current head 24a9b5a differs from pull request most recent head 79830ae. Consider uploading reports for the commit 79830ae to get more accurate results

@@             Coverage Diff             @@
##           develop     #993      +/-   ##
===========================================
- Coverage    96.00%   95.99%   -0.01%     
===========================================
  Files          133      133              
  Lines         3178     3173       -5     
===========================================
- Hits          3051     3046       -5     
  Misses         127      127

Impacted Files	Coverage Δ
libs/RobotKit/include/RobotController.h	`93.59% <0.00%> (-0.04%)`	⬇️
libs/RFIDKit/source/RFIDKit.cpp	`100.00% <0.00%> (ø)`
libs/ActivityKit/include/ActivityKit.h	`100.00% <0.00%> (ø)`
libs/ActivityKit/source/ActivityKit.cpp	`100.00% <0.00%> (ø)`
libs/BehaviorKit/source/BehaviorKit.cpp	`100.00% <0.00%> (ø)`

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

github-actions · 2022-09-02T16:09:08Z

File comparision analysis report

🔖 Info

base: develop / 59051ec + mbed-os-6.15.1+fixes + -std=c++20
head: mmyster/feature/lk-unit-testing-framework-boost-ut / 54810ed + mbed-os-6.15.1+fixes + -std=c++20
toolchain: GNU Arm Embedded Toolchain 10.3-2021.10 - 10.3.1 20210824
enable_log_debug: ON

Target	Flash Used (%)	Flash Available (%)	Static RAM (%)
bootloader	182012 (69%)	80132 (30%)	37456 (7%)
os	445940 (28%)	1118732 (71%)	86152 (16%)

Click to show memory sections

| -          |      Hex |     Bytes |  KiB |
|------------|---------:|----------:|-----:|
| Flash      | 0x200000 | 2 097 152 | 2048 |
| SRAM       |  0x80000 |   524 288 |  512 |
| Bootloader |  0x40000 |   262 144 |  256 |
| Header     |   0x1000 |     4 096 |    4 |
| OS         | 0x17E000 | 1 564 672 | 1528 |
| Tail       |   0x1000 |     4 096 |    4 |
| Scratch    |  0x40000 |   262 144 |  256 |

📝 Summary

Click to show summary

✔️ - existing target
✨ - new target
⚰️ - deleted target
✅ - files are the same
❌ - files are different

Target	Status	.bin	.map	Total Flash (base/head)	Total Flash Δ	Static RAM (base/head)	Static RAM Δ
LekaOS	✔️	✅	✅	445940 (21%)	ø	86152 (16%)	ø
bootloader	✔️	✅	✅	182012 (8%)	ø	37456 (7%)	ø
spike_lk_audio	✔️	✅	✅	135068 (6%)	ø	20984 (4%)	ø
spike_lk_behavior_kit	✔️	✅	✅	198252 (9%)	ø	54536 (10%)	ø
spike_lk_ble	✔️	✅	✅	236652 (11%)	ø	34856 (6%)	ø
spike_lk_bluetooth	✔️	✅	✅	92236 (4%)	ø	18064 (3%)	ø
spike_lk_boost_ut	✨	-	-	367784 (17%)	-	24672 (4%)	-
spike_lk_cg_animations	✔️	✅	✅	152264 (7%)	ø	53064 (10%)	ø
spike_lk_color_kit	✔️	✅	✅	88352 (4%)	ø	20216 (3%)	ø
spike_lk_command_kit	✔️	✅	✅	202716 (9%)	ø	57872 (11%)	ø
spike_lk_config_kit	✔️	✅	✅	138868 (6%)	ø	20976 (4%)	ø
spike_lk_coreled	✔️	✅	✅	87916 (4%)	ø	20104 (3%)	ø
spike_lk_event_queue	✔️	✅	✅	84088 (4%)	ø	18744 (3%)	ø
spike_lk_file_manager_kit	✔️	✅	✅	153984 (7%)	ø	21336 (4%)	ø
spike_lk_file_reception	✔️	✅	✅	331180 (15%)	ø	34072 (6%)	ø
spike_lk_flash_memory	✔️	✅	✅	86712 (4%)	ø	18056 (3%)	ø
spike_lk_fs	✔️	✅	✅	173976 (8%)	ø	47976 (9%)	ø
spike_lk_lcd	✔️	✅	✅	171852 (8%)	ø	53288 (10%)	ø
spike_lk_led_kit	✔️	✅	✅	115708 (5%)	ø	21024 (4%)	ø
spike_lk_log_kit	✔️	✅	✅	84440 (4%)	ø	19248 (3%)	ø
spike_lk_motors	✔️	✅	✅	86008 (4%)	ø	18088 (3%)	ø
spike_lk_qdac	✔️	✅	✅	91508 (4%)	ø	18616 (3%)	ø
spike_lk_reinforcer	✔️	✅	✅	112188 (5%)	ø	21024 (4%)	ø
spike_lk_rfid	✔️	✅	✅	84700 (4%)	ø	18024 (3%)	ø
spike_lk_sensors_battery	✔️	✅	✅	87056 (4%)	ø	19120 (3%)	ø
spike_lk_sensors_light	✔️	✅	✅	83944 (4%)	ø	18056 (3%)	ø
spike_lk_sensors_microphone	✔️	✅	✅	84696 (4%)	ø	18056 (3%)	ø
spike_lk_sensors_temperature_humidity	✔️	✅	✅	90336 (4%)	ø	18032 (3%)	ø
spike_lk_sensors_touch	✔️	✅	✅	91608 (4%)	ø	18296 (3%)	ø
spike_lk_serial_number	✔️	✅	✅	136188 (6%)	ø	20896 (3%)	ø
spike_lk_ticker_timeout	✔️	✅	✅	82584 (3%)	ø	18072 (3%)	ø
spike_lk_update_process_app_base	✔️	✅	✅	148044 (7%)	ø	21912 (4%)	ø
spike_lk_update_process_app_update	✔️	✅	✅	100328 (4%)	ø	19080 (3%)	ø
spike_lk_watchdog_isr	✔️	✅	✅	87940 (4%)	ø	19960 (3%)	ø
spike_lk_wifi	✔️	✅	✅	130640 (6%)	ø	21368 (4%)	ø
spike_mbed_blinky	✔️	✅	✅	57616 (2%)	ø	11496 (2%)	ø
spike_mbed_watchdog_ticker_vs_thread	✔️	✅	✅	84112 (4%)	ø	18920 (3%)	ø
spike_stl_cxxsupport	✔️	✅	✅	83424 (3%)	ø	18144 (3%)	ø

🗺️ Map files diff output

Click to show diff list

spike_lk_boost_ut (click to expand)

Flash used: 367784&nbsp;(17%) / total: 2097152
SRAM used: 24672&nbsp;(4%) / total: 524288

github-actions · 2022-09-02T16:09:09Z

File comparision analysis report

🔖 Info

base: develop / 59051ec + mbed-os-6.15.1+fixes + -std=c++20
head: mmyster/feature/lk-unit-testing-framework-boost-ut / 54810ed + mbed-os-6.15.1+fixes + -std=c++20
toolchain: GNU Arm Embedded Toolchain 10.3-2021.10 - 10.3.1 20210824
enable_log_debug: OFF

Target	Flash Used (%)	Flash Available (%)	Static RAM (%)
bootloader	169636 (64%)	92508 (35%)	30920 (5%)
os	407848 (26%)	1156824 (73%)	79224 (15%)

Click to show memory sections

| -          |      Hex |     Bytes |  KiB |
|------------|---------:|----------:|-----:|
| Flash      | 0x200000 | 2 097 152 | 2048 |
| SRAM       |  0x80000 |   524 288 |  512 |
| Bootloader |  0x40000 |   262 144 |  256 |
| Header     |   0x1000 |     4 096 |    4 |
| OS         | 0x17E000 | 1 564 672 | 1528 |
| Tail       |   0x1000 |     4 096 |    4 |
| Scratch    |  0x40000 |   262 144 |  256 |

📝 Summary

Click to show summary

✔️ - existing target
✨ - new target
⚰️ - deleted target
✅ - files are the same
❌ - files are different

Target	Status	.bin	.map	Total Flash (base/head)	Total Flash Δ	Static RAM (base/head)	Static RAM Δ
LekaOS	✔️	✅	✅	407848 (19%)	ø	79224 (15%)	ø
bootloader	✔️	✅	✅	169636 (8%)	ø	30920 (5%)	ø
spike_lk_audio	✔️	✅	✅	125076 (5%)	ø	14568 (2%)	ø
spike_lk_behavior_kit	✔️	✅	✅	189752 (9%)	ø	48120 (9%)	ø
spike_lk_ble	✔️	✅	✅	228768 (10%)	ø	28200 (5%)	ø
spike_lk_bluetooth	✔️	✅	✅	82948 (3%)	ø	11544 (2%)	ø
spike_lk_boost_ut	✨	-	-	355600 (16%)	-	18000 (3%)	-
spike_lk_cg_animations	✔️	✅	✅	144632 (6%)	ø	46528 (8%)	ø
spike_lk_color_kit	✔️	✅	✅	65712 (3%)	ø	13744 (2%)	ø
spike_lk_command_kit	✔️	✅	✅	192520 (9%)	ø	50944 (9%)	ø
spike_lk_config_kit	✔️	✅	✅	126956 (6%)	ø	14312 (2%)	ø
spike_lk_coreled	✔️	✅	✅	76164 (3%)	ø	13688 (2%)	ø
spike_lk_event_queue	✔️	✅	✅	74736 (3%)	ø	12072 (2%)	ø
spike_lk_file_manager_kit	✔️	✅	✅	139016 (6%)	ø	14416 (2%)	ø
spike_lk_file_reception	✔️	✅	✅	327112 (15%)	ø	27576 (5%)	ø
spike_lk_flash_memory	✔️	✅	✅	63880 (3%)	ø	11448 (2%)	ø
spike_lk_fs	✔️	✅	✅	174616 (8%)	ø	47888 (9%)	ø
spike_lk_lcd	✔️	✅	✅	162236 (7%)	ø	46616 (8%)	ø
spike_lk_led_kit	✔️	✅	✅	103992 (4%)	ø	14608 (2%)	ø
spike_lk_log_kit	✔️	✅	✅	63736 (3%)	ø	12256 (2%)	ø
spike_lk_motors	✔️	✅	✅	62600 (2%)	ø	11488 (2%)	ø
spike_lk_qdac	✔️	✅	✅	78564 (3%)	ø	11816 (2%)	ø
spike_lk_reinforcer	✔️	✅	✅	103480 (4%)	ø	14608 (2%)	ø
spike_lk_rfid	✔️	✅	✅	80920 (3%)	ø	11520 (2%)	ø
spike_lk_sensors_battery	✔️	✅	✅	78196 (3%)	ø	12568 (2%)	ø
spike_lk_sensors_light	✔️	✅	✅	60056 (2%)	ø	11440 (2%)	ø
spike_lk_sensors_microphone	✔️	✅	✅	72496 (3%)	ø	11504 (2%)	ø
spike_lk_sensors_temperature_humidity	✔️	✅	✅	67048 (3%)	ø	11424 (2%)	ø
spike_lk_sensors_touch	✔️	✅	✅	68600 (3%)	ø	11432 (2%)	ø
spike_lk_serial_number	✔️	✅	✅	114244 (5%)	ø	14056 (2%)	ø
spike_lk_ticker_timeout	✔️	✅	✅	69052 (3%)	ø	11632 (2%)	ø
spike_lk_update_process_app_base	✔️	✅	✅	126236 (6%)	ø	15304 (2%)	ø
spike_lk_update_process_app_update	✔️	✅	✅	77632 (3%)	ø	12352 (2%)	ø
spike_lk_watchdog_isr	✔️	✅	✅	82032 (3%)	ø	13288 (2%)	ø
spike_lk_wifi	✔️	✅	✅	116392 (5%)	ø	14808 (2%)	ø
spike_mbed_blinky	✔️	✅	✅	57968 (2%)	ø	11400 (2%)	ø
spike_mbed_watchdog_ticker_vs_thread	✔️	✅	✅	63208 (3%)	ø	12448 (2%)	ø
spike_stl_cxxsupport	✔️	✅	✅	58456 (2%)	ø	11400 (2%)	ø

🗺️ Map files diff output

Click to show diff list

spike_lk_boost_ut (click to expand)

Flash used: 355600&nbsp;(16%) / total: 2097152
SRAM used: 18000&nbsp;(3%) / total: 524288

ladislas

1ère review 👍

du coup, tu n'as pas modifié boost::ut?

spikes/lk_boost_ut/main.cpp

YannLocatelli

Hard to understand but seems powerful to use 👌

spikes/lk_boost_ut/tests/test_expect.cpp

spikes/lk_boost_ut/tests/test_source_location.cpp

spikes/lk_boost_ut/tests/test_tag.cpp

spikes/lk_boost_ut/tests/test_parametrized.cpp

Test boost::ut in the context of functional tests.

- separate tests in different files - create small main.cpp that can be used anywhere - update CMakeLists.txt file - add config.h - use MACROS for start/end statistics

sonarcloud · 2022-10-11T18:37:37Z

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

MMyster force-pushed the mmyster/feature/lk-unit-testing-framework-boost-ut branch 5 times, most recently from a847366 to be75e1f Compare September 5, 2022 16:56

MMyster changed the base branch from develop to mmyster/feature/add-unit-testing-framework-boost-ut September 5, 2022 16:57

MMyster force-pushed the mmyster/feature/lk-unit-testing-framework-boost-ut branch from be75e1f to fd57f64 Compare September 5, 2022 16:59

ladislas requested changes Sep 5, 2022

View reviewed changes

spikes/lk_boost_ut/main.cpp Outdated Show resolved Hide resolved

spikes/lk_boost_ut/main.cpp Outdated Show resolved Hide resolved

spikes/lk_boost_ut/main.cpp Outdated Show resolved Hide resolved

spikes/lk_boost_ut/main.cpp Outdated Show resolved Hide resolved

ladislas mentioned this pull request Sep 7, 2022

Embedding ut - Using a printer when iostream/std::cout is not available boost-ext/ut#527

Closed

ladislas force-pushed the mmyster/feature/add-unit-testing-framework-boost-ut branch from 534be36 to 400749f Compare September 7, 2022 08:12

Base automatically changed from mmyster/feature/add-unit-testing-framework-boost-ut to develop September 7, 2022 08:19

MMyster force-pushed the mmyster/feature/lk-unit-testing-framework-boost-ut branch 3 times, most recently from f1024f0 to b87cd44 Compare September 7, 2022 10:53

ladislas assigned MMyster Sep 7, 2022

ladislas added the 01 - type: task Something to do label Sep 7, 2022

ladislas added this to the Next Release milestone Sep 7, 2022

MMyster force-pushed the mmyster/feature/lk-unit-testing-framework-boost-ut branch 2 times, most recently from e362525 to 6a5f390 Compare September 12, 2022 07:33

MMyster mentioned this pull request Sep 12, 2022

mmyster/feature/check ut status #1016

Closed

1 task

MMyster force-pushed the mmyster/feature/lk-unit-testing-framework-boost-ut branch 2 times, most recently from fef74dd to d6ac5eb Compare September 19, 2022 07:36

ladislas modified the milestones: Next Release, Future releases Sep 20, 2022

MMyster force-pushed the mmyster/feature/lk-unit-testing-framework-boost-ut branch 2 times, most recently from 85fa678 to 0fef3e2 Compare September 23, 2022 10:15

ladislas force-pushed the mmyster/feature/lk-unit-testing-framework-boost-ut branch from 0fef3e2 to c8b9b83 Compare September 26, 2022 12:20

ladislas force-pushed the mmyster/feature/lk-unit-testing-framework-boost-ut branch 4 times, most recently from 6b4d646 to 0ca2806 Compare September 30, 2022 07:02

ladislas requested review from HPezz and YannLocatelli September 30, 2022 10:36

YannLocatelli reviewed Oct 4, 2022

View reviewed changes

MMyster force-pushed the mmyster/feature/lk-unit-testing-framework-boost-ut branch from 0ca2806 to 542354b Compare October 6, 2022 09:23

ladislas force-pushed the mmyster/feature/lk-unit-testing-framework-boost-ut branch from 542354b to fe75819 Compare October 10, 2022 15:42

⚗️ (spikes): Add lk_boost_ut spike

b46b426

Test boost::ut in the context of functional tests.

ladislas force-pushed the mmyster/feature/lk-unit-testing-framework-boost-ut branch from 24a9b5a to 54810ed Compare October 11, 2022 15:52

♻️ (spikes): boost::ut - refactor tests in separated .cpp files

79830ae

- separate tests in different files - create small main.cpp that can be used anywhere - update CMakeLists.txt file - add config.h - use MACROS for start/end statistics

ladislas force-pushed the mmyster/feature/lk-unit-testing-framework-boost-ut branch from 54810ed to 79830ae Compare October 11, 2022 18:29

ladislas merged commit aa98428 into develop Oct 11, 2022

ladislas deleted the mmyster/feature/lk-unit-testing-framework-boost-ut branch October 11, 2022 18:29

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

mmyster/feature/lk unit testing framework boost ut #993

mmyster/feature/lk unit testing framework boost ut #993

MMyster commented Sep 2, 2022 •

edited

Loading

codecov bot commented Sep 2, 2022 •

edited

Loading

github-actions bot commented Sep 2, 2022 •

edited

Loading

github-actions bot commented Sep 2, 2022 •

edited

Loading

ladislas left a comment

YannLocatelli left a comment

sonarcloud bot commented Oct 11, 2022

mmyster/feature/lk unit testing framework boost ut #993

mmyster/feature/lk unit testing framework boost ut #993

Conversation

MMyster commented Sep 2, 2022 • edited Loading

codecov bot commented Sep 2, 2022 • edited Loading

Codecov Report

github-actions bot commented Sep 2, 2022 • edited Loading

File comparision analysis report

🔖 Info

📝 Summary

🗺️ Map files diff output

github-actions bot commented Sep 2, 2022 • edited Loading

File comparision analysis report

🔖 Info

📝 Summary

🗺️ Map files diff output

ladislas left a comment

Choose a reason for hiding this comment

YannLocatelli left a comment

Choose a reason for hiding this comment

sonarcloud bot commented Oct 11, 2022

MMyster commented Sep 2, 2022 •

edited

Loading

codecov bot commented Sep 2, 2022 •

edited

Loading

github-actions bot commented Sep 2, 2022 •

edited

Loading

github-actions bot commented Sep 2, 2022 •

edited

Loading