[ 
https://issues.apache.org/jira/browse/ARTEMIS-3553?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17438895#comment-17438895
 ] 

Robbie Gemmell commented on ARTEMIS-3553:
-----------------------------------------

It's not a duplicate test, it's a specific unit test of the conversion bits 
that were broken, and were not previously being used at all in the modified 
test since its really targeting another area.

I wouldn't add a specific integration test for this. That's precisely how you 
end up with exists now, hundreds of slow and often unnecessary integration 
tests, which then often aren't even run in the typical developers build while 
they make changes, or even the typical CI run after, due to taking so long most 
of them can't reasonably be run in such contexts and needing additional 
profiles to pick and choose test subsets.

We will need to agree to disagree here I think.

> Expiry Fields AMQP Large Message are not translated on AmqpToCore conversion
> ----------------------------------------------------------------------------
>
>                 Key: ARTEMIS-3553
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-3553
>             Project: ActiveMQ Artemis
>          Issue Type: Bug
>    Affects Versions: 2.19.0
>            Reporter: Clebert Suconic
>            Assignee: Clebert Suconic
>            Priority: Major
>             Fix For: 2.20.0
>
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to