Jetty

Implement integration tests for Deployment Manager JMX features

Details

  • Type: Improvement Improvement
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 7.4.0
  • Fix Version/s: 7.4.2
  • Component/s: None
  • Labels:
    None
  • Number of attachments :
    1

Description

This enhancement implement integration tests to verify that Deployment Manager's JMX integration is working correctly.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Michael Gorovoy added a comment -

Implementation of the DeployMgrJmxTest

Show
Michael Gorovoy added a comment - Implementation of the DeployMgrJmxTest
Hide
Permalink
Michael Gorovoy added a comment -

Jesse, please review the patch when you get a chance.

Show
Michael Gorovoy added a comment - Jesse, please review the patch when you get a chance.
Hide
Permalink
Michael Gorovoy added a comment -

Minor comment edit.

Show
Michael Gorovoy added a comment - Minor comment edit.
Hide
Permalink
Jesse McConnell added a comment -

pls merge with the JETTY-1353 test, will close that issue as both this patch and that one are now applied

Show
Jesse McConnell added a comment - pls merge with the JETTY-1353 test, will close that issue as both this patch and that one are now applied
Hide
Permalink
Michael Gorovoy added a comment - - edited

Potentially it is better to keep the two tests separate, because the JmxTest harness expects that the application is always remains in its original deployed state. Perhaps we could rename the Deployment Manager test to something generic, and use it for JMX tests that require that Jetty is restarted before each test runs.

Show
Michael Gorovoy added a comment - - edited Potentially it is better to keep the two tests separate, because the JmxTest harness expects that the application is always remains in its original deployed state. Perhaps we could rename the Deployment Manager test to something generic, and use it for JMX tests that require that Jetty is restarted before each test runs.
Hide
Permalink
Michael Gorovoy added a comment -

Done.

Show
Michael Gorovoy added a comment - Done.

People

  • Assignee:
    Michael Gorovoy
    Reporter:
    Michael Gorovoy
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: