[ 
https://issues.apache.org/jira/browse/ARTEMIS-4583?focusedWorklogId=901745&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-901745
 ]

ASF GitHub Bot logged work on ARTEMIS-4583:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 25/Jan/24 16:02
            Start Date: 25/Jan/24 16:02
    Worklog Time Spent: 10m 
      Work Description: clebertsuconic commented on PR #4758:
URL: 
https://github.com/apache/activemq-artemis/pull/4758#issuecomment-1910505858

   Looks good to me... I think we should merge this




Issue Time Tracking
-------------------

    Worklog Id:     (was: 901745)
    Time Spent: 40m  (was: 0.5h)

> remove artemis-server module test-jar, do some related cleanup
> --------------------------------------------------------------
>
>                 Key: ARTEMIS-4583
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-4583
>             Project: ActiveMQ Artemis
>          Issue Type: Task
>          Components: Tests
>            Reporter: Robbie Gemmell
>            Assignee: Robbie Gemmell
>            Priority: Major
>             Fix For: 2.33.0
>
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> The tests/ subtree of the repo has lots of modules that create a chain of 
> test module dependencies back to the test-jar created artemis-server module, 
> mostly in order to get at the ActiveMQTestBase support class and other helper 
> rules etc. This class is overly complex for the tests in the artemis-server 
> module, with most (~2000 lines) of it unused there and instead actually aimed 
> at system tests that are in the integration-tests, smoke-tests, etc etc 
> modules. We can improve (and eventually speed up) the build by simplifying 
> things and breaking such chains, while moving support code into either the 
> artemis-unit-test module (for use by any module in the codebase) that now 
> exists, or the even older artemis-test-support module (for use in any system 
> tests in the tests/ subtree).



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to