Skip to content

Commit 0c7549e

Browse files
hyanwongmergify[bot]
authored andcommitted
Use ignore_provenance in tests
1 parent 2a42740 commit 0c7549e

File tree

3 files changed

+7
-42
lines changed

3 files changed

+7
-42
lines changed

python/tests/test_tables.py

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -2968,9 +2968,7 @@ def test_zero_mutation_sites(self):
29682968
t1.simplify([0, 1], filter_zero_mutation_sites=filter_sites)
29692969
t2 = ts.dump_tables()
29702970
t2.simplify([0, 1], filter_sites=filter_sites)
2971-
t1.provenances.clear()
2972-
t2.provenances.clear()
2973-
t1.assert_equals(t2)
2971+
t1.assert_equals(t2, ignore_provenance=True)
29742972
if filter_sites:
29752973
assert ts.num_sites > len(t1.sites)
29762974

@@ -3515,7 +3513,7 @@ def test_equals(self):
35153513
t1.provenances.clear()
35163514
t2.provenances.clear()
35173515
assert t1 == t2
3518-
assert t1 == t2
3516+
assert t2 == t1
35193517
assert not (t1 != t2)
35203518

35213519
t1.nodes.clear()
@@ -4253,9 +4251,7 @@ def test_no_effect(self):
42534251
t2 = msprime.simulate(10, mutation_rate=1, random_seed=1).dump_tables()
42544252
assert len(t1.sites) > 0
42554253
t1.deduplicate_sites()
4256-
t1.provenances.clear()
4257-
t2.provenances.clear()
4258-
t1.assert_equals(t2)
4254+
t1.assert_equals(t2, ignore_provenance=True)
42594255

42604256
def test_same_sites(self):
42614257
t1 = msprime.simulate(10, mutation_rate=1, random_seed=1).dump_tables()
@@ -4269,9 +4265,7 @@ def test_same_sites(self):
42694265
assert len(t1.sites) == 2 * len(t2.sites)
42704266
t1.sort()
42714267
t1.deduplicate_sites()
4272-
t1.provenances.clear()
4273-
t2.provenances.clear()
4274-
t1.assert_equals(t2)
4268+
t1.assert_equals(t2, ignore_provenance=True)
42754269

42764270
def test_order_maintained(self):
42774271
t1 = tskit.TableCollection(1)

python/tests/test_topology.py

Lines changed: 2 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -2415,10 +2415,8 @@ def verify_unary_tree_sequence(self, ts):
24152415
py_ts, py_node_map = s.simplify()
24162416
lib_ts, lib_node_map = ts_new.simplify(map_nodes=True, **params)
24172417
py_tables = py_ts.dump_tables()
2418-
py_tables.provenances.clear()
24192418
lib_tables = lib_ts.dump_tables()
2420-
lib_tables.provenances.clear()
2421-
assert lib_tables == py_tables
2419+
lib_tables.assert_equals(py_tables, ignore_provenance=True)
24222420
assert np.all(lib_node_map == py_node_map)
24232421

24242422
def test_binary_tree_sequence_unary_nodes(self):
@@ -2564,34 +2562,9 @@ def verify_permuted_nodes(self, ts):
25642562
# form, and identical to the original.
25652563
simplified, s_node_map = permuted.simplify(map_nodes=True)
25662564

2567-
original_tables = ts.dump_tables()
2568-
simplified_tables = simplified.dump_tables()
2569-
original_tables.provenances.clear()
2570-
simplified_tables.provenances.clear()
2571-
2572-
assert original_tables.sequence_length == simplified_tables.sequence_length
2573-
assert original_tables.nodes == simplified_tables.nodes
2574-
assert original_tables.edges == simplified_tables.edges
2575-
assert original_tables.sites == simplified_tables.sites
2576-
assert original_tables.mutations == simplified_tables.mutations
2577-
assert original_tables.individuals == simplified_tables.individuals
2578-
assert original_tables.populations == simplified_tables.populations
2579-
2580-
assert original_tables == simplified_tables
2581-
assert ts.sequence_length == simplified.sequence_length
2582-
for _ in simplified.trees():
2583-
pass
2584-
25852565
for u, v in enumerate(node_map):
25862566
assert s_node_map[v] == u
2587-
assert np.array_equal(simplified.samples(), ts.samples())
2588-
assert list(simplified.nodes()) == list(ts.nodes())
2589-
assert list(simplified.edges()) == list(ts.edges())
2590-
assert list(simplified.sites()) == list(ts.sites())
2591-
assert list(simplified.haplotypes()) == list(ts.haplotypes())
2592-
assert list(simplified.variants(as_bytes=True)) == list(
2593-
ts.variants(as_bytes=True)
2594-
)
2567+
ts.tables.assert_equals(simplified.tables, ignore_provenance=True)
25952568

25962569
def test_single_tree_permuted_nodes(self):
25972570
ts = msprime.simulate(10, mutation_rate=5, random_seed=self.random_seed)

python/tests/test_wright_fisher.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -579,9 +579,7 @@ def test_simplify_tables(self, ts, nsamples):
579579
node_map = tables.simplify(samples=sub_samples)
580580
small_ts = tables.tree_sequence()
581581
other_tables = small_ts.dump_tables()
582-
tables.provenances.clear()
583-
other_tables.provenances.clear()
584-
tables.assert_equals(other_tables)
582+
tables.assert_equals(other_tables, ignore_provenance=True)
585583
self.verify_simplify(ts, small_ts, sub_samples, node_map)
586584

587585
@pytest.mark.parametrize("ts", wf_sims)

0 commit comments

Comments
 (0)