testng-team/testng

ListenerComparator doesn't work

Closed this issue · 4 comments

SergiyBelevitnevSFLY commented

TestNG Version

7.10.1

Expected behavior

ListenerComparator should order listeners

Actual behavior

ListenerComparator doesn't order listeners

Test case sample

<properties>
  <property>
    <name>listenercomparator</name>
    <value>base.listeners.order.AnnotationBackedListenerComparator</value>
  </property>
  <property>
    <name>listener</name>
    <value>base.listeners.ReportPortalListenerOverwritten, 
      base.listeners.AttachFilesListener, base.listeners.TestRailListener, base.listeners.MixPanelListener
    </value>
  </property>
</properties>

Contribution guidelines

I tried many combinations but had no success. Listeners still are starting without any order

krmahadevan commented

@SergiyBelevitnevSFLY - Thank you so much for filing this issue. This was a miss in the feature implementation. I have raised a PR for fixing this.

  • 🚀1
krmahadevan commented

@SergiyBelevitnevSFLY - This is now fixed and released as part of TestNG 7.10.2.

https://repo1.maven.org/maven2/org/testng/testng/7.10.2/

  • 🎉1
krmahadevan commented

@missedone - I had to release TestNG v7.10.2 to include the fix for this bug.

SergiyBelevitnevSFLY commented

I have checked the code against TestNG v7.10.2 and can confirm that the issue has been resolved. I am grateful for the feature delivery!

  • 👍1