Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unit Tests? #52

Open
GoogleCodeExporter opened this issue Mar 17, 2015 · 2 comments
Open

Unit Tests? #52

GoogleCodeExporter opened this issue Mar 17, 2015 · 2 comments

Comments

@GoogleCodeExporter
Copy link

Hi, I browsed the parallel execution framework. I have done work in past 
related to this and know how to reliably speed up this framework with:

(a) less locking
(b) far less self-imposed sleeps/waits 

However, I do not see any unit tests..  Without unit tests there is no 
mechanism to prove the changes.

I don't expect Load and Performance tests (though would be nice..)  in any case 
I can write those. But need FUNCTION/FEATURE unit tests to prove the features 
working same under the (much higher) load conditions that my new code would 
support.

Why think that no tests?

(a) the tests dir is EMPTY
(b) Even under src: I searched for AbstractSimpleParallelTask: and only found 
it in source code (no testcases..)


Let me know: How did/do you regression test this project ?  And then .. how did 
you verify your AbstractSimpleParallelTask is working in the first place?

thanks


Original issue reported on code.google.com by [email protected] on 29 May 2011 at 10:14

@GoogleCodeExporter
Copy link
Author

Thanks for the interest in the project. 
Unfortunately, as you pointed out there are no unit tests at the moment. The 
only functional tests are done via many BeanShell in scripts/ directory. 

On how do I ensure that AbstractSimpleParallelTask (or related) is working in 
the first place is by using them in actual application code as in the case of 
http://code.google.com/p/metastudio/source/browse/trunk/metastudio/src/org/meta/
math/qm/property/OneElectronProperty.java  or 
http://code.google.com/p/metastudio/source/browse/trunk/metastudio/src/org/meta/
math/qm/TwoElectronIntegrals.java

In view of your interest, I will be adding you to the project. Hope this is 
fine with you :)

Original comment by [email protected] on 4 Jun 2011 at 8:28

@GoogleCodeExporter
Copy link
Author

thx.  I just got real busy so will be bit delayed on doing this, probably
July timeframe.

Original comment by [email protected] on 4 Jun 2011 at 6:50

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant