tests: drivers: rtc: multiple minor issues in rtc_emul mock/fake device #59901
Labels
area: RTC
Real Time Clock
bug
The issue is a bug, or the PR is fixing a bug
priority: low
Low impact/importance bug
Describe the bug
The RTC tests and test mock (emulation) device have multiple minor issues:
The RTC tests that run in the CI/CD platform do not cover the alarm, update and calibration parts of the API by default although such tests exist in the tests tree.The RTC test suite does not build on 64 bit architectures.The RTC emulation device is enabled by default on native_posix for all tests, although it's only required for a single driver.The RTC emulation device is meant for testing purposes only but resides in the main driver tree which clutters devicetree and docs, may confuse users concerning its purpose (it confused me quite a bit), breaks encapsulation and creates redundant code. As customary, mock/fake drivers should be placed in the test tree.UPDATE: Removed items due to review comments on the linked PR, see there. Won't fix.
To Reproduce
./scripts/twister -T tests/drivers/rtc -c --integration
Expected behavior
Impact
minor impact:
missing test coverage - potential for regressionssome avoidable maintenance effort, unnecessary complexitypotential confusion of users re purpose of zephyr,rtc-emul(true, but no longer part of this issue, see update above)a few extra seconds of execution on the CI/CD pipeline plus unnecessarily long "skipped" reportskernel resources, interrupts,mutex)Logs and console output
n/a
Environment (please complete the following information):
dev env, CI/CD
Additional context
See #19030 (comment)
The text was updated successfully, but these errors were encountered: