Fixed test issue with overflow of times #574

gaisser · 2022-03-07T18:41:50+01:00

gaisser commented

2022-03-07 18:41:50 +01:00

Tests of LocalDatapoolManager had an issue with overflowing times. This is solved by using timevalOperations.

Added conversion function from CCSDSTime::CDS_short to timeval
Added unit test for that function
Changed behavior of Clock::convertTimeOfDayToTimeval for Linux and Host (only unix implementation anyway). It now assumes that the input is UTC and not local time.

Tests of LocalDatapoolManager had an issue with overflowing times. This is solved by using timevalOperations. * Added conversion function from CCSDSTime::CDS_short to timeval * Added unit test for that function * **Changed behavior of Clock::convertTimeOfDayToTimeval** for Linux and Host (only unix implementation anyway). It now assumes that the input is UTC and not local time.

gaisser added this to the v5.0.0 milestone 2022-03-07 18:41:50 +01:00

gaisser added the

bug

feature

API Change

labels 2022-03-07 18:41:50 +01:00

gaisser added 2 commits 2022-03-07 18:41:54 +01:00

8b1af232c3 Added Tests for CCSDS Time, fixed LPM Test

Changed behaviour of Host and Linux Clock

fsfw/fsfw/pipeline/pr-development This commit looks good Details

a4f97a7ba7 Fixed another issue of time checks

gaisser requested review from mohr 2022-03-09 12:55:55 +01:00

mohr requested changes 2022-03-09 17:48:10 +01:00

tests/src/fsfw_tests/unit/timemanager/TestCCSDSTime.cpp Outdated

						
				@ -92,0 +111,4 @@

				    CCSDSTime::CDS_short cdsTime;

				    result = CCSDSTime::convertToCcsds(&cdsTime, &timeAsTimeval);

				    CHECK(result == HasReturnvaluesIF::RETURN_OK);