Fixed the bug with the annotation and few fixme's

[sksekar]

Prior this change, the "annotation" printed for each group was the same. With this change, the annotation is derived from the group name (see snapshot below). This also fixes few "fixme's"

[root@dut /tmp]# time ./pci_lmt.par -o json -c lmt.cfg.org auto |  jq -rc '[.test_info.annotation, .device_info.bdf, .lane, .step, .error_count]' | grep -v ",0]"
["CPU Tx -> Retimer Rx (Timing left)","0000:00:01.1",0,6,63]
["CPU Tx -> Retimer Rx (Timing left)","0000:00:01.1",1,6,54]
:
["Retimer Tx -> PEX Rx (Timing left)","0000:01:00.0",3,16,1]
["Retimer Tx -> PEX Rx (Timing left)","0000:e1:00.0",7,16,1]
:
["PEX Tx -> ASIC Rx (Timing right)","0000:09:00.0",0,15,1]
["PEX Tx -> ASIC Rx (Timing right)","0000:0a:00.0",3,15,2]

[mimir-d]

i suggest using proper types and the type checker instead of labeling methods with types (here x_as_str())

[sksekar]

Thanks for the approval. Will address in the next PR (need some time to learn more about the enum types and how it can be used here)

[mimir-d]

that too, yes; the string "TIMING" should be an enum type