Remove local_time from ActiveTextLogger. Fixes: #2815

[AlesKus]

Related Issue(s)	#2815
Has Unit Tests (y/n)	y
Documentation Included (y/n)	y

---

Change Description

Removed invocation of the localtime_r.

Fixed unit tests:

• included them into build, after they were removed #f797734
• fixed the LogFile behavior, which, I assume, has been broken here: #58c2141 ; the intention of include_excl flag was to include the O_EXCL option for opening (see source code); therefore include_excl = false means OverwriteType::OVERWRITE. I've mentioned this behavior in the documentation.

Rationale

Fixes #2815.

Testing/Review Recommendations

Please review the documentation changes as well.
How to test: run fprime-util check in Svc/ActiveTextLogger.

Future Work

1. The component has several TODOs:

• https://github.com/nasa/fprime/blob/devel/Svc/ActiveTextLogger/ActiveTextLogger.cpp#L76
• https://github.com/nasa/fprime/blob/devel/Svc/ActiveTextLogger/test/ut/ActiveTextLoggerTester.cpp#L165
• https://github.com/nasa/fprime/blob/devel/Svc/ActiveTextLogger/docs/sdd.md?plain=1#L30
• https://github.com/nasa/fprime/blob/devel/Svc/ActiveTextLogger/docs/sdd.md?plain=1#L52
  Probably, they worth fixing.

2. The test functions might be split to small test cases.

[LeStarch]

Looks very good! Only two minor touch-ups.

[LeStarch]

I went ahead and made the changes. Should be good once it passes CI!

[LeStarch]

This file is using multiple styles to compare strings, so let us leave as is.