Skip to content

Commit

Permalink
Improve test documentation
Browse files Browse the repository at this point in the history
[skip ci]
  • Loading branch information
Aratz committed Oct 12, 2022
1 parent e4b305a commit fb8fd60
Showing 1 changed file with 10 additions and 1 deletion.
11 changes: 10 additions & 1 deletion tests/test_sync.py
Original file line number Diff line number Diff line change
Expand Up @@ -176,7 +176,16 @@ def test_create_merge_base_branch(self):
assert psync.merge_branch in psync.repo.branches

def test_create_merge_base_branch_thrice(self):
"""Try creating a merge base branch thrice"""
"""Try creating a merge base branch thrice
This is needed because the first time this function is called, the
merge branch does not exist yet (it is only created at the end of the
create_merge_base_branch function) and the if-statement is ignored.
Also, the second time this function is called, the existing merge
branch only has the base format, i.e. without the -{branch_no} at the
end, so it is needed to call it a third time to make sure this is
picked up.
"""
psync = nf_core.sync.PipelineSync(self.pipeline_dir)
psync.inspect_sync_dir()
psync.get_wf_config()
Expand Down

0 comments on commit fb8fd60

Please sign in to comment.