Skip to content

[MRG+2] MAINT: Refactor the converted-image cache #7764

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 19 commits into from
Closed

[MRG+2] MAINT: Refactor the converted-image cache #7764

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
1b24a16
Factor the converted-image cache out of compare.py
jkseppan Jan 7, 2017
650ddf2
Import cbook again
jkseppan Jan 21, 2017
3b5f636
Lock the cache directory
jkseppan Jan 21, 2017
556fd82
Communicate node-specific cache reports to master if using xdist
jkseppan Jan 22, 2017
9f5a06d
Delete the nose plugin
jkseppan Jan 25, 2017
33f2528
Reinstate deleted functions
jkseppan Jan 25, 2017
2136a26
Make conversion_cache private
jkseppan Jan 25, 2017
6995a27
Make globals private
jkseppan Jan 25, 2017
9ea66dd
Improve wording
jkseppan Jan 25, 2017
3751f01
mkdirs throws OSError
jkseppan Jan 25, 2017
a70081e
Improve documentation in response to review
jkseppan Jan 30, 2017
85f54de
Use pytest facilities
jkseppan Jan 30, 2017
0d51e03
Simplify gs and inkscape version checking logic
jkseppan Jan 30, 2017
9de06c0
pep8 fixes
jkseppan Jan 30, 2017
6007b37
There is no nose plugin any more
jkseppan Feb 18, 2017
e52cdd2
Import the pytest hooks in the other conftest.py files
jkseppan Feb 20, 2017
e5184bb
Omit unnecessary tmpdir deletions
jkseppan Feb 22, 2017
f47cbe4
Move pytest_addoption call to top-level conftest.py
jkseppan Feb 22, 2017
996f22b
pep8 fix
jkseppan Feb 22, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
mkdirs throws OSError
  • Loading branch information
@jkseppan
jkseppan committed Feb 20, 2017
commit 3751f01f5ca0a10722a07b4664f39fae79bdb954
2 changes: 1 addition & 1 deletion lib/matplotlib/testing/_conversion_cache.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -191,7 +191,7 @@ def ensure_cache_dir(self):
if not os.path.exists(self.cachedir):
try:
cbook.mkdirs(self.cachedir)
except IOError as e:
except OSError as e:
raise _CacheError("Error creating cache directory %s: %s"
% (self.cachedir, str(e)))
if not os.access(self.cachedir, os.W_OK):
Expand Down
2 changes: 1 addition & 1 deletion lib/matplotlib/tests/test_cache.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -103,7 +103,7 @@ def test_cache_default_dir():

@raises(_CacheError)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use pytest.raises context manager instead of this decorator.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@mock.patch('matplotlib.testing._conversion_cache.cbook.mkdirs',
side_effect=IOError)
side_effect=OSError)
def test_cache_mkdir_error(mkdirs):
tmpdir = tempfile.mkdtemp()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use tmpdir as a parameter to use the pytest fixture.

try:
Expand Down