e040e51d90b1b7af95a56c14199e4238e6912b48 braney Sat Sep 5 09:21:54 2026 -0700 docent: the four regression scripts that were failing, now green These were written yesterday, failed, and were left out of the last commit so the suite stayed green. None of the four was a browser bug. Each was the test being wrong, and each was wrong in a way worth writing into the script. rm36387 had lost the bug. The term the ticket quotes carries a ZERO-WIDTH SPACE (U+200B) between "):" and "n.46G>A", which is how it arrived from a clinical report; hgTracks passed it to encodeURIComponent unstripped, hgSuggest put it in a LIKE, and MySQL died on a collation mismatch. Without that character there is no bug at all and the test passes on any build ever shipped, so the term is now written as a ​ escape. The fixed behavior is also not what the test assumed: the position box submits to hgSearch, which reports no results for an HGVS term. The old script waited for #imgTbl and timed out. rm37805 sat at a position where the track cannot draw. gnomadGenomesVariantsV3_1_1 carries filterValuesDefault.annot pLoF,missense,synonymous, and all 258 v3.1.1 variants in chr17:43044295-43045295 are annotated "other", so the row does not exist. Moved to chr17:43091000-43092000, inside BRCA1 exon 11, which holds 80 missense, 13 pLoF and 29 synonymous. gnomadVariants is also a superTrack, so hideKids is needed to keep its coverage siblings off. rm37326 hovered into a gap. at: sets x from the coordinate but forces y to the MIDDLE of the row, and in pack mode that track is 324px of stacked guides. It now hovers by title:, which lands on the item's own row and waits for that item's tooltip. rm37553 asserted label text that does not exist. "GFF example" is not a label: the file says `track name=GFF example description=` with the name unquoted, so hgCustom takes the name as "GFF". Nor is the description on the page, since hgTracks puts a longLabel in the control's title attribute, which innerText does not see. All 46 tracks were loading the whole time. refs #38252 diff --git src/hg/utils/docent/tests/regress/rm37805.docent.yaml src/hg/utils/docent/tests/regress/rm37805.docent.yaml new file mode 100644 index 00000000000..d0e69020663 --- /dev/null +++ src/hg/utils/docent/tests/regress/rm37805.docent.yaml @@ -0,0 +1,42 @@ +# #37805 -- the gnomAD v3.1.1 tooltip carried a stray closing brace. +# +# hg38/gnomad.ra had an extra "}" immediately after ${chromEnd} in the mouseOver pattern +# for gnomadGenomesVariantsV3_1_1, so the tooltip read +# +# Position: chr17:43091449-43091450} (T/G) +# +# instead of ...-43091450 (T/G). Spotted during QA of #37615. +# +# Two things about reaching a drawn row, both of which cost a run: +# +# * gnomadVariants is a superTrack, so it does not pass its setting down. Its other +# members come up at their own trackDb visibility and draw coverage rows next to this +# one. hideKids on the container leaves only the track named after it. +# * this track has filterValuesDefault.annot pLoF,missense,synonymous, so at a position +# with only UTR and downstream variants it draws NOTHING and has no row at all. The +# first version of this test used chr17:43044295-43045295, where all 258 v3.1.1 +# variants are annotated "other", and failed for that reason and not for the bug. +# chr17:43091000-43092000 is inside BRCA1 exon 11 and holds 80 missense, 13 pLoF and +# 29 synonymous variants, so the row is drawn. +# +# The hover names an item by its TITLE, which for this track is the mouseOver text itself. +# That pins one variant rather than whatever sits at the middle of a packed row, and it +# makes docent wait for THAT item's tooltip instead of settling for a neighbour's. +# +# Both halves of the assertion are needed. tip: is what only holds when the pattern +# rendered correctly -- the buggy build produced "43091450} (T/G)" and would fail it. +# noText: catches a brace anywhere else the pattern might leak one; the window is narrow +# and only this track is showing, so nothing else on the page produces one. +target: genome-test +db: hg38 +position: chr17:43091000-43092000 +reset: true +fast: true +steps: + - go: chr17:43091000-43092000 + - hide: all + - track: {gnomadVariants: hideKids, gnomadGenomesVariantsV3_1_1: pack} + - expect: {rows: [gnomadGenomesVariantsV3_1_1], noText: "Warning/Error"} + + - mouseover: {track: gnomadGenomesVariantsV3_1_1, title: "rs80357218"} + - expect: {tip: "chr17:43091449-43091450 (T/G)", noText: "}"}