-
Notifications
You must be signed in to change notification settings - Fork 2.1k
tests/unittests: Move large unittests to individual tests #10187
Copy link
Copy link
Open
Labels
Area: testsArea: tests and testing frameworkArea: tests and testing frameworkType: cleanupThe issue proposes a clean-up / The PR cleans-up parts of the codebase / documentationThe issue proposes a clean-up / The PR cleans-up parts of the codebase / documentationType: trackingThe issue tracks and organizes the sub-tasks of a larger effortThe issue tracks and organizes the sub-tasks of a larger effort
Description
I propose breaking out a number of large tests from the unittests to their own test. This to decrease the flash size of the full unittest binary a bit.
Tests:
Flash sizes as mentioned below are based on the size of the required flash of the package itself, or when it is not a package, the flash required for the test itself. All flash sizes are measured for the nrf52840dk board.
- tweetnacl (9KB) tests/tweetnacl: Move from unittests to regular test #10184
- qDSA (13KB) tests/qdsa: Move from unittests to regular tests #10185
- libcose tests/libcose: Move from unittests to regular test #10183
- hacl (10KB) tests/hacl: Move from unittests to regular test #10209
- spiffs (16KB) tests/spiffs: Move from unittests to regular test #10210
- gnrc_ipv6_nib (38KB)
- cn-cbor (1.9KB) tests/cn-cbor: move from unittests to regular test #10188
- relic tests/relic: move from unittests to regular test #10189
- heatshrink tests/heatshrink: move from unittests to regular test #10202
- littlefs tests/littlefs: Move from unittests to regular test #10212
Optionally:
- hashes (19KB)
- Bloom (15KB)
- tests-cpp_ctors to remove
DISABLE_tests variables.
Final task
- Remove
LARGE_STACK_TESTSfromunitteststests/unittests: remove unused LARGE_STACK_TESTS #11601 - Update
BOARD_INSUFFICIENT_MEMORY- Updated after all packages moved outside tests/unittests: enable boards with enough memory #11602
Packages that would still remain if all these tests are moved outside are cn-cbor at almost 2KB and heatshrink at 1.5KB removed
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
Area: testsArea: tests and testing frameworkArea: tests and testing frameworkType: cleanupThe issue proposes a clean-up / The PR cleans-up parts of the codebase / documentationThe issue proposes a clean-up / The PR cleans-up parts of the codebase / documentationType: trackingThe issue tracks and organizes the sub-tasks of a larger effortThe issue tracks and organizes the sub-tasks of a larger effort