Skip to content
This repository has been archived by the owner on Sep 10, 2021. It is now read-only.

ConfigProperty default name #233, no longer decapitalize the default derived name from a class #18

Merged
merged 1 commit into from Sep 16, 2017

Conversation

starksm64
Copy link
Contributor

There was a change in behavior regarding the default name of a @ConfigProperty that no longer decapitalizes the class name of the injection site:
eclipse/microprofile-config#233

Prior to this update, the CDIPlainInjectionTest was failing to deploy due to no org.eclipse.microprofile.config.tck.CDIPlainInjectionTest.defaultPropertyBean.configProperty config value being found:

Sep 15, 2017 8:13:21 PM org.apache.webbeans.portable.events.discovery.ErrorStack logErrors
SEVERE: Error while validating Configuration
No Config Value exists for org.eclipse.microprofile.config.tck.CDIPlainInjectionTest.defaultPropertyBean.configProperty
javax.enterprise.inject.spi.DeploymentException: Error while validating Configuration
No Config Value exists for org.eclipse.microprofile.config.tck.CDIPlainInjectionTest.defaultPropertyBean.configProperty
	at org.wildfly.microprofile.config.inject.ConfigExtension.validate(ConfigExtension.java:122)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.apache.webbeans.event.ObserverMethodImpl.invoke(ObserverMethodImpl.java:347)

The test is now setting this value using the key org.eclipse.microprofile.config.tck.CDIPlainInjectionTest.DefaultPropertyBean.configProperty.

With this PR, the TCK is completely passing.

Signed-off-by: Scott Stark starksm64@gmail.com

…derived

name from a class
eclipse/microprofile-config#233

Signed-off-by: Scott Stark <starksm64@gmail.com>
@jmesnil jmesnil merged commit 1fd3277 into wildfly-extras:master Sep 16, 2017
@jmesnil
Copy link
Member

jmesnil commented Sep 16, 2017

thanks scott!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants