twister: setup logging per process, fix logging on macos

[nashif]

not sure why on MacOS it behaves differently, but this seems to fix the issue

Fixes #86237

[nashif]

looks like the fix works, see CI:

[nashif]

@LukaszMrugala can you help with the pytest mocking here, not sure how to resolve this.

[fabiobaltieri]

Seems like the Windows build had the same issue and also work now.

[nashif]

Seems like the Windows build had the same issue and also work now.

oh yeah, that is good. suprised nobody noticed this before ;-(

[PerMac]

lgtm, verified that it improvers logging on windows

[LukaszMrugala]

It fails Unit tests because in the test_projectbuilder_process test there is an options Mock. Since new options are used after this change in the runner, those options default to mock.MagicMock(), which breaks the test.

Adding this piece of code to the test (L1533) fixes the problem (note that the tmp_path must be also added as the test parameter):

    pb.options.outdir = tmp_path
    pb.options.log_file = None
    pb.options.log_level = "DEBUG"

[nashif]

It fails Unit tests because in the test_projectbuilder_process test there is an options Mock. Since new options are used after this change in the runner, those options default to mock.MagicMock(), which breaks the test.

Adding this piece of code to the test (L1533) fixes the problem (note that the tmp_path must be also added as the test parameter):

    pb.options.outdir = tmp_path
    pb.options.log_file = None
    pb.options.log_level = "DEBUG"

does not work, can you please post a patch here

[fabiobaltieri]

not sure why on MacOS it behaves differently

https://en.wikipedia.org/wiki/Think_different

[nashif]

rebased