The TEST_CASE()s and other magic from Catch2 made it harder to debug.

This requires addinge the members path_ and openFlags_.

When using designated initializes to initialize a struct, it is
perfectly valid not to specify all fields. The unspecified fields are
initialized with default member initializers where provided, and empty
list-initialization otherwise.

The normal lfs_file_open() uses malloc() to allocate the file buffer.
Since we do not want to use the heap, we use lfs_file_opencfg() and
provide a statically allocated buffer instead.

Flag varibales are one of the few cases where unsigned types should be
used because they do not represent a real quantity/number but just a
bunch of states/bits.

This reverts commit 1d0bd2f.

In this special case the Swap() function is an abomination, and the
previous version with the Move() function is better.

- Rename `Move()` to `MoveConstructFrom()`
- Make the class const-correct
- Remove `std::cout` and `iostream`
- Make cosmetic changes

They were used to test stuff during development but are now obsolete.

The longest path should be "/programs/65536.zip.lock" which is only 25
characters long, so we can save quite a bit of memory when we reduce the
max path length from the default 256 to just 30.

- Always pass paths as `Path` aka `etl::string` and use `c_str()` to
  ensure that the littlefs functions get null-terminated strings
- Move member variables which are only used in `CreateLockFile()` to
  that function

Eventually, the `File` class will need semaphores and therefore require
the main function from Rodos. I converted it now, because that allows me
to get rid of this `USE_NO_RODOS` nonsense in `Outcome.hpp` too.