DTT1. Change the call to os.getlogin function by getpass.getuser in testing module

[mhamra]

Target version	Related issue	Related PR/dev branch
4.9.0	#5193

Description

While executing a workflow file, the os.getlogin() function generated an exception. See this comment.

The os.getlogin function returns the user's name logged in on the process's controlling terminal. Typically, processes in the user's session (tty, X session) have a controlling terminal. Processes spawned by the workflow do not have a controlling terminal. The recommended way to obtain the current user is using the getpass.getuser function instead. It is a more reliable method.

[rauldpm]

Mmmm, looking at the command used in the comment, I do not have any error using it; it may be your Python/GLIB permission.

In any case, @fcaffieri let's set the issue as a type/change and work on it in DTT2, what do you think?

https://stackoverflow.com/questions/74187896/os-getlogin-in-docker-container-throws-filenotfounderror

[mhamra]

The Python reference for os.getlogin recommends using the getpass.getuser function (see os.getlogin reference).

I recommend this change because some users may get the exception, blocking the test execution.

[rauldpm]

There is only this reference to getlogin, so let's fix it, as you have the fix, create a PR to the #5190 branch and add a test case

╰─➤  grep -R "os.getlogin"
modules/testing/testing.py:        extra_vars['current_user'] = os.getlogin()

[rauldpm]

Approved by @pro-akim and @rauldpm