Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 358253 - Evaluate using Jetty 8 for cluster tests
Summary: Evaluate using Jetty 8 for cluster tests
Status: CLOSED FIXED
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 1.5 M3   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 358251
Blocks: 341761
  Show dependency tree
 
Reported: 2011-09-20 10:33 EDT by Rüdiger Herrmann CLA
Modified: 2011-10-05 13:49 EDT (History)
1 user (show)

See Also:

Attachments
Changes to update to Jetty 8 (9.92 KB, patch)
2011-09-20 10:43 EDT, Rüdiger Herrmann CLA 		no flags Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Rüdiger Herrmann CLA 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 CLA 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 CLA 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 CLA 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