Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

MicroProfile Config 3.1 #8724

Open
Tracked by #8704
barchetta opened this issue May 3, 2024 · 1 comment · May be fixed by #8757
Open
Tracked by #8704

MicroProfile Config 3.1 #8724

barchetta opened this issue May 3, 2024 · 1 comment · May be fixed by #8757
Assignees
@barchetta
Labels
4.x Version 4.x config dependencies Pull requests that update a dependency file microprofile
Projects
Backlog
Milestone
4.1.0

Comments

@barchetta
Copy link
Member

barchetta commented May 3, 2024 •
edited

Upgrade MicroProfile Config to version 3.1
@barchetta barchetta mentioned this issue May 3, 2024
Open
10 tasks
@barchetta barchetta added the 4.x Version 4.x label May 3, 2024
@github-actions github-actions bot added this to Triage in Backlog May 3, 2024
@barchetta barchetta added this to the 4.1.0 milestone May 3, 2024
@barchetta barchetta added microprofile config dependencies Pull requests that update a dependency file labels May 3, 2024
@barchetta
Copy link
Member Author

barchetta commented May 3, 2024 •
edited

The most significant change is clarifying precedence for sources when using profiles plus the addition of a new TCK test. When run against Helidon 4 this new test fails:

[INFO] Running TestSuite
[ERROR] Tests run: 373, Failures: 4, Errors: 0, Skipped: 0, Time elapsed: 10.437 s <<< FAILURE! - in TestSuite
[ERROR] org.eclipse.microprofile.config.tck.profile.OverrideConfigProfileTest.testConfigProfileWithDevAndOverride  Time elapsed: 0.008 s  <<< FAILURE!
java.lang.AssertionError:

Expected: is "dummy"
     but: was "foo"
	at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
	at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:8)
	at org.eclipse.microprofile.config.tck.profile.OverrideConfigProfileTest.testConfigProfileWithDevAndOverride(OverrideConfigProfileTest.java:74)

See #8738 (see also #8737 which is not causing this failure, but might be involved with the fix).

@barchetta barchetta self-assigned this May 6, 2024
@barchetta barchetta moved this from Triage to Sprint Scope in Backlog May 6, 2024
This was referenced May 9, 2024
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@barchetta barchetta

Labels
4.x Version 4.x config dependencies Pull requests that update a dependency file microprofile
Projects
Backlog
  
Sprint Scope
Milestone
4.1.0
Development

Successfully merging a pull request may close this issue.

4.x: Upgrade to MP Config 3.1 and fix an issue with profile specific properties barchetta/helidon
1 participant
@barchetta