Legacy: Automated.c disabled tests are shown as active in cunit xml test list

Originally: https://sourceforge.net/p/cunit/bugs/74/

Trying to inactivate some tests I realized that they still were displayed as "active" in xml output listing, despite correctly removed from run batch. IMHO the issue is that pSuite->fActive is used instead of pTest->fActive on Automated.c L708. Not sure if this was already filed somewhere.

I believe this bug still exists and is in:

      while (NULL != pTest) {
        assert(NULL != pTest->pName);
        fprintf(pTestListFile,
                "        <TEST_CASE_DEFINITION> \n"
                "          <TEST_CASE_NAME> %s </TEST_CASE_NAME> \n"
                "          <TEST_ACTIVE_VALUE> %s </TEST_ACTIVE_VALUE> \n"
                "        </TEST_CASE_DEFINITION> \n",
                pTest->pName,
                (CU_FALSE != pSuite->fActive) ? _("Yes") : _("No"));
        pTest = pTest->pNext;
      }

Where the Yes/No final part should be pTest->fActive

This bug does not apply to JUnit tests