From 4ff6a55bd5e9a13dfae436477d9fcf8bd7909168 Mon Sep 17 00:00:00 2001 From: VirxEC Date: Fri, 12 Jul 2024 18:36:09 -0400 Subject: [PATCH] Fix IsBot always being false Fix unit ball pred unit test Add more to player mapping test to prevent regression --- RLBotCS/Conversion/GameStateToFlat.cs | 2 +- RLBotCS/Server/BridgeMessage.cs | 4 +++- RLBotCS/lib/Bridge.dll | Bin 132096 -> 132096 bytes RLBotCSTests/BallPrediction.cs | 4 ++-- RLBotCSTests/PlayerMappingTest.cs | 11 +++++++---- 5 files changed, 13 insertions(+), 8 deletions(-) diff --git a/RLBotCS/Conversion/GameStateToFlat.cs b/RLBotCS/Conversion/GameStateToFlat.cs index fb51076..1c6aabe 100644 --- a/RLBotCS/Conversion/GameStateToFlat.cs +++ b/RLBotCS/Conversion/GameStateToFlat.cs @@ -159,7 +159,7 @@ ICollisionShape.Cylinder cylinderShape DodgeTimeout = car.DodgeTimeout, DemolishedTimeout = car.DemolishedTimeout, IsSupersonic = car.IsSuperSonic, - IsBot = car.IsBot, + IsBot = car.IsCustomBot, Name = car.Name, Team = car.Team, Boost = (uint)Math.Floor(car.Boost), diff --git a/RLBotCS/Server/BridgeMessage.cs b/RLBotCS/Server/BridgeMessage.cs index 5a7cd10..0a0c359 100644 --- a/RLBotCS/Server/BridgeMessage.cs +++ b/RLBotCS/Server/BridgeMessage.cs @@ -47,6 +47,7 @@ public void HandleMessage(BridgeContext context) CommandId = 0, SpawnId = PlayerConfig.SpawnId, DesiredPlayerIndex = DesiredIndex, + IsBot = false, IsCustomBot = false } ); @@ -84,7 +85,8 @@ public void HandleMessage(BridgeContext context) CommandId = commandId, SpawnId = config.SpawnId, DesiredPlayerIndex = DesiredIndex, - IsCustomBot = IsCustomBot + IsCustomBot = IsCustomBot, + IsBot = true } ); } diff --git a/RLBotCS/lib/Bridge.dll b/RLBotCS/lib/Bridge.dll index 3d15f0bd2a53dfc26817c0935da1c6e99d9d170e..41f0e031fc44710f77a6e94707e7afef48af1916 100644 GIT binary patch delta 9980 zcmZXa349dA(!jfB&)f+icL-;~l4LiBIRymeR8*pJDmUR!kwt;rpC`Ad=%?u8_^PUB1H2c0>8XEJb$3;DPtWXz@=9lUrE@{0 zHFaX#)1S1bXL!xY}rOTnv5q8R%;EP*< zI{N90znZX##epCi!azXPrYn$NTclYx#V$qLFG56U7X4EYR#&x6*aPk9Bjlvo+?Eqi zlWf6`FWQ)E>zEj&V0k5PhCSA0_Zqe+KDr-KQl^MvGRm83!}GB(@Xf@84C@ z*^WF#?tQ;a{8Z9L8&9dr({!2IM~ErLld^SvE7`6-8m7s@`p1Ix8>ZQELwy`nqC;H9 zTvC@T01;@+BhiJa2Br?uEXJFJR^zQ3TF=2xvTIt?87 z*2AvlC~Nv#GHXlU)?~Ek?D7Vx(_{Qr*1sheZ)q*tH$>f=+S{VJ8Zp5KaPsYVeK2(@ z&i*z{Ycdmj@PuV@h|!1wg}m{%?|5j&9>-R>g>hZGzX61VBj_Kw{5-q(>xZPS`IM1j~O^k zY|*Jz`V3P~gIRPQk`ls1&Cyt2BP+MJmfN;RMPXUVNkwkaDR84k9@?H3wgE-VY$IK4 z#wHom6c<7|-T)-XH+HnI9njPYU;{0^yl+=q`OZgOWYmr@SM!QfcZ*K#tt-BX^_yhk zj@C&un8DL2Pc^4SCm)=6=#F*+-bE3Esilj}sH2O&y!-v-r5#Fs93Ts*pdPGX0TtAj z6?L@-c9enCcLmC4w}wD1Z{Ipux~o?{b++(Ww{Bk=!@@(}nhcF;(d*>pU9FR-uR#q~ zmi$%BXeGH$=I(9>@5v#%hhh8P?v%v$Q0>Ls15+&e5bC9~1`8ILBIkb`C&ND;oclgG zVKoBN6D|6^?8Pda1~*7ODhxWiEPRw~CcD+xBA0)Btp!0zJ?_?IP2WO(Q{?(j4#5Yq zY)@R^2dLmud>~iu*^A02KE2&RN{jx1togL{eIy&~m4fF+S3C#@@p-Wo^*BWXU2MjO zbn&mo(VJI;{CENqFALc{*gp^=w$o2>q2az%4?K{x9mV7p!qwU$?7=NG7e3AcLt0o|7X&^W zn&id<@$iw{dmuh>2Rfz-5t*rpt@xC>hEFN~l5Ll8Fse6=v2I#XsGvj!fUw7017o+0KNO$26SXB<)HWdx;~8?(^xY&+CppHLL6>%zlQ5o@#fN$i z{+JXv)Dt+A@d-jIWjz8N0PxwYlPdZe1o}5N`Fn8_2HJV7=peg0>3?kcbP0zeLz;aZ zkK=4@56_1^a^vA7_*5P^oSeMhoY|`3%zI665bQ%p#p>HcrHnh`@{!S?s^A;J1d^Xq z@^DztKd6c|(cLZ8e?=jy4Xgc#& zz#uF_)*VSn{)0MVov^TW<4=UtsbQe$BpLT%L-2>Zek4EPvgzqTPugV<;x2P7r}vM) zCRbyjTz@nn*o(Il27Eg{x~|Ie zNBhE6nRBd6jyUdg?4SaICYz4MXhM@;97{=YIjms ziwSoTSmli43F!6e@e~Ixll5_0xYrHrOo80tyg2RTgomp`0r6I;%jB5Wfm3KAl$Lya<-_{IG`yq9=g7g4zaOjtq?Y{IJq zP7{_AP*G_wEG7stVHrWF2~`Av2z)?V#2Sp*iARP1RHIqFYp`PS>=$i(|GBVm4m8NkO3+ucN{olL9o}K(?eN(QVrv1Cy0s zwu3NP{bfRM7@F~{OiR%8FnRdP6jcB9%MtA__gAW#ir=$>qW zfJdTL;70@zL3swY5SmUt=^++<5Y9)RNCMOBnohpT6yx#RKK0vV@xNBJxrXj#`82*e zCZ#M7g=_XTcr=vYnG={?o?n~t^&5~zUdds?H4dLs#&v{Lk$?%Mfe=v`K^TDT2i|IUSZGUc20a9!T>jVpoXz_Mv$0&iW=5uF0BjUW=yrQd2EY{eX}6 zme(w)Wg0$i3*_i;+jK5KQ^{$ZFWKTfg%{Tk)Qk}#Ep@!sIvCx^;TS28QZtq;>$F(v z-|+3Y&gp1&)ng{!)o4x*wd(F>&0D4>jWq9*&l=QtKgI@Mdc@vdP2aq>O$*o9kgR(adn;?oj`}@sMAkTl_1KW%r?!(sjVHLJ* zetaI=wLi|m)^n;0wvV6cDUY36nbZQDt{-u&HN6uK_mDGYNbn6+s+@efQ0_l{kG%6t zk_9!H_LX{ zWgLqthCUA0ux5i*zkoTpMl>s2BT>ZNRw#cyo8T)%cke~?a6_u)9Sxcr%TwVp}fD{DtdG9IenAK`@3B^$d}F~2UFx;eAuH$q1=8h zI*Jrp2|mjM4GKxUFUWc46MT7^-UT%lZ#7mCUn&$?58Fbm-u0wvc34YvyWhcEb1QDR z#rGYeY~|}+NAR=)P4A4eU9&I4TubmBK~BElkcZC4hpyA~J2;MiqIZfMa-mh%@%X~f zjiN325OoHAf#ZMCxkJ;5ELNL$BMpvGDfVt69Dyi9FLcOYrOg^@4Yh^ZLmi<`Z!Os| zCAN6$h%hoDP46a0U+7%86W^iR@wE;i14e?EedRqX5i>HGIM70u6!e$$wb9~rQN%t~+Ytq1AvYj!hdkE1*lln1TLZVt6;n4mt>0AU=<}y6X*=u&n{wf%De6~o6r~_uE9*ElNee6KG&XOs=>I4X7d!scnVE>o^3y{r%n3@+YYy+3GLd8 zOrNu-LtDVKDu!%fnzVxtr;q0e!?i_hJjt#R+7df$%h54q!L3B3ww!4)Q<6#9F<)3) z;n$|h_Chp~2dNzUH3ZTWc}O#nwuR3QA$r4ZX&Wx^BO5MZp7s{bcU!nv#S<22YdMSf z@Snp2L}#s$sVh@g?PDf4Q@7f|SD$xUw?(@`s=l&81=Q9Lx5J|k`7R%#8*dhlACPuN z3^`0xA^Np!fi%&iv6ghC#U|~I>78~~>|>&S!_&@*4o9fpOmUC)64LBuszG{>sUmiJ zM4C7!zGhk&yE`J^eomZ9lsutnJjX^F^dNS;R5kixNPxeq47_OJN2|&nhWM(#ABNDd zQTU|<#%XrDP<4nL2UPup7hmvkzpa1zInj=3XTWqAjpR`_=RMkZq&$wE!llhgyTUe#Lj>} zT8>WtRV1G95mq=qiSpQf6&=`E;XD$R?*3J{Ovm1|@BKLoUn~Aa?gfx_?LKH>~cV7`DiqJOAeO=6DQrQE%iiGo5 zINy#eas#}kT%GSlmLa`u(!{7rw+1yPl|;=)+RUUfw1D+X3PxpUfjgKe!^LhZ++$KT zk{w1csRW(yI*|`on=*95dS%26>)Zig`-%po47a)iA%uxCY(jD~QHGzmgP@xtv|V(E zKrxfbFaoA1n{!b}r8@$qo3uJ)KGIAkUxoA0*h+T`m}?rJj-8M6vPrYh770sDszO^N ztTt&^$TfFMs5a?vh}F{)>X=kz@bkvMC<#@WD0qg6Dii97hFOYyxb~r*SXj)cN)rze zUvu_cnRsZU2rJXplK}6S1Y&%8B5Y!!%D6m4WY0?wH>FyMm$fUCG z2z5*<`;O4%8w!T9f6(KCz9v=s5L00oqsl%V8cm{okq-M5VfMp3Zs_{0z#rNd*W4cH z#YEYU^?2Y(Cbjc3V4fn*J{#6DsqAy1PT8FQ2A{{43k@dz2H^e5W#;cJtwqAedzn{?Rbu@yk#cN{36fdw#;N!6kNV!kIE)ndBOQwZ%%td5=T z=?n!-suo>gsYz6euCP`S*5WzO9T0~fMpZ2;-QA!q6V+m_ryJbIq-xO}CMZJNtBxKp zok`;PxW?n_0dahI0@DvVgBmL{sg>h8Y#mkpn*wcI2oMyHw^yD8dBa&6oyRKjDzMoZ*&?r=XBfP=?!(hB8rxyEC4KNlbK$ozKA6 z){9l^iXp!6lZ(`Yv~4nHK+exZ>UNp|r*3)+d=_T2jXZm0J_`q{t`|L1621# znL4XTT0e*@B@>|=sC#x-9Jj2m<2tVXq4VrvtUJ4 z)_6z=JJYP3ku@7mR6T;q!t_6=A9;R~H3wc{Qe)1!CHM*`y6pFa>RBT-ZJJ~u>QyIxhi`ubgyzx!1rm{tlXOY57^L5JCOD@ z(;=krE7X0Fh;^PsN-`-c?9c2Mpg+?>Ow4iHi!h~`;%}P|%db*DvUNwg+)R6sihbAF z$R8Ny-xBY9n8G&tyl@&_btZaCzK*o4nXEbUp`n=;I_AUiW(q)?RRB&w{SuHCG*dp( zM8FT5%GeLFjpnCONX5-mg0!)jc9YHG_xuj&t7g)2(PmY)do-GU1rudI7;TI3`vO&q zX-MVtVN;n8PDS961DTkjHAgy4UiNiD^CE~a6Dr{@sd^p5JAJ5XnOK?`%_;z^- zf& zKGqy$&C&YK&%*|1@Hd_yks-eI^az-pQuW76&^H)zNWPQXszD3jL`YLU`*6FmDq|ts z8Mm?G`Fm7Q=o&s9*EIW{wA{Fvb0XbIGZUV}!he}O0znPM#d z=7{lPjL61+<4`6|rYI1jMKN+7w)v>bM|nK<7%j3;Hx`Mmx%fK<{}qa?D*Wr_Pulj| N>mxMnXIPl&_-MwP>wvQA@&mEk;EZzx*<>MfBOy zt43RGQ>xNZRFJ4tZ4FBMELD`Yw2G3ps1`;1&$;(E!Rw!o`@7$B&OOV$_jm71qNL1H zQf66HW_s1Go_;J`(3TtqnHlDgIS;^*q7IrWryZQH< zo5#LD(r9gxe7s1HiH|TgL7Tw8t!@GnMO(-puYN9R+CAkA>b9i(?=^)7C`u^}7&wRY zg^Pe$wYk)_W3kqhfr>#}PH5Ct+><&F?!_svm!ok?GesqXl$N?B3CL6kR`B)PVt9Vd zK3LA3TcTkF@3y5!0@)kHDEHgwBF$!Bh7UQ+pswI+wj_j;jf{*E>aXD6Zi$V57rU6t zN_v>I_vj%LbspTu6Khg~Fgi&^^!KWIecTdo^~P_FhLzOvaa*l?<<^clqhU^ zG?)Dykpek3sMJciO_!5TgZdtisZD4`qco9-w3OMPmgMj80kthHtFVh{R&q~m6Q2HY z7C%tCK8A#DbZs5nJbA!?l~3Do09IYe7k|<%fs_VSm{d|iHlJ3=EBTF25_p%|==R+W ziv2XM+7np#5A;lV1t(D#l}DO?yPol!h8q;Xue;x}35xsMNZJ+-64`tedhk zsB8GL?FrnrEh-9M88VXDqk{Tt_)ptYLq5bV&gNQrn6z~~VMi1G>b3-0>vbp-YrT$k zT2;{we}E6gdao7jI=*pdw;Xa{P^o=zwaKGFrS`$qCa(r{9Z%ntz>jQ?^2gLjgrUdn zY8kQ~yO_>L^e|}~_*=W0@SZzFIvY?X(%De;>1gHz>Q(8Y4Ha2F<~!6=pEN*6ACmsjCjDFCVcCj-@`W} z6jtM=OH|Zq(z^Nfy@y)YV7GCb9l8HtAdQ%g8vgkHIDT;7dZ^*kKX30Lg+ZlORGbo> zV#9Nz@DV)ZQhG~YIOn9-gpVZL^$($PV9G???ORZeMSM&TleU!}zFq#+{o%<@gclv$ zEi{4z5n?QzR5VzaY^{2He`|m(V%K~u+O53eK(}NXYf!0uaM8$@L3s2n+II({dFwB1 z&B#{fDD^T?gNUVVW+pwD$&g=;WW)2Kj?7i?D4+=Nf>lvQ)pX$mM zsEja%8|_b`*1n7`m+k!f|Hi@&{?~tFhma4kGxAp8h2h&ALQ0#%YA&Ly$)G-ot@Z*o zQlO~YNigv~!447>Vq;JTVXLU0@(Bl%lWDl0Mt7GS>95YfzBb`L!AufN;yVv^PTPrH zDybj*E$dynsz0i14WRAj&O@>3G@x*c+^J-tO(ru%-A(0=%-Y1~@M4KZ8%TF!lyH2!;C9~8adRzjZ+QW_C zG>`EJNndOuC=2R@8K2TL-tn8>9uw9`J7NjV7QS%;UL&yT@G5~>ht~-LbXZE@ufrPz zfjW2zEC?w~yh5umXA8bUz9W;Ox>wwJD^IqBAin=(Tp;ZhJabdy6g7xn zJ(+~!Ca0!CFz2W8EWxP6^Y}mft5Z)83_+lIv(d{MihxI}k>QsX7Djbi!+NC5FbII#bXg1=aRyti-(n)rrAIHMLQ) zQkNu?WHZ^ULWz41Hu%Xcn-~4i(nGJQwJ^tsV+()B%LBi8#-qr675nZiA#OpfS*2sf z9Vq*B0J(!@pAH;%sO-~m;|`a7I&9pLvQGzyJ6iT_)Q^$<7+Ml{tOPoN-0`wc=a0LY z?8i~RIriPz*aT4KPySc~**xHE7i{~SeH`02&o0Nd^G|cI-TTvQY2JmNw;_gom% zssdNo9ZmCDL^GzSLJ&iOVqL&*UWn`W4%(@2{jc#o>HQ<_Nn^=R2~B^0s$Z^(q6&kz z`Iw)349~^HY=aL4`D@hkr?teV{TY1sW!GRI+SoVSZ=f%lHMH@w4{1~w)p{(`KCDq- ze-^vA<8pY&#W+t6n!A5R3kRgg!KfkGZzyvY;x_cfo!IEXY%gr=+oxjJ7goJOgRWvD zUAcdMu0$s+}F&~fUmol7)Y7B@ft>p9De#@w1)&rAzt2AjZ)G-Sd7MtIZtRl z!70?*-_v9(Dv`mx8Ws4NM3V8aDcI;cb>5n%wJY6vmV6{)liGB6rreb_t=$ z7;FqS1)GEYf-UYU(qpnhz>e`yiq83)xSe+Xrh1NuibWfP#r z1ZW%^(m%?dO@=LkmJ6B&9}DUtXdY}e0Unk+ah!>jL#^=A26auDSEpyfOrcHLTRJ@_ zr~--7semUGf41C%NC9_8?1=Pd@4^8=S8YVA;5$K=!-(F8Uj@|&`VbV{HleH!kZwIh zn#pP35OP`tZ3OKXAvQvapv6MB864(!(6Y?09@D4+PdeV4p=_ofWB!Pew&~PC=zg`4 zZVU7j)CXtMpVh)M!s!C`!hs9vXDf-WP`TpSUpUDN4e7|qO| zFiljin5p_(pqcT*O(@F?p}cQFvB-1>D*f4Qm@eo9rsmHYV1}Tzm^aZZ5#p!_!IU{d zcUD*$l$V9>a3lpZD)R*$6P6}rk)Y)^(gi8p3?2+0D*^^9ON8=-Fbz>≫&lVJi-7 zir=-C2^udbUZ+gkH^!!{nNIVfiPDfta9pjGLS!LLNBT5$RuIwKWQo`7D_NcNDX+k5d zMS4e2Y0UPpRCbYlE~q@FE-c4ycuzkK z!6A>~34v+qqmFy;pn=l)sJG#1i1pmSpFK!rpR|iC_-mrw{!?HCQmmj-%cII@q?S5O zfw4%bIu*lYq_&N^BBYLj>R495;(&{+Owewa0;^DWQqU4hbbWm3U?qiIYl9;!o7|F zNR@=TUmZcvSCFhQ42DRZWl2z(BMkC&S`oAWX`CQWspa1>WsY!oQCCimS%6fm(@fMw zK&eh%)J4ETojwa{a72Pvr^7);XCzbzk~0$p^@8NgM8T+UDQ%jWU}rSAC3$e|gPk!j zOQ4*ZSh%ATt$i#6en$iAaAsOK+Zb(91qN4>w zeNSHG%(R9Uf=Vr(k+_3fL$Z!zaR;}COr0uiPE#VZ*J-^i&5;OQbjlBOnv$TGPU8a8 z97)h$kgUE9R0xvQw*mWU3QyHP>9j+ZPOCkLDbPcptUe7s(21%~gBnR#{Sc=ElFu;w zLp#60;e-r9RQ)KY6NU4S0NK~AGxzJyboQqsA{75>Qizyyw z9)#$)I%bNq1GE+-=i(tK)rscfAy_O4=VFHQVKCu0PdOK5j!qCHh~{FhvlHYAl5^1+ z`bk3FtA1S|Ul7Ohah2241u6udgHD$9&Mt7X(R7QmD}??;A;|PQq+x=jWmi}vh_03M z&TdfIXc^_|2Ic2yoV4r)n+3_q?+&{K$z9TYAcUSLQ#m!=Awm#MO&ixEkf76Qq@K`L zker!b&`l?rnO@LO63$G9>oIu2duIeBd!`B@WteH|4P}C47JXo`)LA0JoTfgoOsD2y zX^uWnDM)767p}>1;soyt&I^<-CD_~b1hm&_HBvw5El4If06+2RNEr@**^)5B!LEU@ zRVQH2x(2~6L9})SNXG=x+KqJ$hBJ~-m*{v38U)EAhJf{FN~Y98*U%6M*NLv7ArSYo zM=I(58Vap+CEZ^`Ax$T`_l7~1PIT`LgU*6v_0PgQL2`MXg~*HKld4x;!=brOtC2=R zmLOSuJ}kK??lyUQ=fk^FiPg_`6+qM_azk(NJlAMwDu}9o1*xYXxpT+BC`qVW?s^WU z3X;{2h1pUkmT)Z0*NK*JEG$97`sK|&4&Kt0RN*+N6eKGg57z|A3dh3${9TExu+rs* z5jw4osdT*nFA9B=)C-a?&J;Lz-{-xlu<=*&LYB?br$Te@-F%1(nedxW*e!h;)OwQ(Ag;X~ z{~v<1T#~Lryy_c&DqVwl-c<#V-1M9f z(!jY+58P-pxshH1PVX-m7oPX}kA}FsDZl%MCuPil-GXHJ84z}jbTa%*Xf24sch8s! zCElSld~c(0YQ`+6`GW??;juw4@tk_6GCPw9PpkbpCvba zLR;n>ST9KWob!OsGRXbQXW22c45oNpW6M&a_zxj6Mtq0m!!%V)VY^JQ4rNV|~M zG}0lY%Yv4$7|RKy2AwiOu4m2z*KP7rjy3qToezB*DYoqbD7xd*bw)bgNc)kxdhQCP z_-?S^fg%<_U!kL~2ItZA8A0@c_%4##NXD!Mu&j~F{T9IHM)F79*+z;(n!o@9l9wE$ z9)MpcCH6vOijRgPb#0_Vq(zNXM>_lsksS8}Qf(tC?NDcwxC+0kOuhqXc1D0py}vl8`5n-RE05n0epl%YnR`|QrSY-CrIwi zg;3u}w(Ny~-%8Q4lyx-uXDxzAB#d9ml17!F(`BPE>vb5RlhIV}_XdpDX|gGmy#ezi;p=A0egodqbf^A=6(_h#O7B)kK z&2V9pFKqthcLFWP`klmflCYm5>`R6Vm?fZ044x+jFA{?ni^0o;O@**|SJ+evn>E5_ zqtMm}ZLQGm7LvU}a-e$599U-w+~MrvKP&09Gl036x8)+}5woK$Ng#{be`Jw-glN5! z7r}8D;eDnY{?TMaCsIuBL`G*sJH7L$fcY8Iy_?FRAmU8-RR5_-Egli0Mf*;*;}y7+ z8JaN7?JYXCw#(~(w7T(|Vf@T${H^GXYa4EW#E@ISCsFTV|Afu}3Y ztAiV0o6;wbwPhn&4$H^CBiRUSN3(SNJCbFx0+xmU3b2=lZ8kpJvQgNMM5|1efzo`I di!#z?pd^RodgDO(MX6|C9i}M1L3z60{{crh1gQW3 diff --git a/RLBotCSTests/BallPrediction.cs b/RLBotCSTests/BallPrediction.cs index 2e75787..9afc4cd 100644 --- a/RLBotCSTests/BallPrediction.cs +++ b/RLBotCSTests/BallPrediction.cs @@ -40,9 +40,9 @@ public void TestBallPred() var ballPred = BallPredictor.Generate(PredictionMode.Standard, 1, packet.Balls[12345]); - int numSlices = 8 * 120; + int numSlices = 6 * 120; Assert.AreEqual(numSlices, ballPred.Slices.Count); - Assert.IsTrue(ballPred.Slices[numSlices - 1].GameSeconds > 8.9999); + Assert.IsTrue(ballPred.Slices[numSlices - 1].GameSeconds > 5.9999); // comment out to see results of the below test // dotnet test -c "Release" for best results diff --git a/RLBotCSTests/PlayerMappingTest.cs b/RLBotCSTests/PlayerMappingTest.cs index bfb8b35..01266d1 100644 --- a/RLBotCSTests/PlayerMappingTest.cs +++ b/RLBotCSTests/PlayerMappingTest.cs @@ -29,21 +29,24 @@ public void TestSpawnProcess() SpawnId = spawnId, CommandId = commandId, DesiredPlayerIndex = desiredIndex, - IsCustomBot = true + IsBot = true }; _playerMapping.AddPendingSpawn(spawnTracker); var carSpawn = new CarSpawn() { ActorId = actorId, CommandId = commandId, }; - _playerMapping.ApplyCarSpawn(carSpawn); + var metadata = _playerMapping.ApplyCarSpawn(carSpawn); Assert.AreEqual(desiredIndex, _playerMapping.PlayerIndexFromActorId(actorId)); + Assert.IsTrue(metadata.IsBot); + Assert.IsTrue(!metadata.IsCustomBot); - _playerMapping.ApplyCarSpawn(new CarSpawn() { ActorId = 111, CommandId = 222, }); + var metadata2 = _playerMapping.ApplyCarSpawn(new CarSpawn() { ActorId = 111, CommandId = 222, }); Assert.AreEqual(0u, _playerMapping.PlayerIndexFromActorId(111)); Assert.AreEqual(desiredIndex, _playerMapping.PlayerIndexFromActorId(actorId)); - Console.Write("Good"); + Assert.IsTrue(!metadata2.IsBot); + Assert.IsTrue(!metadata2.IsCustomBot); } } }