diff --git a/EBBR.yaml b/EBBR.yaml index a6fb1ffb82d262a64b0d07088285fdceee64c66f..b202c623c916098d1a9519a2c9fe4c7283dcb4f1 100644 --- a/EBBR.yaml +++ b/EBBR.yaml @@ -1959,23 +1959,6 @@ update: result: KNOWN U-BOOT LIMITATION -- rule: Force simple text input ex protocol warnings as known U-Boot limitation - criteria: - descr: Simple Text Input Ex Protocol Test - device path: No device path - group: ConsoleSupportTest - guid: 6647A0E7-483C-4777-A94B-C8BCA3DFC79C - log: SimpleTextInputExBBTestConformance.c - name: SIMPLE_TEXT_INPUT_EX_PROTOCOL.SetState - SetState() returns - EFI_INVALID_PARAMETER with KeyToggleState being NULL. - result: WARNING - revision: '0x00010000' - set guid: 5B40A8A9-F77A-4134-9FF1-417720B765B4 - sub set: SetStateConformance - test set: SimpleTextInputExProtocolTest - update: - result: KNOWN U-BOOT LIMITATION - - rule: Force misc runtime services update capsule conformance failure as known U-Boot limitation UpdateCapsule implementation in U-Boot up to (at least) v2021.07 does not @@ -2175,16 +2158,10 @@ update: result: KNOWN ACS LIMITATION -############################################################################### -# Known SCT limitations # -############################################################################### - -# We force the following tests result as `KNOWN SCT LIMITATION'. They are -# genuine bugs, which are fixed in a more recent version of the SCT or which -# must ultimately be fixed and which we know about. - -- rule: Force simple text input ex protocol warning as known SCT limitation - Unsupported is a valid return value for SetState(). +- rule: Force simple text input ex protocol warning as known ACS limitation + Unsupported is a valid return value for SetState() and this is a limitation + in the SCT. + https://bugzilla.tianocore.org/show_bug.cgi?id=3390 criteria: descr: Simple Text Input Ex Protocol Test device path: No device path @@ -2199,7 +2176,27 @@ sub set: ReadKeyStrokeExFunctionAuto test set: SimpleTextInputExProtocolTest update: - result: KNOWN SCT LIMITATION + result: KNOWN ACS LIMITATION + +- rule: Force simple text input ex protocol warning (2) as known ACS limitation + Unsupported is a valid return value for SetState() and this is a limitation + in the SCT. + https://bugzilla.tianocore.org/show_bug.cgi?id=3390 + criteria: + descr: Simple Text Input Ex Protocol Test + device path: No device path + group: ConsoleSupportTest + guid: 6647A0E7-483C-4777-A94B-C8BCA3DFC79C + log: Status - Unsupported + name: SIMPLE_TEXT_INPUT_EX_PROTOCOL.SetState - SetState() returns + EFI_INVALID_PARAMETER with KeyToggleState being NULL. + result: WARNING + revision: '0x00010000' + set guid: 5B40A8A9-F77A-4134-9FF1-417720B765B4 + sub set: SetStateConformance + test set: SimpleTextInputExProtocolTest + update: + result: KNOWN ACS LIMITATION ############################################################################### # Comments # diff --git a/README.md b/README.md index f883a26b6493e98b2d73de287d50ec34d99091c3..ecac247d29cb919bf199860c3f7a0d8a468bd165 100644 --- a/README.md +++ b/README.md @@ -288,10 +288,6 @@ ones: `KNOWN ACS LIMITATION` Genuine bugs, which are fixed in a more recent version of the ACS or which must ultimately be fixed and which we know about. - - `KNOWN SCT LIMITATION` Genuine bugs, which are fixed in a more recent - version of the SCT or which must ultimately be fixed - and which we know about. ------------------------------------------------------------------------------- Some of the rules just add a `comments` field with some help text. diff --git a/parser.py b/parser.py index ff6fb7616c280cba6d9261e487b6eb3082d39fd0..56019025b0418a8669f1f6f584e6236f8e75b53a 100755 --- a/parser.py +++ b/parser.py @@ -175,10 +175,10 @@ def ekl_parser(file): n += 1 except Exception: logging.error(f"Line {i+1}: {split_line}") - logging.error("your log may be corrupted") + logging.error(f"{red}your log may be corrupted{normal}") sys.exit(1) else: - logging.error(f"Unparsed line {i} `{line}'") + logging.error(f"{red}Unparsed line{normal} {i} `{line}'") if s: logging.debug(f'{s} skipped test set(s)') @@ -193,7 +193,7 @@ def seq_parser(file): magic = 7 # a test in a seq file is 7 lines, if not mod7, something wrong.. if len(lines) % magic != 0: - logging.error("seqfile cut short, should be mod7") + logging.error(f"{red}seqfile cut short{normal}, should be mod7") sys.exit(1) # the utf-16 char makes this looping a bit harder, so we use x+(i) where i # is next 0-6th @@ -284,13 +284,14 @@ def sanitize_yaml(conf): conf[i] = r if r['rule'] in rules: - logging.warning(f"Duplicate rule {i} `{r['rule']}'") + logging.warning( + f"{yellow}Duplicate rule{normal} {i} `{r['rule']}'") rules.add(r['rule']) if 'criteria' not in r or not type(r['criteria']) is dict or \ 'update' not in r or not type(r['update']) is dict: - logging.error(f"Bad rule {i} `{r}'") + logging.error(f"{red}Bad rule{normal} {i} `{r}'") raise Exception() @@ -439,6 +440,10 @@ def uniq(cross_check): h[i]['count'] += 1 + # Transform all counts to string. + for k in h.keys(): + h[k]['count'] = str(h[k]['count']) + # Transform back to list r = list(h.values()) logging.info(f"{len(r)} unique entries") @@ -676,9 +681,11 @@ def ident_seq(seq_file, seq_db): # Try to identify the seq file if h in known_seqs: - logging.info(f"""Identified `{seq_file}' as "{known_seqs[h]}".""") + logging.info( + f"""{green}Identified{normal} `{seq_file}'""" + f""" as "{known_seqs[h]}".""") else: - logging.debug(f"Could not identify `{seq_file}'...") + logging.warning(f"{yellow}Could not identify{normal} `{seq_file}'...") # Read the .ekl log file and the .seq file and combine them into a single