Convert pubchem_test.py to use plain pytest instead of unittest#1278
Convert pubchem_test.py to use plain pytest instead of unittest#1278mhucka merged 1 commit intoquantumlib:mainfrom
pubchem_test.py to use plain pytest instead of unittest#1278Conversation
The failures in CI of the tests may be due to the use of the old Pyton unittest framework. This PR avoids the use of `unittest` in that file.
mhucka
changed the title
pubchem_test.py to use plain pytest instead of unittest
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request refactors the pubchem_test.py file to transition from the unittest framework to pytest. Key changes include converting the test class to a standard Python class, replacing @patch decorators with the monkeypatch fixture, and updating unittest assertion methods to standard assert statements and pytest.approx. I have no feedback to provide as there are no review comments to evaluate.
There was a problem hiding this comment.
Code Review
This pull request refactors the src/openfermion/chem/pubchem_test.py file to migrate from the unittest framework to pytest. Key changes include converting the test class to a standard Python class, replacing unittest.mock.patch decorators with the monkeypatch fixture, and updating unittest assertion methods to standard assert statements and pytest.approx. I have no feedback to provide as there were no review comments.
3 participants
The failures in CI of the tests may be due to the use of the old Python unittest framework. This PR avoids the use of
unittestin that file.