[TOMEE-1218] mixed scope-handling in CdiAppContextsService

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.6.0.1
• Fix Version/s: 1.7.0
• Assignee: Romain Manni-Bucau
• Reporter: Gerhard Petracek
• Created: 21/May/14 10:06
• Updated: 02/Jul/14 15:46
• Resolved: 21/May/14 17:14

Description

CdiAppContextsService#destroyRequestContext calls
#removeThreadLocals which also resets the ThreadLocal for the session-context -> scope handling is mixed.

that causes issues once openejb is used for unit-tests and one test-method needs to simulate multiple requests and therefore uses

        this.contextControl.stopContext(RequestScoped.class);
        this.contextControl.startContext(RequestScoped.class);

only with

        this.contextControl.stopContext(SessionScoped.class);
        this.contextControl.stopContext(RequestScoped.class);
        this.contextControl.startContext(RequestScoped.class);
        this.contextControl.startContext(SessionScoped.class);

it's possible to prevent a ContextNotActiveException afterwards.
however, that starts a new session-scope which might be problematic for the test (and looks strange anyway).