TST: Mock clipboard IO #22715
TST: Mock clipboard IO #22715
Conversation
TomAugspurger added CI 
| This failed. Unless anyone objects, I'm going to just mock the actual clipboard layer part of things. |
| yeah the clipboard tests are annoying |
| Hello @TomAugspurger! Thanks for updating the PR.
Comment last updated on September 16, 2018 at 11:57 Hours UTC |
| OK, I think we're still testing things, but would appreciate a close look at the mocks. Basic idea is to replace Unfortunately, we had some tests in |
pandas/tests/io/test_clipboard.py Outdated
| | ||
| | ||
| # our local clipboard for tests | ||
| _mock_data = {} |
There was a problem hiding this comment.
it’s really odd to keep state outside of this fixture when it’s a function level
why don’t u just make it a module level?
or it seems that function level might be ok with state inside
| @pytest.fixture | ||
| def mock_clipboard(mock, request): | ||
| def _mock_set(data): | ||
| _mock_data[request.node.name] = data |
pandas/tests/io/test_clipboard.py Outdated
| class TestClipboard(object): | ||
| | ||
| @pytest.mark.clipboard | ||
| def test_mock_clipboard(self): |
There was a problem hiding this comment.
can’t u use monkey patch here?
| Moved the mock clipboard to be function-scope, rather than global. |
| sure |
* Attempt to fix clipboard tests * note * update * update * doc
* Attempt to fix clipboard tests * note * update * update * doc
3 participants
No idea if this will work on not.