e497eccfb431be6af38d22c5bc2bb0e745c89b2d braney Sat Sep 5 09:22:17 2026 -0700 docent: eight more regression scripts, one per fixed bug rm35580 "Show placed on its chromosome" errored on the window rm36061 DECIPHER SNVs details missing after a quickLift rm36335 a quickLifted GIAB item click printed a garbled "can't port" rm36340 a search-result click carried the source coordinate rm37491 hgConvert named subtracks the user never showed as not liftable rm37562 a stale lastPosition banner on the lifted search-results page rm37615 a quickLifted bigBed tooltip reported pre-lift coordinates rm37785 a squishyPack track stole a neighbouring track's center label Each asserts the behavior its ticket says is correct, on genome-test, and names in noText:/noUrl: the exact string the buggy build produced rather than a generic "Error". Where the last page has no track image the noText: is paired with a positive text: or url: check, because a bare noText: passes just as happily on a blank page or an early error. Four things found writing these, all of them recorded in the scripts: A ticket's stated position and its stated item can disagree. #36061 says to go to chr7:156,982,676-156,996,015 and click item 517898; that item is at chr7:155,803,191, and decipherSnvs draws nothing at all in the ticket's window. hideKids cannot express "the user deselected this subtrack". It writes a =hide cart variable, and isSubtrackVisible() (hg/lib/trackHub.c) begins with overrideComposite = (NULL != cartOptionalString(cart, tdb->track)) and then forces enabled = TRUE, so the sibling counts as visible to hgConvert and lands back in the "failed to lift" warning. A first rm37491 failed on exactly that and looked like the bug being back. A deselected subtrack is _sel=0 with no visibility variable, which is what hgTrackUi's checkbox writes and what the goto: in that script sets. #37615's fix is not the one the plan for this suite described. The first commit appended a note saying the coordinates came from the source assembly; the second replaced it with the real fix, which puts the TARGET coordinates in the tooltip. A test for the note would fail. decipherSnvs (SNVs) and decipher (CNVs) are two different members of the superTrack decipherContainer, and the row that turns up when the SNVs track draws nothing is the CNVs one, which reads like the same track under a shorter id. refs #38252 diff --git src/hg/utils/docent/tests/regress/rm35580.docent.yaml src/hg/utils/docent/tests/regress/rm35580.docent.yaml new file mode 100644 index 00000000000..293b3c98dc6 --- /dev/null +++ src/hg/utils/docent/tests/regress/rm35580.docent.yaml @@ -0,0 +1,46 @@ +# #35580 -- "Show placed on its chromosome" errored instead of drawing the view. +# +# The reporter was at chr16_KI270853v1_alt:744117-744278, clicked the GRC Patches item, +# and on the hgc page clicked "Show chr16_KI270853v1_alt placed on its chromosome". That +# link puts hgTracks in singleAltHaplo mode, and it came back with +# +# unexpected current window 55471440, expected first window 55471712 +# +# The first attempt at a fix had to be reverted on the v487 branch (#36311, a build patch) +# because it broke the UCSC Unusual Regions track in multi-region mode. The fix that +# actually shipped is in v488, so this asserts the behavior of that one, not the reverted +# one. +# +# The session is kept. Gerardo/RM_35326_bug is on genome-test, and what it contributes is +# hard to build in steps: a position on an alt contig, with the GRC Patches track showing +# and the four recount3 tracks on -- the ticket's own title asks whether recount3 is +# involved, so the assertion below names two of them and would catch the view coming back +# without them. +# +# The item on altSeqLiftOverPsl is called "chr16", which reads oddly next to the alt-contig +# position but is what hgTracks puts in the map box: the item names the chromosome the alt +# belongs to. +target: genome-test +db: hg38 +reset: true +fast: true +steps: + - loadSession: {user: Gerardo, name: RM_35326_bug} + + # A missing session is not an error -- hgTracks answers 200 with an early-error page and + # every noText check on it passes -- so both halves are needed here. + - expect: {noText: "Could not find session", rows: [altSeqLiftOverPsl]} + + - click: {track: altSeqLiftOverPsl, item: chr16} + - expect: {text: "Show chr16_KI270853v1_alt placed on its chromosome"} + + - click: 'a:has-text("placed on its chromosome")' + + # The row list is what makes this tight. The bug produced an error page with no track + # image at all, so noText: on its own would have passed on it; and asking for the + # recount3 rows checks the tracks that #36311's revert was about are still drawn in + # singleAltHaplo mode. + - expect: + rows: [altSeqLiftOverPsl, recount3_gtex, recount3_tcga] + noText: "unexpected current window" + url: "virtModeType=singleAltHaplo"