From: Amanda R. <ama...@gm...> - 2008-12-17 16:07:57
|
I'm taking a look at the bug pasted below, in which not only @DataPoint-annotated fields but all static fields in a @RunWith(Theories.class) class are used as potential arguments for @Theory methods. Is it correct that only fields with @DataPoint annotations should be used? When I tried implementing a fix, all of a sudden there were twice as many unit test failures overall. It looks like many of the unit tests do not use the @DataPoint annotation for all fields they expect to use in Theory methods. I've got a fix ready for bug 2196462, @Theory annotation optional when running with Theories. Can I submit one patch to fix both that and this bug, or is it best if I submit two separate patches? Thanks, ~Amanda |