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_<n>_ 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"