Skip to content

Commit

Permalink
add specific test for issue #759
Browse files Browse the repository at this point in the history
  • Loading branch information
maximlt committed Jul 3, 2023
1 parent de38270 commit f466c9c
Showing 1 changed file with 18 additions and 7 deletions.
25 changes: 18 additions & 7 deletions tests/testpickle.py
Original file line number Diff line number Diff line change
Expand Up @@ -27,19 +27,19 @@ def test_pickle_simple_class(pickler):

@pytest.mark.parametrize('pickler', [cloudpickle, pickle])
def test_pickle_simple_instance(pickler):
p1 = P1()
s = pickler.dumps(p1)
p = P1()
s = pickler.dumps(p)
inst = pickler.loads(s)
assert eq(p1, inst)
assert eq(p, inst)


@pytest.mark.parametrize('pickler', [cloudpickle, pickle])
def test_pickle_simple_instance_modif_after(pickler):
p1 = P1()
s = pickler.dumps(p1)
p1.x = 'modified'
p = P1()
s = pickler.dumps(p)
p.x = 'modified'
inst = pickler.loads(s)
assert not eq(p1, inst)
assert not eq(p, inst)
assert inst.x is None


Expand Down Expand Up @@ -169,3 +169,14 @@ def test_pickle_complex_depends_instance(pickler):
s = pickler.dumps(p)
inst = pickler.loads(s)
assert eq(p, inst)


def test_issue_757():
# https://github.com/holoviz/param/issues/759
class P(param.Parameterized):
a = param.Parameter()

p = P()
s = cloudpickle.dumps(p)
inst = cloudpickle.loads(s)
assert eq(p, inst)

0 comments on commit f466c9c

Please sign in to comment.