ActiveMQ
  1. ActiveMQ
  2. AMQ-3623

Add DefaultTestAppender utility abstract class

    Details

    • Type: New Feature New Feature
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 5.5.1
    • Fix Version/s: 5.6.0
    • Component/s: Test Cases
    • Labels:
      None
    • Patch Info:
      Patch Available

      Description

      Couple tests keep an eye on the logging for specific failures, be good to have a common approach to this.

        Issue Links

          Activity

          Stan Lewis created issue -
          Stan Lewis made changes -
          Field Original Value New Value
          Attachment 0001-Add-DefaultTestAppender-and-modify-existing-tests-to.patch [ 12506292 ]
          Hide
          Stan Lewis added a comment -

          Patch adds DefaultTestAppender and modifies the two existing tests to use it.

          Show
          Stan Lewis added a comment - Patch adds DefaultTestAppender and modifies the two existing tests to use it.
          Stan Lewis made changes -
          Link This issue blocks AMQ-3622 [ AMQ-3622 ]
          Gary Tully made changes -
          Assignee Gary Tully [ gtully ]
          Hide
          Gary Tully added a comment -

          improvement applied with thanks.
          note for the future, every new file needs the apache copyright header

          Show
          Gary Tully added a comment - improvement applied with thanks. note for the future, every new file needs the apache copyright header
          Gary Tully made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Fix Version/s 5.6.0 [ 12317974 ]
          Resolution Fixed [ 1 ]
          Hide
          Claudio Corsi added a comment -

          I looked at the patch and I just noticed that the appender is added to the logging system but it is not removed when the test has completed.

          Since the appender references the test case then the test case instance will never be reclaimed until the logging system is removed and that is when the application ends.

          Would you consider adding the code that remove the added appender when the test has completed.

          Show
          Claudio Corsi added a comment - I looked at the patch and I just noticed that the appender is added to the logging system but it is not removed when the test has completed. Since the appender references the test case then the test case instance will never be reclaimed until the logging system is removed and that is when the application ends. Would you consider adding the code that remove the added appender when the test has completed.
          Hide
          Stan Lewis added a comment -

          Yeah, totally agree, sorry I missed that. We get away with it currently because of the way the tests are run but it's best the test cleans that up, will attach a second patch momentarily.

          Show
          Stan Lewis added a comment - Yeah, totally agree, sorry I missed that. We get away with it currently because of the way the tests are run but it's best the test cleans that up, will attach a second patch momentarily.
          Hide
          Stan Lewis added a comment -

          Yeah, totally agree, sorry I missed that. We get away with it currently because of the way the tests are run but it's best the test cleans that up, will attach a second patch momentarily.

          Show
          Stan Lewis added a comment - Yeah, totally agree, sorry I missed that. We get away with it currently because of the way the tests are run but it's best the test cleans that up, will attach a second patch momentarily.
          Hide
          Stan Lewis added a comment -

          Created AMQ-3628 for the fix since this feature's been added.

          Show
          Stan Lewis added a comment - Created AMQ-3628 for the fix since this feature's been added.
          Gavin made changes -
          Link This issue blocks AMQ-3622 [ AMQ-3622 ]
          Gavin made changes -
          Link This issue is depended upon by AMQ-3622 [ AMQ-3622 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          21h 44m 1 Gary Tully 07/Dec/11 16:59

            People

            • Assignee:
              Gary Tully
              Reporter:
              Stan Lewis
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development