[refactor] Convert unittests to pytest syntax

[teojgo]

• Use pytest syntax and pytest functions for skipping/failing.

• Small fix in ScopedDict tests.

Fixes #1149

[vkarak]

@teojgo Why addresses and not fixes the issue?

[codecov-io]

Codecov Report

Merging #1188 into master will decrease coverage by 0.24%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master   #1188      +/-   ##
=========================================
- Coverage   92.14%   91.9%   -0.25%     
=========================================
  Files          81      81              
  Lines       11930   11569     -361     
=========================================
- Hits        10993   10632     -361     
  Misses        937     937

Impacted Files	Coverage Δ
reframe/core/containers.py	100% <100%> (ø)	⬆️
reframe/core/launchers/mpi.py	98.63% <100%> (+0.03%)	⬆️
reframe/core/schedulers/slurm.py	54.25% <100%> (-0.29%)	⬇️
reframe/core/fields.py	99% <100%> (+0.05%)	⬆️
unittests/test_launchers.py	92.72% <100%> (-1.48%)	⬇️
reframe/core/schedulers/pbs.py	68.29% <100%> (+0.79%)	⬆️
reframe/utility/__init__.py	90.12% <100%> (+0.06%)	⬆️
reframe/core/schedulers/local.py	100% <100%> (ø)	⬆️
unittests/test_fields.py	97.6% <100%> (+0.07%)	⬆️
reframe/core/pipeline.py	92.24% <100%> (-0.35%)	⬇️
... and 40 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 030d1ed...8c3df25. Read the comment docs.

[vkarak]

@teojgo Please also fix the conflicts with the current master.

[pep8speaks]

Hello @teojgo, Thank you for updating!

• In the file unittests/test_typecheck.py:

Line 29:80: E501 line too long (85 > 79 characters)

• In the file unittests/test_versioning.py:

Line 20:39: E701 multiple statements on one line (colon)
Line 21:39: E701 multiple statements on one line (colon)
Line 22:39: E701 multiple statements on one line (colon)
Line 23:39: E701 multiple statements on one line (colon)
Line 24:39: E701 multiple statements on one line (colon)
Line 25:39: E701 multiple statements on one line (colon)
Line 49:39: E701 multiple statements on one line (colon)
Line 50:39: E701 multiple statements on one line (colon)
Line 51:39: E701 multiple statements on one line (colon)
Line 52:39: E701 multiple statements on one line (colon)
Line 53:39: E701 multiple statements on one line (colon)
Line 54:39: E701 multiple statements on one line (colon)
Line 55:39: E701 multiple statements on one line (colon)
Line 56:39: E701 multiple statements on one line (colon)

Do see the ReFrame Coding Style Guide

Comment last updated at 2020-02-21 13:37:25 UTC

[vkarak]

@teojgo Apparently, PEP8 does not like the with Foo: do_sth() syntax. So let's revert back. Sorry for that.

[vkarak]

@teojgo There are still some PEP8 issues. As soon as you fix them, we can merge this PR.

[teojgo]

@teojgo There are still some PEP8 issues. As soon as you fix them, we can merge this PR.

@vkarak they are fixed but it seems pep8speaks has not taken the update into account