Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 358253

Summary:

Evaluate using Jetty 8 for cluster tests

Product:

[RT] RAP

Reporter:

Rüdiger Herrmann <ruediger.herrmann>

Component:

RWT

Assignee:

Project Inbox <rap-inbox>

Status:

CLOSED FIXED

QA Contact:

Severity:

enhancement

Priority:

CC:

fwaibel

Version:

unspecified

Target Milestone:

1.5 M3

Hardware:

All

OS:

All

Whiteboard:

Bug Depends on:

358251

Bug Blocks:

341761

Attachments:

Description	Flags
Changes to update to Jetty 8	none

Description Rüdiger Herrmann

2011-09-20 10:33:11 EDT

The RWT cluster/integration tests are currently running with Jetty 7.4.
For these reasons we should investigate how to move to Jetty 8:
* keep up with the general development at Jetty
* benefit from improvements with Jettys OSGi integration
* get rid of dependencies to servlet 2.x in the tests (Jetty 7.x uses servlet 2.x) that prevent the tests from running in OSGi

Comment 1 Rüdiger Herrmann

2011-09-20 10:43:39 EDT

Created attachment 203686 [details]
Changes to update to Jetty 8

Noteworthy changes:
* Import-Package version were changed to [8.0.0,9.00).
* Calls to ServletContextHandler.addFilter() had to be adjusted because of the changed signature.
For now, the only remaining issue seems to be bug 358251 (a former deprecated method was changed to throw an exception and hence the tests won't pass)

Comment 2 Rüdiger Herrmann

2011-09-27 05:29:18 EDT

Changed the code to work around bug 358251. On server startup, a servlet filter is added to track all sessions that are handled by the server.
Changes are in CVS HEAD.

Comment 3 Rüdiger Herrmann

2011-10-05 13:49:25 EDT

The cluster test fixture now requires/uses Jetty 8.x. The necessary changes cause by changed API in Jetty were made. Dependencies in the bundle manifest were updated. A positive side effect is that the cluster test fixture now only depends on javax.servlet 3.0 (not v2.x and 3.x anymore)
Changes are in CVS HEAD