Some test skips were implemented using the pytest module instead of unittest. This would cause the tests to not be skipped if the test file was run directly.
Approach
Replace all calls to pytest.mark.skip with unittest.skip.
Pre-Merge Checklists
Submitter
[x] Write a helpfully descriptive pull request title.
[x] Organize changes into logically grouped commits with descriptive commit messages.
[x] Document all new functions.
[x] Write tests for new functions or explain why they are not needed.
Purpose
Some test skips were implemented using the pytest module instead of unittest. This would cause the tests to not be skipped if the test file was run directly.
Approach
Replace all calls to
pytest.mark.skip
withunittest.skip
.Pre-Merge Checklists
Submitter
yapf
to format python code.Reviewer