Skip to content

Commit

Permalink
MAINT add comment about temporary workaround
Browse files Browse the repository at this point in the history
in test until we get rid of inspect.getargspect
  • Loading branch information
lesteve committed Sep 29, 2015
1 parent 23ff89c commit 4197b6f
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions joblib/test/test_memory.py
Expand Up @@ -225,6 +225,9 @@ def name_collision(x):
with warnings.catch_warnings(record=True) as w:
# Cause all warnings to always be triggered.
warnings.simplefilter("always")
# This is a temporary workaround until we get rid of
# inspect.getargspec, see
# https://github.com/joblib/joblib/issues/247
warnings.simplefilter("ignore", DeprecationWarning)
a(1)
b(1)
Expand All @@ -246,6 +249,9 @@ def test_memory_warning_lambda_collisions():
with warnings.catch_warnings(record=True) as w:
# Cause all warnings to always be triggered.
warnings.simplefilter("always")
# This is a temporary workaround until we get rid of
# inspect.getargspec, see
# https://github.com/joblib/joblib/issues/247
warnings.simplefilter("ignore", DeprecationWarning)
nose.tools.assert_equal(0, a(0))
nose.tools.assert_equal(2, b(1))
Expand Down Expand Up @@ -274,6 +280,9 @@ def test_memory_warning_collision_detection():
with warnings.catch_warnings(record=True) as w:
# Cause all warnings to always be triggered.
warnings.simplefilter("always")
# This is a temporary workaround until we get rid of
# inspect.getargspec, see
# https://github.com/joblib/joblib/issues/247
warnings.simplefilter("ignore", DeprecationWarning)
a1(1)
b1(1)
Expand Down

0 comments on commit 4197b6f

Please sign in to comment.