11067079630

Committed 27 Sep 2024 08:15AM UTC coverage: 60.788% (+0.3%) from 60.484%

Build # 11067079630

Build Type

push

github

Committed by

LudovicRousseau

Commit Message

Migrate to pytest features

This PR introduces the following changes:

- Migrate the test suite to use pytest features
- Address a duplicated test name in test_Exceptions.py:
  test_ListReadersException.  The duplicate tests have been merged.

Most of this work was mechanical:

- Remove unittest.TestCase subclass definition lines
- De-dent the class methods
- Remove references to self in the function parameters
- Use regular expressions to search and replace assertions:
  - self.assertEqual(x, y) -> assert x == y
  - self.assertRaises -> pytest.raises
  - self.assertFalse(x) -> assert x is False
  - self.assertIn(x, y) -> assert x in y

Some long strings were wrapped to prepare for the possibility of using
code formatters in the future.

STDOUT capture code was replaced with the pytest capsys fixture, which
simplified the code in test_ATR.py.

This PR prepares for the possibility of consolidating the live tests in
src/smartcard/test/ together with the overall test suite, and using
pytest features to control whether the live tests run or not.

This is only a possibility, not a guarantee, but migrating from unittest
code to pytest code helps pave the way.

Run Details

91 of 515 branches covered (17.67%)

3257 of 5358 relevant lines covered (60.79%)

3.42 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

100.0

/test_PCSCExceptions.py

Source Not Available

The file "test_PCSCExceptions.py" isn't available on github. Either it's been removed, or the repo root directory needs to be updated.

LudovicRousseau / pyscard / 11067079630

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source Not Available

Source File
Press 'n' to go to next uncovered line, 'b' for previous