search

TerraME / terrame

TerraME is a programming environment for spatial dynamical modelling
www.terrame.org
GNU Lesser General Public License v3.0
36 stars 13 forks source link

data files for tests and for release #1100

Closed pedro-andrade-inpe closed 8 years ago

pedro-andrade-inpe commented 8 years ago

Think and implement a strategy to separate data files for tests and for release in packages. Some files should not belong to data directory in the release as they have errors that are identified in the tests. However, the implemented strategy should be easy for developers to adapt the installed terrame to work properly with tests.

pedro-andrade-inpe commented 8 years ago

Note that log directories also have this problem. Possibly it could be something that could be added to the installer or as an command line option to download the remaining files necessary to have a complete test environment.

avancinirodrigo commented 8 years ago

@pedro-andrade-inpe, what do you think about to separate the data physically, like data to test were on tests/data. It is a good thing in terms of uncoupling.

pedro-andrade-inpe commented 8 years ago

I prefer to have data/test within the package directory. This directory then could be removed when creating the installer, but it could be accessed through function filePath. Another possible solution is to have a package called data only to store data related to tests.

avancinirodrigo commented 8 years ago

@pedro-andrade-inpe, questions: There are data that belong to the test and other things, what to do? What does happen to description.lua?

pedro-andrade-inpe commented 8 years ago

One initial idea would be to have data/test, with the data that is only used in tests. Data within a directory in data must not be documented and then they could be removed from the release. What do you think?

pedro-andrade-inpe commented 8 years ago

In this case, filePath when it refers to a package and it contains a / in its first argument, if it could not find the file then a different message must be shown describing that the user might not be using the original source code but a release version.

avancinirodrigo commented 8 years ago

Associated to #813. The data need review.