Re: Injecting Array types as fields

2017-12-18 Thread Romain Manni-Bucau
if you call get() can be yes. Happy to help if you can sort out a test like the one i sent. Romain Manni-Bucau @rmannibucau | Blog | Old Blog | Github |

Re: Injecting Array types as fields

2017-12-18 Thread John D. Ament
We register a single bean per injection point. That bean handles the look up but only returns the type of the injection point. But its still a bit odd. I added CDI.current().select(int[].class, new ConfigProperty...) to my test to see what happens. In Weld, I see it call the producer but fail

Re: Injecting Array types as fields

2017-12-18 Thread Romain Manni-Bucau
Isn't it cause we reuse a generic producer? Romain Manni-Bucau @rmannibucau | Blog | Old Blog | Github | LinkedIn

Re: Injecting Array types as fields

2017-12-18 Thread John D. Ament
That's good that this works. However, I wonder if my issue is specific to 3rd Party Bean creation? Same problem seems to be on both OWB and Weld, which makes me think it's an impl issue, but I can't think of anything else required on the impl side. Bean.getTypes() correctly returns

Re: Injecting Array types as fields

2017-12-18 Thread Romain Manni-Bucau
Works for me: public class ArrayProducerTest extends AbstractUnitTest { @Test public void test() { startContainer(StringProducer.class); final String[] produced1 = getInstance(String[].class); final StringProducer[] produced2 =

Re: Injecting Array types as fields

2017-12-18 Thread Mark Struberg
I think it should work. Could you please create a ticket? txs and LieGrue, strub > Am 18.12.2017 um 03:47 schrieb John D. Ament : > > MP Config is adding arrays as a valid type for config properties. In doing > so, without making any changes to Geronimo Config I get

Injecting Array types as fields

2017-12-17 Thread John D. Ament
MP Config is adding arrays as a valid type for config properties. In doing so, without making any changes to Geronimo Config I get this failure (not what I'm expecting, I'm expecting no valid beans at this point): arquillianBeforeClass(org.eclipse.microprofile.config.tck.ClassConverterTest) Time