From 189f270857627a5db851c63f1f5902b71c74fff7 Mon Sep 17 00:00:00 2001
From: Colin <colin.diesh@gmail.com>
Date: Thu, 16 Jan 2025 05:36:27 -0800
Subject: [PATCH] More tests

---
 .../SequenceFeatureDetails/types.ts           |   9 ++++++
 .../src/LGVSyntenyDisplay/components/util.ts  |   1 -
 .../jbrowse-web/src/tests/LGVSynteny.test.tsx |  26 ++++++++++++++++--
 ...to-synteny-from-feature-details-1-snap.png | Bin 0 -> 1960 bytes
 ...nav-to-synteny-from-right-click-1-snap.png | Bin 0 -> 2810 bytes
 5 files changed, 33 insertions(+), 3 deletions(-)
 create mode 100644 packages/core/BaseFeatureWidget/SequenceFeatureDetails/types.ts
 create mode 100644 products/jbrowse-web/src/tests/__image_snapshots__/lgv-synteny-test-tsx-nav-to-synteny-from-feature-details-1-snap.png
 create mode 100644 products/jbrowse-web/src/tests/__image_snapshots__/lgv-synteny-test-tsx-nav-to-synteny-from-right-click-1-snap.png

diff --git a/packages/core/BaseFeatureWidget/SequenceFeatureDetails/types.ts b/packages/core/BaseFeatureWidget/SequenceFeatureDetails/types.ts
new file mode 100644
index 0000000000..230582a836
--- /dev/null
+++ b/packages/core/BaseFeatureWidget/SequenceFeatureDetails/types.ts
@@ -0,0 +1,9 @@
+import type { SequenceFeatureDetailsModel } from './model'
+import type { SimpleFeatureSerialized } from '../../util'
+import type { SeqState } from '../util'
+
+export interface SequencePanelProps {
+  sequence: SeqState
+  feature: SimpleFeatureSerialized
+  model: SequenceFeatureDetailsModel
+}
diff --git a/plugins/linear-comparative-view/src/LGVSyntenyDisplay/components/util.ts b/plugins/linear-comparative-view/src/LGVSyntenyDisplay/components/util.ts
index 6cd4370ea1..1555e48934 100644
--- a/plugins/linear-comparative-view/src/LGVSyntenyDisplay/components/util.ts
+++ b/plugins/linear-comparative-view/src/LGVSyntenyDisplay/components/util.ts
@@ -117,7 +117,6 @@ export async function navToSynteny({
   const l1 = `${featRef}:${Math.floor(rFeatStart - ws)}-${Math.floor(rFeatEnd + ws)}`
   const m1 = Math.min(rMateStart, rMateEnd)
   const m2 = Math.max(rMateStart, rMateEnd)
-  console.log(m1, m2)
   const l2 = `${mateRef}:${Math.floor(m1 - ws)}-${Math.floor(m2 + ws)}${
     horizontallyFlip ? '[rev]' : ''
   }`
diff --git a/products/jbrowse-web/src/tests/LGVSynteny.test.tsx b/products/jbrowse-web/src/tests/LGVSynteny.test.tsx
index b7e73ff23b..72133863d3 100644
--- a/products/jbrowse-web/src/tests/LGVSynteny.test.tsx
+++ b/products/jbrowse-web/src/tests/LGVSynteny.test.tsx
@@ -20,7 +20,7 @@ beforeEach(() => {
 const delay = { timeout: 5000 }
 const opts = [{}, delay]
 
-test('nav to synteny', async () => {
+test('nav to synteny from right click', async () => {
   const user = userEvent.setup()
   const { session, view, findByTestId, findByText, findAllByTestId } =
     await createView()
@@ -38,6 +38,28 @@ test('nav to synteny', async () => {
     expect(v?.initialized).toBe(true)
     expect(v?.views[0]?.coarseVisibleLocStrings).toBe('ctgA:29,221..34,669')
   }, delay)
-  await new Promise(res => setTimeout(res, 1000))
+  expectCanvasMatch(await findByTestId('synteny_canvas', ...opts))
+}, 60000)
+
+test('nav to synteny from feature details', async () => {
+  const user = userEvent.setup()
+  const { session, view, findByTestId, findByText, findAllByTestId } =
+    await createView()
+
+  await view.navToLocString('ctgA:30,222..33,669')
+  await user.click(await findByTestId(hts('volvox_ins.paf'), ...opts))
+
+  const track = await findAllByTestId('pileup-overlay-strand')
+  fireEvent.mouseMove(track[0]!, { clientX: 200, clientY: 5 })
+  fireEvent.click(track[0]!, { clientX: 200, clientY: 5 })
+  fireEvent.click(
+    await findByText('Launch new linear synteny view on this feature'),
+  )
+  fireEvent.click(await findByText('Submit'))
+  await waitFor(() => {
+    const v = session.views[1] as LinearSyntenyViewModel | undefined
+    expect(v?.initialized).toBe(true)
+    expect(v?.views[0]?.coarseVisibleLocStrings).toBe('ctgA:1..50,001')
+  }, delay)
   expectCanvasMatch(await findByTestId('synteny_canvas', ...opts))
 }, 60000)
diff --git a/products/jbrowse-web/src/tests/__image_snapshots__/lgv-synteny-test-tsx-nav-to-synteny-from-feature-details-1-snap.png b/products/jbrowse-web/src/tests/__image_snapshots__/lgv-synteny-test-tsx-nav-to-synteny-from-feature-details-1-snap.png
new file mode 100644
index 0000000000000000000000000000000000000000..54d157d79e2c0d9449c91fef3fb9bc71c421fff6
GIT binary patch
literal 1960
zcmXX{2~<;88on<i5CbGoD<Hcp2GNru<FJ&)un7UfQX>&D5P<?wBPfwoA%cT-DU4u&
zKvkFqfkFWdAVdNItW1Wj1`0}*<3d>^z!2F5nfp4Pcg}n7-uK`C|L%AH?|at*{k=5_
z`UC)gCe6q58vs;H;d>Nb72eZ>7`gDF8sX>d3AWIyR8o--fEtVD>BdO@arRYv9OK1K
z)z>S>1=0FPG;qRrLLBlB7L{P7aM0iB=yo>I;L4ST<JY1^x1@ysSZBN*u6ndhlhZ`=
z$T}aV-(Xgt*)-YjGC7uVH`^q0!Jg7@UpKJU&lQim0sgt;!XD<d@<@|!AtruD`yLhm
z^_ljZ54&Um<?2Xt@lzrm+#K2`d(2y7iZ{lHcpG9O6|if<gneeAAwpTAc!Nxz&I4^v
z0{qbF4CTiz9&RtvY>QyQ(^!Gs%E9S(WUuPa6H`kq2k$wg<8EdF42TrvQ+g6yE|Teq
zhFzHVWIm8S-!SJ!>uyLKwb)p!7SPL|<)x(4w(Qg}H~Huar7d@&us3aA-(|409pX6L
zwsm+uOek~duIG!lnF1S~@QS{E-9*P6IWd?d)x@JyG0au_0JOp|PnIS30wBD;kh0b4
z-kf}IJUEY(#lXdL(M@Tjx4u2o?JR4HJVuqk>Xom|SpId-M7f&(Qd#<f3Q7}Wrfu7w
zw%e;=7`+;JFm$G1oF(2@t8MLS(eeXj9;aRE3iFgke<cswdvQE6HbtSQ6G(<coLXMb
z-kSFT$a@Pup#D<8N!ZGktXCP6hGcKCpiw+M=bfi0mvvI`fhK5p10-h0$Hg*$8S(}Q
zUtdBid2zk&fuc%>Z`_it3r>mdQ0DV`nXQLbxZ;ieqpehy0w(5$-Df#ow2=81D;%<M
zLtb#{XshDE!4eh3+8lnTxA~X6GR~#nSzPu75-kXIcZ^H>m|O!NsEd|;P)Ny5o$I@C
zpygiE$O!h>Wk`AB>N)xc{R|s5%#a5PWuN>{9{ti<739bT<*1{&K~Oi{ND?(FJv5P2
zGgyWrx?4Bq5_fmJ%u%b9h{1!~%Vu1i)2_QOL=mx-=*gFF+DJ!|BR-ETjf8!lz~*XD
zlkFpQ%x%~QR9c3~)J$)lvksN7gjKA*C9?`tmfuDqSgQbt;+rv{(PHXL^15qa*q|lJ
zR~G>WF;1*8O99E*Fx`ojzW<-Z+waJ`KwxK2L)O(?3rC)Yx|JVX`Ni<s>yCjCowc7q
zJ3LZ$Yz0RYI;A&s?H!XE0-<VfE31>fdYHf4Dd!~eC)z$j2``1wyhXujl^Q~lvU$NB
zS2Ge!scyAzaQrez2e-IaePqE;{L&7BKdA;ol-AsZ|K76ri$~s0X*600FUn-jhmAL9
z{=}9Z`Z;D7mcJ+Mmfel(IW9&_hI)Mv+U=dBa&?DEn^3_I6wGr^h^FQbw^0RiIAM}z
zT&#j*Ge;uu;0P~R%=a@Cz%i<ruV;llwDQg6Dkz!c3zYAG&5k&Eu+!-UnC1^0l5GiN
z#J%~>b<HFFd4>d2Y5jZFmk3+YHjp7=t?ef#m`bIv_OZfZW9MW+g?XiR(U&)fpr(s6
zst9iTc*|l%9(7U|tCs_hZ6<K$_s6>X)i4^cmHBP6<vktWr7CFC7(U(h?yuV5>Dx0=
zPJ|5oj-dT)TbOie(2Ui#GbBTjzx%PdI$aw3{prhC{;0&aBxTy{tke*1qXijVxAjxb
z`lMCE4Vb<=VS$t$8jw%)`%LaB;MneEVe7Y{nR;gAxEi(-z0EgNtifJKr@A0Fxvh1^
zu)BxjUaDJyp@_pI<JR5<pWIyJC;9)iF0F+83~VndgIZti4o8NcKtro}Pak~-yFx&I
z0kvFH2)9;UL{4!nOJ9Fth_4SD46#8<uguG%)7(YdC#d|t|4ju(tU9t1$%|#19(keD
z%5?YnZI#!Jv_Ny!U_(B^WNr1piU%~IC2cBregA&^(wtWgR*QWJ0&Tt~;W-w?nq2z&
z*$R%)fl&eW=j`6FGSfKclRD=~R=$wS8EC`y+vm&FF!*}o!7yipZS*P6?t|Q`=EXxE
zB4wbWuZ=7i=Zaqnp3gETH=S5>{u?eBujdF{oLrEod}B_V6Uu6#EVs~_IV0ujdcOsY
z$}Pbll)m&Rsxj52ezHORpo+%X1f3vo?01F<Jo0agd!c>~e-BAup0j89+ttLKt&m-U
zdr9XGyAy!@(xV207K8XGEu$%vsc_o><!ttHFX}4mje;9{z=`f8j!05w^28+Ow35Wa
zs-ittxN*Uw%iDGClk)*J%p&Sy1CI2PSDHjqn$jD}Cj+8f{gM7Sb;YdPz*x!;gYF$K
zNSi)i%A7da^pLBIox19^15gqsJd*OF3dSGI<9;K-ZlOH9Lm60<h2ObN0YwD$z2L<7
z(c>cn=@A?vcFOy+85|W@rj$^UE>3+q*iJ7@Y9mWeL4TB{49E#4x(?9GYUVOJ|2@M#
SQe}<)dI8PL-?P@8o%ug4$Ufl!

literal 0
HcmV?d00001

diff --git a/products/jbrowse-web/src/tests/__image_snapshots__/lgv-synteny-test-tsx-nav-to-synteny-from-right-click-1-snap.png b/products/jbrowse-web/src/tests/__image_snapshots__/lgv-synteny-test-tsx-nav-to-synteny-from-right-click-1-snap.png
new file mode 100644
index 0000000000000000000000000000000000000000..439664cf1a31c9c9b4bf04b380deec7742fff0ae
GIT binary patch
literal 2810
zcmYM02~bn#8io^=uuBMFA*et&0R`CtsL0~NVL7Z<OpsL&Lxczw5piJ=#0nveki!z7
z5&<I$MWk4yf_OuTHEgn6z@kFX0wpXKlxrz!y?@X<oy?qMGIPHFoA-O)=lfaz^7qtG
zC8}aD7!AY=4#Z&OjlgfZG7da54p8&K7cSbz6UKa!eJa~;7Gf}}cMy0V_3YKfackcj
zhlr1t|8SftKi<&sBkrJ^yTwL3+&B8!9hJY}(QYDI?)Q{ye}u80H&B3%q4q6uqI%3A
z(Zg4<R9Eo;?5P7kjk1~I-;I`xRm9C`cD_iJT#)3$y;r-0sS?RKevTP^RSl(<kO>zQ
z|K7)7iyf;LOb}EssDi^IH)iF&gI;SzuB~BN?}Y&u)><KisUV*#Sbl%0S5vrRP`CLQ
zeTSDbDJhNyqk@{H%Lj(eT5?y*mVQ~nE@@>c`6l{D6^iW7#UCqz5R;>d@X*;6@j^k_
zg?x7YQNnTI7Vdl%-yelIEG#@UJ8nE^X5%0Ts#BwcK{=yjq|Pv`5#o%h%5Pw~sTpDn
z@vEpxhRJ}Ltq<|pHA@8b=vK`Y46YXVUh7>bwc739<!rR<q8-keJ^_wviNbL0)6O6Z
z_|olFE^|%FuE!Og#P_36Ei=9YwRheeRV;G-p=b#^V6vpojZ|4EEHxY4HVvVWRJT6U
zi+E)lCBG{zw1dr2qs-HHsJzzULbc%|sRs(Ed)SbFCHgWuIySvnZ1`wk?OV5b6e_+^
zZmovSRjI|>Rt5;dvb*;y1w%>jRIbQ{Fy5p|MgrS@y-1d()O0O$gzYRpG#>NA!Ko@I
zcL*UWSs0-c;{?{x{fnJSpq{Gshr{Z(`^WfXWN2#$mYaOah2lh?F~~S~(u0vLGO3oG
zr@;fF^}PV1sh_@nZX^PmPB^aMAGY#G2^4p!Je|EQZdZ_AqE|d^wNkGsl9aT6GFMbB
zJMMgB&|av00xEN%+&vuhq2Jy1$A{AcV74tsFx(a9Dfb>)h(X`?uZh2!{Gj4&TZy2_
z2Qy;5&xa@llb1~fmzHt;XYw`?9F6bx3M)t`)Lx;BM?URzXYhKz<!QGhAvFOEj<D>d
zA1XLxo9VT7N0H!kDKy^xPPRNjf>DASdR)JSkdc>`H-}RqqS)9rxbrc7ocyL#Y)6SK
znC14kq7XuhQiPdz<0z>Exttl=pJ}n_8oS=T@kECYXDE9oG{dh-qgM3_e|MYrgs*tL
zFme3R3SpX97zI9G5yuyXEXh?T^B(zgb7WI%&cVXWLVZHprO|}-CjQL2rhXX}Fy3bS
zGugU0XEM>%e9nq%$ok+ta|*ZNwhvkkoHd+W_~`3AN9#?@Y1P@nV3R~<Y!T#;V+{os
z_HlP9c6w@@&LWe{6s~w}M`w%Dw<74oHEVL9_UTL-;XQ2H?sBJHpY$>MrdXo4RJ9m_
z3f7tweVFCjC~b7cgQ(d0d0-!nv#&ngg=!Cfm_*CzS)zaE8Ju!Bq*Q8_<TZ7vmm+^A
z&kLRV8MLAhF=;GcQnvN|R`afy<sRJV=1?9)`wk1&wMna|=Z=2A#oT$@v%4E|!+Gg?
z2zq!%jZHn~16@(Mtl3U45&sq)`%w**&7Kym2-jWTQkFnb!)cBS9U2c&-LHJ)bD`oM
z3~8iKofUD>pzpNBW9T6+g%w^5iUk^+(>5LHQkCZuGz?pt^+^&fN3*xO@-?z7ad^7;
z+6|^`N#c~pDbCO9n8>&QlG%?T(Z)BiTst^3#QSy>189rZ{gwBAs6BBa1)+^#gOCMn
z4MQS3U)ee8;wd=^T6)jL2enTN+AA<GH&03ZX)rv-$tojtlrQVL6Bx6OktN!Z#iwb~
z#ENB>gfRHY$B80noDVd}+tApN8DrY*nzs`A6vT(PBvP$_q~rRaSSjgkY46R41GOGs
z@E~)<c#*YJ;MhjwYUWF~W;?$qw#t*P_}#!lt@@E?1x?9WmbevG@?z~F&VQBDJf<!Q
zjalai6|xN1nTItBC~5m=a$ey=nAQKFp<S$+YyY(mlwqppIvi_=4<^4+%5r6e1de&4
z5G`959~XX}o+9ow_%IUqSzX|>z*NxU$)DV+1ERX?azp?K^lY)+1yiUt;Q_1IBjFot
z4&0eToz)6Ha#5nZ-~>8M02Cz_Ba~D+Lt||ytb#jcw46H&((3@+K>Vf+N>TP*Bazz+
zEh+<|Il|gKd(dHcqSw1~_>1O4mdRN+fGf3F?$3es;sEP9Deg12W1N|xf(RYAd7!Mu
zJCu3`hmX_Sh{S8Zh_&JjI3(_k);JqIz!S8-!CJNFTQxq)bRAqW<^=!a*47w9Z*-*r
z6l`(uV`zf+if2*4=2k!}>Q`X0&x1)PeQnab$n#pO^?IRul5gfhBU!q5X~cSHZ65d(
zpWer)=r!#CSfRS$@Y0&s1+%gSMnn#cp3(oQI*rs348=o!^){0Qv8x44f&BK300Gcz
z?xs+l;0-|K1ieWwvBIw2hm#nxpfhIlrdLf7G0(=r=PT}4Pj_=>0_`>ztm1YVB}KeD
z0@b=a7%Bd{UcMe(<$`*|1uO1jfQ^3r0TW(2Sa$LjMnmix@d8?Ad)4RmDa94!g_3_7
z&THt_YGyG%%;==tWzE_ftwI7fQr5tMCOOjStQ_@VM)ksVuewq;h&Wq(-?9v~SI6O-
zpWBzYBwoKS*x71zdGzUvZ6xH6o}63Hj<ZCz9ZSa2+q;EnU?s?c4<;i`sn5KXex81>
zYrCG1+@8}D9Wbyv>!39SlGdmjF?oPz3Ombvh;?S}sU;vJ(%TQPL8(BPZD1^MvkZ0x
zsb8G7F9~Se&ez2|Na%y6w#Oz*>`k8vx3K)cewFrvI`<q2=^*mt;+ay>Q=t|2`zB2j
zWcqE8N8kSqgL*r-O*xT%r$3Aq76Fbu>r>4wKbF^P1ssE#&u5Bu0w*)g<a5hoz9bZJ
zNL*i2B%W&v)C{!m8^G?+YwU%`Jh_51DW6rWpYUexkk6ev(HeVy?N`kNL}PNGRdCrZ
z^VUR(SG~5hBpr_sn`Bpkd%0Ji^U1Z2>PB@p*U1&+0yPWwu>IyimWMGo{7C<2Zjqax
zNY!Xui)&fvT-SFxBR!v?g@(qJ3*}9BDeXXf+YZ8kAA_+6TzpY!VVRx1&yWvaXE6j6
zqH5#tG|6XVp)0uloY7DIQxaEeGL6^LRoAeg(L4GYu!x{#HB-m~7aI_-66!lHB&C5l
zmAh^aZD440p~=T(rS^eR{SgGGd*R5?K={VR5CfS1S9~$m>I9FrS@vh_igLY8#|edZ
zNN?f#?nsDHghy;Z49fyE9-;Odzuc>&H}VC`%P|4dVUWyf2=RJ;UIpm_ezipO7^qIv
zj9jJi*R#lNXgPsW0SV!$uSl{$%$9(a<i}7e>0VYTGt9gF5EU@bJv>MUe(aA!I>C9r
zwZN~AU3}?=n$P{^&y}b+r`*gHylj%IUY2sljO|V=6$_p@rdOFKO)H{;=XRN~&ayO?
rzH#$GnTt)qI&J=w!^c_$XZ|$QYQO|+M)!4t%Pa<Q_lFzyM_v3Mj2r%H

literal 0
HcmV?d00001