Fix for issue #289 - Githubissues

BristolTopGroup / NTupleProduction

Software for particle physics to create nTuples from AOD files

Other

2 stars 8 forks source link

Fix for issue #289 #293

Closed kreczko closed 7 years ago

kreczko commented 7 years ago

adds test for the bug
renamed __create_tar_file to _create_tar_file so it can be accessed in the test
fixed the issue by making _have_fresh_tar_files into a class variable (instead of instance)

In this implementation the tar files will be created once per command execution, e.g.

ntp run datasets=all will create the tar files once
running ntp run datasets=all right after that will create the tar files again.

mention-bot commented 7 years ago

@kreczko, thanks for your PR! By analyzing the history of the files in this pull request, we identified @EmyrClement and @DouglasBurns to be potential reviewers.