josephw commented on a change in pull request #256: MNG-6685: convert maven-model-builder to JSR 330 URL: https://github.com/apache/maven/pull/256#discussion_r296222975
########## File path: maven-core/src/test/java/org/apache/maven/project/artifact/MavenMetadataSourceTest.java ########## @@ -21,7 +21,9 @@ import org.apache.maven.repository.RepositorySystem; import org.codehaus.plexus.PlexusTestCase; +import org.junit.Ignore; +@Ignore Review comment: All test methods are commented out since a73c5abc9c584 and b8c6c01864e. This change prevents the `setUp` from causing a problem, without trying to fix a test that doesn't currently run, but stops short of deleting the test. The commenting-out is from 2009; is this a good point to drop the class? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
