cdi-tck icon indicating copy to clipboard operation
cdi-tck copied to clipboard

Create TCK coverage for BeanManager.getContexts()

Open manovotn opened this issue 2 years ago • 2 comments

Related specification issue - https://github.com/jakartaee/cdi/issues/628

Tracking issue for TCK tests for this feature. Note that the feature is scheduled for next CDI version (beyond 4.0). The test should probably use a custom scope with custom context as all impls can (in theory) have multiple contexts per given built-in scope making assertions on that unreliable.

manovotn avatar Jul 18 '22 09:07 manovotn

Also we probably should have 2 variants of the same test -- one for context registered through BCExtension, and another one for context registered through PExtension.

Ladicek avatar Jul 18 '22 09:07 Ladicek

I think the BCE variant would be enough.

With PE, you don't even need the method to begin with. Plus BM needs to have references to all instances of contexts anyway since you can already browse them when active. Therefore, so long as registration works (for which there should be different test already), you will see it there.

manovotn avatar Jul 18 '22 10:07 manovotn