{"action":"closed","issue":{"url":"https://api.github.com/repos/cbeust/testng-eclipse/issues/307","repository_url":"https://api.github.com/repos/cbeust/testng-eclipse","labels_url":"https://api.github.com/repos/cbeust/testng-eclipse/issues/307/labels{/name}","comments_url":"https://api.github.com/repos/cbeust/testng-eclipse/issues/307/comments","events_url":"https://api.github.com/repos/cbeust/testng-eclipse/issues/307/events","html_url":"https://github.com/cbeust/testng-eclipse/issues/307","id":199346021,"number":307,"title":"TestNG eclipse plugin doesn't show proper exception if @BeforeMethod (using ITestResult as parameter) fails","user":{"login":"jimpatel90","id":8328041,"avatar_url":"https://avatars.githubusercontent.com/u/8328041?v=3","gravatar_id":"","url":"https://api.github.com/users/jimpatel90","html_url":"https://github.com/jimpatel90","followers_url":"https://api.github.com/users/jimpatel90/followers","following_url":"https://api.github.com/users/jimpatel90/following{/other_user}","gists_url":"https://api.github.com/users/jimpatel90/gists{/gist_id}","starred_url":"https://api.github.com/users/jimpatel90/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jimpatel90/subscriptions","organizations_url":"https://api.github.com/users/jimpatel90/orgs","repos_url":"https://api.github.com/users/jimpatel90/repos","events_url":"https://api.github.com/users/jimpatel90/events{/privacy}","received_events_url":"https://api.github.com/users/jimpatel90/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":8,"created_at":"2017-01-07T06:39:15Z","updated_at":"2017-01-11T23:53:48Z","closed_at":"2017-01-11T23:53:48Z","body":"### Plugin Version\r\n6.10.0.201612030230\r\n\r\n### Steps to reproduce:\r\nTry to run below class using eclipse (TestNG eclipse plugin) and observe report generated in eclipse.\r\n\r\n```\r\nimport org.testng.Assert;\r\nimport org.testng.ITestResult;\r\nimport org.testng.annotations.BeforeMethod;\r\nimport org.testng.annotations.Test;\r\n\r\npublic class TestPlugin {\r\n\r\n\t@BeforeMethod\r\n\tpublic void setUp(ITestResult result) {\r\n\t\tAssert.fail();\r\n\t}\r\n\r\n\t@Test\r\n\tpublic void testPlugin() {\r\n\t\t// To some tests here\r\n\t}\r\n}\r\n\r\n```\r\n\r\n### Expected behaviour\r\nIn result, setUp should show as failed with correct exception(In this case, java.lang.AssertionError)\r\n\r\n### Actual behaviour\r\nTest is shown as fail(instead of setUp) without giving actual exception. Also, it doesn't generate other reports (HTML and JUnit XML)\r\nIt gives following exception:\r\n\r\n```\r\njava.lang.RuntimeException\r\n\tat org.testng.internal.TestResult.toString(TestResult.java:251)\r\n\tat org.testng.internal.TestResult.toString(TestResult.java:236)\r\n\tat org.testng.reporters.TestHTMLReporter.generateTable(TestHTMLReporter.java:118)\r\n\tat org.testng.reporters.TestHTMLReporter.generateLog(TestHTMLReporter.java:303)\r\n\tat org.testng.reporters.TestHTMLReporter.onFinish(TestHTMLReporter.java:44)\r\n\tat org.testng.TestRunner.fireEvent(TestRunner.java:1200)\r\n\tat org.testng.TestRunner.afterRun(TestRunner.java:991)\r\n\tat org.testng.TestRunner.run(TestRunner.java:604)\r\n\tat org.testng.SuiteRunner.runTest(SuiteRunner.java:366)\r\n\tat org.testng.SuiteRunner.runSequentially(SuiteRunner.java:361)\r\n\tat org.testng.SuiteRunner.privateRun(SuiteRunner.java:319)\r\n\tat org.testng.SuiteRunner.run(SuiteRunner.java:268)\r\n\tat org.testng.SuiteRunnerWorker.runSuite(SuiteRunnerWorker.java:52)\r\n\tat org.testng.SuiteRunnerWorker.run(SuiteRunnerWorker.java:86)\r\n\tat org.testng.TestNG.runSuitesSequentially(TestNG.java:1264)\r\n\tat org.testng.TestNG.runSuitesLocally(TestNG.java:1189)\r\n\tat org.testng.TestNG.runSuites(TestNG.java:1104)\r\n\tat org.testng.TestNG.run(TestNG.java:1076)\r\n\tat org.testng.remote.AbstractRemoteTestNG.run(AbstractRemoteTestNG.java:132)\r\n\tat org.testng.remote.RemoteTestNG.initAndRun(RemoteTestNG.java:236)\r\n\tat org.testng.remote.RemoteTestNG.main(RemoteTestNG.java:81)\r\n\r\n```\r\n\r\nNote: If I remove ITestResult result from setUp, it works fine(gives proper exception).\r\n\r\n### Operating System\r\n Windows\r\n"}}
{"actor":{"display_login":"missedone"}}