Remove useless config method in testng-failed.xml

Hi Beust,


Currently, when we run a testng suite like this:

    @BeforeClass
    public void beforeClass(){
        System.err.println("Before class");
    }
    
    @BeforeSuite
    public void beforeSuite(){
        System.err.println("beforeSuite");
    }
    
    @BeforeMethod
    public void beforeMethod(){
        System.err.println("beforeMethod");
    }
    @Test
    public void test1(){
        int a = 1/0;
    }

The test case will fail and final testng-failed.xml looks like this:
'''
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE suite SYSTEM "http://testng.org/testng-1.0.dtd">
<suite name="Failed suite [Failed suite [xx]]">
  <test name="ggsdfd(failed)(failed)" preserve-order="true">
    <classes>
      <class name="MockTest">
        <methods>
          <include name="test1"/>
          <include name="beforeClass"/>
          <include name="beforeMethod"/>
          <include name="beforeSuite"/>
        </methods>
      </class> <!-- MockTest -->
    </classes>
  </test> <!-- ggsdfd(failed)(failed) -->
</suite> <!-- Failed suite [Failed suite [xx]] -->
'''

The config method was added in this file too,I dont know why we need this.

My patch is reduce those config methods.

Look forward for your reply.

Br,
Tim