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

Add option to hash based on method instead of class #2

Open
winhamwr opened this issue Feb 5, 2013 · 4 comments
Open

Add option to hash based on method instead of class #2

winhamwr opened this issue Feb 5, 2013 · 4 comments

Comments

@winhamwr
Copy link
Contributor

winhamwr commented Feb 5, 2013

For cases with test fixtures, it's likely that splitting by class will result in the best speed. When there are no gains from grouping tests by class, though, this can result in a less-even, and thus slower, distribution.

$ export NOSE_DISTRIBUTED_NODES=2;
$ export NOSE_DISTRIBUTED_NODE_NUMBER=1;
$ export NOSE_DISTRIBUTED_CLASS_HASHING=0;
$ nosetests long_test_suit
@simon-weber
Copy link
Contributor

This is something we'd like, too =)

Any thoughts since you suggested this? The interface you described should be easy enough to implement (though I think it'd need to fall back to the test's file if the class isn't available, eg a nose test function or generator).

@simon-weber
Copy link
Contributor

Oh, maybe things changed at some point? Currently tests aren't distributed by class, but individually; I'm looking for distribution by class.

@simon-weber
Copy link
Contributor

though I think it'd need to fall back to the test's file if the class isn't available

actually, it's probably best to just keep the current fallback -- I doubt many people use file-level setup and teardown.

@winhamwr
Copy link
Contributor Author

winhamwr commented Dec 4, 2014

Currently tests aren't distributed by class, but individually; I'm looking for distribution by class.

It would be wonderful if we could switch between per-method and per-class distribution. Pull requests definitely welcome :)

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

2 participants