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

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

                Author: ASF GitHub Bot
            Created on: 21/Nov/24 13:47
            Start Date: 21/Nov/24 13:47
    Worklog Time Spent: 10m 
      Work Description: clebertsuconic commented on code in PR #5363:
URL: https://github.com/apache/activemq-artemis/pull/5363#discussion_r1852123640


##########
tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/server/AutoCreateExpiryResourcesTest.java:
##########
@@ -59,6 +59,7 @@ public void setUp() throws Exception {
 
       // set common address settings needed for all tests; make sure to use 
getMatch instead of addMatch in invidual tests or these will be overwritten
       server.getAddressSettingsRepository().addMatch("#", new 
AddressSettings().setAutoCreateExpiryResources(true).setExpiryAddress(expiryAddress).setExpiryDelay(EXPIRY_DELAY));
+      
server.getAddressSettingsRepository().addMatch(String.valueOf(expiryAddress), 
new AddressSettings().setExpiryDelay(-1L));

Review Comment:
   There are other tests in this test class that will use this:
   
   
https://github.com/apache/activemq-artemis/blob/14c564a48144b0e052c6e0eeee64ba0f94744335/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/server/AutoCreateExpiryResourcesTest.java#L136-L138
   
   
   I will use a prefix rather than everything.





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

    Worklog Id:     (was: 944853)
    Time Spent: 1h 10m  (was: 1h)

> Fix AutoCreateExpiryResourcesTest and document expiry of expiry situation
> -------------------------------------------------------------------------
>
>                 Key: ARTEMIS-5170
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-5170
>             Project: ActiveMQ Artemis
>          Issue Type: Task
>            Reporter: Clebert Suconic
>            Assignee: Clebert Suconic
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 2.39.0
>
>          Time Spent: 1h 10m
>  Remaining Estimate: 0h
>




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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
For further information, visit: https://activemq.apache.org/contact


Reply via email to