WASdev / standards.jsr352.tck

Home of the TCK for the Java Batch standard (JSR 352, Batch Applications for the Java Platform). #JavaEE7
Apache License 2.0
1 stars 9 forks source link

Test Added for Issue#32 #44

Closed jvfullam closed 8 years ago

jvfullam commented 8 years ago

Travis build failed upon pull request. Double checking that build succeeds on local machine.

jvfullam commented 8 years ago

Merged updates to WASdev master into my feature branch (issue#32). Travis build passes now.

jvfullam commented 8 years ago

The test I wrote turns out to be implementation specific. It assumes that unresolved @Inject statements will have a default value. This is not the behavior of CDI injection.

We need a new test strategy for determining that parent-level properties are out of scope of child level artifacts.

jvfullam commented 8 years ago

Reopening this for now... it seems okay to try:

@Inject @BatchProperty String notPresentInBatchletPropertiesOfXML;

and assume that the String will resolve to a value of null.

scottkurz commented 8 years ago

Thanks for the helpful comments on the test strategy.