From ef2c9861ef41df68f657799497ed69798e053e51 Mon Sep 17 00:00:00 2001
From: epriestley <git@epriestley.com>
Date: Tue, 21 Jan 2014 14:02:48 -0800
Subject: [PATCH] Add yet more unit tests for Subversion

Summary: Ref T4327. This adds a subpath/partial repository where we import only a subdirectory, and adds tests for it.

Test Plan: Ran unit tests.

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T4327

Differential Revision: https://secure.phabricator.com/D8019
---
 .../engine/__tests__/data/CHD.svn.tgz         | Bin 0 -> 9437 bytes
 .../PhabricatorChangeParserTestCase.php       | 109 +++++++++++++++++-
 2 files changed, 108 insertions(+), 1 deletion(-)
 create mode 100644 src/applications/repository/engine/__tests__/data/CHD.svn.tgz

diff --git a/src/applications/repository/engine/__tests__/data/CHD.svn.tgz b/src/applications/repository/engine/__tests__/data/CHD.svn.tgz
new file mode 100644
index 0000000000000000000000000000000000000000..0a4d0a5700414b36e6c72445d35dc7310d9f464a
GIT binary patch
literal 9437
zcmV<3BqG}%iwFR|;N4OH1MPiTa~w&M9@*q{@0Qt`$yN{hI3ciY3TV}pR~=}>YB40b
zn<5EN0LY%1UWyx;8CA$(R%T5e0E-&QOfs4DBlH9GTlAtQ{RaJoebIy5JtB`fzye5A
zksM^O*#t5pBRo9ZJwEriTJ1Y)Pn}Mu*Xc0&w`<YA9o>}ws>C#1@Aix?ys%hD(=Efi
z!>sj_cIQqSCp-dD#3b@WocLn8JRf8m4&PIbYDfO>7m`|SCk%$`6$0#c%eqeg8)na_
z`hOFt)pqT5n*iw4EBC+N>tO$DMz8AsO{7+P7)E2Btg{Z_hiRJc>wkS-|9d9%e`lRU
zW$Bjpzg7Fx6jbR`rdInb4By0S9RfaJ{O|U(%KmR6wc38@e1`qst@?i>sntF{*njlw
zpq0E`8wLP=bl*4r8>Tha|E6wstMPvm>GaHt8T`lDNot>qDE7jDMPd@hUJ^#r?^zlP
zmYfMzkIw_{j=iAHk}&k)e@wjP@VlsaNsF;ZAq&EUao24ySNI|k4Hiah9J=0c%7&gV
zVs<7X!O{SJjM+3y*_$A|U}xb)4GQDuq31FujG}asc)^G&EeL7VYA-$bk)0*UWVhXp
zb81_OH;TNt6-J{r8&Z`}ix)^-iy112b9u__ziT)Ba1^)jx0|kGmHz9RF{l6Cj#cUZ
zMiS{iY9|^2RAA6|m)X;@InY{0>n$p?&l3@~YCnY+0^T*4m&9NMyf_hoLj^?QPpKC{
z0jP&gIG%t4+Me$vQ+DAcXSJC^by=w3k?&!#pPCi7G8Ov27S*S8ef_sI&8+l)6RFbw
z+nqB1A8=OEgdQ6Mm`lbJ|0e5Lq5tUY&*^`!V`-KCZz4VTS8dx1+VNSf_JBP<Iyr4V
zJbL!*@hSW1=;%r90X#ef9A1(RhVp{C?gi)J8&GvV1fu}{gAyJ#<4B}I3`HbpWav9E
zEc6`$e9k6OIEwg~ZO2aJO_Bx}6XB!@w|#IPfGi$O;aTFecA$`ekD<1DiIh6qMmuwH
z<~e7wj?9p8mIx4HFjqTdc*vQ6GKQh=hZk6R7`Xxs6^};gSOiJDOO0jh7wr{0J^)TM
zpYNak1P`|HTQFp2=<-l4g;%m8Khg1%mn={8Nsx|hU`#kvp3J#NEcqh{oZ?j?+?|}o
zt{C#vPuN8m5qDg$qYl<VwiJs%#yRBCU=z~-z5wqF+^sky9*dlx@mZMqE{jFN!ax95
zC%)%+i4V*lg3)FUk3`&}s(&jSp1F8g2XkPc9|`VG$%dmZ2aF29JL3Tk5jHw;E|lC*
z^}-l{ZE{7{TI%bJ8kCh$08W_;!f`tH7D|9pXS_@wUe{Tg$Z-RhekS}0lCAtr%!u+J
zF8paT=OjxaA!M750uX=~f%lfhQy81Z4JtZ7f6&Kz9pR=C$q5$D&&8}%K?zc}SqR@n
z8Ksol4OMSKN!U|ZbeN9Lki0RUs`pR=zQplS_(SxUC1HnQg4FY5PK?JW6SgRcKPn1V
z1bm6MV-VuA5Cqxx3)-N<?PNOf9Pax?UUH9J(mTfEod$Iq)ktr#A}E5|OfEU`r|H9R
z4xACu*o#R>(I0mlbkpa(@SMLtdmsWAan54&=at-{rY?v9d{=IC%jVE<knEX<z<@dm
zm7OM3fX!piVV)~E^%gvdp=_XnDXp0iw1OROC?uNk0_Z_c7AAsoFA4+Hc92u#B*2#m
zr~>>>QW3`>pg$la0YRIf4U$$37y|F1S4ftXI-<Em15qyZoJSt@ULu1b(50w@@EcZ@
z#wke6IrmeM>nwCkoZ7jB4Y@~=6`!X8*fbR?$t>-;#~HWd&`%R-k&qFOhsgRm$TRdR
z8jdsoeFxJ4A1JD0X#@ETw;HMuPU=r0CB3BBlFV%kq@Xkeqn)K~j8!lEB%~^V5-YaJ
zYBH0K>{8mOMjB`jj2M$h5SV;G9H*F#`aUM1h4;;i2vs|v0%PErK(PvAG=<9YEDc&Q
zPC~8%Mdv|_Pq=d?C_+TJ+B_bXB1r87^yN((!&))Hp{680ZX+chQj`-0N#xnHh1F^`
z8d&zWzSXwi$CvPfzE!J@q0@mAlWAsvx4xx(TlB-rE!M`q2J+e}C$8AIDYQI;|Ih=x
zarCzu|7m7-KK^5Ms`$@F(jxwQdi3zg67+|%kz+p)@*MXOG=gWCfrj`E;6JSEb3%VF
zP7aP|(H{-BIBGk96$qZdLnOOEd{}BM$AAkoqJ#`37lAN*AEPgxgvD5jBf!b&qsNDA
zn<HZZZq2&X4FG&F{vqni`b8B3qk9QKHnJfW3IR?93JhUhz<z#4h9r?AjC5$wS~je!
zQ6C16%w7X_01gAA{pn8Yb3(XMh)ej^7C>&+MV~9gRpgO%{wRP;YO^_bE4B1ifm`aM
zS!hd-t_5sA4pQyTzznoZ3THfECjuBI#tLN8H!$Rpo<|9cehuaeqHipx=D1Xt3s4v(
zgu5lQLhnjg1*=*wSgF#Dry2YgiE|L>Nz*yw!ART~5?q1*Ortv&|I<5Wufl&DNsIUo
zT#o0*N6(uNf7(C%@n8uOoRyMG<AHg^3p0x-nUZae<&eu=7sqv49pJ%7VA15Ffh7QP
z20mtE#B~9rgpawMo^8XGpkyP*k{b>r=8Tk35m=a92>ynGhxn0k8UCc9O2HNAff$Rx
zFSFGZS;wey%vTIM{&4@<K~YYYr>I=AEUVml#r7YbK0Z1uSDt@_`Rn_2+1d4Hb=BRp
zGNk^fE_-|@V;CV@NyN+jr;L5~9g!LOZc!qIPfD;hN5qR@55LJ|^ot2;&b9R8@ZOx1
zZw~p*tJkf<fODhhEE=49Sp__+fah%scq*5~*s5j24g6+q)#Nw=KU*_F(ZY=D3z5;x
z>Dc-<D++&>1y!!dRniRp2lZP=1b9{aw`24c=l?qu{@+Ae#Q!f2FNpw4>5#@wj=yse
zV3}c4F<@ES7ZL?_LA0W9di6MP#><tFU?^l8(Cx=Xg0txAl1Ok)$nA&)7aesY-oZz~
zcO~JQgYB70-72`Q2EAFBt{#5zVcSTI!}Hm&tuo_P;I>M4O85`Fk2Nj-T!sI-mN~!w
zyW6$&3jb{)&Emh~gXPOVbKzKs_S#~Ip6tx_vE(g|+30F#i}O|lb(i2lgf&+T>VnD6
z=_|v*40OIXmb*qcc>24;=4>#v9FRpOaXU;YT4Ia&C80$)6w!F|^ytaK;Yk(|B?F!S
z|8Z0;@Znr6lwqhut`<P|oO^*BvV06UZk3AN{l{QS61aswxdJc`1|_Hr42S^#a4H(U
z+iB!Q#3j~&U?3gyv_V%;8N{M|eefxTrreAXz6W3f-j8|oMs7Xel0-!`luv~aoaGrI
z7>16xBO@yG{(yrAP}H865kvWek74XRe%utX!zT*PHdGU6Vt{2BCl)ZS@&tD<WOcCm
zpX){DEMuuM0F6LrgNi9<%jx~((D(5Ypo%b7#meH8@Kqz1&Ai7zB)?Zaf-JQ36^8E+
zx^4$y(0nJNaA&3fknbo#!h=v1hV=6{AUmLk=tUqL1Q9(rK9<`@k~9hmPsi~+&_fWJ
zJoJgnRTS|b5@ScgJ{*%5R%U&W9A`bgW9;zg^kA2r{v_w$a7yII<Nass*^84?e8-+0
zJ$n4Z?<oVAnFc%f-NS?Dr}S}A$PY)yIGyzP1gB>(Kz49^`uO04ZTlWj4lLXCLo|Ez
zBTs?MXs5+gAyiDZJNV*k3ce|{3EPjlo5sZ!UG6qXHy1jgC5=!Z?Q9Bt3q4*M2ap>A
z_vt#abE4E&?nuFEDuN(Nj^@E3Q@e`sBUKm(jmr~&<uv@nG{YqPnCz;cLefbV6&?$W
zCwlQ1yKBs)^Ahu+P~8Wv?KTZcxP@4?lZsFZfsdqUW;RU5Cu8JZzBwxLMrR2_JRS%S
zrG$zo>PF6@lYm2wRBox9=r}{0WEg-XPYn>D%fr-P^xR5=->q@n3bb1IZN;-IyKU8^
zN;RqSMNg`L@z28q&(M;^X`~E;L?|P$IC0EJqQHQ0h*~AyCav1Z%fqKIzU;NrIFfvo
zCcO5odP5~NaY;JVv{eKR{>=Wd2G(o@)U&N$c=O$U^ADZo;MErUd)9Wvc{@mbpZ)pI
zGE$}?c@NJ7!ljHfIo?PEnqv)$$C0H#<0%7?B0@8)dP-#r)tKUC{hWa)aN$7yP=K?g
zC9RtLrYY@Elb9O%t%+X_fl%v~XLN?6*@o;x%N%S>7EvYJ%@OpKXl&K!i_T{rLMi+I
zYg+!la{ZTOE%^V2R_*`XNLuj!mrVXB(YVz2N1N^V013~PPb8gxxnlJCu0OlB?>|2g
z^r=06mASkkil$(KITSP#0FcuxtJZW$wr1d4O(dz+TNt`II6N(VA>n!sQ%l*cSKxOY
z>_3(L4cRsH?($j?CPh=K2Jmo#zD!m?L*nYr&ZTZ8v}uwiXqczKBIGYR$SypVUY2~X
zeC7;|+Cm;Zpay`FZ!Mvc+F`<Qb(B2Thbhzo#yG^`94Lhn^1cT)k;8#G2p5l1@a6$D
zC{Iv^C`KZ{^3+6I8hC$7L86@~jAJ$Al)6h`h#~RDUzRtoR5DY@YdiDwJ$JGeD5oAu
zxRUc;`*3G_gR@_`5xGqs)ZDyR_T)O=(@oBKq1BvUAtXgzO=!_kBNFdHmElE@#n=Gp
z4Tbc;XqjsRdOoNp3X{pG6WQ1}34O3A+oCmUsgVGq7P9zeQ-&g1tyX=9<aYZRpV~s&
z6WJoLs8U7CGjh3-AW6}`*|{86{|L?9qbG=*YbOVf<Zod3YBjXPuaAB{gtq@v34hPd
z)ghB|_?@S3v0zsJ_eV&;E~`U^Z=rB#Tw6VAx9k6;xOpMK>S1I8q$x(*HH^YBQzVjV
zMxu_6uQK-qvQ<Wz#C9?f!EOzV$S*)uLHZWkV_RG7)%W;807NPjJoIXX*;0#<u{eV2
zJOMmi*!{Zf+~`B9R+E{R)PIDHyYONd7A`~`G;f4Q)&;UU^i+uy!t>IcWkzwgCSa6d
zTL)1T0yn7}@`VFxfdq)+xBsCa=RdZ#nD+O-(QC?c_|!;mc`D=oHEsV`h5s#Me*RDI
z=)LOv=Z&NV{QutV9}7Zq`F!BA?H@Q$eCWoH+)+T_aqW#Ca(G)@4}YzFAB!FX(sFbA
zK4hSNao>jwXncxIAF^f%NLiVtEA9U9vRKSVZvk1{7;*U)kn2F>nJpqXti;zw$E6}L
zUW~9q7S&o7TwXb8Ri3PZ0toUT3uU9{4-?9GB3u6R{_)}C!yk7s;_KyrS%p0dU7|@O
zDwyC|R+yYx!{RWArsf>vULWad<y_6<;#A1Yln!ETz{c<`0b6k9ywZoXyb%-;;8P-Z
zpd(lFO*Al?#zaT%@p)=p<a~pXLiE>O9KJj_My)w`1U5zRKuS49Zh6+#!_+VL5|l}c
z{8I2}&c*jkx}*T=a=99T_a&*onH(VQ=knB7-Wr<&BuOW8-(z(30wL5#%`<+IojZ&5
z-YR$K(tSIN@2Ux=YJ%x@OfX$<*N*bZvJE)kk3HLBO+K?f2Og@W2f3T3oV4#5dTG+p
zdWedKa<a_yOCtzk8RV-N5DrA)nP+<-QCsY@_q#rqRQCVYwEbu0{13dX)_;DYwBY|O
z-Tspc$2I4G^4%1hm<3wg5cEaP17#a-R?Y+|<}RNLqLrO9-;`!lWJP)B6wcglj}P{r
zWFoLQC1gh>M{;p4=nI<<$=iQzpHJz+r<fDj=$@dGQr#v;U|~m4@$9pm8@b+u2==5M
zp)F{A#5BiW*gS`d_$yoSk#ieTUfw^uakUCJIiv9<&tz2TLsEwS5e2X70)UnCKgK-%
z@9A1K|Fe-aga1!X_m7uf08kc>YwQ8b)|?O~$gmv^+Tjp}BnkL0yEvfm5OBc#z(WIy
zbHB<>xYDTs6w6+_kFW^tab+wN46q4<V+4wLf>b4HJ(g@k;U7HQf4=|Q$4?*2%^k>4
zAMmQ$#)JolVyGV&?(hkx+gLD$g0v{ix-2PhCjsmC6w|_8VbIsWhn*ZL?vixO5P-Fg
z5y8-nGU5Vze&%QbOF};Q0pog9K`dtt3a}`t!_x+Q!DETFw#FhF3EY*jRWxPV%7<PQ
z!6*a7<+e6tkckFhwb^WrMF{C|O~`Jkl3k4CH>tE{QJvZ-M^Z53zmyI)$SOf#-OTtr
z%TYPw5s~7|0@D!DFKtSRL8K#`7UFT7V(*8b6971I)>O7;rbB(2iwST$4;2N)V7c_M
zLi8E4BjICkG-oqTb4*J-#loiq!HY&+=FnwMyi&<}e5r#61*t<L*6iRZGlkJjFG$7p
z=k!zo*o8?rndBypW|b~Aa#!5V_My{4GxlpE{@<*{ye%H!3=+zHz?rAjscqH#&n=n%
zDIG+TZI7Yd%@J;(X*Nr($({FRlhinKT#LOkG4!{`{U!pOChefX^GN2hH|1-o?Z#j5
z&YRC8wc4)T2AqNBWnFcw@c;0)-2c;i9lZb9GOGEXjU=<ul3Sff|K(NLYpVn2c;Bo4
z`uzPrreRk4zma6td_Kj=$M}qcPYZ52P+!@ETa%>z$4zib#JcKOJ^ows`fr-m`L7#E
z!+021>VA8Z(*I#h=RK~cjurZkT0gJ<y=wg5OnUI(0W0o+S7%<t%9{>`-YAv3t+TBM
zKb-t<Qi{s8YPDa+V(ehF!EJ&cj{$~PHCd{N1q)T&n!;^)PNs6oE#Ek$fsbXxCEz~Q
zLWSU&PB=3pL>Aq+xVUJ!JQ#5h*l{~Au}u*iiqFV%n-tXn6_aPfxI1kVFVR9#C#<nS
z_Zng(Ix8yAM5QRAWxSGt8AG)vTry{j`yFuZo!ZRgDO=$#ZYA)dI8p3jdm!w+RtNrQ
zyPBqJ8f3t8S`*PMh}vW0X+gCUMw6C`J7K%&J3V7_Rw41YIDD8E3>3Q>R0kq&*iT0%
zr@POOj!#>&?8*BD_o-l55I`YCzmp3*9C@7jiHD)HJnRE1^F0-DmK}TL$vye5obr+(
z*Y7Z-`yCTOaXL-81Pb-a?xx)?PQqTic>G8O*znlt(s5prPCDz0)^lKY6;UYJ96EU`
zJ5T@@!~`-*&K<{dMUY_d6(oT|-Q1^12_UK9!Wmw!2n7XUc`^=JEH}(>H%5Z!q^_7^
zQ28MP<u<~@=o0?YL=_!qND*u=i>60uoK5=0s9W@zQmCJC5{pX?G$Y~D+@8SY8!U93
z6a`aGAV9IDfTNU#n6^PbkPFoEnUMyb??cP1F)m6)sU?3(Rxx5a!IjsM2i5tKznNix
z>bGe#HrAkyft2$qU>`tLRpWM~(&3yIeO{t15L5|KsB8dH0fePNc6fz6`$3+eNTKH@
z&dPv>0JjA22ztMXf(jHU%M;SMkJtiGQ#xmmBtN%WB;HbKh+F(T^jw_liD-Wz3Sx>=
zQP5<RExBQd1R!pyL-FufIXVrUv)Ctjb3&Z;3$g=?uv*D*JlfLBj<UqXSs=SOjfWbk
z6l|^%q7U={H$(}=a+;CCvnOJ;TZ&r7Vv43qfpWz}1SsDuov0bY*n5}l8l!RwO;IyW
z`6czC{YT*>F;f#~bW#*o+nxA0Ox0{ei8pd<qO=TX{Tg!WfKKTgLid%24Ljn2b0!7N
zRXP!o;AK-9_%JtU#dLcyn=Ar~=d)6GFRrA;ooYC5D9t`vFc3)6JfMZsp`%0WHJqkW
zJJ3MpZ5a3^Kbw-DYQ>ne0>zp3k_0z9RTMalC+we#NX;z|OFQ&Rvr+21W|{ImkYuoD
zpB#D<{u_HiGYz2qZ(tCQ#Q6=^ah>?DTK}(WMuq=2k*ZkF?M|fsXqag9ZMpUu>R6%w
z-A>8>(~TbJzoqvo|8FDd<lj975X+>;#Ed)l@7%q6=N}k@SHFV4_exps!r%LIzu#R^
z{IBjb|L>i@x%dC>eD&2o-}&mF@BOcvrM^m4`k?fmkM4f+cfbDi-TyMCl}um&Uy}jM
zfB5R*@xlJ-0XyCQ?b8F6KW5vEp?i47P7i)3kK#Uj@$@Nse*E~^{_*eGlY`$kAV<Dw
z_3`29!H);Wc}9E*5EPf&EPF+8ec21~b^_>CxUZMyfDAj@S6~0#!LRS$@gUpV_)k7)
z^=oimL;C%7-hkIy{v+!4{lCHg-81jp```Ee$Gs8!P^BtW>Agw2|Mjc8e|7h6pKF{C
zduErrT<87HklP)DTYYDsSv_kw6n(wlx43ifUw)(CgHIrJ4O?i2shO_bAA-NxwRO$x
z_Imxn&=dog8{qiZ=D@zk{`PCoXsSlD2R1hWeb^ee&7R<<YiT`)o0e|cLv5hjmZ|r|
z;H$s=x^wsISAPwkbR9z<_Dx&sJ9=jb^b9*a+W|U-=m;RLJLq=Y{;>Zq?j!uafA86y
zd;jy^yL<l!eyCEFs`O#0aqrcYjlzA*d}VWhnXhaI{`I|Ae|=R8p!`2N-6n3Yhb`#$
zS=RgFKlAf{RtMuh)_Ta9O}E_t%ldy^eUIt-{cmcTUiJSb67_!)@gSC`8LUGNuG9Z=
z{jb$Et*ZYwk!1hB4Vr6Q1Ad+Ozv_P&6RY^&CQ`Nj`*x>!{ih?~Z@7-t`aeJZclB;H
z|F@Bp_dhrQNw^*$c>VDo<e*oL|C>nE|7q&EYpi3X{+kQyzdL%R{~JkS&=ZE<HJi@B
z)|#fxwI;WRc2l=CZkZOh-JVrJgumQG`cDfP*D><Tbp7$)(sZ-Z|4pPK{?l1o1nBzX
zzo}bQ{BI+v-2dy+x6X?FpIX;g*#Bu%{{KeO^QZfd4{Ha9kMQsD!SkBu?lRn)5OvyG
zuZG(Kck#@hnuAM#;Z+T9j)i~Wj|qQtS0D6xuI3E7L$lxGwlKLS^p5Vj9m6(7&A|oy
zy9|$&(0WYk^eQy*Ii<4xYip`wmHv12UatRIw*R}@|GAO$=-}z;J|5(H_s*UBU;XOt
zf4y&WK4S}-8d?Lbtq+tnz<m_+=0F2;pmo6>*u9=HG|awj^tEoc&wI9G>z$4v`hshN
zA!k>en1qTu?bmv4KSm)>pw`JW3PZ9#U{R#;dcw>ay?qHJ)My`DndoJ<4PF>!<2H%J
zjM-DI)AH77{Q)yTGMUTu9v2R`I|IY+8^b~0<z2I{3)}7Lwq+FdZjRR1Sb74D!Yi<v
zL?!NDk=p=N^J+C*{%UkLc*JU1|MfN1u~PqgTA}}4rT-QF-$*L!|JPrAbN9b}{il>W
zZ)Pokj*6kT4edP^Kt~JEXI-Z8o@NRcI&oltUiUk8-|m5$h`!eAImUopbsYoKnp)Wc
zxGB(esRf|7WeZRc{Q(9-Z-XPU+(1a%p<1u!t=9$`>vn-EZWxZPL#r&_?b)JZm@e40
zfop4a-|RZ(3JU>^UTz_%fnXuPK{5t67LzIKzi~rl&Zianul2Oj{GZX4`d{Jyjij>v
zfAiH}-Tm+1*gSwgnMN$y2R@mA_Lme!ZyKl<M!Rdg$37U$(4~PeI!?FS>%r)0n3l<1
z-5vJ2x@$OKBy_VgU{_rg1vTsK*WFy9<e_$wzB%1OI?aqu{k?|FKtncQJ?0J#q3HwO
z({<kOT8?8_#sFNJ?!W@b!d-UNrCyRwYmQFE=SvKlRg>)~OjeoAPgu96zK0AwnPH}V
z^F}RuXZDTDojQY=Ai5ZVVOHPnbJOVctiA@_GVE}-r*~{;039{#X`R`|tivoL4`ku=
z9>|?-#WJ&Pw0oVwa<eThx@xeIH`p5V8T<#%tRY;@(gb%b!?bmy&pReaBOuXk#{i48
z!fZnqli5ZxrNLHaTQ;HgS}*5XFz#j1t-Us{kXLnotp+V-ed9yt@yqsqU5o!$*?&;=
z`S_n<bSwM6iIlr|O|uo5?M~S+D(xxh<bq$cZm&<udhYw-i4Rz<a+hw(Se7kOuVjfb
zo-8+}X4~kPi^isGqvrT!k{_vc5G$LA5jk*S*tG_t*XaTX?sm=LVAwY_*ZrghR<>-J
zfi>HfbqOw(wpX>-%v)>%;wJ;k4Y$iRkb^#qDxz=e0;I$04TqL%cZa>pJ!hmDjBH{w
zjlv)<+E^(QxoiNPtXs`pu1d4(zt=Mdbp81sOS8;s{$~?uzW>)T=B|qWbh`8BKUlg^
z#eX-Fp0Ium?@)Dlg5#&&tLXs0njNjFYo{H3*X-|FMyscH2ZsI!8WKJ=#Z|h=G_U{b
zm;+il|7#g@^FN^f)%vfEq$dDJ$pGL%XJLd;uv05|QCn#r2D?_TWmuZ2_x}L7do^_<
zGi2`90Io5DW0m3fOw+vnuVW7ATKeCW`d`ieZYF)K{#%`0ebDNgJ<ZUT>pwZDmHvOG
zX<q-=F$c6-|INAiAJG45{%0fUqxE0g)jF+#-Uk3Mum1zme;Hz`48Z4^=JkIaa{#OK
zU)Sg3{}uk<RJsBEH@dBX*|9o<dHpx7+yPK$HK$4Jxk8$R2IEc?#&LdFR$Lj0&8HS#
z0yk`9@4s7D9i8{@|Ij<Vh4UXQtHS>qNweqw&t8xyO@jAM9eqyIE_l7{xjb82o*n-<
ziL=uq@#uM19+vLQOHVo7J3J2E*Cb%C;nUetsR%llTb8yT<8jLufxKr}9<Cn?d6cxg
zM>d`<AiwBR-n5*ZF1?+d?!8mKa8aJw?-iHvW<>|`x=wZa^g;!A7<G&7&(Th2lG9P$
zkr?6`(0J586v3K!D7Y7le0kkzqj)-s!gPWxp#$Dki(EX$nlp7fBKw6h#ji5riq9dJ
z@A1n%qjzH2cfP;mPH8-1J*Eq#p~dxYXEM~GEpJ3of;_1{2!rW3Oyi}WXD{bJ<-s%z
z#7ySz>UjLN>+r}ePW|3zAVJXC>f!+}I2YNi$rE6&_cr@HN`b9(%6$Dl)m!X1#WTUP
z1HVTR4-z_{pKaHp`i?q*zCn-Zo=$-;E=q4edV=gCSt&a8`VJnwpYgNyfc-HPY>%}c
z?)~l$?UU2JN01e$Yd_sP3Gkxkhh8#mAE{rCJio!7CT^>J^kVN=#DYiAtj9r|cuAUw
z_QS)ye+$L!L;Ls$kFQtVO*h^~(eyQl@ave4QQwnK8uVNKo_?cTlB`mm#&6&eUX=Mu
zdl-gon@4%EkTJS|Uw$J$ir~lZ*dFCA6l51ZdDkAM@9?vKSR{u^_>MmR)OM>%H<Q%(
zkJqVQtZNVWit)c=b{F=4_p1G$n@R5*|4Asq$i3RoAA4iZ=TUj!&$FY9wTi)AUfV!r
zWOd2$zI^!vUhI;M&pwCuc=N(9H1@nI#%j9rM&1*X5sO#FAa0;*h2`Mgpwhs|(>)dw
zcrNb~sj2cxR+9b&`|d;nuV>CoAN`v>{?wgzgGuTCOi$f(9V_(TG|b+@`7gRv>Hj9u
zb?^Vm1ckX=i~(e-`<_T2@mej9?va;Qa!XJp4T2ZX-qNxyFN9an60^(8a?~eh0E$Lu
z^3E?*?=~nk-73Uh96yce?wUZBs_Yhh?M0FB#W`TW)=rI{y!4>P{9^R{;-OaJjUq3m
z`_Y$P9EPT0Bm%%_{<7Q56rW~VSibay%1nb9P@Uhtv;>O-LtX;jbmeVb@;09>f~#8%
zwuRtztC4@d1z1~bWw!uH^BIdIp?iR&9p!Wd72XIm<amz_UP6Nqy{QP^BS4G~@vim!
zA=Q|h*36<dT|2vBY(_r((3ug9EPsA+8(ngQ{R6bV2z;yqyF9x&jZ2pe^$%&6pq{g~
zY-x65Q#~#!A|dkOXOKU|bJvStj6jyFfg`)YDUr7{l|`6Hkt5fTo$Q}H#R9WCeYRtv
zMxmWXNwV;wAUbDVT_+~Le^I)eN=QFYGCq-)%awRPJBngQl=-?Re{*Mi&v#el_@XNv
zkvoLahrDfK54f3>s-oW48S=iPg$}uDXJmu=vQ8;@igz?+m&672q#!{yA>#nvW_Eci
zuzL7hT)#gSqZ+MRQ7iNa{aDKlqw0t(b`a;BFI@`<+T@Ry-u)znEx8C6Z@iQCl<>=v
z(unC(b@`OMVsGY~T&yxplh+cSdyZge<+w5sS+l87X(h|17<v>>e`?VkgxH8Z_H4?2
z!21f>adE9)?U$IxxhY)6m5Vb$ckE%mUZ59VV*SHul*XiSRDkc<5m!cXw&^YQNRf-O
zf{S}wMU@#CB$dP%fxPArN1%LEf^0#UFgx{pbr&7gE$^K}jgQmGB#e^p>C(wF;k=QT
zKGyRRnoTuE)QO*d7`~B&l31xr)VPZhT`$@---QUI&pF^>uq~~6jmE|%vM{^0ZgEV>
z)IiE=>|xX5Boe_WIm<_1;iO=rrvQkjB2sov-ZUtgPP9PNCQ*3qxu9wtHWnQ3-weUh
zInk6VYcNedmhuoJVuYnOi6PiB9F-f{U<9CkL|2@FYoO84;)Id9e^8PjwO<T}*h9d>
zHz2saB%IKAUNPGitx*fWUSE*~KQ(Q2GhvZ!r$wTx>c4zQ!+Xn0h3~NrKICs_AL>^3
jFA1?%sY+F<QkAMyr7BgaN>!@Tho%1y9M%tP02l!P;I_kn

literal 0
HcmV?d00001

diff --git a/src/applications/repository/worker/__tests__/PhabricatorChangeParserTestCase.php b/src/applications/repository/worker/__tests__/PhabricatorChangeParserTestCase.php
index 616de159d4..54bbdd74c8 100644
--- a/src/applications/repository/worker/__tests__/PhabricatorChangeParserTestCase.php
+++ b/src/applications/repository/worker/__tests__/PhabricatorChangeParserTestCase.php
@@ -952,6 +952,110 @@ final class PhabricatorChangeParserTestCase
       ));
   }
 
+  public function testSubversionPartialParser() {
+    $repository = $this->buildBareRepository('CHD');
+    $repository->setDetail('svn-subpath', 'trunk/');
+
+    id(new PhabricatorRepositoryPullEngine())
+      ->setRepository($repository)
+      ->pullRepository();
+
+    id(new PhabricatorRepositoryDiscoveryEngine())
+      ->setRepository($repository)
+      ->discoverCommits($repository);
+
+    $viewer = PhabricatorUser::getOmnipotentUser();
+
+    $commits = id(new DiffusionCommitQuery())
+      ->setViewer($viewer)
+      ->withRepositoryIDs(array($repository->getID()))
+      ->execute();
+
+    $this->expectChanges(
+      $repository,
+      $commits,
+      array(
+        // Copy of a file outside of the subpath from an earlier revision
+        // into the subpath.
+        4 => array(
+          array(
+            '/',
+            null,
+            null,
+            DifferentialChangeType::TYPE_CHILD,
+            DifferentialChangeType::FILE_DIRECTORY,
+            0,
+            4,
+          ),
+          array(
+            '/goat',
+            null,
+            null,
+            DifferentialChangeType::TYPE_COPY_AWAY,
+            DifferentialChangeType::FILE_NORMAL,
+            0,
+            4,
+          ),
+          array(
+            '/trunk',
+            null,
+            null,
+            DifferentialChangeType::TYPE_CHILD,
+            DifferentialChangeType::FILE_DIRECTORY,
+            0,
+            4,
+          ),
+          array(
+            '/trunk/goat',
+            '/goat',
+            '1',
+            DifferentialChangeType::TYPE_COPY_HERE,
+            DifferentialChangeType::FILE_NORMAL,
+            1,
+            4,
+          ),
+        ),
+        3 => array(
+          array(
+            '/',
+            null,
+            null,
+            DifferentialChangeType::TYPE_CHILD,
+            DifferentialChangeType::FILE_DIRECTORY,
+            0,
+            3,
+          ),
+          array(
+            '/trunk',
+            null,
+            null,
+            DifferentialChangeType::TYPE_ADD,
+            DifferentialChangeType::FILE_DIRECTORY,
+            1,
+            3,
+          ),
+          array(
+            '/trunk/apple',
+            null,
+            null,
+            DifferentialChangeType::TYPE_ADD,
+            DifferentialChangeType::FILE_NORMAL,
+            1,
+            3,
+          ),
+          array(
+            '/trunk/banana',
+            null,
+            null,
+            DifferentialChangeType::TYPE_ADD,
+            DifferentialChangeType::FILE_NORMAL,
+            1,
+            3,
+          ),
+        ),
+      ));
+  }
+
   private function expectChanges(
     PhabricatorRepository $repository,
     array $commits,
@@ -1023,7 +1127,10 @@ final class PhabricatorChangeParserTestCase
       ksort($dicts);
       ksort($expect_changes);
 
-      $this->assertEqual($expect_changes, $dicts);
+      $this->assertEqual(
+        $expect_changes,
+        $dicts,
+        pht('Commit %s', $commit_identifier));
     }
   }