You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Kristian Rosenvold (JIRA)" <ji...@codehaus.org> on 2011/01/12 21:13:59 UTC
[jira] Closed: (SUREFIRE-498) POJO tests have only partially
implemented fixture support
[ http://jira.codehaus.org/browse/SUREFIRE-498?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Kristian Rosenvold closed SUREFIRE-498.
---------------------------------------
Resolution: Fixed
Fix Version/s: 2.7.2
Assignee: Kristian Rosenvold
Patch applied with modifications in r1058301
> POJO tests have only partially implemented fixture support
> ----------------------------------------------------------
>
> Key: SUREFIRE-498
> URL: http://jira.codehaus.org/browse/SUREFIRE-498
> Project: Maven Surefire
> Issue Type: Bug
> Components: JUnit 3.x support
> Reporter: Christian Gruber
> Assignee: Kristian Rosenvold
> Fix For: 2.7.2
>
> Attachments: fixture.diff, fixture.diff
>
>
> The current support for POJO executes each test method, and before and after each test method there is a method on PojoTestSet executed for fixture setup and teardown. In the executeMethod() method, there are try/catch blocks around a delegated method (setUpFixture() and tearDownFixture()) but the implementation of those two methods is empty. So under no circumstances is setup or teardown able to occur, except with in each method.
> I've attached a patch which looks up the setUp() and tearDown() public methods on the pojo test class, and if such exist, executes them appropriately. All the try/catch logic is already there - it just needed the last step of deciding what these methods were going to be called, and calling them.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira