search

vepadulano / PyRDF

Python Library for doing ROOT RDataFrame analysis
https://pyrdf.readthedocs.io/en/latest/
9 stars 7 forks source link

Clear global sets in unittests #73

Closed vepadulano closed 5 years ago

vepadulano commented 5 years ago

The includes_* sets of paths are not cleaned after the execution of the analysis. Even though this shouldn't be a problem in a normal user environment, during the execution of unittests it can happen that different tests using different sets of headers are actually trying to include the paths remaining from previous tests in the global sets.

A new function clear_global_sets has been added to reset these sets of paths when needed. The function is then called at tearDown time in some tests that included headers or libraries.

vepadulano commented 5 years ago

addresses #61

vepadulano commented 5 years ago

After talking with @etejedor we decided that a global function, even if flagged as for internal use, is not necessary for the moment. Instead the global sets are cleared when needed in the tearDown methods of the unittests