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/rm37326.docent.yaml src/hg/utils/docent/tests/regress/rm37326.docent.yaml new file mode 100644 index 00000000000..eeae69aaf17 --- /dev/null +++ src/hg/utils/docent/tests/regress/rm37326.docent.yaml @@ -0,0 +1,34 @@ +# #37326 -- the CRISPR tracks showed no mouseOver text at all. +# +# The fix is on genome-test and the assertion is that a tooltip really comes up over a +# CRISPR guide and carries the guide's scores, rather than coming up empty. +# +# The tooltip text is the bigBed's own _mouseOver field (trackDb: mouseOverField +# _mouseOver), which reads "MIT Spec. Score: 62, Doench 2016: 92%, Moreno-Mateos: 79%" -- +# not the scoreLabel ("MIT Guide Specificity Score") that trackDb shows on the details +# page. 46 of the 117 guides in this window carry "Sequence is not unique in genome" +# instead, so a guide with real scores has to be picked deliberately. +# +# Hover by TITLE, not by position. An earlier version used +# `mouseover: {track: crispr10KTargets, at: chr17:43044494}`, which raised no tooltip: +# `at:` sets x from the coordinate but forces y to the MIDDLE of the row, and in pack mode +# this track is 324px of stacked guides, so the cursor landed between them. `title:` +# matches the item's own map box, lands on its row wherever that row is, and makes docent +# wait for THAT item's tooltip rather than settling for a neighbour's. +# +# The guide named here is chr17:43044497-43044520 and its mouseOver string is unique in +# this window, so a match cannot be the wrong item. If the CRISPR data is rebuilt and its +# scores change, this fails with the tooltip it did find, which is a readable break. +target: genome-test +db: hg38 +position: chr17:43044295-43045295 +reset: true +fast: true +steps: + - go: chr17:43044295-43045295 + - hide: all + - track: {crispr10KTargets: pack} + - expect: {rows: [crispr10KTargets], noText: "Warning/Error"} + + - mouseover: {track: crispr10KTargets, title: "MIT Spec. Score: 62, Doench 2016: 92%"} + - expect: {tip: "MIT Spec. Score: 62, Doench 2016: 92%, Moreno-Mateos: 79%"}