Skip to content

Commit 4516073

Browse files
committed
Fix ret type
1 parent 3c480fb commit 4516073

1 file changed

Lines changed: 14 additions & 10 deletions

File tree

c/tests/test_tables.c

Lines changed: 14 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -10580,6 +10580,7 @@ static void
1058010580
test_check_integrity_bad_mutation_parent_topology(void)
1058110581
{
1058210582
int ret;
10583+
tsk_id_t ret_trees;
1058310584
tsk_table_collection_t tables;
1058410585
const char *sites = "0 0\n";
1058510586
/* Make a mutation on a parallel branch the parent*/
@@ -10610,22 +10611,25 @@ test_check_integrity_bad_mutation_parent_topology(void)
1061010611
ret = tsk_table_collection_build_index(&tables, 0);
1061110612
CU_ASSERT_EQUAL_FATAL(ret, 0);
1061210613

10613-
ret = tsk_table_collection_check_integrity(&tables, TSK_CHECK_TREES);
10614-
CU_ASSERT_EQUAL_FATAL(ret, 1);
10615-
ret = tsk_table_collection_check_integrity(&tables, TSK_CHECK_MUTATION_PARENTS);
10616-
CU_ASSERT_EQUAL_FATAL(ret, TSK_ERR_BAD_MUTATION_PARENT);
10614+
ret_trees = tsk_table_collection_check_integrity(&tables, TSK_CHECK_TREES);
10615+
CU_ASSERT_EQUAL_FATAL(ret_trees, 1);
10616+
ret_trees
10617+
= tsk_table_collection_check_integrity(&tables, TSK_CHECK_MUTATION_PARENTS);
10618+
CU_ASSERT_EQUAL_FATAL(ret_trees, TSK_ERR_BAD_MUTATION_PARENT);
1061710619

1061810620
parse_mutations(reverse_mutations, &tables.mutations);
10619-
ret = tsk_table_collection_check_integrity(&tables, TSK_CHECK_TREES);
10620-
CU_ASSERT_EQUAL_FATAL(ret, 1);
10621-
ret = tsk_table_collection_check_integrity(&tables, TSK_CHECK_MUTATION_PARENTS);
10622-
CU_ASSERT_EQUAL_FATAL(ret, TSK_ERR_MUTATION_PARENT_AFTER_CHILD);
10621+
ret_trees = tsk_table_collection_check_integrity(&tables, TSK_CHECK_TREES);
10622+
CU_ASSERT_EQUAL_FATAL(ret_trees, 1);
10623+
ret_trees
10624+
= tsk_table_collection_check_integrity(&tables, TSK_CHECK_MUTATION_PARENTS);
10625+
CU_ASSERT_EQUAL_FATAL(ret_trees, TSK_ERR_MUTATION_PARENT_AFTER_CHILD);
1062310626

1062410627
/* Now check that TSK_CHECK_MUTATION_PARENTS implies TSK_CHECK_TREES
1062510628
by triggering an error with reversed sites */
1062610629
parse_sites(reverse_sites, &tables.sites);
10627-
ret = tsk_table_collection_check_integrity(&tables, TSK_CHECK_MUTATION_PARENTS);
10628-
CU_ASSERT_EQUAL_FATAL(ret, TSK_ERR_UNSORTED_SITES);
10630+
ret_trees
10631+
= tsk_table_collection_check_integrity(&tables, TSK_CHECK_MUTATION_PARENTS);
10632+
CU_ASSERT_EQUAL_FATAL(ret_trees, TSK_ERR_UNSORTED_SITES);
1062910633

1063010634
tsk_table_collection_free(&tables);
1063110635
}

0 commit comments

Comments
 (0)