[pytest] Relocate Session Level Fixtures To Test Files

[tahmed-dev]

Description of PR

Summary:

Commit b0237f1 introduced session level fixtures, however indirect
import of those fixtures is not being respected. Relocation of
fixture import to pytest test files solves this issue.

signed-off-by: Tamer Ahmed [email protected]

Fixes # (issue)

Type of change

• Bug fix
• Testbed and Framework(new/improvement)
• Test case(new/improvement)

Approach

What is the motivation for this PR?

Fixing failures when running test case with indirect import of session level fixture

How did you do it?

Relocate import to test files

How did you verify/test it?

ran with and without this issue test_qos_sai. Issue is resolved with this fix

tests$ pytest qos/test_qos_sai.py --testbed=vms12-t0-s6000-1 --inventory=../ansible/str --testbed_file=../ansible/testbed.csv --host-pattern=str-s6000-acs-14 --module-path=../ansible/library --disable_loganalyzer --skip_sanity
================================================================================================================================================== test session starts ==================================================================================================================================================
platform linux2 -- Python 2.7.12, pytest-4.6.9, py-1.8.1, pluggy-0.13.1
ansible: 2.8.7
rootdir: /var/host-acs-mgmt-repo/tests, inifile: pytest.ini
plugins: ansible-2.2.2
collected 1 item                                                                                                                                                                                                                                                                                                        

qos/test_qos_sai.py .                                                                                                                                                                                                                                                                                             [100%]

============================================================================================================================================== 1 passed in 396.14 seconds ===============================================================================================================================================

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

[lgtm-com]

This pull request introduces 8 alerts and fixes 8 when merging bcc3dfa into eefb131 - view on LGTM.com

new alerts:

• 8 for Unused import

fixed alerts:

• 8 for Unused import