Flaky XmlMetadataTemplateTest fix #11225
Open
+0
−9
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What this PR does / why we need it: Some obsolete code in the XmlMetadataTemplate class that tried to find the PidProvider for a 10.5072/FK2* identifier was causing intermittent failures - depending on whether other tests had added/remove a DOI provider for that authority/shoulder. This was seen in #10567 and #11222 recently. This PR removes the obsolete code which is doing the PidProvider lookup.
Which issue(s) this PR closes:
Special notes for your reviewer: FWIW: This code's from #10632 where the pidprovider was originally used as a way to call a method in the DatasetFieldServiceBean (since XMLMetadataTemplate isn't a bean). That was replaced by
dataverse/src/main/java/edu/harvard/iq/dataverse/pidproviders/doi/XmlMetadataTemplate.java
Line 651 in 3aea148
Suggestions on how to test this: Basic regression testing w.r.t. creating/publishing a DataCite DOI dataset? Nominally there should be no effect except for no future test fails.
Does this PR introduce a user interface change? If mockups are available, please link/include them here:
Is there a release notes update needed for this change?:
Additional documentation: