diff options
author | Ned Batchelder <ned@nedbatchelder.com> | 2017-01-16 08:21:56 -0500 |
---|---|---|
committer | Ned Batchelder <ned@nedbatchelder.com> | 2017-01-16 08:21:56 -0500 |
commit | 050d768cd646a87a5c1578791546a71501bf518f (patch) | |
tree | 975e2654b395116058039f1c1c8c02d0f925c68b | |
parent | c00a96412f2d8a425fb07b2a23dde76e10554137 (diff) | |
download | python-coveragepy-git-050d768cd646a87a5c1578791546a71501bf518f.tar.gz |
Oops, don't assume decorator arguments can be hashed.
-rw-r--r-- | coverage/misc.py | 2 | ||||
-rw-r--r-- | tests/test_misc.py | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/coverage/misc.py b/coverage/misc.py index 797a16d5..7f85a0eb 100644 --- a/coverage/misc.py +++ b/coverage/misc.py @@ -71,7 +71,7 @@ if env.TESTING: def _decorator(func): argnameset = set(name.strip() for name in argnames.split(",")) def _wrapped(*args, **kwargs): - vals = set(kwargs.get(name) for name in argnameset) + vals = [kwargs.get(name) for name in argnameset] assert sum(val is not None for val in vals) == 1 return func(*args, **kwargs) return _wrapped diff --git a/tests/test_misc.py b/tests/test_misc.py index fc95744a..e69c2cdd 100644 --- a/tests/test_misc.py +++ b/tests/test_misc.py @@ -102,10 +102,10 @@ class ContractTest(CoverageTest): return (a, b, c) assert give_me_one(a=17) == (17, None, None) - assert give_me_one(b=17) == (None, 17, None) + assert give_me_one(b=set()) == (None, set(), None) assert give_me_one(c=17) == (None, None, 17) with pytest.raises(AssertionError): - give_me_one(a=17, b=23) + give_me_one(a=17, b=set()) with pytest.raises(AssertionError): give_me_one() |