This commit adds a copy constructor that takes a non-const argument.
This is required because otherwise the templated constructor would take
precedence over the copy constructor when supplied with a non-const argument.

Currently the templated constructor is functionally the same as the new copy
constructor. However, adding the new copy constructor avoids ambiguity, and
means we can add some safety by casting the argument to const before passing
it to the base class constructor.

Fixes cppcheck warning. The overload is required to avoid the
possibility of an incorrect raw pointer copy.

re #18885

re #18844

re #18444

Refs #17513

For cases where file logging is enabled then the framework simply uses the
specified path as is or defaults to the appdata directory on all
platforms. This has the advantage of moving it outside if the install
directory.

data search directories from test, that are restored to initial settings after test

re #14524

This should allow the logger.is functionality to work correctly.

re #14524

re #15286

re #15286