Skip to content

Conversation

@dependabot
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Jul 12, 2025

Bumps org.apache.commons:commons-lang3 from 3.12.0 to 3.18.0.

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
    You can disable automated security fix PRs for this repo from the Security Alerts page.

Bumps org.apache.commons:commons-lang3 from 3.12.0 to 3.18.0.

---
updated-dependencies:
- dependency-name: org.apache.commons:commons-lang3
  dependency-version: 3.18.0
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot added dependencies Pull requests that update a dependency file java Pull requests that update Java code labels Jul 12, 2025
@qqmyers
Copy link
Member

qqmyers commented Jul 12, 2025

Re: test fails: Looks like commons-lang3 now changes ɨ to i, so the keys in these two lines (and in translations of this properties file) need to be updated (or parsing changed):

controlledvocabulary.language.magɨ_(madang_province)=Magɨ (Madang Province)
controlledvocabulary.language.magɨyi=Magɨyi

@pdurbin pdurbin moved this to Ready for Triage in IQSS Dataverse Project Jul 15, 2025
@ofahimIQSS ofahimIQSS moved this from Ready for Triage to Ready for Review ⏩ in IQSS Dataverse Project Jul 15, 2025
@ofahimIQSS ofahimIQSS added the Size: 10 A percentage of a sprint. 7 hours. label Jul 15, 2025
@pdurbin
Copy link
Member

pdurbin commented Jul 17, 2025

When I run mvn test -Dtest=MetadataBlockTest#testLanguageBundle on develop I get basically no output and the test passes:

[INFO] -------------------------------------------------------
[INFO]  T E S T S
[INFO] -------------------------------------------------------
[INFO] Running edu.harvard.iq.dataverse.MetadataBlockTest
Missing count:0
Unused Count:0
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.077 s -- in edu.harvard.iq.dataverse.MetadataBlockTest
[INFO] 
[INFO] Results:
[INFO] 
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0
[INFO] 
[INFO] 
[INFO] --- jacoco:0.8.11:report (jacoco-after-unit) @ dataverse ---
[INFO] Loading execution data file /Users/PDurbin/github/iqss/dataverse/target/coverage-reports/jacoco-unit.exec
[INFO] Analyzed bundle 'dataverse' with 1113 classes
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------

However, when I switch to this branch, I get the following output and errors:

[INFO] -------------------------------------------------------
[INFO]  T E S T S
[INFO] -------------------------------------------------------
[INFO] Running edu.harvard.iq.dataverse.MetadataBlockTest
Missing key:magɨ_(madang_province) controlledvocabulary.language.magi_(madang_province)
Missing key:magɨyi controlledvocabulary.language.magiyi
Unused key:controlledvocabulary.language.magɨyi
Unused key:controlledvocabulary.language.magɨ_(madang_province)
Missing count:2
Unused Count:2
[ERROR] Tests run: 1, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.080 s <<< FAILURE! -- in edu.harvard.iq.dataverse.MetadataBlockTest
[ERROR] edu.harvard.iq.dataverse.MetadataBlockTest.testLanguageBundle -- Time elapsed: 0.068 s <<< FAILURE!
org.opentest4j.AssertionFailedError: expected: <0> but was: <2>
	at org.junit.jupiter.api.AssertionFailureBuilder.build(AssertionFailureBuilder.java:151)
	at org.junit.jupiter.api.AssertionFailureBuilder.buildAndThrow(AssertionFailureBuilder.java:132)
	at org.junit.jupiter.api.AssertEquals.failNotEqual(AssertEquals.java:197)
	at org.junit.jupiter.api.AssertEquals.assertEquals(AssertEquals.java:150)
	at org.junit.jupiter.api.AssertEquals.assertEquals(AssertEquals.java:145)
	at org.junit.jupiter.api.Assertions.assertEquals(Assertions.java:531)
	at edu.harvard.iq.dataverse.MetadataBlockTest.testLanguageBundle(MetadataBlockTest.java:122)
	at java.base/java.lang.reflect.Method.invoke(Method.java:568)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)

[INFO] 
[INFO] Results:
[INFO] 
[ERROR] Failures: 
[ERROR]   MetadataBlockTest.testLanguageBundle:122 expected: <0> but was: <2>
[INFO] 
[ERROR] Tests run: 1, Failures: 1, Errors: 0, Skipped: 0
[INFO] 
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------

@cmbz cmbz added FY26 Sprint 1 FY26 Sprint 1 (2025-07-02 - 2025-07-16) FY26 Sprint 2 FY26 Sprint 2 (2025-07-16 - 2025-07-30) labels Jul 17, 2025
@github-actions

