SimonEnsemble / PorousMaterials.jl

Julia package towards classical molecular modeling of nanoporous materials
GNU General Public License v3.0
50 stars 11 forks source link

split `runtests.jl` suite into multiple files #25

Closed SimonEnsemble closed 6 years ago

SimonEnsemble commented 6 years ago

(@ahyork plz for when summer starts!) current the runtests.jl is a mess:

SimonEnsemble commented 6 years ago

nevermind; Travis CI wants all tests in one file. Still, would be great if each test suite were independent of the others! so e.g. framework from one test is not used in a test down the road.

ahyork commented 6 years ago

This is resolved with the latest pull request improving speed. Each test set is in its own file now. Because they are each inside their own module as well, they are tested in different global scopes, stopping the issue of one variable being used multiple times in different tests when they are all included in runtests.jl.