UnitTestFrameworkPkg/ResultReportLib: Remove invalid index string indicator

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2535

The UNIT_TEST_STATUS and FAILURE_TYPE have used 0 as status, so use 0 as
unknown is confused, remove it from array enumeration but keep it
location in the array.

Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Sean Brogan <sean.brogan@microsoft.com>
Cc: Bret Barkelew <Bret.Barkelew@microsoft.com>
Signed-off-by: Guomin Jiang <guomin.jiang@intel.com>
Reviewed-by: Shenglei Zhang <shenglei.zhang@intel.com>
Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
Reviewed-by: Bret Barkelew <bret.barkelew@microsoft.com>

diff --git a/UnitTestFrameworkPkg/Library/UnitTestResultReportLib/UnitTestResultReportLib.c b/UnitTestFrameworkPkg/Library/UnitTestResultReportLib/UnitTestResultReportLib.c
index 687a04f55dae3af2ea99e228209b56d73de02b8c..eba68e330c764f9f1063be1977505c438a3f2d7e 100644 (file)
--- a/UnitTestFrameworkPkg/Library/UnitTestResultReportLib/UnitTestResultReportLib.c
+++ b/UnitTestFrameworkPkg/Library/UnitTestResultReportLib/UnitTestResultReportLib.c
@@ -65,7 +65,7 @@ GetStringForUnitTestStatus (
 {\r
   UINTN  Index;\r
 \r
-  for (Index = 0; Index < ARRAY_SIZE (mStatusStrings); Index++) {\r
+  for (Index = 0; Index < ARRAY_SIZE (mStatusStrings) - 1; Index++) {\r
     if (mStatusStrings[Index].Status == Status) {\r
       //\r
       // Return string from matching entry\r
@@ -87,7 +87,7 @@ GetStringForFailureType (
 {\r
   UINTN  Index;\r
 \r
-  for (Index = 0; Index < ARRAY_SIZE (mFailureTypeStrings); Index++) {\r
+  for (Index = 0; Index < ARRAY_SIZE (mFailureTypeStrings) - 1; Index++) {\r
     if (mFailureTypeStrings[Index].Type == Failure) {\r
       //\r
       // Return string from matching entry\r