This comment has been minimized.

1 similar comment
@github-actions

This comment has been minimized.

@coveralls
Copy link

Coverage Status

coverage: 23.225% (+0.06%) from 23.161%
when pulling c9acc2e on dependabot/maven/modules/dataverse-parent/org.apache.commons-commons-lang3-3.18.0
into 7e16ed3 on develop.

@cmbz cmbz added the FY26 Sprint 3 (2025-07-30 - 2025-08-13) label Jul 31, 2025
@stevenwinship stevenwinship self-assigned this Aug 7, 2025
@stevenwinship stevenwinship moved this from Ready for Review ⏩ to In Review 🔎 in IQSS Dataverse Project Aug 7, 2025
controlledvocabulary.language.magɨ_(madang_province)=Magɨ (Madang Province)
controlledvocabulary.language.magɨyi=Magɨyi
controlledvocabulary.language.magi_(madang_province)=Magɨ (Madang Province)
controlledvocabulary.language.magiyi=Magɨyi
Copy link
Contributor

@stevenwinship stevenwinship Aug 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same issue Phil saw

In citation.tsv the old values are still present:

language	Magɨ (Madang Province)	gkd	3946	gkd
language	Magɨyi	gmg	3947	gmg

For me MetadataBlockTest is failing:

Missing key:magɨ_(madang_province) controlledvocabulary.language.magɨ_(madang_province)
Missing key:magɨyi controlledvocabulary.language.magɨyi
Unused key:controlledvocabulary.language.magi_(madang_province)
Unused key:controlledvocabulary.language.magiyi
Missing count:2
Unused Count:2

org.opentest4j.AssertionFailedError:
Expected :0
Actual :2

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made the change to citation.tsv as well to fix the failure

@github-actions

This comment has been minimized.

1 similar comment
@github-actions
Copy link

github-actions bot commented Aug 7, 2025

📦 Pushed preview images as

ghcr.io/gdcc/dataverse:dependabot-maven-modules-dataverse-parent-org.apache.commons-commons-lang3-3.18.0
ghcr.io/gdcc/configbaker:dependabot-maven-modules-dataverse-parent-org.apache.commons-commons-lang3-3.18.0

🚢 See on GHCR. Use by referencing with full name as printed above, mind the registry name.

Copy link
Contributor

@stevenwinship stevenwinship left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did some extensive testing with the new tsv file both for new installation and upgrade. No issues seen

@github-project-automation github-project-automation bot moved this from In Review 🔎 to Ready for QA ⏩ in IQSS Dataverse Project Aug 7, 2025
@stevenwinship stevenwinship removed their assignment Aug 7, 2025
@ofahimIQSS ofahimIQSS moved this from Ready for QA ⏩ to QA ✅ in IQSS Dataverse Project Aug 8, 2025
@ofahimIQSS ofahimIQSS self-assigned this Aug 8, 2025
@ofahimIQSS
Copy link
Contributor

tests passing - merging

@ofahimIQSS ofahimIQSS merged commit ee3af4a into develop Aug 8, 2025
16 checks passed
@github-project-automation github-project-automation bot moved this from QA ✅ to Merged 🚀 in IQSS Dataverse Project Aug 8, 2025
@ofahimIQSS ofahimIQSS deleted the dependabot/maven/modules/dataverse-parent/org.apache.commons-commons-lang3-3.18.0 branch August 8, 2025 13:07
@ofahimIQSS ofahimIQSS removed their assignment Aug 8, 2025
@pdurbin pdurbin added this to the 6.8 milestone Aug 11, 2025
pdurbin added a commit that referenced this pull request Aug 11, 2025
@scolapasta scolapasta moved this from Merged 🚀 to Done 🧹 in IQSS Dataverse Project Aug 12, 2025
@cmbz cmbz added the FY26 Sprint 4 FY26 Sprint 4 (2025-08-13 - 2025-08-27) label Aug 16, 2025
ofahimIQSS added a commit that referenced this pull request Aug 28, 2025
…e-note

reload citation.tsv when upgrading, a follow up to #11632
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file FY26 Sprint 1 FY26 Sprint 1 (2025-07-02 - 2025-07-16) FY26 Sprint 2 FY26 Sprint 2 (2025-07-16 - 2025-07-30) FY26 Sprint 3 (2025-07-30 - 2025-08-13) FY26 Sprint 4 FY26 Sprint 4 (2025-08-13 - 2025-08-27) java Pull requests that update Java code Size: 10 A percentage of a sprint. 7 hours.

Projects

Status: Done 🧹

Development

Successfully merging this pull request may close these issues.

7 participants