[ https://jira.codehaus.org/browse/MNG-5075?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=283161#comment-283161 ]
Jesse Glick commented on MNG-5075: ---------------------------------- If you mean {{ant -f maven-core/pom.xml test}} or {{mvn clean install test integration-test}} then both pass with the patch. > MavenProject.getParent throws undocumented ISE > ---------------------------------------------- > > Key: MNG-5075 > URL: https://jira.codehaus.org/browse/MNG-5075 > Project: Maven 2 & 3 > Issue Type: Bug > Components: Embedding > Affects Versions: 3.0.3 > Reporter: Jesse Glick > Attachments: MavenProject-getParent-ISE.diff > > > http://bugzilla-attachments-197994.netbeans.org/bugzilla/attachment.cgi?id=107899 > shows a stack trace encountered when calling {{MavenProject.getParent}} on a > project with some errors (probably POMs missing in the local repository). > This method has no Javadoc comment, so it is hard to know exactly what it is > permitted/supposed to do, but {{hasParent}} implies that {{null}} is a valid > return value, and there is no {{throws IllegalStateException}} clause. The > attached patch brings the behavior in line with that signature. (I think I > got the {{PlexusTestCase}} infrastructure working with all the required > wiring but it may be possible to simplify the test case.) > Cleaner might be to just declare {{getParent}} (and also {{hasParent}}?) to > throw {{ProjectBuildingException}}, though this would be a > source-incompatible change. (Only binary-incompatible for clients which are > already catching {{IllegalStateException}}!) -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira