7aba31f14aed7f2620e4746b54f9569a5c93e1ad braney Sat Sep 12 15:07:34 2026 -0700 docent: ten regression tests for quickLift, on hgTracks and on hgc, refs #38252 Fourteen scripts here already lift something -- they are the ones that call `convert: {quicklift: true}` -- so these take the parts of the lift that had no test. Five read the lifted image and five read a details page: rm38032 the target keeps the source's track order. First use of `ordered:`, which was added to expect: for this bug rm38042 a ClinVar CNV running past the chains quickLift loads is clipped rather than dropped, so the spanned-item merge still has it rm37646 a lolly composite subtrack lifts, and its map boxes still carry its own track name -- the string the stale pop pointer clobbered rm36048 the spanned-item merge still works on a lifted DECIPHER track rm37815 "Hide all default tracks on the target" hides all six of hs1's own tracks and keeps the lifted one rm36059 a lifted GENCODE Versions item gives the real details page, in destination coordinates, with no "Can't start query" rm36370 a lifted knownGene click renders GeneReviews and Methods, the two sections the ticket says were missing rm36125 a lifted RefSeq item's page, and its Predicted Protein link returning SHH's peptide instead of a blank page rm36942 the Alignment Differences description, reached from a difference item: the four colors and the figure rm38146 the same page with a GenArk assembly as the SOURCE, down to the base alignment that reads query bases out of a two bit file All ten are assertion-only: every fix shipped long ago. make test is 57 of 57 green in 10m18s, up from 7m34s -- each script costs a convert, about 17 seconds, because no URL builds a quickLift hub. README.txt gains what the batch cost. A lifted row and map box carry a per-run hub__ prefix, so rows: matches by suffix and a has: selector must use a substring. Never assert a count an otto reload can move: rm38042 and rm36048 both read the merged-item box and leave its count (45 for ClinVar today) to a comment. And a details page prints the track's own labels whether or not it worked, so each hgc assertion names something only the fixed page has. Three candidates were rejected: #38033's "(N items could not be lifted)" label is only in the drawn image and the page JSON, where no expect: check reaches it; #37970 needs a broadPeak track and hg38 has none; #37974's center-label drag is pixels. diff --git src/hg/utils/docent/tests/regress/rm37815.docent.yaml src/hg/utils/docent/tests/regress/rm37815.docent.yaml new file mode 100644 index 00000000000..276a36748e0 --- /dev/null +++ src/hg/utils/docent/tests/regress/rm37815.docent.yaml @@ -0,0 +1,43 @@ +# #37815 -- a QuickLift landed the user on a target assembly showing its own default +# tracks as well as the lifted ones, so the view they had built on the source was buried. +# f80b3826a92 adds a "Hide all default tracks on the target" checkbox to the Convert page, +# on by default when QuickLift is enabled. +# +# The commit is in two halves and both are needed for this to pass. hgConvert appends +# hideTracks=on to the QuickLift link when the box is checked; hgTracks then has to leave +# the QuickLifted tracks visible while it hides the target's defaults, which is the change +# in hg/hgTracks/hgTracks.c. Without the first half hs1's own tracks are drawn as well; +# without the second, hideTracks=on takes the lifted track with it and the target comes up +# empty. `exact: true` fails on either. +# +# hs1 opens with six tracks of its own at this position, measured on genome-test on +# 2026-09-12: catLiftOffGenesV1, ncbiRefSeq, ncbiRefSeqCurated, ncbiRefSeqPredicted, +# cactus11way and multiz11way. So the row list below is a claim about all six being gone, +# not a list that happens to match an empty target. +# +# Docent's `convert:` re-checks the box after the Assembly menu reloads, because the menu +# reload reverts it -- the reason `hideDefaults:` is a verb option rather than a click. +proof: + - "assertion-only 2026-09-12 -- written from the ticket and from f80b3826a92, after the fix shipped" + +target: genome-test +db: hg38 +position: chr7:155799529-155812871 +reset: true +fast: true +steps: + - go: chr7:155799529-155812871 + - hide: all + - track: {mane: pack} + - expect: {rows: [ruler, mane], exact: true} + + - convert: {to: hs1, quicklift: true, hideDefaults: true} + - open: lift + + # The lifted track, the synthetic Alignment Differences track that hubConnect adds, and + # nothing of hs1's own. + - expect: + rows: [ruler, quickLiftChain, mane] + exact: true + noRows: [catLiftOffGenesV1, ncbiRefSeqCurated, cactus11way, multiz11way] + noText: "Warning/Error"