Re: Overview of ETS 3.0 tests

Hi Ilan,

Ilan Schnell wrote:
> The reason why test_all.py does not pick up the integration tests
> is because nose does not pick them up.  test_all.py just runs nose
> in integrationtests as a subprocess.  If I go to integrationtests and
> run nose manually nothing happens either (at least with nose 0.10.3).
> Different version of nose behave differently, and its causing a lot of
> trouble, but essentially test_all.py can do whatever it wants.  For
> example to run the command line options you can say something like
>   retval = subprocess.call(['python', 'test_image_plane_widget.py', '-h'])
>   ...
>   self.assertEqual(retval, 0)
> test_all.py should however run all the test suites, i.e. if one fails

Well, you do look for certain directories in the test_directories method 
and I think it may be easier to modify that to do the job rather than 
explicitly list each file.  Perhaps adding the ability to specify a list 
of relative paths of directories to run nose in would make this convenient.

> the others should still run and at the end it prints out a summary before
> checking if all went well.  Note also that test_all needs to be a unittest,
> such that it works when you say: python setup.py test

I thought python setup.py test would work with nose if you use 
nose.collector in setup.py?

> I assume that:
> ./enthought/mayavi/tests   are more "internal" tests
> ./integrationtests         are graphical tests

enthought/mayavi/tests contains unittests.

integrationtests contains integration tests.  These are not unit test 
(in the sense of the definition), take longer to run, may or may not be 
graphical but typically are graphical.

> If this is true, I will move test_csv_sniff.py and the corresponding folder
> with test files to ./enthought/mayavi/tests

Yes, that would be great.  You'll also have to move the csv_files and 
use pkg_resources to locate them in the tests.  Thanks.

> I hope you had a god trip back to India.

Yes I did, thanks!  Its a long flight but not too bad.

cheers,
prabhu