CivicSpleen / ambry

A comprehensive data package manager
BSD 2-Clause "Simplified" License
4 stars 5 forks source link

Update setup.py test commands #138

Closed ericbusboom closed 8 years ago

ericbusboom commented 8 years ago

We now have a re-organized test environment, described at:

https://github.com/CivicKnowledge/ambry/blob/master/doc/README.testing.md

However, it isn't quite finished: we need new test commands. The commands should be:

Additionally, all of the command should take these mutually exclusive options:

As described in the test README, these options will set the AMBRY_TEST_DB environmental variable to select which type of database to use.

nmb10 commented 8 years ago

Done.

nmb10 commented 8 years ago

Additionally, all of the command should take these mutually exclusive options: --sqlite or -s. Test with a sqlite database --postgres or -p Test with a postgres database.

I'm a developer, and I want to run both. How should I do that?

ericbusboom commented 8 years ago

On Nov 11, 2015, at 4:42 AM, Kazbek notifications@github.com wrote:

Additionally, all of the command should take these mutually exclusive options: --sqlite or -s. Test with a sqlite database --postgres or -p Test with a postgres database.

I'm a developer, and I want to run both. How should I do that?

Run the command twice.

They don’t actually have to be mutually exclusive; you could detect which options are selected, and then have the test program arrange for multiple runs. So, if you select both, it would run the tests twice.

eric.


Civic Knowledge Data for community development Eric Busboom, eric@civicknowledge.com, (619) 363 2607 http://civicknowledge.com

nmb10 commented 8 years ago

Now tests can run over both databases.

python setup.py test --postgres --sqlite

In such a case all tests run on postgres, then all tests run on sqlite.