[Enabler] [conftest.py] Hot fix for testing the collection in Ansible 2.17.1

[rexemin]

SUMMARY

Fixes #1670.

ISSUE TYPE

• Enabler Pull Request

COMPONENT NAME

• tests/conftest.py

ADDITIONAL INFORMATION

Instead of passing the environment variables through the interpreter string, now we decorate the Ansible engine function that computes the environment string.

Some changes were needed to fix a couple of tests. Most changes were related to escaping shell commands and were failures even when running on dev with Ansible 2.16. Changes directly related to this PR are the ones made to tests for zos_copy and zos_fetch, as well as the users.py helper.

Changes to the config

This changes how the config should look like in Jenkins. When testing on Ansible >=2.17.1, we need to remove the PYTHONSTDINENCODING variable and change the value of _CEE_RUNOPTS to "FILETAG(AUTOCVT,AUTOTAG) POSIX(ON)" (notice how we will not use an extra pair of single quotes now).

The new config for Jenkins should be like this:

_BPXK_AUTOCVT: "ON"
_CEE_RUNOPTS: "FILETAG(AUTOCVT,AUTOTAG) POSIX(ON)"
_TAG_REDIR_ERR: txt
_TAG_REDIR_IN: txt
_TAG_REDIR_OUT: txt
LANG: C
ZOAU_HOME: "ZOAUDIR"
ZOAU: "ZOAUDIR/bin"
ZOAU_ROOT: "ZOAUDIR"
LIBPATH: "ZOAUDIR/lib:PYTHONDIR/lib:/lib:/usr/lib:."
ZOAUTIL_DIR: "ZOAUDIR"
PYTHONPATH: "ZOAUDIR/lib"
PYTHON_HOME: "PYTHONDIR"
PYTHON: "PYTHONDIR/bin"
PATH: "ZOAUDIR/bin:PYTHONDIR/bin:/bin:/usr/sbin:/var/bin:/usr/lpp/java/java180/J8.0_64/bin"

When running Ansible 2.16 or 2.15, the only change needed is to remove the pair of single quotes from _CEE_RUNOPTS, and thus the config should look like this:

_BPXK_AUTOCVT: "ON"
_CEE_RUNOPTS: "FILETAG(AUTOCVT,AUTOTAG) POSIX(ON)"
_TAG_REDIR_ERR: txt
_TAG_REDIR_IN: txt
_TAG_REDIR_OUT: txt
LANG: C
ZOAU_HOME: "ZOAUDIR"
ZOAU: "ZOAUDIR/bin"
ZOAU_ROOT: "ZOAUDIR"
LIBPATH: "ZOAUDIR/lib:PYTHONDIR/lib:/lib:/usr/lib:."
ZOAUTIL_DIR: "ZOAUDIR"
PYTHONPATH: "ZOAUDIR/lib"
PYTHON_HOME: "PYTHONDIR"
PYTHON: "PYTHONDIR/bin"
PATH: "ZOAUDIR/bin:PYTHONDIR/bin:/bin:/usr/sbin:/var/bin:/usr/lpp/java/java180/J8.0_64/bin"
PYTHONSTDINENCODING: "cp1047"

Running locally

To run locally, the same config should work, but in case you're getting garbage as output when using Ansible 2.16 or 2.15, try removing PYTHONSTDINENCODING and see if that fixes the issue.

[rexemin]

---

[rexemin]

---

[rexemin]

---

[rexemin]

[fernandofloresg]

@rexemin I see this PR is ready for review but no reviewers were assigned. I'm reviewing this, just one question, I see the screenshots are tested with 2.17.1, is this not breaking testing functionality for 2.15 and 2.16 ?

[rexemin]

Thanks for reviewing, it should still be on draft mode, that's a mistake on my part. I've been testing the changes for 2.16, but I still need to make sure it works fine with 2.15 too

[rexemin]

The whole test suite passed on 2.16.7. Still testing on 2.15 to make sure nothing breaks.

[AndreMarcel99]

Your change also cover a lot of warning we had on pytest, great

[rexemin]

The test suite was successful on Ansible 2.15.5.