Added tests for the existing behavior of the parse method, including an exception test in the case of a missing plugin.xml file.

Issue #20112 - Reformatting according to Spring Java standards.

Comment From: mikesmithson

latest commit has proper formatting of PluginXmlParser.java classs.

Comment From: mikesmithson

Thanks for the review of the PR. One question I had was cloning the repo and simply ran './gradlew build' was not successful even though I didn't make any changes to the code. The buildSrc directory was also in a state of non-compilation which made formatting and making changes next to impossible. Any help?

Comment From: wilkinsona

Sorry to hear that. CI is green at the moment so ./gradlew build should have built cleanly. Can you give us some more information about your OS, the JDK you are using, and the build failure please?

Comment From: mikesmithson

Sorry to hear that. CI is green at the moment so ./gradlew build should have built cleanly. Can you give us some more information about your OS, the JDK you are using, and the build failure please?

@wilkinsona - I am using MacOS Catalina 10.15.3 running Oracle JDK 1.8.0.92. I am currently seeing the following test failures:

1) AetherGrapeEngineTests > resolutionWithCustomResolver() : Caused by: org.eclipse.aether.resolution.DependencyResolutionException: Could not find artifact io.spring.docresources:spring-doc-resources:zip:0.1.1.RELEASE in nexus (https://repo1.maven.org/maven2) why isn't it looking in repo.spring.io/release?

2) LiquibaseAutoConfigurationTests > changelogXml() : Caused by: org.xml.sax.SAXParseException; lineNumber: 7; columnNumber: 124; schema_reference.4: Failed to read schema document 'https://www.liquibase.org/xml/ns/dbchangelog/dbchangelog-3.0.xsd', because 1) could not find the document; 2) the document could not be read; 3) the root element of the document is not .

Comment From: snicoll

@mikesmithson it looks like your local environment is redirecting traffic to a local Nexus of yours. The Spring Boot build needs public access to the internet in order to build. The first issue could be fixed by adding https://repo.spring.io/release to the list of repositories. As for the second, I can't say for sure.

Comment From: snicoll

@mikesmithson thank you for making your first contribution to Spring Boot.