NPE on com.mycila.testing.testng.MycilaTestNGTest.shutdown

oghenez / mycila

Automatically exported from code.google.com/p/mycila

0 stars 0 forks source link

NPE on com.mycila.testing.testng.MycilaTestNGTest.shutdown #24

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago

MycilaTestNGTest throws an exception on the shutdown() method if you have
tests classes without any method as part of the suite.

It seems that the @BeforeClass on the prepareTestInstance() is not executed
if the class has no tests to run for the suite, which will make the
testNotifier to not be initialized.

The @AfterSuite is executed regardless, making the shutdown() method throw
a NPE since testNotifier is null.

Original issue reported on code.google.com by daniel.m...@gmail.com on 4 Mar 2010 at 6:02

GoogleCodeExporter commented 8 years ago

Hi,

I will have time to check this next week or next week-end. In the meanwhile, if 
you
have time to checkout / suggest a patch to apply it will be very nice :-)

Thank you,

Mat'

Original comment by mathieu....@gmail.com on 5 Mar 2010 at 1:47

GoogleCodeExporter commented 8 years ago

Original comment by mathieu....@gmail.com on 7 Mar 2010 at 11:55

Changed state: Accepted
Added labels: Component-mycila-testing, Milestone-Release2.5, Priority-Medium, Type-Defect

GoogleCodeExporter commented 8 years ago

Original comment by mathieu....@gmail.com on 7 Mar 2010 at 11:56

Added labels: Milestone-Release2.4
Removed labels: Milestone-Release2.5

GoogleCodeExporter commented 8 years ago

version 2.5 deployed and also includes junit and testng upgrade

Original comment by mathieu....@gmail.com on 8 Mar 2010 at 12:16

Changed state: Deployed