All of lore.kernel.org
 help / color / mirror / Atom feed
* [Fuego] [PATCH 1/6] bsdiff: add test_pre_check for command "bsdiff bspatch"
@ 2017-11-06 15:42 Liu Wenlong
  2017-11-06 15:42 ` [Fuego] [PATCH 2/6] curl: add test_pre_check for command "curl" Liu Wenlong
                   ` (5 more replies)
  0 siblings, 6 replies; 17+ messages in thread
From: Liu Wenlong @ 2017-11-06 15:42 UTC (permalink / raw)
  To: fuego

This test depends on "bsdiff bspatch" being already installed on the target.

Signed-off-by: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
---
 engine/tests/Functional.bsdiff/fuego_test.sh | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/engine/tests/Functional.bsdiff/fuego_test.sh b/engine/tests/Functional.bsdiff/fuego_test.sh
index 6ebe3c7..dfc3f80 100755
--- a/engine/tests/Functional.bsdiff/fuego_test.sh
+++ b/engine/tests/Functional.bsdiff/fuego_test.sh
@@ -1,5 +1,12 @@
 tarball=bsdiff.tar.gz
 
+function test_pre_check {
+    is_on_target_path bsdiff PROGRAM_BSDIFF
+    assert_define PROGRAM_BSDIFF
+    is_on_target_path bspatch PROGRAM_BSPATCH
+    assert_define PROGRAM_BSPATCH
+}
+
 function test_deploy {
     put data/*.modified data/*.original $BOARD_TESTDIR/fuego.$TESTDIR/;
 }
-- 
2.7.4




^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [Fuego] [PATCH 2/6] curl: add test_pre_check for command "curl"
  2017-11-06 15:42 [Fuego] [PATCH 1/6] bsdiff: add test_pre_check for command "bsdiff bspatch" Liu Wenlong
@ 2017-11-06 15:42 ` Liu Wenlong
  2017-11-07 19:09   ` Bird, Timothy
  2017-11-06 15:42 ` [Fuego] [PATCH 3/6] commonAPI_Dbus: using common script to generate dbus test program Liu Wenlong
                   ` (4 subsequent siblings)
  5 siblings, 1 reply; 17+ messages in thread
From: Liu Wenlong @ 2017-11-06 15:42 UTC (permalink / raw)
  To: fuego

This test depends on "curl" being already installed on the target.

Signed-off-by: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
---
 engine/tests/Functional.curl/fuego_test.sh | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/engine/tests/Functional.curl/fuego_test.sh b/engine/tests/Functional.curl/fuego_test.sh
index 90d24dd..582544e 100755
--- a/engine/tests/Functional.curl/fuego_test.sh
+++ b/engine/tests/Functional.curl/fuego_test.sh
@@ -1,3 +1,8 @@
+function test_pre_check {
+    is_on_target_path curl PROGRAM_CURL
+    assert_define PROGRAM_CURL
+}
+
 function test_run {
     report "if curl -o test.html www.baidu.com > /dev/null; then echo 'TEST-1 OK'; else echo 'TEST-1 FAILED'; fi;\
     if curl -i -r 0-1024 http://www.sina.com.cn -o sina_part1.html > /dev/null; then echo 'TEST-2 OK'; else echo 'TEST-2 FAILED'; fi;\
-- 
2.7.4




^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [Fuego] [PATCH 3/6] commonAPI_Dbus: using common script to generate dbus test program
  2017-11-06 15:42 [Fuego] [PATCH 1/6] bsdiff: add test_pre_check for command "bsdiff bspatch" Liu Wenlong
  2017-11-06 15:42 ` [Fuego] [PATCH 2/6] curl: add test_pre_check for command "curl" Liu Wenlong
@ 2017-11-06 15:42 ` Liu Wenlong
  2017-11-07 23:54   ` Bird, Timothy
  2017-11-06 15:42 ` [Fuego] [PATCH 4/6] commonAPI_SomeIp: using common script to generate someip " Liu Wenlong
                   ` (3 subsequent siblings)
  5 siblings, 1 reply; 17+ messages in thread
From: Liu Wenlong @ 2017-11-06 15:42 UTC (permalink / raw)
  To: fuego

Signed-off-by: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
---
 .../commonAPI_Dbus.tar.gz                          | Bin 6320 -> 0 bytes
 .../tests/Functional.commonAPI_Dbus/fuego_test.sh  |  34 +++++++++++++--------
 2 files changed, 22 insertions(+), 12 deletions(-)
 delete mode 100755 engine/tests/Functional.commonAPI_Dbus/commonAPI_Dbus.tar.gz

diff --git a/engine/tests/Functional.commonAPI_Dbus/commonAPI_Dbus.tar.gz b/engine/tests/Functional.commonAPI_Dbus/commonAPI_Dbus.tar.gz
deleted file mode 100755
index 7f1b491803d44b4fb25b16ea50294e9bd1a4b6b7..0000000000000000000000000000000000000000
GIT binary patch
literal 0
HcmV?d00001

literal 6320
zcmV;h7*FRPiwFQyY=l?<1MNNiQ{y<Y^DFr)wAm^y@R)o<?gE=v49xCQ#mrE^-0r?N
zuToB;5Z^d<o<HDnTkPL{-TJU)TTYx10&{C^)lQJ5R!eHNx?8Q*(3wsh`}FFfe?ADj
z+MDM{bx57gA^zLw9M;u;WqQ+S)!U8w(NUug<r>Xa>+lUZd=_jHDe!#5CFG6o)6t@G
z%f5f3X&9gXpQ&Xzzc{WnI)4{<H-0cU9T_v9y4A^Swr(3Z!AD2!toc7`HjZ%qo6UCX
zu+xJ4X1(4%dPC})>}Xc{lh1!`zqC(oC#FZnrbWqv;gLIPQ`dm*lEIw#6H3k`jgX)C
zC&%@wQ0=n^WgPde3N_4ov&w17nBgy%l2)}*ZB_4^^~Pbn)BI~4{)^QbyM{e9c-eaO
zZ%`VFwj0gH-x`fp8;VNIZk)g!A{rw~estU^fjlo5{DTgC;yVmQpwOvDoH6_3gYz%b
zvJ7$+3@md<F3lmeJ$gXytE5@2BMuj1VvwOTn}=1dE?M7g$0sAx^IdZg;0QdJ{san$
z;W;3`IRP0OHgN{NVZuK)C58_GC%!-H)@oDUauvE)1CTXnt_JPZO5dAfdql_N?9+!2
zpFZMpV=JxCmRkS(-RGPB&F$xR{nPW)tK0Y2{hzO{`lauoh6ydKR0*20hgJYXRgM-j
zE@WK6W$jm7>gRN3IrAyCeO8W$gy6>HJC4YRR3cO^K0=cpPcQqBb#-xhdVBHd<DLd(
zUAfZt)E=2*1=o|1#@+(i)WW}Nh#ULgkH!$_ZjRXXR;j1(FzE4&bFy_ItXwxftUc&)
zke~zY_>^p8O1+r@RDa*-*U9fnM!3o#x8%=fl);ltce~gu|9`6MCvl4CkxX{a%O96Y
zL(A~I>~a@2{LeP55)w<7%=n7CH(ft4ED~={R5>Vd(h_Btpiug~#0z#=wWWwuN<>LR
z$M$^v;GB*|E`_e`_6`2)KvUE6N8PSBF<d(8&wTep3+c*r{y9H$Y#aEI>DWC0ISl_=
z5i0iRo#9ag;=%7dm;$W$$MTM2(OL})%i-s(4IIZJ6T|z_r9dj)SKpZd#xS(LNj<Ek
zR#{rO#PQ*vm&_gS&bbM6>klW?y)_0F-G%+d*W^n~GNz0H_XS^$1Bz!?<dEGv=7@|a
zunqGbcC!zLZQRk(1we;wC+c~r1Q|XMqIPb7qJ9WIj`T@5vE44N(QY?fnB8ui@FN;f
z+W0$lJ)n>9QxCfn)nn5-hV?GW#T#eyun_SN)Soz`Uts&X1{O5e;hqz3My-A@v*-yY
zTo?}fuD9Ap&3t0Ph&`@Th0h-!JkOXj<aOgjX0GGIa-^efjMzD;4Phbf<KXil^RO`(
zeb@13d<p+LwayLSsO<3y#|$8&sU<vV!73nLI<L<OG6(6i+VL!?{J5N5$Jz8!o|0hI
zY;vg0niK46>^jpv%10-ez<MMn3+JqRXFZ<1d49b*ujcr7mu|+QqdQ?Qh|I{^H^zqL
z(LFV9rHwGzlFU__i+^C<A;0s?M@CtX1U46q6q5XaJf$QAJp8hf@cPo)(4Q}5>i=w&
zJVon|61V?!GW7pOv(;<}{lDF+H(GD%jibg<tI+?qkqZ6)73lw0*Pnj<EsxHx6x3?+
zY|-ML#FhPL*4{U+%+l%10>UFFsMca3Qms}=Ka_Od|I>$=gzd`fr6e8Jbs^tK0WT%z
zgxg$ZiEsoHvX2f4L@rCVVQ?x7qz6`@2MUTs@1V~@8~Gy%L25Hn(T~29P(`?fZ$i?A
zRNLR@sPB_A!?L_kH0H$Y&F$gDb!;bqFLY|yphCcE7Bx!zpHoXp$1}r)d4PsI0=*!n
zJqu!)Z2iIVm@?tHs?@g~`)vxw11t|Rptzc>Xn9fMQa^B!M(+(Ppb<W|$bO&~H=ybS
z>FN_|v+BHc0R=F%M}X<b93nAMXhTT80I+=X`Rwfd%}p6t5Y#t^6E?KS2dL{p9w@)W
z84d#%=Bx^149nePpJCOr5(nhd$IIV1V+=8q;3a|{^<^JwF`a{HrE*3ps*K6B8xSRt
z?hjNMQtAJ0-xT6qlzEV-4@=7fP4^(JNn@Q0d+d;X00aO0K<vP>fG!2`EukDE2kD3@
z1BL6}9Kj3?0nWe}ehmi(Gt`-LQ9BK(2)_cIfp$e<7Q#uBL(eD1Bcp)z73*V-s+LR=
z1Q5-<gt|Hb%=1#f#JGozfk?*7h&o2oY|6#~kgG!bY`YEDQbg>5CFtGo`QQK@iG7O=
zUkLfa=at0pvl1S%QHB5+L+_MPGNSb~ffWPr#bzoeO;5oJ>L*$%jg;|Iq?iRnexsH<
zlcds@?XO6}DfV|Q*g;%|5vou$f>Q^G1bCDWvJIbr8M0osHw8AMA&;#9{dlBs47Jod
z>Wkm1?H~h37beKmqJ-0?poPl>iPQFB;e5572OH-NCegSzObeBBJ)X?8XL_fy)P`|d
zvGsSBGyDp)kOK>7{lPV6Gia7?`u+qK0yG(ToMtN{`gYHycObV3)If|g$2fDiO%r3_
z1il1|-~_IwW%~0BI$*i7fBG&ZH*-$?@9$w|m~Aqd8Sdq4b<!9Pk*MyuIglXryHnP|
zcn>7x^*WL?JBCsZ>rv)c-nE!jEGhbEM65{q5!S1;ki*KJ^pZK}bZi6`5>2UAL#ub<
zJGlT!lA@82jSv=ENw`cRkRko`REz1#5??nox#+-G_BpK49B?&<{h7nIeqfHCuN}-v
z2zYM~aEpvZ;UKRM5X9Wa73K5%LztF%*JzH(Rr%X|9TFIJG&VzTdm&;13vF{z3mnYx
z%EOgo6JLEWpMATR<HaQ=-LP>K<k&@!*Bz(q%{~9{q{V>DZm309N0#le`SH@o0-1=#
z%5D?0q@0ik!>G>cm@eGZSDN$m_P=P_b;80Ea9^;GuhfYJJN}~HEEE)=TPvO>EIm$8
z>|!r%Q2oZzLjOO-du8~9g8=5_*Ut3+YcyLOX8(uGPOH^;17ADsW?}!|Mk?(8TeAP>
za=XiIE01y`czv{?1B*m+@LP0VKouAmvuwd*Uqe>Bpon`H;9uD#-nou3Ld8K}ge2Lx
zg5+wf|D~75e|*a1|2NdVhlTSb#xK$m|L;b-*_8g@?K+lg9JO18|Myl><jZP0!1kyy
zRsCE23FM6Oc)W(c5-zd{CuFqCa85@`frrXrwBSKyJcevgc*zu)YbaqZFsWPzwr@^p
zkGT8?`GKUrGC>vu&sq)s4ND61#}ZqW#JNYNbFm_z4OV_?e3focx(d?2P_*~9Ck>PW
zOr;E3ieZ%xNZF^J5C4WOAgZA`(-W=oBN#ONaR){=a1r)HC-BM12`S$M%uqNEENf1#
z1r*rH1nlq1SW0+ujVxi2e3;<oUqOQ}jC!rcc0AcK^wF}gA81;o_ka_guNk)YfcA2S
zj02nb5V3Ouk6?c-V)|r2fw@qRd}W6h)zBUl$M+Y*^pOtGY|8+kep5LeIIhq8e0F<%
z`Cn&yNxWmw5Z^v0R+vbO(pewiaeEg(vVv?t#~yi=R((%Q&0~qEDoh=6AIYC=%gN^d
z=%C2WtWVto7W@Cyn}<ij|EJk(HQPrh|2I4Jg8y$L74rX!$^YsKi{H&TF+5<tklV}e
zJfnr1^yn7Z73J)KpBgcn)6c-yE$a3w*kQ%n)J#}&`jrT()=E^7Ck83=sVJ~ZGSJll
zTd@D1Stj6pG3t^2ko`<prBawP>r=e#>u3P>v(Pea(^nmM^}fN{>n=U!t;~K(<Ptu_
z`g#W>6HPR^M+N&JkjNH})z0r~rj}dZ5{D(Ru4+$4@(q6c97Gu5B|LTwq3@GEb6JSX
zcD#Z()PU}k%e*7JY(;z!ro&89zfN^mmsio&Cj0#n|8*03a(AV(nDV7xiIwA2{w-^?
znfw1{Tz^?ww*NPpb-DkyI}P0b4;#&PvHx!)75o27`TxGZy!_Pv<<s@$d8X%YvPkK<
zJ$jex`Dk49p64!W(xW%wTPwX)6n6nM5r7sab`o|O&a^G;Z{liKC{swSJYG62py<3^
z*xA*w6lyr^EI}xFJXR@3izBL2=3APk`K1C@9DEu=rEx&&YaO;le@E+{ZFAZ~xFm_b
zG(RiSYANse-&3T4W#@mLPN@GMHV<3;{IA&mw~~tee=GaHMgU+}AQ}JU?DPEn)#az(
zKD_^Un<Wn{tMSaz!4tw>Y*tcObod$Wl)95i=IQUfl0(wezP$D!sdi^xMl53eFXaFH
z4(O9l0GIIpPP<-@?7xi;p8s`Pje`GgBNhDrMg2d*Th{qqKT7$79Q4T0r7)A4HY=qS
z11;=qD#8M)E;~YRS#XdVb)phQFHpfS-R9!s#qGuE<;DNJzplt5Qw<E(W(rjiQc9c;
zM<=u#eKvADCI}4HPPV28W3#iHIna9vlXLh%waveVX({b=(?b*?rk{k;@7yLUj|vi&
zWu>#P%i~@qwZm+0M*m?ol+8hUCQ@-|#J0|QA$`xefq`GHSPEgkSCQA)_Ku$qKQT&`
zXTU8^EF#)itaN6L8|s>dW&R6$CWR_*aou%Cg7i{gdj}iEG9RWnLmS2LjOFyYUG$Ym
z1bK?bKHJA_E8e@ZT!iIhej=m7Z&&^|zc)d;WH*JfI0ljHYcCE!SrPFMy2Y%k8J+pW
zYDZ^8@p2%7_<zSst`()mW86t3?{yaq)7sk2mi#nS@TDx=(l*h8J*|Z4X4w;8JKq3n
z`rmN9{vWC6Ws$xo#E;F;f5_z$tA8iNq!PBM;dBYQe~fo0lC4=hy-I?y#`t6&0S?;z
z^kB>5g#h??<EZcN5+}_3VEhAvBWORFw&TXHaLJUY53Id$oK*m!K!JJ;_wZ+y%_O71
zYdntMdRplJ$Gml)?f|~T|GVDlB;&sy9n}l}zpbR@j@qh+_|&k?SODb6X}pxKpY%S%
zJaJ^JEBr@hX`=I3;+0;^U5WQwIxXF>zl1k+ERQ)k^C&vZg_*U(GQTSyM?gb==&(b7
zNfiIfUYGE^82l<A0>i{ge6h;{g}CRGT2dTh%lTg%|4)Vzdm4=`_5W?xW&FQpr%`A2
z-y;6sc2Z&geKqm_-ksjO&mROZDy=mx%K(7e@)x?NXiM$oaUlwA-s(moFOe8lJQr2=
z=XW&<nVoR_OoRta1_e+&lUM~Aw?>#5cu!7n&#Gt@f38jJO0$5ntL*Oe^u~lR^7`uz
zBo_(iuRL5BM|AUnjgyP>rTP1^l;iw&Bm4gX`M=d_$oRjlPNT!(|90Ak#s0sIRP6t+
zVE<>n8o7f2Crjsv1e`nmz-xK#M;vvo{NT>I&jcw#NnT+if-YXYSYQ4KG0@8&y70sq
zk&Lsg-V2{$5QopwQueZMFoZ9-q&>bPT!riRs}+j&UauGw#!8J1+s5p>06i875<T6X
z=K%?o3BL43`i-Vc`H!2+HWB_~Y5b2)y(#U#?PeR}KOHp>i}+7lNsHru$c-(^32cy#
zMtC!MBQZe~|2K`DITa!#@nBV+a3SPG6B2~15;4S(u|lM<$pmEBYb6MWiK2O;-k$U@
zkO<$7e~3bZ3<6UYNL1)0!-ODGthk8090p<zuR6mo9>tq2U=iXXZeu>1P!=FD76>^N
z83gNzz#!iVq!WQ4_i8oZ(K1d5AOSr(P@_CY5&5kLW6mQ=^FSaw0SnVnY1H?0R`3u)
zwOUQ#mApi)Y~jQ3=W)_tFL3$5H*jgHm;DNbhVnaS0HZxrB!E~&oB$T4J`9dO#%XkH
z5lsGp^62kUhKAV~n<MKj>JL>-Pk#c9LkJbpz8`B(vFx+W6#o|mEO~Ogl3eG1e>|;?
zHO~J!og)6zR#I{PSDgPXUjI*z1F&@cH`w}b9X24l)#m5_h5c_UNgAf2n?m?1-^GWk
z%l^-iAxi(1Y%VTbEDjDy*+;_l9=<TV_~lX=)g#N~5%HJfEK5UHk!%ygX1NaT@M5h>
zU3gzV5BQwJ+EPHS_+L+C0ZY#RTW#3vh5i4qb%gtWr`0I*zpbRg{{IT>|Lh^u^Y=fV
ze!k4S{JXfk{ty7ya8ii#LV+!>v6?+svrn%0qf#D!*NU-MGxo{~(?v6bR$<Uc1e3sy
za3|sL%_&d3on;d`o%!=b69))Eywu}?YR)50Leqx4uq)oQ68h$FV%T^59W4F?mYD9a
zIKmSogmdZAAvNz=uu9hFPxsVy`Ab?zbAT|uhcpKwaBOQ16k{_Al<#dBNS26~XeM%M
z0R5UZg^&6?I02~dgGfaTb$FpkAK+NkAXBt8au{9O6rTXI=j>%YF$D;8Kw-#CSR;sa
zO}lrV%hE)7{#ov5@uCa&%rJhO5HlA<Qrl3-v}5(Xd0im6Cr6Ca8pF4wRxq!Q#OvgI
z{xreN7SEfy)VQ*uP9vr@W1-3MiP`isNrUXG(b7bm)#FmegXv4+x9NftN^~?qb9B-E
zHh%D+^4|Y|Lx+X(--cqqEs_6@4#oQq=>Jh?@n4SW^+NvJN-E^P*CPMrmHQGtA2H9#
zQidS7UFVJ!d_g6b*}RLVkXo;~MCVDP8BC<>Or;Eofy^gtW5FW|3#Wc&_`?bA+&~#1
z?TUMK;wf8C$cqM-1-alx!xIo?z{#oQg@PfrZrJRxWDiD*3}j>&29HHk4+G9{k#+Po
z{)8nU;|#DFcB-OCyFjyYLgn+Z0vnVD$tj4&vzaB}K+$_@Irzw=H(^p}C|BYWq7Hi}
zHzIi^iDV0vLRwquG8+qOE2M{%V++bGp--(wARAEws2pX84~7l%IJyvV@bOVK=F6E7
zqm)NB&)ER6CorMwzKKKtGv@)3%4dY76vsxI%W)p5u`@O-&g6DfOe9E@^M^18(WAK@
zwa|M7bQg7YS?L-RLj*xeWU!7+F;u_d&K30sz;TLH^&EQ75OS%-__Am+A_D}h95~^}
zi}OpQUqEuEQO+1~052sltdj^3@gE!o7ku$sfbC+1w?O!y5ruRaJ@3p02>x&$SPxjC
zxp<ZI{{!mz@)cx8?M!82eZt;h_<w{>2(8{#`5OmN)|53WWyj{S;SKdq!q7|~Vcqx2
zdsTrL5c)8IlY^QQJ3a-S%l%4>F+0RnM@}?>Z$1wwwS~es;^;&TCt5w@rFi2dBDN{M
zMp*xZIyr#F_P|C_<ai8E?SjNrC}#hjsig*>?tH~JpMh)R^RNbz02BySVq`<nTGua9
zMW`+<ji;HYAbqSds5d%sX0ty9NeZ>v5IxS4P+?VGv=w|uFvgO2kT=ahv7=%Eh~-#a
zFwn-pL8HM-4zymkPO!n%LwzSEE&HYcsl?PydCj+%&^)tU>#${<@yA$!XHdL{;+Npt
z;!R|X$;SK`IB|(L(7C}R{&qafrlbsM{!wbk{#IX1sTbYC=DGjMlZhY|fc`i}3>$#2
z6vW7QB=Psg+e_>EtXX|_y!C9ehw^(-HP*S1bf3*e@J(F*SbYCOna-y;0xmiK?Ho1^
z!}Fh`Cg}godcC;+v7J=t|1TE*Z}H6xb?rSve88BwT^GDgwcYFD$gFooAO+y*w*}S%
zj@@Ig+yg96JmrBE^93eUM^7Ia?|^WyIinOR05z|~EDl$w)Qtx%L@0~m77}gLLKPKF
z>_X-9g}5ydbKS})6T0UvQsj>{H&YZ?JZ=-l_d#(;JSD9OSffR8p#cF_&0|(z|4fK5
zntY@^ECTtigMof=_J5sP=Z0@29xWG-4y12(m}*_DNPY;3x_lx8TU_lwd?{<h?l0xu
z7rPt}n{;Bhcd`o8ale$AC<K270sJ1BE^iJpoWO^iTCc1$TH~$804GtA^6#?tnYbLa
z?<JZyZPxl?5JI2Hh9Wx=M_BC!%Z|N}lp6Qyu+N<2lFu?ktmr4|hqeHAdL=A|YCN%S
zcZqva#In+gRro0xk}it|qf$w_h~rn;jU%CbSmJ;Hd6fCNq<#PrF$Z;HkhA~%<wlH6
zr3TLGt@d$wl&v+YM4r~UCPA))#CS6RsX>>84|r&^9ZbnkU26xzOYeo7gDr?caLJ`i
zIzAY;Xt&<YNzt0B$p=robP@ixS%z4`nv&~wWn|(tb-2V!B%hhfgui^-d?i;hSAdw3
ztV%oGjG%>zh^ZH%#)jq5kgAu^T$Uila2?HICjq(_S9v)7$Jb`B!tp&I@n&_jR*A>3
z`yKvMA{gS*CXXmz={Vqud_)Ojis?~V<QmH3!b}$0Rnx|YB`XYH4f{vb*TH;+Z3sHR
zR>#0`tUOC_Vt7Bgl#ZzP)purB;Smn&XsyP{aU;N#CCv!n_*;M5uSi8IQjv;Oq#_lm
mNJT1Ak&0BLA{D7fMJiH}id3W`6{$#nzVvT4*9}Ji$N&I8dw_fZ

diff --git a/engine/tests/Functional.commonAPI_Dbus/fuego_test.sh b/engine/tests/Functional.commonAPI_Dbus/fuego_test.sh
index 485d279..97ae6c9 100755
--- a/engine/tests/Functional.commonAPI_Dbus/fuego_test.sh
+++ b/engine/tests/Functional.commonAPI_Dbus/fuego_test.sh
@@ -1,25 +1,35 @@
-tarball=commonAPI_Dbus.tar.gz
+#!/bin/bash
 
-function test_build {
-    $CXX -std=gnu++11 HelloWorldService.cpp HelloWorldStubImpl.cpp HelloWorldDBusStubAdapter.cpp HelloWorldStubDefault.cpp HelloWorldDBusDeployment.cpp -rdynamic -lCommonAPI -lCommonAPI-DBus -ldbus-1 -I $SDKTARGETSYSROOT/usr/include/CommonAPI-3.1/ -I $SDKTARGETSYSROOT/usr/include/dbus-1.0/ -I $SDKTARGETSYSROOT/usr/lib/dbus-1.0/include/ -o HelloWorldService
-    $CXX -std=gnu++11 HelloWorldClient.cpp HelloWorldDBusProxy.cpp HelloWorldDBusDeployment.cpp -rdynamic -lCommonAPI -lCommonAPI-DBus -ldbus-1 -I $SDKTARGETSYSROOT/usr/include/CommonAPI-3.1/ -I $SDKTARGETSYSROOT/usr/include/dbus-1.0/ -I $SDKTARGETSYSROOT/usr/lib/dbus-1.0/include/ -o HelloWorldClient   
+
+tarball=../commonAPI/commonapi.tar.gz
+source $FUEGO_CORE/engine/tests/commonAPI/common_api.sh
+
+function pre_check {
+    is_on_target libCommonAPI.so COMMON_API /lib:/usr/lib:/usr/local/lib
+    assert_define COMMON_API
+    is_on_target libCommonAPI-DBus.so COMMON_API_DBUS /lib:/usr/lib:/usr/local/lib
+    assert_define COMMON_API_DBUS
 }
 
+
 function test_deploy {
-	put HelloWorldService $BOARD_TESTDIR/fuego.$TESTDIR/
-	put HelloWorldClient $BOARD_TESTDIR/fuego.$TESTDIR/
+    put build/project/commonAPI_Dbus/HelloWorldService $BOARD_TESTDIR/fuego.$TESTDIR/
+    put build/project/commonAPI_Dbus/HelloWorldClient $BOARD_TESTDIR/fuego.$TESTDIR/
+    put `find build -name 'libdbus-1.so*'` $BOARD_TESTDIR/fuego.$TESTDIR/
 }
 
 function test_run {
-	report "cd $BOARD_TESTDIR/fuego.$TESTDIR; export PATH=$BOARD_TESTDIR/fuego.$TESTDIR:$PATH; chmod 777 *;
-	if (./HelloWorldService &); then echo 'TEST-1 OK'; else echo 'TEST-1 FAILED'; fi;\
-	if ./HelloWorldClient; then echo 'TEST-2 OK'; else echo 'TEST-2 FAILED'; fi;
-	pkill HelloWorldServi"
+    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; \
+        export PATH=$BOARD_TESTDIR/fuego.$TESTDIR:\$PATH; \
+        export LD_LIBRARY_PATH=\`pwd\`:\$LD_LIBRARY_PATH; chmod 777 *;\
+        if (./HelloWorldService &); then echo 'TEST-1 OK'; else echo 'TEST-1 FAILED'; fi;\
+            if ./HelloWorldClient; then echo 'TEST-2 OK'; else echo 'TEST-2 FAILED'; fi; \
+                pkill HelloWorldServi"
 }
 
 function test_processing {
-	log_compare "$TESTDIR" "2" "^TEST.*OK" "p"
-    	log_compare "$TESTDIR" "0" "^TEST.*FAILED" "n"
+    log_compare "$TESTDIR" "2" "^TEST.*OK" "p"
+    log_compare "$TESTDIR" "0" "^TEST.*FAILED" "n"
 }
 
 
-- 
2.7.4




^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [Fuego] [PATCH 4/6] commonAPI_SomeIp: using common script to generate someip test program
  2017-11-06 15:42 [Fuego] [PATCH 1/6] bsdiff: add test_pre_check for command "bsdiff bspatch" Liu Wenlong
  2017-11-06 15:42 ` [Fuego] [PATCH 2/6] curl: add test_pre_check for command "curl" Liu Wenlong
  2017-11-06 15:42 ` [Fuego] [PATCH 3/6] commonAPI_Dbus: using common script to generate dbus test program Liu Wenlong
@ 2017-11-06 15:42 ` Liu Wenlong
  2017-11-08  0:04   ` Bird, Timothy
  2017-11-06 15:42 ` [Fuego] [PATCH 5/6] commonAPI_C++: using common script to generate commonapi c++ test Liu Wenlong
                   ` (2 subsequent siblings)
  5 siblings, 1 reply; 17+ messages in thread
From: Liu Wenlong @ 2017-11-06 15:42 UTC (permalink / raw)
  To: fuego

Signed-off-by: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
---
 .../commonAPI_SomeIp.tar.gz                        | Bin 6533 -> 0 bytes
 .../Functional.commonAPI_SomeIp/fuego_test.sh      |  44 +++++++++++++--------
 2 files changed, 27 insertions(+), 17 deletions(-)
 delete mode 100755 engine/tests/Functional.commonAPI_SomeIp/commonAPI_SomeIp.tar.gz

diff --git a/engine/tests/Functional.commonAPI_SomeIp/commonAPI_SomeIp.tar.gz b/engine/tests/Functional.commonAPI_SomeIp/commonAPI_SomeIp.tar.gz
deleted file mode 100755
index b00f19aaa659a2b9a4af3be583a980098db12d7b..0000000000000000000000000000000000000000
GIT binary patch
literal 0
HcmV?d00001

literal 6533
zcmV;08G7a)iwFR)Z-iI?1MNL)Q{%?6`!)U*6Q3%M;j(@|Hiq~XVM*dDc9#Nja&vM|
zxr9A{vO@BeWC+<Jzx{U4OCxC{%V5lIqS>uNmgdnj)6>)4)7>-mZf-pH;AA{G^KR&P
z-gveC)Pyt`^w{54uZ@3OgI<&U7U@;1)9kjI{Xrj|o2^c-(|kpG&j3p*ErQVU33>IA
z`mX2AgRFdENN1aXXTxcloc|xG<9I)NzB6MlPELI9_xt*EK7V{0`Sagu_4*y0|8}?A
z>9<?`SIu@49$%5>`ns8&{^s*fL~86*<l_!GzqSK%X*-nMSpm7CF7++wKDoFj;WZ_P
z5@*N{{FCPgb*=bM0p#`kt2&glZ|yqAF>8)bJ|&%ctKO;KwgDHKgZ7&y{4<um^euO4
z@iNW&H&6<SbX)D#H?3Bus})H?e&#LwDS|np<a^J*A@D9(T>M6-AqhR!g^>C;0r4)`
zk9Xc5w&PgjWO3oxQ*vxisT<H;a$6_udQ*pOd`T=a_2&1g=*ck~Bi9Sb%nm}|zF6Q$
z-Pz$aWDvu+OMdYdWNNv@y9g~C{<xG_p;qyA7|w@{#tnz04&84+)eUIAp|#&okN>dU
zsk4CoRn6=5dOokJ%vz~S^7V{f+Ae@tJN)qO-G}$M^vCbdk51no98ciY$@uu-eEi}4
zc2tU&Ao=;D?9`pvmz6ZA`wDLw9Ms0aZ06G-IERT297H38dt`un;Q|cH08rUo>xKq%
z3sB&;HEEJhh7SrQ;j8)_gtOr=xVC&co6JK$f_FIN01t<3bVp?BQ&<qjp|7dxOlj)5
zK}ZZ0i=iM}6N^9X7O1Bvz=xje0`l9QJAwwL>ese5Hkf=GE_|0)dt-g16JGbev7@7!
z0+r}>?x`vtqM;qXAD@p8j>rE#I<5Ki3RVjBjkP90${wl+Ylg!s8rHU7k8D`Bewdzt
zs^d*9XBg3;sOe!ygRq(&?B?&ScBi`we|!DGn_d1J|8#qOt&15Tes4_yu6qTT>@uGX
zhCJ_Rc<s&7uxg-T>9GsNE&*k0+i4)Ta`4bR55}y+;g^rei8h8N$at@eSWBV`KI!8Y
zjohtg&$5Ouu<qHHn%)`M3*0LJ5{b^P4$h(FIA@_1E^v%&oL|>+qL;gg@<6fz>k6t!
zjX0JWyO$o>fod+joTc-Y&K>XGx^U>eR{5@Jkj~gAUf_nh;fT!GGwVl9^agmQ(4J1F
zRuGz|Dp|+QsOByl2PSqqd+n?PMln{k=YTJUMKIlS>hGs8+te8KZOgI$z@8oWR|{+~
zNSDkWjm+f}8>Y-$1^RC+J4(GBrC0J_yT7qFqu~%X5r@<5pMk$Sh58IcatGNy(}wb<
zN#CU3Us?|EnyJ?{wxr2_6aIHdwha9OrpQ`2p`-(b9YR~X>5nGjlU0LQS&8K#Uz2YW
z`GEBg|7AG~*tYb2NItQdyj%dcLHCW6akPxo9t8L9G>6`v>G`nU3v2ojD86_qsp(5l
zQ^8-BL{)!N5>-i5`I`QM3F@zkzWfKoYa_>JWcb##ZWWbI&@N10DcEX87z~s<v3fB*
zq&po+i{WDK&{|@{Fw*~md^Ldb;m-XR^#fqhYYPCo)t`i0#8WTdS-*MyE-`+xUC-YZ
zbUr>FiSn>}wf3?yU@efLX%wd(NTXrWDr99qeMIG@RerIwRR4PxE-nsc)*LVfwKI=t
zV~PI1*KaHRZ@brm@9kEv+bs3Jn@EkF$|Lo?kJIz6px;%Hty`bFc0KMA{q0Vp@(=qm
zp~C#}==k`<<mV5k$8RTRAKo2}PbO#QKYcejczbYiesntd@#JJuNvmE96{{H4!?2LK
zUbwQF9=M?XSL%Ip^t~D_;k+K#_ZBO$G_O#)EzNg+Kc~UcT!)@}X<zYdm4XWBl;Sx<
z;T!@4tUlAGo*1J2_zb26qg08#N@eO;L6DboEI0tRQ4PtQuasNc4;Pk0;!TPoa_1L$
zmx59GRN+~NtY86P;;i3e6^!dqD6U#3^yWJ&pjb%&&%pJt3jQq8ZOkk<BHqx_vE>cE
zwE?%n={5Dwkzw2d${iZsOA$PCKw0&npn37C;;(D6Ta0O&QCM*HE$Xr@7g)X780rU-
zP6OT}E;gDP{DFq5;yA#OoH88Z8X87Q`<Rv*3Pm%dwIH}Q^=X71iOMN$y2uxAoJ~VE
z)CtXx3GjkIGelzCvd-j>R{K~)B6w7x%7{oUOL~0B+-Fn1bW!IZDhbl>CLBi#VJ9me
ze+9)KiqO3{y(_Rd6I724C3RVof)ZT%-pvGAgw!#aR^5mcriQh3ovb9&a}dU-x0xiI
zkq*LmhNBOZ`TS7_un6@v)&f5`G+c0eVAz!@?U|%J!4KgVjeA3QCJdl#?ro#|e~$d8
zED4_42ABT^S@u8ePP?V#zxJT>3chywy;A<$L@MRKmy-W9QgCwm;eUR~l=#dUjWQlv
zvPe#ap!d9z-rB_?dNX$b@yI?3NSKM#>vb|wTm8}RA<QI@=U|lD>RH#t_Acd^)Hbg)
z+?hGl2qrdu6A%bo*0#=V4RSU<VM#Qj4!wdt3m*N?WDSzzmLg;HohW7&1=>i2XzU1s
zxE+G}-9RyO9GOwae(ig%w}3D7#&Ut%mDYPZe0X-qf#Zm+dv5tK571CRpclk;=Zn}&
zw*Fvw+%V?2u9bH^_jRg{J6Im%g5qj&qUA-!$q>A`Mb&cz&pDExNPC_veob9goVPA&
z0Z82$KsvLhh)fjP5R4yGSUvmc@bKvDtO_Iu%G=XxHnd0wsPDtOYoG$&bh_|i&gwA6
zu-pUo8CE^Zu}eO@KmLUiMg^IyUIG}<Q1*cv?MZ}VlGEp5FG9X0@|<q32<BJ?P)AJJ
z-1lzn8O)G8b5H|=FZ7x7QIt|xgkJ&AK*XRh3u@Bj(DR82$ZWy-isi9H-H0X$1c+u{
z*19?Z+!w`wYwH#^1|kVBL+Th#YX{&F?z8Pyt)&Rr7mmPpE98R%cqH~MHhik)amWja
z;b%DlB%=%fatXcDM#+lS(>1IZs9$WRa?;EgT;S|}V~6xH6i$)9Q$Y^mJd9xXqY=FE
z07-yGg}~eJ378}4WqZ@YWjGYD6<{8Z6pTttyP~0Zu8DJ~y=)kLm>}B-5>A^27A_M+
zPB(;w^U?M0T%0$UMC;bF9h6MXa9(&GEQ;Vj=2|mME4Kcf<4r#TE);+QTEFwH`5c<%
zn|^o=3jvy31RQ5;Gx~a4npiT>fQ_>+aprKFCf0?w2xYZcoq*ML?C?GV57=ATKXaFo
zn>ok+qg$96raw!*RD1cOI%!R(h*Y;+aYYdO{TtT7cn@UFkLyTML8iDK)}zd?ylXMl
zP?GgAuviiIBh)u(CWqzSA0^+s<>&Oc<{J(8pZre7z)4cnX0Z{%Oe-;$NdPjqzmcxq
zTbbh<Dn*5<`q~b!YjzK?+SAG0V_QGB!Z8hIO;x<T4X{OO5H-k;uMpVWhZXtr;zO9`
zc^}bS!i->9o1X^<Ce=%7hF*^pU;>I<Ta8_A%mRyV)!|yn0I%p^Zp<c6*pV5Aje{V^
zE~@#l<CML*7ayK98<5!zJ?rYgvOTsqTn1hs6R=p`O+uFB6N*3>m02Cq+)aI@IZto@
zdDCuU7M=im&OW|UCvtZDyx!z83g}j`nks#%E?v`$y|hC03rl(SU(~;~r}SyQznJp%
z|IJpn*;V@gPJ4j*|9)ps>i;*9A`2MXgYBAHH@a0!cnxo6W;+@6lrW`9m>eM|F5FQo
zivZbyXmtaVum`V!#UVGA?JAm*!$hAh+|a%O(ZS~r<Xe*dD*W#D8V&fXatelbiF6`i
z9+8`Su>zotJeNpWK-yV|_9Cx8QB94zAy#)ul{+0qEg+~$fDJ^eT<M0Q9yhCC0~bL*
z1)-Mg@004;f{BKg3&*(!fs)q(LZhhr>ni3F2iG%4$e?%Ec-&J%5g$gq(O@#7Y#I9K
zIM@#qrYVS=OrvVK!5wNyJaV~kx!2s3sR^;aAl!%Kf&w|A0r|+(RgvRz9NwQTJ48G{
zL7i2AJ6i4F!t+Dk=fm^U<NrF`PQo3l4dI<|^n{6YD4kEBJ}%*KB_+HDaNJo?>olbm
z@<WB_n-&`NK9IllmXSyQ(Mp30+Utn_kV*gBy?%85+v{~%{D(nl|GSY?p8sy>{MWoY
z%Js!I$Y__f0(4}rUyq6(o*L^-nxBc+HBI-#wT4XQ_z|eQL;Xn&JFMB!*a>caQp<8u
zm#!L8W0Vq~MTRYs0j{aC9B$yyu>tP!Yy`4CdrU|rDae`h5pVl68i1WFux!^3(<WN^
zOoX!hLUS$lD1l3uKbgzzl1wmB<q~C_0DB@^*wZ_|WtdtneI<Qv^sX8;W5otPt_PFO
zjiNqgGdT&#gdJJMIXzw=@+GZSc}IBOTI74mJo2+p{WR5G)4Q6nG})O5-!FW*E$j#?
zQh}<zOlq-w9LrY{W?!1%C(=^?pU5!u(M-S+|DRTCpya=9uaEJcTHS7G|Fw}+?*A{h
z|0mZMx7jMV;YV+y8YjpI0Lejvpx42iPVo>_xU4ddVfIxi<8rbH7h_Y76ur~h|Ds9Z
z-!kRU|8<!Dvb0S8Yc<<~{x{qGE_`qGJFRX>|2L6J`u`dDe-`ld<Tk(0ZP9De^J?yr
z>o)jJ+VCc<X{CLNW*~qnV(EY{Jhg1s!5(sPWeSqY4vEK2+cV@&i=-y|7Is>-C&gG^
zcXoWXxae7{n!?Rhxe#M$eyK<uuRdKvt+h*<k2+q9en%^v2{y)Kw<J=&GzBZuQ_?E(
z-(&fKW%A#kmDvA5{_FJ1{eKgw-2XSS{~H(p;s3|zDJQ3Ik4}z1{POPT{dpEgu)NGO
za|k&B6XJwov}Rr*Pux{|rAb_6*8RbkBo-UD-Q_d?xb?r}{Ewa9Kh*$aiTnq|fyMuC
zw+5Z&pvmk%27{9RZzL@@+o+^JB#)Kwo#4z(ng7hpO-x4-ywHoCLvSk?c^YxXesK{Z
z_KKM@aPKH)sKA<GFTbwt8LTu^fBu=s{?F|p;qPMg*8vU~C6?oV-KsF#h@Xh)IfbaQ
z>ijSB|CY|APr<RJ=l|_?WdGG_;rV~7SKj}ykyP@3pN;?bcL!%j#ht*T+{VKN=>Wbd
zebFt&;9DcJ3Q0wh&@Bn25t-%0-=a^#^Q)Q^aWlJ&C(Xb$%~Z@CTli7%p6ughmJZ(%
zIaO>)7YA*-+1ehN9V7OnL-*(HIvmBu{LCFhIiTy0<C0vCFHO-`OH1`XYls4~ME}!i
z^_yz{?{)^|{ZAWA<^KN}oc{>nK0gdhZvH~iU|uThXVJi%Mm8ZYd}mQ6A*xLT(|bJM
zpb^E&S2>YpqBt_7@>MF{OjgjfB~jvXuH|`Z(6}{S!?Rb(I=Rz(AP(_9TlQxA^JQpM
zoQv47nFFO`m*er#@r@Q1J?6AudA=BTrF-6In{xO+u2*}y``=RkzkWCJ|7&%6sQ>Nu
zOaG6Jq<sHBxm`wY7A|l{GxSV&q)(uJKV`}xFcCesh;ty>H#h*2Rr~`j=@}@QNyY%f
zE-kC@7^_(%s>%Kc-3Iwq{%zz4c(Jf$hD3o;(kT#uV%bGx9TjLtxiY{LkJ7}Jr*LSC
z>zWU*DRY321-eQ30cvi5UkjiUj{e(?22gkD6$n5;k9PGaK2gLq=gzw45reqf-&YIf
zlvistw@un`RYSdAPr;RRlF+vKse0T?`44bd-tkRdXzXRT6kk#0d<cy9RO1t3MR5vP
zATkyH>=Gx^b49qYJIdpGNlqH(<I<ivuTes%8~XYK$P|M0kedEjdzy|P3nv^DpV>b#
zMgOrYYrX`0iTsaA+JAH~7A#1C-EOPA|9cb3JpZ=={IKWs-vZBldK);w{eCxl|L^em
z?+jX<Hu`_I2F-SP{=1R%Nf4YWi*yGY5E&A=CGiiAcXb8i6+W=v(M?KKhu%_$6`0YB
z#g%r8ZWYA~&!OxiZVLK0q?7pJ6RGk@>MR$#wyUaNFjbv7q4tS8a0{TU`P`B5-6Ukc
zs*>^Bo~p=Su!{JW`8u#LI#}Y;->N-hd78hsI^9lmdA0^nR<&Y-Tv~5?Fwp8{PyZ49
z5mj9I-eSH&hlDR$vdOHuR2f8bswJ6zjr~fuoUB0gtkX5zq0GB4q8Es)c$;!o)`ja+
z$5t58X#sJlrm{Z7-ux$HKw%(5j<p%{qu$Q!VCvmc|K5P%Dg4y1^9BY!8s>JpTZav-
z&j0rGjC0TDc{<JRH_g}=-wVB|$JfweCa!FMc<w!4Pets1p6Um(%>Ji8h|d4}ZRY>q
zY_&@IzlkIj(~)5rUPv&0cXB-WF;YXDzmliKg^78-leC^y8Um@iiSjR2k$$|H3RGoy
z6xZz;ZIj|zvK5o-HAg~@Us&--z1og><S~HfnkYoU_#bOH|I1weo$jC?#s3(f{dcq1
zEYJTolFIYH&%pkN-2wCV==+18jx$Za^Ybse@=sIgNiARDBQO@Oc!!+upyC0KYtD$R
z9gDKdbXA!_D=>r<R+E5^@LRQ1%<(|PPIn2tnTPj@CU&s~5ve(#p@=^PO<U4<Tf{UM
z`l#u(<zDeP=sX0vm~NFF>k}x1bLrD5wQpJID%R%@z!&;Fk~-oX0F3dl?*R!s*SQCb
zaTx~6c<UCTCBh||iA<dA@SZh=L7nfsg)_r|-BN+1bbNv(-NmtLz)R8A%wu@zQVap_
z-m{w(#1vqq7ZirfhBbmvH;g;!TbY|Iz3ne|2UYQ7#5rG1B$YLVOxe@##WMwxd2+-U
z<AK_StdE(E7fJy@%IJ)bIG;a5Fq8aw)0Y~TVDxFk^vGB!a(rUZy)#jR?C8-_MAWst
zO4hA$`SDv!!HFa|1{X?Gw4e1KX==3hFL0W<g#TMh41gv4U%#i_|JCa-{a>qJ>i;&9
zO8)P2;s1*AeF=N7m>qQ`gAv@U0a&grxTGrUH}MvH>M7^p0(s^JQ|Eg(l5503rq^sM
z!9z<2C!1YEihDKS0`Rxgw}D^#LtI`I2oBNW+#hGvs5%qXhL{{Ufno)*b<1VfV+Ali
z<ihu?87d1|eJq-)ayR5m>g;v=(h>m1yTE4HIjhFs0+z}tlotjGXpsLS2fz$R%5lhg
zOC1ld6$!2x->EoByfnsRHzP$z&UlY(HZp#RYK~2t9>z}a3n>dZ<WiJPy@3VGyIz>v
zawixk?OK($Y=GFMC(!lKMkIil3ji(U#Zi)PV<XKuw}3R*sXJz8oH~jkvP$Iar!WW6
zHAw+=(CQ7O6@7MD?gnE#1VTz+u#8JFRIlaVYXT2c$1zfuZ0Icm$fpjY#G=Uv4X|SE
z02)7z-yS3W0+4fqIL43zs3(D8y^bm(;hm?!f_J=$YKK_hH6T97KngFT3%%F?!4L0_
z^?+r%7x&Qo-+~6Ay!VP>I}?&vpRjkRnaA1*q4ldek8uonZCRpL^<2&kp3!h$Ip)5H
zbw8?Z*9BmJ=+gvD9?DAW!~>)(|0A(3nOTQEa-s>m<?Dh{SBQu+4o=i?qSZs5i#J{&
zVq@ZKg!NAdkQcDn?${`bHvxmwxFB&AirK$qf~W<cdmr(ZFQC(SA&`YA02!1JjASTU
z>*htOvC);K@l+B8q-ACX_C_`+Z1!(}cS1Q<fyYS_im2KhMuP4L!dMYkFr^77c2q0?
zu^j6;18rS+s3v&Hfi_FiPY&*xn6zw9BT|bAnleh}C?RX+nB+psIO7ko49_5HPsNj<
z+u{~1^vGwHO@V%gBUh2cZwrLkl$aqEK3WOc-};M*@uFMUJhy0DLxk6a{$wwP3~Q)g
z%ZL{BB=LLg?WLuA)~xw;y!C8FhxVLTjCD?-?z3qHzC;7(rT$;q1V7dkc!~X2zX#uh
z{NEe!_^-Xv{&zE}l>c8i{;R&e;zu<vCcmd#u7nTKIZRmK;PgL_0AIlSD0WIP^(mlf
z&VU+SrJm?1wH`;QM^+78;}>UcO#%^r9199-;k-*~l0!49sa_HtF2XBjVFbtT$LHgN
z<MF?bPHXaZnw%<4;qBu?8zGRez<)imUA*Zanka6s7TI}du@i^<)*O|U+nwzaZdA6*
zf8(ESuP;nE44?>SL<NAlq$ldgA;T+vFd0FX9r#h{I0)HVij&*PD+rPq#=?&+9ycc|
zmvW29Q<v-+y$QA2;L{w#<<BUxo#K+VWN4mjKyy_!u(0D1FXc@9Ktm;Dvx6X^ilhF%
z8qeWyiQP|x4rz%+eHs<(2Fz-dFDdE7A*^l1!7#?`<i~r4o^#at8TJ+?NSHcbu;ctL
zqy<Q=`<l2iTJ58e&SKUmq}$^PB?(-m;6z9<D>3E_gw(*x%)6Q=3?qUJ&Ki&V%5S+X
z0Q1hMCId!n@2<N-ju0}QY=QL{AAoaI1#cC`Wkca)6xkr1g<lbpxk<9d<f?rHK=HCB
zc;Y$4jU|PeC_*k3O?V6&u<pHo^$mY$B6kMn$h^C2cls<vzBum`RTqDq#nBYGX&RdZ
zHiY#{gUp1JwYVAjC-~R={Y{g;ArHH-=Nf6`s^mp8EPqIVlR(L6Xtp(D@vkah`du((
rDpQ%tRHibOsZ3=mQ<=(CrZSbOOl2xlnaWhAzfSspNV{Xd0LTCU88!z)

diff --git a/engine/tests/Functional.commonAPI_SomeIp/fuego_test.sh b/engine/tests/Functional.commonAPI_SomeIp/fuego_test.sh
index c47d56b..3b2c880 100755
--- a/engine/tests/Functional.commonAPI_SomeIp/fuego_test.sh
+++ b/engine/tests/Functional.commonAPI_SomeIp/fuego_test.sh
@@ -1,28 +1,38 @@
-tarball=commonAPI_SomeIp.tar.gz
-
-function test_build {
-    $CXX -std=gnu++11 HelloWorldService.cpp HelloWorldStubImpl.cpp HelloWorldSomeIPStubAdapter.cpp HelloWorldStubDefault.cpp HelloWorldSomeIPDeployment.cpp -rdynamic -lCommonAPI -lCommonAPI-SomeIP -lvsomeip -I $SDKTARGETSYSROOT/usr/include/CommonAPI-3.1/ -o HelloWorldService
-    $CXX -std=gnu++11 HelloWorldClient.cpp HelloWorldSomeIPProxy.cpp HelloWorldSomeIPDeployment.cpp -rdynamic -lCommonAPI -lCommonAPI-SomeIP -lvsomeip -I $SDKTARGETSYSROOT/usr/include/CommonAPI-3.1/ -o HelloWorldClient   
+#!/bin/bash
+
+
+tarball=../commonAPI/commonapi.tar.gz
+source $FUEGO_CORE/engine/tests/commonAPI/common_api.sh
+
+function test_pre_check {
+    if [ -z "test" ];then
+        is_on_target libCommonAPI.so COMMON_API /lib:/usr/lib:/usr/local/lib
+        assert_define COMMON_API
+        is_on_target libCommonAPI-SomeIP.so COMMON_API_SOMEIP /lib:/usr/lib:/usr/local/lib
+        assert_define COMMON_API_SOMEIP
+        is_on_target libvsomeip.so VSOMEIP /lib:/usr/lib:/usr/local/lib
+        assert_define VSOMEIP
+        is_on_target libvsomeip-cfg.so VSOMEIP_CFG /lib:/usr/lib:/usr/local/lib
+        assert_define VSOMEIP_CFG
+    fi
 }
 
 function test_deploy {
-	put HelloWorldService $BOARD_TESTDIR/fuego.$TESTDIR/
-	put HelloWorldClient $BOARD_TESTDIR/fuego.$TESTDIR/
-	put ./fidl/vsomeip.json $BOARD_TESTDIR/fuego.$TESTDIR/
+    put build/project/commonAPI_SomeIp/HelloWorldService $BOARD_TESTDIR/fuego.$TESTDIR/
+    put build/project/commonAPI_SomeIp/HelloWorldClient $BOARD_TESTDIR/fuego.$TESTDIR/
+    put build/project/fidl/vsomeip.json $BOARD_TESTDIR/fuego.$TESTDIR/
 }
 
 function test_run {
-	report "cd $BOARD_TESTDIR/fuego.$TESTDIR; export PATH=$BOARD_TESTDIR/fuego.$TESTDIR:$PATH; export VSOMEIP_CONFIGURATION_FILE=./fidl/vsomeip.json; chmod 777 *;
-	if (VSOMEIP_APPLICATION_NAME=service-sample ./HelloWorldService &); then echo 'TEST-1 OK'; else echo 'TEST-1 FAILED'; fi;\
-	if (VSOMEIP_APPLICATION_NAME=client-sample ./HelloWorldClient); then echo 'TEST-2 OK'; else echo 'TEST-2 FAILED'; fi;
-        pkill HelloWorldServi"
+    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; export PATH=$BOARD_TESTDIR/fuego.$TESTDIR:\$PATH; \
+        export VSOMEIP_CONFIGURATION_FILE=./vsomeip.json; chmod 777 *; \
+        if (VSOMEIP_APPLICATION_NAME=service-sample ./HelloWorldService &); then echo 'TEST-1 OK'; else echo 'TEST-1 FAILED'; fi; \
+            if (VSOMEIP_APPLICATION_NAME=client-sample ./HelloWorldClient); then echo 'TEST-2 OK'; else echo 'TEST-2 FAILED'; fi; \
+                pkill HelloWorldServi"
 }
 
 function test_processing {
-	log_compare "$TESTDIR" "2" "^TEST.*OK" "p"
-    	log_compare "$TESTDIR" "0" "^TEST.*FAILED" "n"
+    log_compare "$TESTDIR" "2" "^TEST.*OK" "p"
+    log_compare "$TESTDIR" "0" "^TEST.*FAILED" "n"
 }
 
-
-
-
-- 
2.7.4




^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [Fuego] [PATCH 5/6] commonAPI_C++: using common script to generate commonapi c++ test
  2017-11-06 15:42 [Fuego] [PATCH 1/6] bsdiff: add test_pre_check for command "bsdiff bspatch" Liu Wenlong
                   ` (2 preceding siblings ...)
  2017-11-06 15:42 ` [Fuego] [PATCH 4/6] commonAPI_SomeIp: using common script to generate someip " Liu Wenlong
@ 2017-11-06 15:42 ` Liu Wenlong
  2017-11-07 23:56   ` Bird, Timothy
  2017-11-06 15:42 ` [Fuego] [PATCH 6/6] core: back to the original path once "test_build" done Liu Wenlong
  2017-11-07 19:04 ` [Fuego] [PATCH 1/6] bsdiff: add test_pre_check for command "bsdiff bspatch" Bird, Timothy
  5 siblings, 1 reply; 17+ messages in thread
From: Liu Wenlong @ 2017-11-06 15:42 UTC (permalink / raw)
  To: fuego

Signed-off-by: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
---
 .../Functional.commonAPI_C++/commonAPI_C++.tar.gz  | Bin 11657 -> 0 bytes
 .../tests/Functional.commonAPI_C++/fuego_test.sh   |  59 ++++++++++++---------
 2 files changed, 34 insertions(+), 25 deletions(-)
 delete mode 100755 engine/tests/Functional.commonAPI_C++/commonAPI_C++.tar.gz

diff --git a/engine/tests/Functional.commonAPI_C++/commonAPI_C++.tar.gz b/engine/tests/Functional.commonAPI_C++/commonAPI_C++.tar.gz
deleted file mode 100755
index a5a6670887348b30e94a3feaef0423665314fd1a..0000000000000000000000000000000000000000
GIT binary patch
literal 0
HcmV?d00001

literal 11657
zcmY+}gLj?n(+B#-Y1p`F>@;ZD*l6rDwry+2wr$(CZEH7n8ly?}eV*TY);epQzhSPK
z`CgwPi9-A&sne_R3Fb=I)A?Yu;kj>0f#tYLnO1!2&4D)KyJn*8Z%g;zRj!16TqA*!
zk|Ma!FP{_}_q9MzYy5cdK?5Ttw6-FYM<xE;Uf0*T*ZOnsUA$o4POdCs;B~I1F89!K
zi0vEd4S3dKUJbyRQ_s=!W<Ey`we<m?MTh3pjsLM6mlBv_ZiE);jU@NZO{+b9!}~Xl
zc|ZcVo`E;aqaJ0$6A!?Do5P9>3C2M4*Kw@}eGz2_@1SPgHDCjMo7>mX5#yzgpTj&#
zJcZ|OtZxY)#hVMV<JZS|=@zQr(p3-X(oc9&`Kjd#nHtNxDQi*35_}#H)25+Zi?vQd
zpr_<)HAlc@&(q;gJYA(8GfIsoS*H<1%#wb}6Op_?;FKFb+vb(ysH%p8AlWQhyJhwq
z`{Jd&mcPjnK+LCoi8OYnjXXDJ`h<RrFaCAm1)aHg`BFv6zn1i&m`NUQQ-j|IhyvXA
z?#}g}7|0V8vc7B#=t{m@{~IF!z_Axvhen=9JZw(a(>6@a&lA2$E?KrItxeWO2^oDJ
z$(l^FmL#5jV#o4d;bvDbz0U_C?8u=0j{4q3^-;1-Q9yp1aNoq4W6>)k^~MyAw6Am;
zC$ks4GSYU!t+XZo)4n?axOS&?Mc&SjalE=TmQ9VQ)_%Tv+3E8V$FFnwJs;0>wY9I$
z?fP-~0#BztNOeDEXX6C2d%U{{PK>xE{r+)vT+QTMXk{*oa@%V~FRw265U(4)K*964
zIDRAB!GhV5LARhK7ovWy=H=DVV1~QqUE+A%xe=C^dZ7~xTkzpSv#;sTRlrmFvcyZw
z7C2_S*@eZ5Y@_jIwAOxnUv_T#c`*Dd(y^|vTPf~ai+*8tBP_{VE<zIcw&$}da}wx^
z&NoQGKw_%jcw|?T<mx;9+x+^xlxU*#GAp`CLf(EYzqj|(-sS20B}f6x3H=7I7D@C~
zW<_f9y3%7cdW_R;dEJ-a(O?5!$qV+pR<LQ?u^WBlcN_(8^!eGW9k6=#^zZHFuL0LT
zv%HTO;8R>BJ4?{;-x~b+N7~O=a%M%sG;;*JosTD$S~!c$&lw0CwT6=%ZcAd<hts0>
zTwjx7$MhkxtLI;{XpL)FXSFc#Zn`$@9p`r5X8Ee#L@pBiEX!j8n)<t-$Q_q9)F-%G
zb)H!p3#pzGf(|Y;?cSaAr_i<4EOvx2Du~DU%y(N9p7PVJ1<lX0*OAvi&9ypjir;;}
z%Tj3%SW^z>-V4FgXZD-1@b;)TvI%Tiv(?P3s404*yTyBqx_IzzoZeT}LBDp@W6JWG
zSfUKt@!TBJGdD^nFZ&yGPQdGpMSsj9a6-?rH+m8}d@y&CGb+X2VmxUo+L9imA#-;2
z`!gndp`yd4Z%=dm;GVEMl&nMIM&DAHqpa2sxqlxmN6>W{UOV|RneQp}pfQuN`5y+6
z)g=TGzWoj&wAP#-3(c=4DgElHt$f4QcQvVK8o;T2YFDta41d@#6Hnrr%pZ<R(Sup8
z@0(U}gA(?sO<=vsv-)j%>n991ABc5VmcW?Z<m=Q`X=U$*U8RwVsr5hPT}6QGISmx(
zVSD%zjRNjmt=TSV=|Rp@PizfMfFGKzsnPQhC?Oz{iCS>l+eE$UXTAogUad%vc7mQc
zc@YD|?HK|tXC1y>ebMjrG5(lBe~P$GI(QxU0X|9kC^!mRs&bA``6LqnYg@$i7}2an
zT_d6fy)3but9_K_L3a*fDmq$O*w*TQ7uJ>Q;6Zu-f(+HqL0v)e&sA=COhK*xz;b*w
zJ)4pLc2^#{aUQx78(lA)*J{TY=fT%4R=|OdHQh9c!rUBK4&?U?c{lNTy`DMS7<srG
zQUJ@v$;bW>$nMR9csR1ul>DwMX<UguH`M5qWzwlCMfgzx;CO)ved`*%nf2n%yr+(6
zWv5c?Le2#Eeq7cwuQdGC<>h41tSN@(De;(9<dP7AVw`g+x7Kw@Cmd)tNS<_5teVlG
zN4Zzdbwu+gRyA%kpg-FETv&3HXp+Jg#r5jBDZ*;ix2CbkGNTcgPI=K*8R*E5G*W3Q
zV%^&$LV)~vkJEv_*<WyyodC-@HvG5hdRz4%7Q(R_dsj;k`Z)7-|11&yE;0sRlc?n_
z2KYWvoJyZltciAsqs?%D`?-9omCxYu*j!P-=FDweYk2sVN{HZk0t1(n14CK#SbOMb
z!=gYzNvP~C;`g5pwQ_!$2BP~u%2~5ZRP<#-@p3JtuPhjPi55Hwv~AU_uSr%UQ<AXs
zrj>Hx^Nq!CVGPhbhapis=7ovOA9j<+78M&(_cZ##PYR|_J%@hkMvhip#8=W=MZfQI
z{_x!VEgdH>uANBUr!Ic5Th~~diG=WtjJ~|Oo)ef?2QlyNl@povf>b{<G`)S32@w4H
zQ`X^K%v`;+**x)%OE6d;`x^BuLpiXqqb=qRC^-_eR9EalqXuFcFF~C>b`q1I9VPU`
zzNhjX@FXb9qxBB#JOP?ibGU`<fI$<<FVml6Sc!Fu+jE?R|5{S@v_y$N?F@!QNT4YR
z4T=?S*d$i)ohyK%3@8n~(P%1)ayfEH{))s(5#1{f<z$FF<-6Tvg$wOB#=4qW$2;FR
z$`p27ZF0_=%@zR#X{20sWUQm$2op1<{21}@RbtNYEsAn5DizxpHQV{RJq-9h?PjF6
ziSLhF$m1^YMJ~V7Vdwj)o(5Djj4C#!1)O8|2<vTPt)K?6dos5j-77!XJBRK7Dixul
z(eY=;?Oo&R%$M#Ci4|wrIfQLLqpObGkJTp&H#J@+INHZ!p|kCbJ(I=)wAmNofXAu4
z*f<3yri*$wVLx*>nJ(OCC|<-rJB*jm5SvyMgXzrqvHES%fB0(rat#W*J^Vn3ZFC~5
zQa34cGsObDSu+haVM?}MXb<9RYPff25G&gl9gFKrJR>x)n=ynK!HclL@*+ai@k#tU
zchn92G}{*zo3C)(7KIlZ{lO-^#kM2w=|z^=)SIy35_-v6zR-NBIZ$n@{DdfCHr+1D
zG{?<OP{eb-UfYu`*a}Z2Qzu<G4YL8B!*h(AzKw@X;imgL3M#?vG0M>6Kbz&xsOa?+
z+Ukw+ef-iRh1ur*V7fmObQLSe?kzt(=h`Dk5r;OtIil~rGD{i6Mf$fH!qoTIq~*%V
z#LX7O*M4#a6<Z`5ZN?DYf>Hg@UE}49KfxS+l^Qfu86PwaOj?RE*z<XKM?fD#g<t4w
zBgCUpVarkPMUNuq!>m}9RT<grbN<+nNJpkB_unLewr=pZ+9)8|85HrSOC!r_is|_9
zQcu@VUi6P~u5~=TNvB>->5CR?&>$>Zam+boF(#gGEA&04J`3Z{`Y3y5W>p21_9dIY
zO2;&tv1O}1%Q}iXJR!I@2Tb6GC#qYA*ODgbpxY`PtFJZ(TS=PxqA;jUe>gnKWlj`W
zqSO4#!g5oHv+i991TMf6_zsC2R!tV@hK178CGtY>&zwpt-YML@4XUc?k@lAjjmL39
zRcu*5e7qjm=m}NKIZOt+x`ec=;S+MYy#o4={VK=v(*6#z&vl8oX5tGg<uC=oV4YKO
zi`3C1GRuqC){F|UnFWlfw{+3r|4F%;j@dYL<Yg6YW5E;yk5JWZDr|&#zg|l<L1oM+
zCaKY+<~AIivED#u1XC^AC9>uy(25#nm+_Qzgn5;)D3Ac2Lv}{E0zpsN&fUrsPV(>F
ztB0&er)o-aDSG8H>JLV9HHsgDxpz<cZVrE^FDrZdUJfsStsK8DrbnRX8u0_f8?>he
z^hMCL`aAk?)!JUjw?szR?agf4&`^LZOW89gFr4<ciCS7w5{SapIhj8r+<em@bTa48
z-Bxi*KYf;Aa=I3BL0m__W0V3N`MD$1;J(wVIEA(e@nEnkjPFq7-78Au$`p^2UNU(2
zHms2_l@4rwxp*s{%|<);7O(ofKeSt|)J0S&F5~xnY*a;FfT9c<y3~i0l=#p^$%R0m
z(1bKpK|<PxE0M}=)J4aX>3o=W_qAnyIC{^nl@h+vGK#H>t_=dx5=VHpYLS)a9*mWw
zk5kV>`r3AM<>!VT(rt;u$xzf`v<ZpI+DM|bqz8x%H^wXs_Od0EN8!)d+ILbTTLH)o
z<b7_ei)SZdF@E=ZRxWdVtu0fjB5{&KdLpJC4EHXd&BNv((yK~Lc^K}uM?Z34MdH`%
z2Z=35`g+edi0Cl{OoVdq1|G;qe*@vM+B^etd|Pi|)EyvGL877X6be7DQ7t_7a%M~F
zbxOs!qw^IXw4>;vAZsmY8cSlB=6+?c*$+?e#vVI5qd=f3e?ItI)FN7WpG~1&ly&?k
zPU;XUVHeuGFGrq#wF1aWSt`<r&X@ASkHo!S4_7nxM9dPa#)n!&%demV_v3r`(sbVR
z>ys39M!8c2mV5v|#!tfX7%&S8V^Xh69zjdRuNjSwS%vd~U}p-ZxJ_Ak3f2;JH7WTx
z({Nv@3)j^zsjV1qxyo_x)fEtd6`iH7P!`UOEGICNPsh|1qLoMhU7svLgo{n8I;L5k
z_t08yBt)<w!vc;vzaJ=D!k2oj6Z}dJQE_MA-UQhmZsY=U?vM-m$PJqmzMF$K#|RY@
zSc56yp!~JH!Q4G6T6fJ7;-QBs)zKr$y&t2c*cBi06+qN35<s7~_y$;&_C0)01bD7~
zfPt+Y!@f5hAEGC(tq%~^oErdwg5**8@Mp~YRndOPeCUqsI_yOCb0yH5$y8e}qlJeq
zC&xv5*;bjl)v+PNQa2;5xJ;n5+YBIc)uQ~l(Kf$T2}3s8*w}QtkyvP6WEYrK#YQ%&
z=vyF}!yuq5Qq-idNv~Ts0ve;wLE;#-=JZt-ct&gW6*$q<h0xETV^4sJV%;D?@0?$4
z=Kfdldl1D*UjWn}2*^dybn9&5Y~-~|e>c1LBYp$}`^3`-jSPXAtmCKb>NNdfMAuz4
zdqmZ8r{H_yu+c$-Hs4)U4Iuptdic@T2_+fuj*ta20{nIR)$>#?cO~eRa|P*3C|kEF
zN`>U>)9>-lW@KRk2^pE`wd5M5=3QY1YRvl_`-zYdvG8GftI7WEqVfXhn$X_&PCtYC
z#rUh&q||qXF5|&ns!4ljqIntikvhGcEL%!@vE^hP)e0WS%6ngkwPgzr(qEu!XB|?t
zH1G%}u{{v9Io!AdF^vLmT}4w%ck+K6tqzU?9X^4Rdrx<N-PA;AhEO(8sLF^KikWpx
zw4Em1JXBJ5<YefMy0STlv!ytU9s6NdQ-yh?m+Wa9l|>Tm?dKD6@^)c084m|Ioce9l
z4zX|?BfEbw4Xz~_6W5QO>nT$A{x6@Zi3EI~2OfBaTKRyto&^rb5`5m(D*$~BYIn#(
zhe^un#r`(!Jm8lKcg)`S+4CmHG$Xj39j&yD2b^yc)S}KU$;J|kQqd#5H)9e|i<3Wm
z6nylmGvEF*WRJVtbZ5JQ@lTouQ)<tKZwE@uaEGKz7CtReR`#&O7$*{g*PvzJlgn3k
z?dNX1x8!3WJ=GTHxlaV>GZcIRE*{{q0sH1hV0}F4FF+GJ^)j;vb8Qiu`<eb9Y~}5D
z6|{hFUUv8=LQWPApD2I7!!YjQ)22&Jej57D7<u!(N)vwN&>_$+=#m!TaD(mE;vo-6
zH7w+XBUfVQmRL;?;$2LE@?%M+?j-FH)mC2f@v&Axc!8aln?zT1gn7@zs^R|kw^ggY
z3%O2+6k4mfuJC0z-1<7`ar{eA`WCo~%5~D(WwVRgww<A1!>06fF^4kQ*pC`ST>Osk
zmC=5kZ0_e%p=M%TIhz3NKkw3;&k@yJu-M_UgL%3KLjQ=vkiF~id(-|D8}ilWbi93|
z3N9_E5%iZG&_pjdQ&=uLePL-j&VLz(M8LfXa$GhA`KLt#F;67?!+~%Cj_fw}2O#_X
z?`)7~viCiPOaRH(K844@KWxbF=@o2z048wt(me+@t{>Z;`v^S!P6*_4$ae;y#74yk
zr7nXmz}x*#NF=9VN6L~;Yw*^O8RoXSR8DE^9iB+MUhAbe5886DL*6iYUlNa~a#@AN
zdTr-7gGvE+iX$xNKgVL4Q6cm<JFuoeJa^JU?)kX>eBgS*D^K$mB3a~r+V02fa&K+&
zb>OHVb9z`)oGJA2L8q>w1mzzej|#iQdc$&mnDB7E4f`%U&^3)JQ~z0+$Q-_p8`eL|
ze1tZc^70i$F<teE)$30(Vn3=bmSL*bm6#3JL*NMWqB|=a&H|aa(7Komo7s-OjJG+{
zWWPSU-B_gWRlHVA&+?wWv}bkg_+g>dQ@^?AQ|r7y$t+nv1B>!LpWnyxUTshrPxwsU
z%E|=pSIRm+6kG-4$=~w!XnO$?x3^^oLTvDV10X<xw$ylt6-CnF>3HAgpK*042Z$;!
z2s!D-hsKh>iWxSsA>t0E3Q=%Z7Os+=Px|~Rx>ag|awBr$F_VI9Y_7&5SkHED`yTO6
zM=6mJ)q7mZHNd(CsuTP+`a(is1)u`p+W%W!eRu|dsBwRGwjbzW$+Ps#-S1yvJSb9H
z!Qxd~KR~H{pWkUfXhbyBTAovzCZJKq^>;=-4kB5|h{ZAn?r#~R*f2ug9g3$Na<Ape
z^)XB?WoHWsLJ(4AYw#_{82ib;@{O9K9cDro_<-3kZR)YR7wo|Xl*^2(ZIx5H&13g|
zJa`l-q7%X|(H`q@%%6Sce1q{17Vv3rRvEKXCGqO0IUJn|CA4q?c?4f%+++QBL;@&X
zeX^-`#(J#=Wc1dy%kCGOFrat6ixY5Oz8Dm!bGT%{Si@G>;VUd+C1hg%dYj@Grt(AS
zJDCzz*2FTI4ETmO2TOUs5zaD-+=8k}qC&uw3~fN)bW%OG@q*tkdp9PhYb6_~6o!$9
z)1W8@P|o+TmBDYQEGToGZwfQgTj>b+MSaeej5l^DU@!Qn2BcLsya5<HHu_r-z0oJ^
z8_-YyCSW~Z^si6Yy9(GEZ#n|>kO1}6ze|@rm~P80D8C$<;0-!j{}_>I_C;`>sDRms
ze_*@YF}nnSXxe|lvi6ineAd(I<XUG(`JTJNjOX9i0)z0<0=oYrPImHzarA>lZ6?*d
zP#Q0Zf-||Abk)w2olgz%L$nqeoLSTupDMASTGB;ktb9_JBrN0q6+o&gc&<hQAvdX_
z6d-;bt`LUA5kj#M(vOjy^<4e?sz%ve;!}1=$zl<$KtVB*lzm6MVPnD?ZWSw+g=9ST
z8j>7B==6QDbhkh7C<M=}b^F^vaqw^M0wS@XT%jcj1RAJl9mFMN*jP*-f-B3>j*wz;
z`yCeUqh%#*3~_RE!626KA<B@}C^dW1n(`tUl{VUg0h(e`_<Iiykw>k8BJvv{N1I=j
zi7~zkpOI|Ru7|rPJ-!(XOv17I@$qtxy&{YKK-vGALs;*PdAhpQPKwF@g2edsAO??j
z6s68^KL~~L#FTJ8%I(Yhp5Q$0yjUC`+6=&W#f<4#cWP56f@zD@iuUtfgNMxRzI1{)
zmaK-tiT%8HtbIU+mnS*4!`^U5;k-*u{;TXm#;>HfM$DeuPv7ZZB&)5hRtug8>!6E#
zbT?YZ#D}>lXlga89@9&VQq~X!)7QThTpd6k;xM!#Y(~AyNuII)kkkqBdH7pFY6W!$
zk%+#7dt--(ZFE2qg8mMBz=2)odQbj){ZxxAH!@XN!^~`YcO5<?7D99*Pdg@kx1JhI
zUWqvr{+HLsxcc^s>=`OH!6e=%{n(l8*w$JfRV+SpiL8>dIr!&gc*d=zaCXeVQlW`#
zi$=3psUOSH6p-%;e?E&{;R7Sjw#0DA{C5?mD&?aNCUl7=%|c)0SlEIGB__x=w|qEQ
zaygN!%yDh4g*;XsXb>VM>O=2Rf-RI3n)}xD^tC$6Rx<Tztmo8RX#4IP7e*&4-b}Ot
zm=^jw3V}4%?IXDW<jcNLn3vHbU?6<I?`c!}4LrlA{R(8<>aXCWbXR(Eyz(D8jrfKO
z^!tbWKm(Q#Z>YNCFxz08`a>-<mkNHVSIj*mYsD2UM~aIsY5bYau^hi0W1(onokdnI
za-S(C)0R^0M5Iq2M9l5se{*ni@q6kk`%6I1BPX9g?eS)^Hi{Ll{iiRNi})8$I6~w`
z)rcE0lG}c|_)LOS#?FR-;V;v-OZ)dS!LLqOXfWTm@s%)1gC2%axl8XOb!A7BO<@G3
zi4e$mL!6z|(S`GAxDNbmq_(rGRTmk_6?B~p9;j8NgA|a1r}DPO-R?+jdDs;=3OXvk
zX{<{P%dlYn$3NJOXX0PfM>YluF6VCB`>$zo(JUjx{W6oS;iIjx<<ndeu{qolO}?`5
zZan4|9NAcXpqL}$P|C(IZ8+c}TYgr)C?Kzut@LHTZc1klb<zGWtq-h7qXmCyY8EJ?
zEauUp+*Ri@Jf)UAvtiy=$31F=o$*cz{S>{vqwgMIBZA1cfqB8Q0Y##S)2-t5gMjsd
zw$YI1Gcrb^r{v<~aHVcQzKI?d{{%LlkyHiUq)nQWf<_3c#`A{$M?XncD=&<*iNH=u
zx<E)0(**Cx2$jCMBfBZNoN-Ej=)Bwa<cwnz3d>Lk?|a?H7N_kS!JQ?^tj+<krmMFS
z%9Nnxp@j<Tb*Ao!#{2IpnP|8qDj-jXacIu%eP2NesPil1Cv@A4uU#T}kkAI|<oG%c
zNa{+H{-zdCSESYz^#j)HSIlqVALk%1+h*Vjt?daQ@NE5;_um$ByxUPolMV54adNrc
zd+^0Kj*pf#MGd%^7^w*%OC)?7b(OC(+#D95WGs$>lp%`I((6Yyyk=w4ova8qtktlx
z>C79nT=rFzUQFXnZ2!^x7sL)pNR?;)ZORZe@>3~QI6NU(E0x;`o=mFs=g8q$j!OO&
zy~)f$A|36&O1AfRt=qW$P-A5SmI%Kj@1wULnY=zE>&(9g;}ZJX<IPw)>r@Q)Gp*uu
znceX3tWzz#3|b$|h-?#i*~FwB_jFCfhWBu-_*kPIZ8$56D)aq}yn^`#^=l)gqRrvp
zRDR)d_*LIevl3m<6yirxt%afB4|g&QPv$?vkO)H@KLa1L{0-z)hR`#UJ4sGz!lB3F
zSrnQof}#q_-MTbwXM_CWTb10ukZWS?^)0lx#UZE6P4!Zx5)rBKwmWrdTTN?y`s&~+
z{Evgdp>)oNlv33IJk1k$R{`*E@NXW($IM~v$6cD+0R0~rdvU#P-VrFrj~yiiKyO*`
zdmrX?oyXmWAkPl*Gav*L?^Y&ptN%+Ih>Q7OU&D<9ul;?;!6jcnd1_wzUCh>C#`*0W
z!a@xhm}2%UIo6cy*)w~LXv$6r*=c`>*<3JJ0^hOppC@Wh5-bc!9bEe%e>>-P1b6@e
zI?@^}l-0q07p4~&Z-(+KMp?|DG){zFWt%j%qNO^xp(qJI!2JKP@08QY+>=x!ST3Px
z6vFKU=iXfp{S~1+{1u*S*C@k7YAp}TNqf6Ak)$VC#T0Q!PeabGyBjt(-vK}T?A3ou
z0R<>C?`y$I?Z=Fn7cC3t)(YGjcAK9N$JXe3KYMw3xNAhy{xWtyR6F9^j|S}#77O+N
zw>{lOmJtEZ=cjkPRu<_=vD?7@`B*UmEvlKsJht#5;fxI9lyn}S@%9GL<!YU_%Tz)U
z{IV0S(z-Z3VmXk?gdpd_PmmrzowgW*;LIMgB$oB*c05YCO2L1uk<H)mt0V#uPX(5<
zRzPy>Nw34nev_vB|0IvsYun(YpKt#J_ewzX4!<oM&jWUO0lH1;_&yL#Sei-m)=LB^
zTAz|c$@GNpqh!Av#+J7Q{e5+dKQwHo#eODrW$@>xyR(TM>d#ldwJDUeLitVJ`SV#K
z^4Is0-E|D0<{XXH0>!|ekU|QRI4~h(3YMk}Z(#8C^c9`h^{5GMy~;Z4thllVw);ZM
zvQp#Cy;gzDN=7jNjVGW*NP&2y2C-5n)Ml3B_~}B|a7;o<rc+r<9m0w^qHN9#hs*i@
z^SWjHfOG1{v^xg<iiA}ZeuRa&SYsj~-oNQ(Dur-`hb!L-tB9!9j2|Eg`kG6aS`E;}
zCZ${$+vsb0B^iXVmDPQF=@{+D+KkIy*5C<EXbyO+-$p-Np;_DeA}U#Grf)}D`AYW*
zykl5@u{-{MI>(UsKb>PBe8F-x%skO8;F-~ADW6Rn2pDPY@(ISyw~`NO=xm3_z$&zI
zPvIJ_az9=|=#$h!{?-|jK>F{;56>09NrCMtcZE#UahuT0W}b$JXhwy!0K9QZh6J~y
z<TD=;e53=umG3kB%k(jx|6b+@+_0K%@S9Jv4^qA*IPagdP^T#gdGA`aeWo(Lv90@3
zDTs3kU~8vZDH)U}=nE16JzE5~FW?}hy0BYNSB@~K3IGo`tpI{qK-s`$ZF4Q3A3+*t
z38^Gw;Q605i*A2%W}E54(0IQTZAbaUKW3g10e4NxJ~W@~r=`MsW9Y^F9C7b}4TXsh
zATg+*8WU2*43Wqi1O0vS{&<St2T&2T-~Pt%z25c}YJ0FT8Q*GQKI)2nVTJ>s6B=au
znSSdX-FE+XMZ^5w@vlfooyuagUe5<rh{!W`l{~1PDOtF`xQK=R7y5e($(nz$v}@@n
zX(M&!@)u+H=;i~XbJOOEDXQPtdQ7^{3Ed|p{ZgY-^5RfM;y26&<u@UE_ARQa+|d6g
zca>^a-uV&xYyZg|at5_f1Q!>2>lzgrx#)nKQP?3xPqzgf@o%~wL2b0b0Hqd2#!p9C
zqYEh{V%0NaB)fUQtH&C4k8En_gZzd`NWgHvu?lV6S=V|BPB*+$x99q4i~H-mPvzM_
z>o`1xeRp)(C4>a#QB4pA=|;aRh^!4bbMxL2*zfx^);<D&|Ch@1&;hE&!u;W~Y8-Mp
z_+J|gwtU76oD|a1q8;AMLN=WB1CcG3Nb1@HsGL?2-}XmI31of*ZO{F3R<J6O4gce%
zPNjIkHm-%0-UCHyD$d@*EYAfODz@y9&*3)~o*TQ#NwgtqdtOR<ueuv+`X}w%d^Qhy
z$iyNCY$F+kfO91Q={@RSX~SJ--p0&!p(u>2|ARKr1g7o8J?Qt6{y*#-AjeKS8~9K{
zm<49b*9h$XUwQ&ME<r5yYstT!CY#?^l9qEf3)?hiC60_<*n6ux#hJc~G2uFlTv8Uy
zI>eqBdH-Dd0aty|AI~$B|Fyx+(}APq(nhhKl47wu>&wjfr6ilG{<r~EltSA4>1EQB
zbeiog__nibEc+HyKCf`DC(ky|YR>8zw9+0>_|~@|ISB+^=Z{?jttY^*+^g5VgnMxR
zCq=w*uI`hE!-xI%9VUBuuvzFQhRNIE?E&+Cz3McT@viq|nvyfKM6G{eOoxf=u0QI0
zq^$TdqNtJe0h}k$F-+$ms=)w6@VsrS0n)}M@?l>F-ns_<J3_qyO`9OezVo|M5>Muh
zZ{xz>$hG~fmg%4wyS4BJp(B6PMPgJP{4z1nO!Y5MHJ#^CDK--pX1l{8ov+g~mGPSA
ztW$2Eb6m-s)bvwyy=OAiu&Nt9`w{t>4PE5Qy)+3&9WK*M)ADkvu^lgN)4+?P=%XIV
zv?-4`f_<8Wch!<U_{tMry1oMzhvMbW+37(~a)`k@cQX|YvIK!l@$9tY1kAPJ6s!u)
z@riB=%{b-nOUjsGfLk@SsF|`^DR6c;Yr-SC74zG;3g(*)dR&dk#kmp2>f`ff6#RYd
zJN<nYsc~Zxd4_&a_+v{JgODc11bXR+%_KhfmC|X<U2MXoKTr%LO0yQ4NCE#)b9imE
z-pp~lq3nld`&`1XxyZ9{^`siH_-LnKbL9i2y(D<(^+98q?wMdYz7h904eFS(tO8Ps
zi^;6nFI_gf0TX`%^?h6m0&Al@hM|7pp1;I=jy?6kzTIruzj%k!n7|r<hy&~a&DecV
zF>;zm^Lu@ZJ6a>A!FPuBT#>H#j1sm9$s5eN15Su)Jy6uzRrS>$ZynSoL4o?;ShI9*
zeF719MxwU-M!w~pa>o2L-mu`on4{k&Fy#I|xtjbEH@!S;(lE@*pVfhB-NvldSqyTt
zJ|E__I-FK?DgVq?6ZSvF{DMb!RbUzfDW(FJuG8z?Bkr<<5M8k3ZmZN!C@C4smXqHx
zwWC2e?}AVaYOb+3o50f+G*HKT-dq83j{@2<%&P!jmKPA1NH;eX0V-PML_B&sy=T5B
zrgK>eAB()=UJV^{Pt52)#H4K$?5RlD-fiD2gGVn9f~HNCVi$Fbz7AEoF<%^xRYayO
zo1DEKu9l4-zhEMe{9!h$&!zK~!sVlX^xwYE^+(TXg%8lR|C%=hJbMuZ09ypEzxcf%
z=&iSToOl0GF`ffNMLTAZwBPHTRkFY27C}$Coy2cNll#wXrC}g;X)!|+M%rwVUiB<U
z5^4H9>~k&`%mkPAMGiq|fW82-+{XQ=K|F@eA}*MvX8Yl7gjKwYDGfDIqawmiJIfDG
zT7gu!uw-iqHeQi);6y;OZ@!EY7)v&<8USJMgQ7Qe3xF&4w{T$j8oljBfxz?bR&aO#
zw@=fnpJ--*&66>mH1Z=$crH5-aRFK_B><T$3|{h!G91e=rUqHyMllL&c1s~*USpxD
zqCGkdBIRK0fxWRsd{qerrUgDVk1Qglc#`v>>E~ev-JCr<(c4m))?X+*#|PS3(3I5v
z@han_>2WYYJ~I)k{-o~OjO#^k@`zuc7&q0Ga*f6Oo(H^4!WU5x4_6WgUD$c9bdzv(
zQs-?rR9w#FmnSg<k_6FRLio#$P<i`9D)pLWSWzGUzSEyxWeh@5vhIIUO!xX63yK*!
z^(K^s449q%4GariXm>llwoWF=3{8c6!bC8q3-%LK$}8L`3;PbCfoFKN=}yGC1|a2H
zV@S`SompJKl49-&V($&8n|{;<!YH;wcwIi8;XSU_E}Gte8i+kboTBVVuE^Jvzn3#3
zuMcAy7^V>E)iC6={sFS@AK^NGSqB~b4wVDW6H4yFpVNXE<V4jHZE%LX+==-1UbS!<
zZ*n~enX`u`b%a75>!OwZL-%XI`(r`z6hP=9;|CCSYTnxt{(G+=sDGeuh>wR4Z$){t
zY&oF&r7R{Whm~_#9a{X>#!0JhwPtZT_4&$VQP$#Z>NFUpA(Mp}`M(jrMf?xq%qSU*
z<wN+~QY9|C#2?CkH^fohNb-|~AEP*8{wHxmKjJJ$j%NNRarImp46wK)b{T|I2P%kT
zto7!t&I$u`mRaDPE#ZE0%e`!vKo;Uf3R!5dGl}7-hYNh9hc>RfOmUPt7Qs({AGxZ*
zd;xR_=J0cshFCHyUs~~VL!H)5;@Z#Wv+f5Q6sTkV#c?km#QJ%9g3jrHpMk1v(dH)+
zl5)v6bNga4)HD2SRDOi!Y|in^tFRev`J-71L)l-4Fn^0iv(XK4QmQFdp)vAs69~ej
zYeL#9WMV$kpEK__k>h5b<^{_OI$Oo_<1oe?R1+_=LYLzGY0K#U495*WPl&Y%-ee+P
z?s*pcvseHPw6VhLKmnYDH!tWGz(VN@Q$Lj5Yu|tW8OYy~8<6R%z|1j)A-Cp~0W7EW
zw$cEu>LvXJ=k1}+h4<(}h=?UT4E~FIV3-)`y|!2VADoaV(Z2e^4aVGUW;Xw=BR^NZ
zWZUsq@u=ItXvd}_)bPc<!*Nt9<M0-3V$@VZ5&H!C02I-wB8ziOhtP)V`?ZNLsY219
zkd=9DoRF;$ZhCmt;JBk@QclTO|DSpcI(lf7J#D%qL9(H3$;c@_RH0uUEwi^hCROAq
zvnmd^wF)L|pqEpf(Z3Jj9%>fL)dY}bZQ)Kk9g^*85Jw-pM<4t#R{Uu-Rned%9Y2G$
zDa;N_&Cc)SfZ0DD!Yn2J)o@^BA=k`DM^ft{7Bxwwmk16^iSq81?Q0O1g`Gv0MGtgi
z6i+9+Yg8kdj|h?xbDTMaY99_C-99?@m{~F1$@+70e!tsU?{rmk85C}{4%gmTQ<8Ib
zpU~kTY4*Ydl=t@lyPg<i@`a$f`9O?5&j#b^n{U(tl44GJqkB1)2m>S#7+Wa0hw-9B
z2zU{V0s}5%){TyJfy40P3xq_5X4r0*T{9X+@*<otrt>e$?#*H>8WsDJw#IYPKf5F>
zcr4%2g~Oslu{s6eV<Z90i%h-!O8&8`{`5KZ)+@&V$Jy}+`c*4_vkHx)pM!DVTYp)+
zwR3sGMO_6vk%O?5d18<;)yYUcyy;T4ggOBXV}YUCm_0+rZ0LBQ$l-R>5$m09Nu1J7
zweYeE(=Qxq_)VO>j-^E)gluf-mPEc*s)GJXvF4*UH1JnzK2m!xn32#sxc)R_wz8rc
zdXg0Qxs3(04XrKsDp6nSpwuf%p>LlXRN7zV*(cbxn$fIBF@R*r8`!|-Xkh=1BQ7F3
z-DWv9K<)=IOwM=N#D)Vy_L1o?=Ok2N7WVfW!Ct!z)wvsWK2MaAQ1c{|)I;AU)iKFg
zi5DPx-oBQad1}^vIo15yR|W60Q@;pk%C3{n5PK1y<LDqHlYi9MU>Vv0B2*q2Pk`-G
z`F-tr6)<L-Q-gpw-ury!P+VL1p#%&>cI=?*R9v5_2{Tr7V3px~dfWP~LX7V%lD%o^
z4-5F1jkyg(TAruJJ>G(Gv@uH*zt4Z6q9&<ToXCP@99F(_K#ss@QKTC2*Dd!EIM~Jf
zyi^m-zR1U^5HHB=QA%sdvQ0hwEbJB@mdNpG^{dqMZ_@tC*JQZMJWcWk>Wi~4N&RpA
z68ZXsAAR5s8Z{G3;VIAgfM)ZsqT`Y1Lv8Mgi?Hm5krl{Qoh!`NhqdormAb+pHIddp
z%(#f@w*EyjPM~0#%nQFev>H+OO&UhGrQEr638R{Ip+OGz0D!xquyNE5QLNg%wYYi}
zckra{!d?ic9I#?Ly$}m7R_)_7SFxfj3o{LGD<rv;JcmBL!dd%T>0xx>7~rYgZCV0H
z|0)$$Je3*bNaw3Zd1ie{8Nm#nx=zE)e}1XAxDEB~IKKj2x;0PSPRxc5hs#>|%9v$x
z6-m=b=0Z7$bTwcV-0ZJf7B&2avU!Okz@xf0Ddch1<lt+1FE!vkF(}8gfls1n-=pI6
zi=gowN@36i6L*Uth_T>zgsXN&x`{j%;{>)>ddKu#RNM^Sexhc<Ft+(`@1L`u7*#c7
z*|1z~I|>-hT?ewKjD~4z?j^XS40fH>i8+8IYv)CShwCJhmeP(@Nu%`-G(hDGilb$c
z!nl5hTYuzRIJ9x(9F1uLff=!}S>w1J4>5eIjiMLomSAEp(h*K9nZ+kvBK6+CshWXp
zYA?P9Th6@iUtgn;JwrqZHmw@Y+EWLUspM$-2hxbQ?B8tiu9|#X=bk3KASA=1Hawp!
Hpg#RSLPc@Y

diff --git a/engine/tests/Functional.commonAPI_C++/fuego_test.sh b/engine/tests/Functional.commonAPI_C++/fuego_test.sh
index 94d74c4..0776809 100755
--- a/engine/tests/Functional.commonAPI_C++/fuego_test.sh
+++ b/engine/tests/Functional.commonAPI_C++/fuego_test.sh
@@ -1,40 +1,49 @@
-tarball=commonAPI_C++.tar.gz
+#!/bin/bash
 
-function test_build {
-    $CXX -std=gnu++11 ./commonAPI_Dbus/HelloWorldService.cpp ./commonAPI_Dbus/HelloWorldStubImpl.cpp ./commonAPI_Dbus/HelloWorldDBusStubAdapter.cpp ./commonAPI_Dbus/HelloWorldStubDefault.cpp ./commonAPI_Dbus/HelloWorldDBusDeployment.cpp -rdynamic -lCommonAPI -lCommonAPI-DBus -ldbus-1 -I $SDKTARGETSYSROOT/usr/include/CommonAPI-3.1/ -I $SDKTARGETSYSROOT/usr/include/dbus-1.0/ -I $SDKTARGETSYSROOT/usr/lib/dbus-1.0/include/ -o HelloWorldService_Dbus
-    $CXX -std=gnu++11 ./commonAPI_Dbus/HelloWorldClient.cpp ./commonAPI_Dbus/HelloWorldDBusProxy.cpp ./commonAPI_Dbus/HelloWorldDBusDeployment.cpp -rdynamic -lCommonAPI -lCommonAPI-DBus -ldbus-1 -I $SDKTARGETSYSROOT/usr/include/CommonAPI-3.1/ -I $SDKTARGETSYSROOT/usr/include/dbus-1.0/ -I $SDKTARGETSYSROOT/usr/lib/dbus-1.0/include/ -o HelloWorldClient_Dbus
+tarball=../commonAPI/commonapi.tar.gz
+source $FUEGO_CORE/engine/tests/commonAPI/common_api.sh
 
-    $CXX -std=gnu++11 ./commonAPI_SomeIp/HelloWorldService.cpp ./commonAPI_SomeIp/HelloWorldStubImpl.cpp ./commonAPI_SomeIp/HelloWorldSomeIPStubAdapter.cpp ./commonAPI_SomeIp/HelloWorldStubDefault.cpp ./commonAPI_SomeIp/HelloWorldSomeIPDeployment.cpp -rdynamic -lCommonAPI -lCommonAPI-SomeIP -lvsomeip -I $SDKTARGETSYSROOT/usr/include/CommonAPI-3.1/ -o HelloWorldService_SomeIp
-    $CXX -std=gnu++11 ./commonAPI_SomeIp/HelloWorldClient.cpp ./commonAPI_SomeIp/HelloWorldSomeIPProxy.cpp ./commonAPI_SomeIp/HelloWorldSomeIPDeployment.cpp -rdynamic -lCommonAPI -lCommonAPI-SomeIP -lvsomeip -I $SDKTARGETSYSROOT/usr/include/CommonAPI-3.1/ -o HelloWorldClient_SomeIp
+
+function pre_check {
+    is_on_target libCommonAPI.so COMMON_API /lib:/usr/lib:/usr/local/lib
+    assert_define COMMON_API 
+    is_on_target libCommonAPI-DBus.so COMMON_API_DBUS /lib:/usr/lib:/usr/local/lib
+    assert_define COMMON_API_DBUS
+
+    is_on_target libCommonAPI-SomeIP.so COMMON_API_SOMEIP /lib:/usr/lib:/usr/local/lib
+    assert_define COMMON_API_SOMEIP
+    is_on_target libvsomeip.so VSOMEIP /lib:/usr/lib:/usr/local/lib
+    assert_define VSOMEIP
+    is_on_target libvsomeip-cfg.so VSOMEIP_CFG /lib:/usr/lib:/usr/local/lib
+    assert_define VSOMEIP_CFG
 }
 
 function test_deploy {
-	put HelloWorldService_Dbus $BOARD_TESTDIR/fuego.$TESTDIR/
-	put HelloWorldClient_Dbus $BOARD_TESTDIR/fuego.$TESTDIR/
-
-	put HelloWorldService_SomeIp $BOARD_TESTDIR/fuego.$TESTDIR/
-	put HelloWorldClient_SomeIp $BOARD_TESTDIR/fuego.$TESTDIR/
-	put ./commonAPI_SomeIp/fidl/vsomeip.json $BOARD_TESTDIR/fuego.$TESTDIR/
+    report "cd $BOARD_TESTDIR/fuego.$TESTDIR/ && mkdir -p commonAPI_Dbus commonAPI_SomeIp fidl"
+    put build/project/commonAPI_Dbus/HelloWorldService $BOARD_TESTDIR/fuego.$TESTDIR/commonAPI_Dbus
+    put build/project/commonAPI_Dbus/HelloWorldClient $BOARD_TESTDIR/fuego.$TESTDIR/commonAPI_Dbus
+    put build/project/commonAPI_SomeIp/HelloWorldService $BOARD_TESTDIR/fuego.$TESTDIR/commonAPI_SomeIp
+    put build/project/commonAPI_SomeIp/HelloWorldClient $BOARD_TESTDIR/fuego.$TESTDIR/commonAPI_SomeIp
+    put build/project/fidl/vsomeip.json $BOARD_TESTDIR/fuego.$TESTDIR/fidl
 }
 
 function test_run {
-	
 
-	report "cd $BOARD_TESTDIR/fuego.$TESTDIR; export PATH=$BOARD_TESTDIR/fuego.$TESTDIR:$PATH; export VSOMEIP_CONFIGURATION_FILE=./fidl/vsomeip.json; chmod 777 *;
-	if (./HelloWorldService_Dbus &); then echo 'TEST-1 OK'; else echo 'TEST-1 FAILED'; fi;\
-	if ./HelloWorldClient_Dbus; then echo 'TEST-2 OK'; else echo 'TEST-2 FAILED'; fi;
-	pkill HelloWorldServi
 
-	if (VSOMEIP_APPLICATION_NAME=service-sample ./HelloWorldService_SomeIp &); then echo 'TEST-3 OK'; else echo 'TEST-3 FAILED'; fi;\
-	if (VSOMEIP_APPLICATION_NAME=client-sample ./HelloWorldClient_SomeIp); then echo 'TEST-4 OK'; else echo 'TEST-4 FAILED'; fi;
-    pkill HelloWorldServi"
+    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; export PATH=$BOARD_TESTDIR/fuego.$TESTDIR:$PATH; \
+        export LD_LIBRARY_PATH=\`pwd\`:\$LD_LIBRARY_PATH; \
+        export VSOMEIP_CONFIGURATION_FILE=./fidl/vsomeip.json; chmod 777 *; \
+        if (commonAPI_Dbus/HelloWorldService &); then echo 'TEST-1 OK'; else echo 'TEST-1 FAILED'; fi;\
+            if commonAPI_Dbus/HelloWorldClient; then echo 'TEST-2 OK'; else echo 'TEST-2 FAILED'; fi;\
+                pkill HelloWorldServi
+
+    if (VSOMEIP_APPLICATION_NAME=service-sample commonAPI_SomeIp/HelloWorldService &); then echo 'TEST-3 OK'; else echo 'TEST-3 FAILED'; fi;\
+        if (VSOMEIP_APPLICATION_NAME=client-sample commonAPI_SomeIp/HelloWorldClient); then echo 'TEST-4 OK'; else echo 'TEST-4 FAILED'; fi;\
+            pkill HelloWorldServi"
 }
 
 function test_processing {
-	log_compare "$TESTDIR" "4" "^TEST.*OK" "p"
-    	log_compare "$TESTDIR" "0" "^TEST.*FAILED" "n"
+    log_compare "$TESTDIR" "4" "^TEST.*OK" "p"
+    log_compare "$TESTDIR" "0" "^TEST.*FAILED" "n"
 }
 
-
-
-
-- 
2.7.4




^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [Fuego] [PATCH 6/6] core: back to the original path once "test_build" done
  2017-11-06 15:42 [Fuego] [PATCH 1/6] bsdiff: add test_pre_check for command "bsdiff bspatch" Liu Wenlong
                   ` (3 preceding siblings ...)
  2017-11-06 15:42 ` [Fuego] [PATCH 5/6] commonAPI_C++: using common script to generate commonapi c++ test Liu Wenlong
@ 2017-11-06 15:42 ` Liu Wenlong
  2017-11-06 16:39   ` Liu, Wenlong
                     ` (2 more replies)
  2017-11-07 19:04 ` [Fuego] [PATCH 1/6] bsdiff: add test_pre_check for command "bsdiff bspatch" Bird, Timothy
  5 siblings, 3 replies; 17+ messages in thread
From: Liu Wenlong @ 2017-11-06 15:42 UTC (permalink / raw)
  To: fuego

It's possible that function "test_build" will change the path,
such as Funcitonal.glib2.
We need back to the current path before "touch fuego_test_succesfully_built"

Signed-off-by: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
---
 engine/scripts/functions.sh | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/engine/scripts/functions.sh b/engine/scripts/functions.sh
index da3415b..209b152 100755
--- a/engine/scripts/functions.sh
+++ b/engine/scripts/functions.sh
@@ -284,12 +284,16 @@ function build {
     if [ -e fuego_test_succesfully_built ]; then
         build_duration=0
     else
+        # it's possible that function "test_build" will change the path, such as Funcitonal.glib2
+        # we need back to the current path before "touch fuego_test_succesfully_built"
+        ori_path=`pwd`
         build_start_time=$(date +"%s.%N")
         call_if_present test_build
         ret=$?
         build_end_time=$(date +"%s.%N")
         build_duration=$(python -c "print $build_end_time - $build_start_time")
         if [ $ret -eq 0 ]; then
+            cd $ori_path
             touch fuego_test_succesfully_built
         else
             abort_job "ERROR: test_build returned $ret"
-- 
2.7.4




^ permalink raw reply related	[flat|nested] 17+ messages in thread

* Re: [Fuego] [PATCH 6/6] core: back to the original path once "test_build" done
  2017-11-06 15:42 ` [Fuego] [PATCH 6/6] core: back to the original path once "test_build" done Liu Wenlong
@ 2017-11-06 16:39   ` Liu, Wenlong
  2017-11-07  5:10   ` Bird, Timothy
  2017-11-08  0:21   ` Bird, Timothy
  2 siblings, 0 replies; 17+ messages in thread
From: Liu, Wenlong @ 2017-11-06 16:39 UTC (permalink / raw)
  To: fuego

[-- Attachment #1: Type: text/plain, Size: 1815 bytes --]

Hi,

The attached patch file will be better for this issue.
Thanks.

Best regards
Liu

-----Original Message-----
From: fuego-bounces@lists.linuxfoundation.org [mailto:fuego-bounces@lists.linuxfoundation.org] On Behalf Of Liu Wenlong
Sent: Monday, November 06, 2017 11:43 PM
To: fuego@lists.linuxfoundation.org
Subject: [Fuego] [PATCH 6/6] core: back to the original path once "test_build" done

It's possible that function "test_build" will change the path, such as Funcitonal.glib2.
We need back to the current path before "touch fuego_test_succesfully_built"

Signed-off-by: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
---
 engine/scripts/functions.sh | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/engine/scripts/functions.sh b/engine/scripts/functions.sh index da3415b..209b152 100755
--- a/engine/scripts/functions.sh
+++ b/engine/scripts/functions.sh
@@ -284,12 +284,16 @@ function build {
     if [ -e fuego_test_succesfully_built ]; then
         build_duration=0
     else
+        # it's possible that function "test_build" will change the path, such as Funcitonal.glib2
+        # we need back to the current path before "touch fuego_test_succesfully_built"
+        ori_path=`pwd`
         build_start_time=$(date +"%s.%N")
         call_if_present test_build
         ret=$?
         build_end_time=$(date +"%s.%N")
         build_duration=$(python -c "print $build_end_time - $build_start_time")
         if [ $ret -eq 0 ]; then
+            cd $ori_path
             touch fuego_test_succesfully_built
         else
             abort_job "ERROR: test_build returned $ret"
--
2.7.4



_______________________________________________
Fuego mailing list
Fuego@lists.linuxfoundation.org
https://lists.linuxfoundation.org/mailman/listinfo/fuego



[-- Attachment #2: 0001-core-back-to-the-original-path-once-test_build-done.patch --]
[-- Type: application/octet-stream, Size: 1060 bytes --]

From cb0960773682c5cde1645c8c25587af80bdc41fb Mon Sep 17 00:00:00 2001
From: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
Date: Mon, 6 Nov 2017 11:17:23 -0500
Subject: [PATCH] core: back to the original path once "test_build" done

It's possible that function "test_build" will change the current path, such as Funcitonal.glib2.
We need back to current path before "touch fuego_test_succesfully_built"

Signed-off-by: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
---
 engine/scripts/functions.sh | 1 +
 1 file changed, 1 insertion(+)

diff --git a/engine/scripts/functions.sh b/engine/scripts/functions.sh
index da3415b..c7dba25 100755
--- a/engine/scripts/functions.sh
+++ b/engine/scripts/functions.sh
@@ -290,6 +290,7 @@ function build {
         build_end_time=$(date +"%s.%N")
         build_duration=$(python -c "print $build_end_time - $build_start_time")
         if [ $ret -eq 0 ]; then
+            cd "$WORKSPACE/$JOB_BUILD_DIR"
             touch fuego_test_succesfully_built
         else
             abort_job "ERROR: test_build returned $ret"
-- 
2.7.4


^ permalink raw reply related	[flat|nested] 17+ messages in thread

* Re: [Fuego] [PATCH 6/6] core: back to the original path once "test_build" done
  2017-11-06 15:42 ` [Fuego] [PATCH 6/6] core: back to the original path once "test_build" done Liu Wenlong
  2017-11-06 16:39   ` Liu, Wenlong
@ 2017-11-07  5:10   ` Bird, Timothy
  2017-11-08  0:21   ` Bird, Timothy
  2 siblings, 0 replies; 17+ messages in thread
From: Bird, Timothy @ 2017-11-07  5:10 UTC (permalink / raw)
  To: Liu Wenlong, fuego

Thank you for these patches.  I had hoped to review them today, but ran out of time.
I will try to review them tomorrow and get back to you with feedback and/or
notices of acceptance.

Sorry I couldn't get to it today.
 -- Tim

> -----Original Message-----
> From: fuego-bounces@lists.linuxfoundation.org [mailto:fuego-
> bounces@lists.linuxfoundation.org] On Behalf Of Liu Wenlong
> Sent: Monday, November 06, 2017 7:43 AM
> To: fuego@lists.linuxfoundation.org
> Subject: [Fuego] [PATCH 6/6] core: back to the original path once
> "test_build" done
> 
> It's possible that function "test_build" will change the path,
> such as Funcitonal.glib2.
> We need back to the current path before "touch
> fuego_test_succesfully_built"
> 
> Signed-off-by: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
> ---
>  engine/scripts/functions.sh | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/engine/scripts/functions.sh b/engine/scripts/functions.sh
> index da3415b..209b152 100755
> --- a/engine/scripts/functions.sh
> +++ b/engine/scripts/functions.sh
> @@ -284,12 +284,16 @@ function build {
>      if [ -e fuego_test_succesfully_built ]; then
>          build_duration=0
>      else
> +        # it's possible that function "test_build" will change the path, such as
> Funcitonal.glib2
> +        # we need back to the current path before "touch
> fuego_test_succesfully_built"
> +        ori_path=`pwd`
>          build_start_time=$(date +"%s.%N")
>          call_if_present test_build
>          ret=$?
>          build_end_time=$(date +"%s.%N")
>          build_duration=$(python -c "print $build_end_time -
> $build_start_time")
>          if [ $ret -eq 0 ]; then
> +            cd $ori_path
>              touch fuego_test_succesfully_built
>          else
>              abort_job "ERROR: test_build returned $ret"
> --
> 2.7.4
> 
> 
> 
> _______________________________________________
> Fuego mailing list
> Fuego@lists.linuxfoundation.org
> https://lists.linuxfoundation.org/mailman/listinfo/fuego

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [Fuego] [PATCH 1/6] bsdiff: add test_pre_check for command "bsdiff bspatch"
  2017-11-06 15:42 [Fuego] [PATCH 1/6] bsdiff: add test_pre_check for command "bsdiff bspatch" Liu Wenlong
                   ` (4 preceding siblings ...)
  2017-11-06 15:42 ` [Fuego] [PATCH 6/6] core: back to the original path once "test_build" done Liu Wenlong
@ 2017-11-07 19:04 ` Bird, Timothy
  2017-11-09  7:22   ` Liu, Wenlong
  5 siblings, 1 reply; 17+ messages in thread
From: Bird, Timothy @ 2017-11-07 19:04 UTC (permalink / raw)
  To: Liu Wenlong, fuego

> -----Original Message-----
> From: Liu Wenlong on Monday, November 06, 2017 7:43 AM
> This test depends on "bsdiff bspatch" being already installed on the target.
> 
> Signed-off-by: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
> ---
>  engine/tests/Functional.bsdiff/fuego_test.sh | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/engine/tests/Functional.bsdiff/fuego_test.sh
> b/engine/tests/Functional.bsdiff/fuego_test.sh
> index 6ebe3c7..dfc3f80 100755
> --- a/engine/tests/Functional.bsdiff/fuego_test.sh
> +++ b/engine/tests/Functional.bsdiff/fuego_test.sh
> @@ -1,5 +1,12 @@
>  tarball=bsdiff.tar.gz
> 
> +function test_pre_check {
> +    is_on_target_path bsdiff PROGRAM_BSDIFF
> +    assert_define PROGRAM_BSDIFF
> +    is_on_target_path bspatch PROGRAM_BSPATCH
> +    assert_define PROGRAM_BSPATCH
> +}
> +

Looks good.  Thanks.

I have a question about this interface.  It seems like this will be a common pattern:
is_on_target_path, followed by assert_define.

Would it be useful to have an API that does both operations?
I'm thinking of adding something to the dependency checker, so that the
following declaration in fuego_test.sh would perform the same functionality:

NEED_PROGRAMS="bsdiff bspatch"

What do you think?

The reason for adding this into the dependency checker, is to allow static
evaluation of test dependencies, for (eventually) automatically building
a testplan for a board, based on tests that will work on it.

>  function test_deploy {
>      put data/*.modified data/*.original $BOARD_TESTDIR/fuego.$TESTDIR/;
>  }
> --
> 2.7.4
In the mean time, I have applied this patch.
 -- Tim

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [Fuego] [PATCH 2/6] curl: add test_pre_check for command "curl"
  2017-11-06 15:42 ` [Fuego] [PATCH 2/6] curl: add test_pre_check for command "curl" Liu Wenlong
@ 2017-11-07 19:09   ` Bird, Timothy
  0 siblings, 0 replies; 17+ messages in thread
From: Bird, Timothy @ 2017-11-07 19:09 UTC (permalink / raw)
  To: Liu Wenlong, fuego



> -----Original Message-----
> From: Liu Wenlong on Monday, November 06, 2017 7:43 AM
> 
> This test depends on "curl" being already installed on the target.
> 
> Signed-off-by: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
> ---
>  engine/tests/Functional.curl/fuego_test.sh | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/engine/tests/Functional.curl/fuego_test.sh
> b/engine/tests/Functional.curl/fuego_test.sh
> index 90d24dd..582544e 100755
> --- a/engine/tests/Functional.curl/fuego_test.sh
> +++ b/engine/tests/Functional.curl/fuego_test.sh
> @@ -1,3 +1,8 @@
> +function test_pre_check {
> +    is_on_target_path curl PROGRAM_CURL
> +    assert_define PROGRAM_CURL
> +}
> +
>  function test_run {
>      report "if curl -o test.html www.baidu.com > /dev/null; then echo 'TEST-1
> OK'; else echo 'TEST-1 FAILED'; fi;\
>      if curl -i -r 0-1024 http://www.sina.com.cn -o sina_part1.html > /dev/null;
> then echo 'TEST-2 OK'; else echo 'TEST-2 FAILED'; fi;\
> --
> 2.7.4
Applied.  Thanks.
 -- Tim


^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [Fuego] [PATCH 3/6] commonAPI_Dbus: using common script to generate dbus test program
  2017-11-06 15:42 ` [Fuego] [PATCH 3/6] commonAPI_Dbus: using common script to generate dbus test program Liu Wenlong
@ 2017-11-07 23:54   ` Bird, Timothy
  0 siblings, 0 replies; 17+ messages in thread
From: Bird, Timothy @ 2017-11-07 23:54 UTC (permalink / raw)
  To: Liu Wenlong, fuego

See comments inline below.

> -----Original Message-----
> From: Liu Wenlong on Monday, November 06, 2017 7:43 AM
>
> Signed-off-by: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
> ---
>  .../commonAPI_Dbus.tar.gz                          | Bin 6320 -> 0 bytes
>  .../tests/Functional.commonAPI_Dbus/fuego_test.sh  |  34
> +++++++++++++--------
>  2 files changed, 22 insertions(+), 12 deletions(-)
>  delete mode 100755
> engine/tests/Functional.commonAPI_Dbus/commonAPI_Dbus.tar.gz
> 
> diff --git
> a/engine/tests/Functional.commonAPI_Dbus/commonAPI_Dbus.tar.gz
> b/engine/tests/Functional.commonAPI_Dbus/commonAPI_Dbus.tar.gz
> deleted file mode 100755
> index
> 7f1b491803d44b4fb25b16ea50294e9bd1a4b6b7..0000000000000000000000000
> 000000000000000
> GIT binary patch
> literal 0
> HcmV?d00001
> 
> literal 6320
> zcmV;h7*FRPiwFQyY=l?<1MNNiQ{y<Y^DFr)wAm^y@R)o<?gE=v49xCQ#mrE
> ^-0r?N
  [binary stuff omitted]
> mNJT1Ak&0BLA{D7fMJiH}id3W`6{$#nzVvT4*9}Ji$N&I8dw_fZ
> 
> diff --git a/engine/tests/Functional.commonAPI_Dbus/fuego_test.sh
> b/engine/tests/Functional.commonAPI_Dbus/fuego_test.sh
> index 485d279..97ae6c9 100755
> --- a/engine/tests/Functional.commonAPI_Dbus/fuego_test.sh
> +++ b/engine/tests/Functional.commonAPI_Dbus/fuego_test.sh
> @@ -1,25 +1,35 @@
> -tarball=commonAPI_Dbus.tar.gz
> +#!/bin/bash

I don't think we should use this bash invocation line.
I see that some scripts have this bash header line, but I don't
think, in general, that this makes sense. It looks like I've even
added this line into some of the tests that I wrote.  I probably
copy and pasted this without thinking about it.   However, now
that I think about it I would like to move away from using it.

Although fuego_test.sh looks like a regular shell script, it really
is not one. fuego_test.sh relies on a lot of external environment setup,
and only includes certain explicitly supported variables
and shell functions.  It is not intended to be executed as a standalone
script, but always be source'ed by main.sh (or loaded and processed
by ftc or the overlay generator).

I think we should instead move the other way, and remove
#!/bin/bash from all fuego_test.sh scripts.
(In fact I just did so, with a commit to remove this from all remaining
scripts where it is used.)

It does no harm, since it is just a shell comment in our usage of 
fuego_test.sh, but I believe it's confusing.  So I think it's better
to get rid of it.

> 
> -function test_build {
> -    $CXX -std=gnu++11 HelloWorldService.cpp HelloWorldStubImpl.cpp
> HelloWorldDBusStubAdapter.cpp HelloWorldStubDefault.cpp
> HelloWorldDBusDeployment.cpp -rdynamic -lCommonAPI -lCommonAPI-
> DBus -ldbus-1 -I $SDKTARGETSYSROOT/usr/include/CommonAPI-3.1/ -I
> $SDKTARGETSYSROOT/usr/include/dbus-1.0/ -I
> $SDKTARGETSYSROOT/usr/lib/dbus-1.0/include/ -o HelloWorldService
> -    $CXX -std=gnu++11 HelloWorldClient.cpp HelloWorldDBusProxy.cpp
> HelloWorldDBusDeployment.cpp -rdynamic -lCommonAPI -lCommonAPI-
> DBus -ldbus-1 -I $SDKTARGETSYSROOT/usr/include/CommonAPI-3.1/ -I
> $SDKTARGETSYSROOT/usr/include/dbus-1.0/ -I
> $SDKTARGETSYSROOT/usr/lib/dbus-1.0/include/ -o HelloWorldClient
> +
> +tarball=../commonAPI/commonapi.tar.gz
> +source $FUEGO_CORE/engine/tests/commonAPI/common_api.sh

OK - this is something that I think need discussing and evaluation.
There are two issues:
 1) using source in common with another test
 2) using test functions in common with other tests

I know this is how netperf and OpenSSL handle this, but I'm not
sure I like this method and want to proliferate it into more tests.
We might end up just proceeding with this method of sharing items
between tests, but I'd like to discuss it first before moving ahead with
this patch.

The shared portion of this solution is in a patch which is off-list,
which makes it hard to discuss, but I'm going to try anyway.

Sorry for the long train of thought.  I'm kind of rambling through
my thought process here...

One reason I find this problematical is that it makes it hard to
package a test.  For most other tests, all the materials for the test
reside in the test's "home" directory.  And so a test can be packaged
up and sent to another user somewhat easily.  This puts material
needed for the test outside the test's home directory.  I don't mind
putting source somewhere else, but when we share source, we should
probably come up with a more formal mechanism for doing that
(e.g. integrated with a source cache, or something like that, that
could also be used to provide offline access to gitrepos).

---
The intent is to share the source and the test_build function between
the tests: Functional.commonAPI_C++, Functional.commonAPI_Dbus,
and Functional.commonAPI_SomeIp.

The proposed solution does in fact do this, but does not end up sharing
a build directory (and neither do the netperf and OpenSSL versions of
this solution either).  I think we may want to have other cases, like
sub-tests of LTP (posix, realtime, general, stress, etc.).  We currently
put all these under a single Functional.LTP Fuego test, with multiple specs.
In the future, if we have a good method of sharing source and 
build directories, it might be good to split these out into
different Fuego tests.

Also, the setup for the build phase of this test is quite complicated, 
and there are special cases inside the test_build function in
common_api.sh (in the off-list patch).  The special cases
essentially pull in whole other build scripts, for the dbus and someip
cases, which makes me wonder why they should be in common.


> +
> +function pre_check {
> +    is_on_target libCommonAPI.so COMMON_API /lib:/usr/lib:/usr/local/lib
> +    assert_define COMMON_API
> +    is_on_target libCommonAPI-DBus.so COMMON_API_DBUS
> /lib:/usr/lib:/usr/local/lib
> +    assert_define COMMON_API_DBUS
>  }
This looks OK.

> 
> +
>  function test_deploy {
> -	put HelloWorldService $BOARD_TESTDIR/fuego.$TESTDIR/
> -	put HelloWorldClient $BOARD_TESTDIR/fuego.$TESTDIR/
> +    put build/project/commonAPI_Dbus/HelloWorldService
> $BOARD_TESTDIR/fuego.$TESTDIR/
> +    put build/project/commonAPI_Dbus/HelloWorldClient
> $BOARD_TESTDIR/fuego.$TESTDIR/
> +    put `find build -name 'libdbus-1.so*'` $BOARD_TESTDIR/fuego.$TESTDIR/
>  }
Using a 'find' to create the arguments to the put function call is a bit different.
Is the location of libdbus-1.so not known?  Does it depend on the Fuego 
PLATFORM?

> 
>  function test_run {
> -	report "cd $BOARD_TESTDIR/fuego.$TESTDIR; export
> PATH=$BOARD_TESTDIR/fuego.$TESTDIR:$PATH; chmod 777 *;
> -	if (./HelloWorldService &); then echo 'TEST-1 OK'; else echo 'TEST-1
> FAILED'; fi;\
> -	if ./HelloWorldClient; then echo 'TEST-2 OK'; else echo 'TEST-2
> FAILED'; fi;
> -	pkill HelloWorldServi"
> +    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; \
> +        export PATH=$BOARD_TESTDIR/fuego.$TESTDIR:\$PATH; \
> +        export LD_LIBRARY_PATH=\`pwd\`:\$LD_LIBRARY_PATH; chmod 777 *;\
> +        if (./HelloWorldService &); then echo 'TEST-1 OK'; else echo 'TEST-1
> FAILED'; fi;\
> +            if ./HelloWorldClient; then echo 'TEST-2 OK'; else echo 'TEST-2 FAILED';
> fi; \
> +                pkill HelloWorldServi"
This looks OK.
>  }
> 
>  function test_processing {
> -	log_compare "$TESTDIR" "2" "^TEST.*OK" "p"
> -    	log_compare "$TESTDIR" "0" "^TEST.*FAILED" "n"
> +    log_compare "$TESTDIR" "2" "^TEST.*OK" "p"
> +    log_compare "$TESTDIR" "0" "^TEST.*FAILED" "n"
The second log_compare is not needed.  The test only
has two testcases.  If we get 2 OKs, then we know there
are no failures.

Overall - some items are OK, but the general approach is quite complicated.
I'd like to do some testing, but I'm hampered by the fact that I'm missing the shared
libs on my target.  I think I should probably install AGL so I can test this properly.
Let me continue looking at it (but please respond to my questions above).

Thanks,
 -- Tim


^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [Fuego] [PATCH 5/6] commonAPI_C++: using common script to generate commonapi c++ test
  2017-11-06 15:42 ` [Fuego] [PATCH 5/6] commonAPI_C++: using common script to generate commonapi c++ test Liu Wenlong
@ 2017-11-07 23:56   ` Bird, Timothy
  0 siblings, 0 replies; 17+ messages in thread
From: Bird, Timothy @ 2017-11-07 23:56 UTC (permalink / raw)
  To: Liu Wenlong, fuego

This has similar issues to patch 3 and 4.

> -----Original Message-----
> From: fuego-bounces@lists.linuxfoundation.org [mailto:fuego-
> bounces@lists.linuxfoundation.org] On Behalf Of Liu Wenlong
> Sent: Monday, November 06, 2017 7:43 AM
> To: fuego@lists.linuxfoundation.org
> Subject: [Fuego] [PATCH 5/6] commonAPI_C++: using common script to
> generate commonapi c++ test
> 
> Signed-off-by: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
> ---
>  .../Functional.commonAPI_C++/commonAPI_C++.tar.gz  | Bin 11657 -> 0
> bytes
>  .../tests/Functional.commonAPI_C++/fuego_test.sh   |  59 ++++++++++++--
> -------
>  2 files changed, 34 insertions(+), 25 deletions(-)
>  delete mode 100755
> engine/tests/Functional.commonAPI_C++/commonAPI_C++.tar.gz
> 
> diff --git
> a/engine/tests/Functional.commonAPI_C++/commonAPI_C++.tar.gz
> b/engine/tests/Functional.commonAPI_C++/commonAPI_C++.tar.gz
> deleted file mode 100755
> index
> a5a6670887348b30e94a3feaef0423665314fd1a..00000000000000000000000000
> 00000000000000
> GIT binary patch
> literal 0
> HcmV?d00001
> 
> literal 11657
> zcmY+}gLj?n(+B#-Y1p`F>@;ZD*l6rDwry+2wr$(CZEH7n8ly?}eV*TY);epQzhSPK
> z`CgwPi9-A&sne_R3Fb=I)A?Yu;kj>0f#tYLnO1!2&4D)KyJn*8Z%g;zRj!16TqA*!
> zk|Ma!FP{_}_q9MzYy5cdK?5Ttw6-
> FYM<xE;Uf0*T*ZOnsUA$o4POdCs;B~I1F89!K
> zi0vEd4S3dKUJbyRQ_s=!W<Ey`we<m?MTh3pjsLM6mlBv_ZiE);jU@NZO{+b9!}
> ~Xl
> zc|ZcVo`E;aqaJ0$6A!?Do5P9>3C2M4*Kw@}eGz2_@1SPgHDCjMo7>mX5#yzg
> pTj&#
> zJcZ|OtZxY)#hVMV<JZS|=@zQr(p3-
> X(oc9&`Kjd#nHtNxDQi*35_}#H)25+Zi?vQd
> zpr_<)HAlc@&(q;gJYA(8GfIsoS*H<1%#wb}6Op_?;FKFb+vb(ysH%p8AlWQhyJ
> hwq
> z`{Jd&mcPjnK+LCoi8OYnjXXDJ`h<RrFaCAm1)aHg`BFv6zn1i&m`NUQQ-
> j|IhyvXA
> z?#}g}7|0V8vc7B#=t{m@{~IF!z_Axvhen=9JZw(a(>6@a&lA2$E?KrItxeWO2^o
> DJ
> z$(l^FmL#5jV#o4d;bvDbz0U_C?8u=0j{4q3^-;1-
> Q9yp1aNoq4W6>)k^~MyAw6Am;
> zC$ks4GSYU!t+XZo)4n?axOS&?Mc&SjalE=TmQ9VQ)_%Tv+3E8V$FFnwJs;0>w
> Y9I$
> z?fP-
> ~0#BztNOeDEXX6C2d%U{{PK>xE{r+)vT+QTMXk{*oa@%V~FRw265U(4)K*96
> 4
> zIDRAB!GhV5LARhK7ovWy=H=DVV1~QqUE+A%xe=C^dZ7~xTkzpSv#;sTRlrm
> FvcyZw
> z7C2_S*@eZ5Y@_jIwAOxnUv_T#c`*Dd(y^|vTPf~ai+*8tBP_{VE<zIcw&$}da}
> wx^
> z&NoQGKw_%jcw|?T<mx;9+x+^xlxU*#GAp`CLf(EYzqj|(-
> sS20B}f6x3H=7I7D@C~
> zW<_f9y3%7cdW_R;dEJ-
> a(O?5!$qV+pR<LQ?u^WBlcN_(8^!eGW9k6=#^zZHFuL0LT
> zv%HTO;8R>BJ4?{;-
> x~b+N7~O=a%M%sG;;*JosTD$S~!c$&lw0CwT6=%ZcAd<hts0>
> zTwjx7$MhkxtLI;{XpL)FXSFc#Zn`$@9p`r5X8Ee#L@pBiEX!j8n)<t-$Q_q9)F-%G
> zb)H!p3#pzGf(|Y;?cSaAr_i<4EOvx2Du~DU%y(N9p7PVJ1<lX0*OAvi&9ypjir;;}
> z%Tj3%SW^z>-V4FgXZD-
> 1@b;)TvI%Tiv(?P3s404*yTyBqx_IzzoZeT}LBDp@W6JWG
> zSfUKt@!TBJGdD^nFZ&yGPQdGpMSsj9a6-
> ?rH+m8}d@y&CGb+X2VmxUo+L9imA#-;2
> z`!gndp`yd4Z%=dm;GVEMl&nMIM&DAHqpa2sxqlxmN6>W{UOV|RneQp}pf
> QuN`5y+6
> z)g=TGzWoj&wAP#-
> 3(c=4DgElHt$f4QcQvVK8o;T2YFDta41d@#6Hnrr%pZ<R(Sup8
> z@0(U}gA(?sO<=vsv-
> )j%>n991ABc5VmcW?Z<m=Q`X=U$*U8RwVsr5hPT}6QGISmx(
> zVSD%zjRNjmt=TSV=|Rp@PizfMfFGKzsnPQhC?Oz{iCS>l+eE$UXTAogUad%vc
> 7mQc
> zc@YD|?HK|tXC1y>ebMjrG5(lBe~P$GI(QxU0X|9kC^!mRs&bA``6LqnYg@$i7}
> 2an
> zT_d6fy)3but9_K_L3a*fDmq$O*w*TQ7uJ>Q;6Zu-
> f(+HqL0v)e&sA=COhK*xz;b*w
> zJ)4pLc2^#{aUQx78(lA)*J{TY=fT%4R=|OdHQh9c!rUBK4&?U?c{lNTy`DMS7<sr
> G
> zQUJ@v$;bW>$nMR9csR1ul>DwMX<UguH`M5qWzwlCMfgzx;CO)ved`*%nf2
> n%yr+(6
> zWv5c?Le2#Eeq7cwuQdGC<>h41tSN@(De;(9<dP7AVw`g+x7Kw@Cmd)tNS<
> _5teVlG
> zN4Zzdbwu+gRyA%kpg-
> FETv&3HXp+Jg#r5jBDZ*;ix2CbkGNTcgPI=K*8R*E5G*W3Q
> zV%^&$LV)~vkJEv_*<WyyodC-@HvG5hdRz4%7Q(R_dsj;k`Z)7-
> |11&yE;0sRlc?n_
> z2KYWvoJyZltciAsqs?%D`?-9omCxYu*j!P-
> =FDweYk2sVN{HZk0t1(n14CK#SbOMb
> z!=gYzNvP~C;`g5pwQ_!$2BP~u%2~5ZRP<#-
> @p3JtuPhjPi55Hwv~AU_uSr%UQ<AXs
> zrj>Hx^Nq!CVGPhbhapis=7ovOA9j<+78M&(_cZ##PYR|_J%@hkMvhip#8=W
> =MZfQI
> z{_x!VEgdH>uANBUr!Ic5Th~~diG=WtjJ~|Oo)ef?2QlyNl@povf>b{<G`)S32@w
> 4H
> zQ`X^K%v`;+**x)%OE6d;`x^BuLpiXqqb=qRC^-
> _eR9EalqXuFcFF~C>b`q1I9VPU`
> zzNhjX@FXb9qxBB#JOP?ibGU`<fI$<<FVml6Sc!Fu+jE?R|5{S@v_y$N?F@!QN
> T4YR
> z4T=?S*d$i)ohyK%3@8n~(P%1)ayfEH{))s(5#1{f<z$FF<-
> 6Tvg$wOB#=4qW$2;FR
> z$`p27ZF0_=%@zR#X{20sWUQm$2op1<{21}@RbtNYEsAn5DizxpHQV{RJq-
> 9h?PjF6
> ziSLhF$m1^YMJ~V7Vdwj)o(5Djj4C#!1)O8|2<vTPt)K?6dos5j-77!XJBRK7Dixul
> z(eY=;?Oo&R%$M#Ci4|wrIfQLLqpObGkJTp&H#J@+INHZ!p|kCbJ(I=)wAmNo
> fXAu4
> z*f<3yri*$wVLx*>nJ(OCC|<-
> rJB*jm5SvyMgXzrqvHES%fB0(rat#W*J^Vn3ZFC~5
> zQa34cGsObDSu+haVM?}MXb<9RYPff25G&gl9gFKrJR>x)n=ynK!HclL@*+ai@
> k#tU
> zchn92G}{*zo3C)(7KIlZ{lO-^#kM2w=|z^=)SIy35_-v6zR-NBIZ$n@{DdfCHr+1D
> zG{?<OP{eb-
> UfYu`*a}Z2Qzu<G4YL8B!*h(AzKw@X;imgL3M#?vG0M>6Kbz&xsOa?+
> z+Ukw+ef-iRh1ur*V7fmObQLSe?kzt(=h`Dk5r;OtIil~rGD{i6Mf$fH!qoTIq~*%V
> z#LX7O*M4#a6<Z`5ZN?DYf>Hg@UE}49KfxS+l^Qfu86PwaOj?RE*z<XKM?fD#
> g<t4w
> zBgCUpVarkPMUNuq!>m}9RT<grbN<+nNJpkB_unLewr=pZ+9)8|85HrSOC!r_i
> s|_9
> zQcu@VUi6P~u5~=TNvB>-
> >5CR?&>$>Zam+boF(#gGEA&04J`3Z{`Y3y5W>p21_9dIY
> zO2;&tv1O}1%Q}iXJR!I@2Tb6GC#qYA*ODgbpxY`PtFJZ(TS=PxqA;jUe>gnKWlj`
> W
> zqSO4#!g5oHv+i991TMf6_zsC2R!tV@hK178CGtY>&zwpt-
> YML@4XUc?k@lAjjmL39
> zRcu*5e7qjm=m}NKIZOt+x`ec=;S+MYy#o4={VK=v(*6#z&vl8oX5tGg<uC=oV4
> YKO
> zi`3C1GRuqC){F|UnFWlfw{+3r|4F%;j@dYL<Yg6YW5E;yk5JWZDr|&#zg|l<L1o
> M+
> zCaKY+<~AIivED#u1XC^AC9>uy(25#nm+_Qzgn5;)D3Ac2Lv}{E0zpsN&fUrsPV(
> >F
> ztB0&er)o-
> aDSG8H>JLV9HHsgDxpz<cZVrE^FDrZdUJfsStsK8DrbnRX8u0_f8?>he
> z^hMCL`aAk?)!JUjw?szR?agf4&`^LZOW89gFr4<ciCS7w5{SapIhj8r+<em@bTa
> 48
> z-Bxi*KYf;Aa=I3BL0m__W0V3N`MD$1;J(wVIEA(e@nEnkjPFq7-
> 78Au$`p^2UNU(2
> zHms2_l@4rwxp*s{%|<);7O(ofKeSt|)J0S&F5~xnY*a;FfT9c<y3~i0l=#p^$%R0
> m
> z(1bKpK|<PxE0M}=)J4aX>3o=W_qAnyIC{^nl@h+vGK#H>t_=dx5=VHpYLS)a9
> *mWw
> zk5kV>`r3AM<>!VT(rt;u$xzf`v<ZpI+DM|bqz8x%H^wXs_Od0EN8!)d+ILbTTLH)
> o
> z<b7_ei)SZdF@E=ZRxWdVtu0fjB5{&KdLpJC4EHXd&BNv((yK~Lc^K}uM?Z34M
> dH`%
> z2Z=35`g+edi0Cl{OoVdq1|G;qe*@vM+B^etd|Pi|)EyvGL877X6be7DQ7t_7a%
> M~F
> zbxOs!qw^IXw4>;vAZsmY8cSlB=6+?c*$+?e#vVI5qd=f3e?ItI)FN7WpG~1&ly&
> ?k
> zPU;XUVHeuGFGrq#wF1aWSt`<r&X@ASkHo!S4_7nxM9dPa#)n!&%demV_v
> 3r`(sbVR
> z>ys39M!8c2mV5v|#!tfX7%&S8V^Xh69zjdRuNjSwS%vd~U}p-
> ZxJ_Ak3f2;JH7WTx
> z({Nv@3)j^zsjV1qxyo_x)fEtd6`iH7P!`UOEGICNPsh|1qLoMhU7svLgo{n8I;L5k
> z_t08yBt)<w!vc;vzaJ=D!k2oj6Z}dJQE_MA-UQhmZsY=U?vM-
> m$PJqmzMF$K#|RY@
> zSc56yp!~JH!Q4G6T6fJ7;-QBs)zKr$y&t2c*cBi06+qN35<s7~_y$;&_C0)01bD7~
> zfPt+Y!@f5hAEGC(tq%~^oErdwg5**8@Mp~YRndOPeCUqsI_yOCb0yH5$y8e
> }qlJeq
> zC&xv5*;bjl)v+PNQa2;5xJ;n5+YBIc)uQ~l(Kf$T2}3s8*w}QtkyvP6WEYrK#YQ%&
> z=vyF}!yuq5Qq-idNv~Ts0ve;wLE;#-=JZt-
> ct&gW6*$q<h0xETV^4sJV%;D?@0?$4
> z=Kfdldl1D*UjWn}2*^dybn9&5Y~-~|e>c1LBYp$}`^3`-
> jSPXAtmCKb>NNdfMAuz4
> zdqmZ8r{H_yu+c$-Hs4)U4Iuptdic@T2_+fuj*ta20{nIR)$>#?cO~eRa|P*3C|kEF
> zN`>U>)9>-lW@KRk2^pE`wd5M5=3QY1YRvl_`-
> zYdvG8GftI7WEqVfXhn$X_&PCtYC
> z#rUh&q||qXF5|&ns!4ljqIntikvhGcEL%!@vE^hP)e0WS%6ngkwPgzr(qEu!XB|
> ?t
> zH1G%}u{{v9Io!AdF^vLmT}4w%ck+K6tqzU?9X^4Rdrx<N-
> PA;AhEO(8sLF^KikWpx
> zw4Em1JXBJ5<YefMy0STlv!ytU9s6NdQ-
> yh?m+Wa9l|>Tm?dKD6@^)c084m|Ioce9l
> z4zX|?BfEbw4Xz~_6W5QO>nT$A{x6@Zi3EI~2OfBaTKRyto&^rb5`5m(D*$~BYI
> n#(
> zhe^un#r`(!Jm8lKcg)`S+4CmHG$Xj39j&yD2b^yc)S}KU$;J|kQqd#5H)9e|i<3W
> m
> z6nylmGvEF*WRJVtbZ5JQ@lTouQ)<tKZwE@uaEGKz7CtReR`#&O7$*{g*PvzJl
> gn3k
> z?dNX1x8!3WJ=GTHxlaV>GZcIRE*{{q0sH1hV0}F4FF+GJ^)j;vb8Qiu`<eb9Y~}5D
> z6|{hFUUv8=LQWPApD2I7!!YjQ)22&Jej57D7<u!(N)vwN&>_$+=#m!TaD(mE;
> vo-6
> zH7w+XBUfVQmRL;?;$2LE@?%M+?j-
> FH)mC2f@v&Axc!8aln?zT1gn7@zs^R|kw^ggY
> z3%O2+6k4mfuJC0z-
> 1<7`ar{eA`WCo~%5~D(WwVRgww<A1!>06fF^4kQ*pC`ST>Osk
> zmC=5kZ0_e%p=M%TIhz3NKkw3;&k@yJu-M_UgL%3KLjQ=vkiF~id(-
> |D8}ilWbi93|
> z3N9_E5%iZG&_pjdQ&=uLePL-
> j&VLz(M8LfXa$GhA`KLt#F;67?!+~%Cj_fw}2O#_X
> z?`)7~viCiPOaRH(K844@KWxbF=@o2z048wt(me+@t{>Z;`v^S!P6*_4$ae;y#74
> yk
> zr7nXmz}x*#NF=9VN6L~;Yw*^O8RoXSR8DE^9iB+MUhAbe5886DL*6iYUlNa~
> a#@AN
> zdTr-
> 7gGvE+iX$xNKgVL4Q6cm<JFuoeJa^JU?)kX>eBgS*D^K$mB3a~r+V02fa&K+&
> zb>OHVb9z`)oGJA2L8q>w1mzzej|#iQdc$&mnDB7E4f`%U&^3)JQ~z0+$Q-
> _p8`eL|
> ze1tZc^70i$F<teE)$30(Vn3=bmSL*bm6#3JL*NMWqB|=a&H|aa(7Komo7s-
> OjJG+{
> zWWPSU-
> B_gWRlHVA&+?wWv}bkg_+g>dQ@^?AQ|r7y$t+nv1B>!LpWnyxUTshrPxwsU
> z%E|=pSIRm+6kG-4$=~w!XnO$?x3^^oLTvDV10X<xw$ylt6-
> CnF>3HAgpK*042Z$;!
> z2s!D-
> hsKh>iWxSsA>t0E3Q=%Z7Os+=Px|~Rx>ag|awBr$F_VI9Y_7&5SkHED`yTO6
> zM=6mJ)q7mZHNd(CsuTP+`a(is1)u`p+W%W!eRu|dsBwRGwjbzW$+Ps#-
> S1yvJSb9H
> z!Qxd~KR~H{pWkUfXhbyBTAovzCZJKq^>;=-
> 4kB5|h{ZAn?r#~R*f2ug9g3$Na<Ape
> z^)XB?WoHWsLJ(4AYw#_{82ib;@{O9K9cDro_<-
> 3kZR)YR7wo|Xl*^2(ZIx5H&13g|
> zJa`l-
> q7%X|(H`q@%%6Sce1q{17Vv3rRvEKXCGqO0IUJn|CA4q?c?4f%+++QBL;@&X
> zeX^-
> `#(J#=Wc1dy%kCGOFrat6ixY5Oz8Dm!bGT%{Si@G>;VUd+C1hg%dYj@Grt(AS
> zJDCzz*2FTI4ETmO2TOUs5zaD-
> +=8k}qC&uw3~fN)bW%OG@q*tkdp9PhYb6_~6o!$9
> z)1W8@P|o+TmBDYQEGToGZwfQgTj>b+MSaeej5l^DU@!Qn2BcLsya5<HHu_
> r-z0oJ^
> z8_-YyCSW~Z^si6Yy9(GEZ#n|>kO1}6ze|@rm~P80D8C$<;0-
> !j{}_>I_C;`>sDRms
> ze_*@YF}nnSXxe|lvi6ineAd(I<XUG(`JTJNjOX9i0)z0<0=oYrPImHzarA>lZ6?*d
> zP#Q0Zf-
> ||Abk)w2olgz%L$nqeoLSTupDMASTGB;ktb9_JBrN0q6+o&gc&<hQAvdX_
> z6d-
> ;bt`LUA5kj#M(vOjy^<4e?sz%ve;!}1=$zl<$KtVB*lzm6MVPnD?ZWSw+g=9ST
> z8j>7B==6QDbhkh7C<M=}b^F^vaqw^M0wS@XT%jcj1RAJl9mFMN*jP*-f-
> B3>j*wz;
> z`yCeUqh%#*3~_RE!626KA<B@}C^dW1n(`tUl{VUg0h(e`_<Iiykw>k8BJvv{N1I
> =j
> zi7~zkpOI|Ru7|rPJ-!(XOv17I@$qtxy&{YKK-
> vGALs;*PdAhpQPKwF@g2edsAO??j
> z6s68^KL~~L#FTJ8%I(Yhp5Q$0yjUC`+6=&W#f<4#cWP56f@zD@iuUtfgNMxRz
> I1{)
> zmaK-tiT%8HtbIU+mnS*4!`^U5;k-
> *u{;TXm#;>HfM$DeuPv7ZZB&)5hRtug8>!6E#
> zbT?YZ#D}>lXlga89@9&VQq~X!)7QThTpd6k;xM!#Y(~AyNuII)kkkqBdH7pFY6
> W!$
> zk%+#7dt--
> (ZFE2qg8mMBz=2)odQbj){ZxxAH!@XN!^~`YcO5<?7D99*Pdg@kx1JhI
> zUWqvr{+HLsxcc^s>=`OH!6e=%{n(l8*w$JfRV+SpiL8>dIr!&gc*d=zaCXeVQlW`
> #
> zi$=3psUOSH6p-
> %;e?E&{;R7Sjw#0DA{C5?mD&?aNCUl7=%|c)0SlEIGB__x=w|qEQ
> zaygN!%yDh4g*;XsXb>VM>O=2Rf-RI3n)}xD^tC$6Rx<Tztmo8RX#4IP7e*&4-
> b}Ot
> zm=^jw3V}4%?IXDW<jcNLn3vHbU?6<I?`c!}4LrlA{R(8<>aXCWbXR(Eyz(D8jrfK
> O
> z^!tbWKm(Q#Z>YNCFxz08`a>-
> <mkNHVSIj*mYsD2UM~aIsY5bYau^hi0W1(onokdnI
> za-S(C)0R^0M5Iq2M9l5se{*ni@q6kk`%6I1BPX9g?eS)^Hi{Ll{iiRNi})8$I6~w`
> z)rcE0lG}c|_)LOS#?FR-;V;v-
> OZ)dS!LLqOXfWTm@s%)1gC2%axl8XOb!A7BO<@G3
> zi4e$mL!6z|(S`GAxDNbmq_(rGRTmk_6?B~p9;j8NgA|a1r}DPO-
> R?+jdDs;=3OXvk
> zX{<{P%dlYn$3NJOXX0PfM>YluF6VCB`>$zo(JUjx{W6oS;iIjx<<ndeu{qolO}?`5
> zZan4|9NAcXpqL}$P|C(IZ8+c}TYgr)C?Kzut@LHTZc1klb<zGWtq-
> h7qXmCyY8EJ?
> zEauUp+*Ri@Jf)UAvtiy=$31F=o$*cz{S>{vqwgMIBZA1cfqB8Q0Y##S)2-
> t5gMjsd
> zw$YI1Gcrb^r{v<~aHVcQzKI?d{{%LlkyHiUq)nQWf<_3c#`A{$M?XncD=&<*iNH
> =u
> zx<E)0(**Cx2$jCMBfBZNoN-
> Ej=)Bwa<cwnz3d>Lk?|a?H7N_kS!JQ?^tj+<krmMFS
> z%9Nnxp@j<Tb*Ao!#{2IpnP|8qDj-
> jXacIu%eP2NesPil1Cv@A4uU#T}kkAI|<oG%c
> zNa{+H{-
> zdCSESYz^#j)HSIlqVALk%1+h*Vjt?daQ@NE5;_um$ByxUPolMV54adNrc
> zd+^0Kj*pf#MGd%^7^w*%OC)?7b(OC(+#D95WGs$>lp%`I((6Yyyk=w4ova8q
> tktlx
> z>C79nT=rFzUQFXnZ2!^x7sL)pNR?;)ZORZe@>3~QI6NU(E0x;`o=mFs=g8q$j!O
> O&
> zy~)f$A|36&O1AfRt=qW$P-
> A5SmI%Kj@1wULnY=zE>&(9g;}ZJX<IPw)>r@Q)Gp*uu
> znceX3tWzz#3|b$|h-?#i*~FwB_jFCfhWBu-_*kPIZ8$56D)aq}yn^`#^=l)gqRrvp
> zRDR)d_*LIevl3m<6yirxt%afB4|g&QPv$?vkO)H@KLa1L{0-z)hR`#UJ4sGz!lB3F
> zSrnQof}#q_-
> MTbwXM_CWTb10ukZWS?^)0lx#UZE6P4!Zx5)rBKwmWrdTTN?y`s&~+
> z{Evgdp>)oNlv33IJk1k$R{`*E@NXW($IM~v$6cD+0R0~rdvU#P-VrFrj~yiiKyO*`
> zdmrX?oyXmWAkPl*Gav*L?^Y&ptN%+Ih>Q7OU&D<9ul;?;!6jcnd1_wzUCh>C
> #`*0W
> z!a@xhm}2%UIo6cy*)w~LXv$6r*=c`>*<3JJ0^hOppC@Wh5-bc!9bEe%e>>-
> P1b6@e
> zI?@^}l-0q07p4~&Z-
> (+KMp?|DG){zFWt%j%qNO^xp(qJI!2JKP@08QY+>=x!ST3Px
> z6vFKU=iXfp{S~1+{1u*S*C@k7YAp}TNqf6Ak)$VC#T0Q!PeabGyBjt(-
> vK}T?A3ou
> z0R<>C?`y$I?Z=Fn7cC3t)(YGjcAK9N$JXe3KYMw3xNAhy{xWtyR6F9^j|S}#77O
> +N
> zw>{lOmJtEZ=cjkPRu<_=vD?7@`B*UmEvlKsJht#5;fxI9lyn}S@%9GL<!YU_%Tz
> )U
> z{IV0S(z-
> Z3VmXk?gdpd_PmmrzowgW*;LIMgB$oB*c05YCO2L1uk<H)mt0V#uPX(5<
> zRzPy>Nw34nev_vB|0IvsYun(YpKt#J_ewzX4!<oM&jWUO0lH1;_&yL#Sei-
> m)=LB^
> zTAz|c$@GNpqh!Av#+J7Q{e5+dKQwHo#eODrW$@>xyR(TM>d#ldwJDUeLit
> VJ`SV#K
> z^4Is0-
> E|D0<{XXH0>!|ekU|QRI4~h(3YMk}Z(#8C^c9`h^{5GMy~;Z4thllVw);ZM
> zvQp#Cy;gzDN=7jNjVGW*NP&2y2C-
> 5n)Ml3B_~}B|a7;o<rc+r<9m0w^qHN9#hs*i@
> z^SWjHfOG1{v^xg<iiA}ZeuRa&SYsj~-oNQ(Dur-`hb!L-tB9!9j2|Eg`kG6aS`E;}
> zCZ${$+vsb0B^iXVmDPQF=@{+D+KkIy*5C<EXbyO+-$p-
> Np;_DeA}U#Grf)}D`AYW*
> zykl5@u{-{MI>(UsKb>PBe8F-x%skO8;F-
> ~ADW6Rn2pDPY@(ISyw~`NO=xm3_z$&zI
> zPvIJ_az9=|=#$h!{?-
> |jK>F{;56>09NrCMtcZE#UahuT0W}b$JXhwy!0K9QZh6J~y
> z<TD=;e53=umG3kB%k(jx|6b+@+_0K%@S9Jv4^qA*IPagdP^T#gdGA`aeWo(
> Lv90@3
> zDTs3kU~8vZDH)U}=nE16JzE5~FW?}hy0BYNSB@~K3IGo`tpI{qK-
> s`$ZF4Q3A3+*t
> z38^Gw;Q605i*A2%W}E54(0IQTZAbaUKW3g10e4NxJ~W@~r=`MsW9Y^F9C7
> b}4TXsh
> zATg+*8WU2*43Wqi1O0vS{&<St2T&2T-
> ~Pt%z25c}YJ0FT8Q*GQKI)2nVTJ>s6B=au
> znSSdX-
> FE+XMZ^5w@vlfooyuagUe5<rh{!W`l{~1PDOtF`xQK=R7y5e($(nz$v}@@n
> zX(M&!@)u+H=;i~XbJOOEDXQPtdQ7^{3Ed|p{ZgY-
> ^5RfM;y26&<u@UE_ARQa+|d6g
> zca>^a-
> uV&xYyZg|at5_f1Q!>2>lzgrx#)nKQP?3xPqzgf@o%~wL2b0b0Hqd2#!p9C
> zqYEh{V%0NaB)fUQtH&C4k8En_gZzd`NWgHvu?lV6S=V|BPB*+$x99q4i~H-
> mPvzM_
> z>o`1xeRp)(C4>a#QB4pA=|;aRh^!4bbMxL2*zfx^);<D&|Ch@1&;hE&!u;W~Y8
> -Mp
> z_+J|gwtU76oD|a1q8;AMLN=WB1CcG3Nb1@HsGL?2-
> }XmI31of*ZO{F3R<J6O4gce%
> zPNjIkHm-%0-
> UCHyD$d@*EYAfODz@y9&*3)~o*TQ#NwgtqdtOR<ueuv+`X}w%d^Qhy
> z$iyNCY$F+kfO91Q={@RSX~SJ--p0&!p(u>2|ARKr1g7o8J?Qt6{y*#-
> AjeKS8~9K{
> zm<49b*9h$XUwQ&ME<r5yYstT!CY#?^l9qEf3)?hiC60_<*n6ux#hJc~G2uFlTv8
> Uy
> zI>eqBdH-Dd0aty|AI~$B|Fyx+(}APq(nhhKl47wu>&wjfr6ilG{<r~EltSA4>1EQB
> zbeiog__nibEc+HyKCf`DC(ky|YR>8zw9+0>_|~@|ISB+^=Z{?jttY^*+^g5VgnM
> xR
> zCq=w*uI`hE!-
> xI%9VUBuuvzFQhRNIE?E&+Cz3McT@viq|nvyfKM6G{eOoxf=u0QI0
> zq^$TdqNtJe0h}k$F-+$ms=)w6@VsrS0n)}M@?l>F-
> ns_<J3_qyO`9OezVo|M5>Muh
> zZ{xz>$hG~fmg%4wyS4BJp(B6PMPgJP{4z1nO!Y5MHJ#^CDK--
> pX1l{8ov+g~mGPSA
> ztW$2Eb6m-
> s)bvwyy=OAiu&Nt9`w{t>4PE5Qy)+3&9WK*M)ADkvu^lgN)4+?P=%XIV
> zv?-
> 4`f_<8Wch!<U_{tMry1oMzhvMbW+37(~a)`k@cQX|YvIK!l@$9tY1kAPJ6s!u)
> z@riB=%{b-nOUjsGfLk@SsF|`^DR6c;Yr-SC74zG;3g(*)dR&dk#kmp2>f`ff6#RYd
> zJN<nYsc~Zxd4_&a_+v{JgODc11bXR+%_KhfmC|X<U2MXoKTr%LO0yQ4NCE#
> )b9imE
> z-pp~lq3nld`&`1XxyZ9{^`siH_-LnKbL9i2y(D<(^+98q?wMdYz7h904eFS(tO8Ps
> zi^;6nFI_gf0TX`%^?h6m0&Al@hM|7pp1;I=jy?6kzTIruzj%k!n7|r<hy&~a&Dec
> V
> zF>;zm^Lu@ZJ6a>A!FPuBT#>H#j1sm9$s5eN15Su)Jy6uzRrS>$ZynSoL4o?;ShI9
> *
> zeF719MxwU-M!w~pa>o2L-mu`on4{k&Fy#I|xtjbEH@!S;(lE@*pVfhB-
> NvldSqyTt
> zJ|E__I-
> FK?DgVq?6ZSvF{DMb!RbUzfDW(FJuG8z?Bkr<<5M8k3ZmZN!C@C4smXqHx
> zwWC2e?}AVaYOb+3o50f+G*HKT-dq83j{@2<%&P!jmKPA1NH;eX0V-
> PML_B&sy=T5B
> zrgK>eAB()=UJV^{Pt52)#H4K$?5RlD-
> fiD2gGVn9f~HNCVi$Fbz7AEoF<%^xRYayO
> zo1DEKu9l4-
> zhEMe{9!h$&!zK~!sVlX^xwYE^+(TXg%8lR|C%=hJbMuZ09ypEzxcf%
> z=&iSToOl0GF`ffNMLTAZwBPHTRkFY27C}$Coy2cNll#wXrC}g;X)!|+M%rwVUiB
> <U
> z5^4H9>~k&`%mkPAMGiq|fW82-
> +{XQ=K|F@eA}*MvX8Yl7gjKwYDGfDIqawmiJIfDG
> zT7gu!uw-
> iqHeQi);6y;OZ@!EY7)v&<8USJMgQ7Qe3xF&4w{T$j8oljBfxz?bR&aO#
> zw@=fnpJ--*&66>mH1Z=$crH5-
> aRFK_B><T$3|{h!G91e=rUqHyMllL&c1s~*USpxD
> zqCGkdBIRK0fxWRsd{qerrUgDVk1Qglc#`v>>E~ev-JCr<(c4m))?X+*#|PS3(3I5v
> z@han_>2WYYJ~I)k{-
> o~OjO#^k@`zuc7&q0Ga*f6Oo(H^4!WU5x4_6WgUD$c9bdzv(
> zQs-
> ?rR9w#FmnSg<k_6FRLio#$P<i`9D)pLWSWzGUzSEyxWeh@5vhIIUO!xX63yK*!
> z^(K^s449q%4GariXm>llwoWF=3{8c6!bC8q3-
> %LK$}8L`3;PbCfoFKN=}yGC1|a2H
> zV@S`SompJKl49-
> &V($&8n|{;<!YH;wcwIi8;XSU_E}Gte8i+kboTBVVuE^Jvzn3#3
> zuMcAy7^V>E)iC6={sFS@AK^NGSqB~b4wVDW6H4yFpVNXE<V4jHZE%LX+==
> -1UbS!<
> zZ*n~enX`u`b%a75>!OwZL-%XI`(r`z6hP=9;|CCSYTnxt{(G+=sDGeuh>wR4Z$){t
> zY&oF&r7R{Whm~_#9a{X>#!0JhwPtZT_4&$VQP$#Z>NFUpA(Mp}`M(jrMf?x
> q%qSU*
> z<wN+~QY9|C#2?CkH^fohNb-
> |~AEP*8{wHxmKjJJ$j%NNRarImp46wK)b{T|I2P%kT
> zto7!t&I$u`mRaDPE#ZE0%e`!vKo;Uf3R!5dGl}7-
> hYNh9hc>RfOmUPt7Qs({AGxZ*
> zd;xR_=J0cshFCHyUs~~VL!H)5;@Z#Wv+f5Q6sTkV#c?km#QJ%9g3jrHpMk1v(
> dH)+
> zl5)v6bNga4)HD2SRDOi!Y|in^tFRev`J-71L)l-4Fn^0iv(XK4QmQFdp)vAs69~ej
> zYeL#9WMV$kpEK__k>h5b<^{_OI$Oo_<1oe?R1+_=LYLzGY0K#U495*WPl&Y
> %-ee+P
> z?s*pcvseHPw6VhLKmnYDH!tWGz(VN@Q$Lj5Yu|tW8OYy~8<6R%z|1j)A-
> Cp~0W7EW
> zw$cEu>LvXJ=k1}+h4<(}h=?UT4E~FIV3-
> )`y|!2VADoaV(Z2e^4aVGUW;Xw=BR^NZ
> zWZUsq@u=ItXvd}_)bPc<!*Nt9<M0-3V$@VZ5&H!C02I-
> wB8ziOhtP)V`?ZNLsY219
> zkd=9DoRF;$ZhCmt;JBk@QclTO|DSpcI(lf7J#D%qL9(H3$;c@_RH0uUEwi^hCR
> OAq
> zvnmd^wF)L|pqEpf(Z3Jj9%>fL)dY}bZQ)Kk9g^*85Jw-pM<4t#R{Uu-
> Rned%9Y2G$
> zDa;N_&Cc)SfZ0DD!Yn2J)o@^BA=k`DM^ft{7Bxwwmk16^iSq81?Q0O1g`Gv0M
> Gtgi
> z6i+9+Yg8kdj|h?xbDTMaY99_C-
> 99?@m{~F1$@+70e!tsU?{rmk85C}{4%gmTQ<8Ib
> zpU~kTY4*Ydl=t@lyPg<i@`a$f`9O?5&j#b^n{U(tl44GJqkB1)2m>S#7+Wa0hw-
> 9B
> z2zU{V0s}5%){TyJfy40P3xq_5X4r0*T{9X+@*<otrt>e$?#*H>8WsDJw#IYPKf5F
> >
> zcr4%2g~Oslu{s6eV<Z90i%h-
> !O8&8`{`5KZ)+@&V$Jy}+`c*4_vkHx)pM!DVTYp)+
> zwR3sGMO_6vk%O?5d18<;)yYUcyy;T4ggOBXV}YUCm_0+rZ0LBQ$l-
> R>5$m09Nu1J7
> zweYeE(=Qxq_)VO>j-^E)gluf-
> mPEc*s)GJXvF4*UH1JnzK2m!xn32#sxc)R_wz8rc
> zdXg0Qxs3(04XrKsDp6nSpwuf%p>LlXRN7zV*(cbxn$fIBF@R*r8`!|-
> Xkh=1BQ7F3
> z-
> DWv9K<)=IOwM=N#D)Vy_L1o?=Ok2N7WVfW!Ct!z)wvsWK2MaAQ1c{|)I;AU
> )iKFg
> zi5DPx-
> oBQad1}^vIo15yR|W60Q@;pk%C3{n5PK1y<LDqHlYi9MU>Vv0B2*q2Pk`-G
> z`F-tr6)<L-Q-gpw-
> ury!P+VL1p#%&>cI=?*R9v5_2{Tr7V3px~dfWP~LX7V%lD%o^
> z4-
> 5F1jkyg(TAruJJ>G(Gv@uH*zt4Z6q9&<ToXCP@99F(_K#ss@QKTC2*Dd!EIM~J
> f
> zyi^m-
> zR1U^5HHB=QA%sdvQ0hwEbJB@mdNpG^{dqMZ_@tC*JQZMJWcWk>Wi~4
> N&RpA
> z68ZXsAAR5s8Z{G3;VIAgfM)ZsqT`Y1Lv8Mgi?Hm5krl{Qoh!`NhqdormAb+pHId
> dp
> z%(#f@w*EyjPM~0#%nQFev>H+OO&UhGrQEr638R{Ip+OGz0D!xquyNE5QL
> Ng%wYYi}
> zckra{!d?ic9I#?Ly$}m7R_)_7SFxfj3o{LGD<rv;JcmBL!dd%T>0xx>7~rYgZCV0H
> z|0)$$Je3*bNaw3Zd1ie{8Nm#nx=zE)e}1XAxDEB~IKKj2x;0PSPRxc5hs#>|%9v
> $x
> z6-m=b=0Z7$bTwcV-0ZJf7B&2avU!Okz@xf0Ddch1<lt+1FE!vkF(}8gfls1n-=pI6
> zi=gowN@36i6L*Uth_T>zgsXN&x`{j%;{>)>ddKu#RNM^Sexhc<Ft+(`@1L`u7*
> #c7
> z*|1z~I|>-hT?ewKjD~4z?j^XS40fH>i8+8IYv)CShwCJhmeP(@Nu%`-
> G(hDGilb$c
> z!nl5hTYuzRIJ9x(9F1uLff=!}S>w1J4>5eIjiMLomSAEp(h*K9nZ+kvBK6+CshWXp
> zYA?P9Th6@iUtgn;JwrqZHmw@Y+EWLUspM$-
> 2hxbQ?B8tiu9|#X=bk3KASA=1Hawp!
> Hpg#RSLPc@Y
> 
> diff --git a/engine/tests/Functional.commonAPI_C++/fuego_test.sh
> b/engine/tests/Functional.commonAPI_C++/fuego_test.sh
> index 94d74c4..0776809 100755
> --- a/engine/tests/Functional.commonAPI_C++/fuego_test.sh
> +++ b/engine/tests/Functional.commonAPI_C++/fuego_test.sh
> @@ -1,40 +1,49 @@
> -tarball=commonAPI_C++.tar.gz
> +#!/bin/bash
> 
> -function test_build {
> -    $CXX -std=gnu++11 ./commonAPI_Dbus/HelloWorldService.cpp
> ./commonAPI_Dbus/HelloWorldStubImpl.cpp
> ./commonAPI_Dbus/HelloWorldDBusStubAdapter.cpp
> ./commonAPI_Dbus/HelloWorldStubDefault.cpp
> ./commonAPI_Dbus/HelloWorldDBusDeployment.cpp -rdynamic -
> lCommonAPI -lCommonAPI-DBus -ldbus-1 -I
> $SDKTARGETSYSROOT/usr/include/CommonAPI-3.1/ -I
> $SDKTARGETSYSROOT/usr/include/dbus-1.0/ -I
> $SDKTARGETSYSROOT/usr/lib/dbus-1.0/include/ -o HelloWorldService_Dbus
> -    $CXX -std=gnu++11 ./commonAPI_Dbus/HelloWorldClient.cpp
> ./commonAPI_Dbus/HelloWorldDBusProxy.cpp
> ./commonAPI_Dbus/HelloWorldDBusDeployment.cpp -rdynamic -
> lCommonAPI -lCommonAPI-DBus -ldbus-1 -I
> $SDKTARGETSYSROOT/usr/include/CommonAPI-3.1/ -I
> $SDKTARGETSYSROOT/usr/include/dbus-1.0/ -I
> $SDKTARGETSYSROOT/usr/lib/dbus-1.0/include/ -o HelloWorldClient_Dbus
> +tarball=../commonAPI/commonapi.tar.gz
> +source $FUEGO_CORE/engine/tests/commonAPI/common_api.sh
> 
> -    $CXX -std=gnu++11 ./commonAPI_SomeIp/HelloWorldService.cpp
> ./commonAPI_SomeIp/HelloWorldStubImpl.cpp
> ./commonAPI_SomeIp/HelloWorldSomeIPStubAdapter.cpp
> ./commonAPI_SomeIp/HelloWorldStubDefault.cpp
> ./commonAPI_SomeIp/HelloWorldSomeIPDeployment.cpp -rdynamic -
> lCommonAPI -lCommonAPI-SomeIP -lvsomeip -I
> $SDKTARGETSYSROOT/usr/include/CommonAPI-3.1/ -o
> HelloWorldService_SomeIp
> -    $CXX -std=gnu++11 ./commonAPI_SomeIp/HelloWorldClient.cpp
> ./commonAPI_SomeIp/HelloWorldSomeIPProxy.cpp
> ./commonAPI_SomeIp/HelloWorldSomeIPDeployment.cpp -rdynamic -
> lCommonAPI -lCommonAPI-SomeIP -lvsomeip -I
> $SDKTARGETSYSROOT/usr/include/CommonAPI-3.1/ -o
> HelloWorldClient_SomeIp
> +
> +function pre_check {
> +    is_on_target libCommonAPI.so COMMON_API /lib:/usr/lib:/usr/local/lib
> +    assert_define COMMON_API
> +    is_on_target libCommonAPI-DBus.so COMMON_API_DBUS
> /lib:/usr/lib:/usr/local/lib
> +    assert_define COMMON_API_DBUS
> +
> +    is_on_target libCommonAPI-SomeIP.so COMMON_API_SOMEIP
> /lib:/usr/lib:/usr/local/lib
> +    assert_define COMMON_API_SOMEIP
> +    is_on_target libvsomeip.so VSOMEIP /lib:/usr/lib:/usr/local/lib
> +    assert_define VSOMEIP
> +    is_on_target libvsomeip-cfg.so VSOMEIP_CFG /lib:/usr/lib:/usr/local/lib
> +    assert_define VSOMEIP_CFG
>  }
> 
>  function test_deploy {
> -	put HelloWorldService_Dbus $BOARD_TESTDIR/fuego.$TESTDIR/
> -	put HelloWorldClient_Dbus $BOARD_TESTDIR/fuego.$TESTDIR/
> -
> -	put HelloWorldService_SomeIp $BOARD_TESTDIR/fuego.$TESTDIR/
> -	put HelloWorldClient_SomeIp $BOARD_TESTDIR/fuego.$TESTDIR/
> -	put ./commonAPI_SomeIp/fidl/vsomeip.json
> $BOARD_TESTDIR/fuego.$TESTDIR/
> +    report "cd $BOARD_TESTDIR/fuego.$TESTDIR/ && mkdir -p
> commonAPI_Dbus commonAPI_SomeIp fidl"
> +    put build/project/commonAPI_Dbus/HelloWorldService
> $BOARD_TESTDIR/fuego.$TESTDIR/commonAPI_Dbus
> +    put build/project/commonAPI_Dbus/HelloWorldClient
> $BOARD_TESTDIR/fuego.$TESTDIR/commonAPI_Dbus
> +    put build/project/commonAPI_SomeIp/HelloWorldService
> $BOARD_TESTDIR/fuego.$TESTDIR/commonAPI_SomeIp
> +    put build/project/commonAPI_SomeIp/HelloWorldClient
> $BOARD_TESTDIR/fuego.$TESTDIR/commonAPI_SomeIp
> +    put build/project/fidl/vsomeip.json
> $BOARD_TESTDIR/fuego.$TESTDIR/fidl
>  }
> 
>  function test_run {
> -
> 
> -	report "cd $BOARD_TESTDIR/fuego.$TESTDIR; export
> PATH=$BOARD_TESTDIR/fuego.$TESTDIR:$PATH; export
> VSOMEIP_CONFIGURATION_FILE=./fidl/vsomeip.json; chmod 777 *;
> -	if (./HelloWorldService_Dbus &); then echo 'TEST-1 OK'; else echo
> 'TEST-1 FAILED'; fi;\
> -	if ./HelloWorldClient_Dbus; then echo 'TEST-2 OK'; else echo 'TEST-2
> FAILED'; fi;
> -	pkill HelloWorldServi
> 
> -	if (VSOMEIP_APPLICATION_NAME=service-sample
> ./HelloWorldService_SomeIp &); then echo 'TEST-3 OK'; else echo 'TEST-3
> FAILED'; fi;\
> -	if (VSOMEIP_APPLICATION_NAME=client-sample
> ./HelloWorldClient_SomeIp); then echo 'TEST-4 OK'; else echo 'TEST-4
> FAILED'; fi;
> -    pkill HelloWorldServi"
> +    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; export
> PATH=$BOARD_TESTDIR/fuego.$TESTDIR:$PATH; \
> +        export LD_LIBRARY_PATH=\`pwd\`:\$LD_LIBRARY_PATH; \
> +        export VSOMEIP_CONFIGURATION_FILE=./fidl/vsomeip.json; chmod
> 777 *; \
> +        if (commonAPI_Dbus/HelloWorldService &); then echo 'TEST-1 OK'; else
> echo 'TEST-1 FAILED'; fi;\
> +            if commonAPI_Dbus/HelloWorldClient; then echo 'TEST-2 OK'; else
> echo 'TEST-2 FAILED'; fi;\
> +                pkill HelloWorldServi
> +
> +    if (VSOMEIP_APPLICATION_NAME=service-sample
> commonAPI_SomeIp/HelloWorldService &); then echo 'TEST-3 OK'; else echo
> 'TEST-3 FAILED'; fi;\
> +        if (VSOMEIP_APPLICATION_NAME=client-sample
> commonAPI_SomeIp/HelloWorldClient); then echo 'TEST-4 OK'; else echo
> 'TEST-4 FAILED'; fi;\
> +            pkill HelloWorldServi"
>  }
> 
>  function test_processing {
> -	log_compare "$TESTDIR" "4" "^TEST.*OK" "p"
> -    	log_compare "$TESTDIR" "0" "^TEST.*FAILED" "n"
> +    log_compare "$TESTDIR" "4" "^TEST.*OK" "p"
> +    log_compare "$TESTDIR" "0" "^TEST.*FAILED" "n"
>  }
> 
> -
> -
> -
> --
> 2.7.4
> 
> 
> 
> _______________________________________________
> Fuego mailing list
> Fuego@lists.linuxfoundation.org
> https://lists.linuxfoundation.org/mailman/listinfo/fuego

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [Fuego] [PATCH 4/6] commonAPI_SomeIp: using common script to generate someip test program
  2017-11-06 15:42 ` [Fuego] [PATCH 4/6] commonAPI_SomeIp: using common script to generate someip " Liu Wenlong
@ 2017-11-08  0:04   ` Bird, Timothy
  0 siblings, 0 replies; 17+ messages in thread
From: Bird, Timothy @ 2017-11-08  0:04 UTC (permalink / raw)
  To: Liu Wenlong, fuego

This has similar issues to patch 3.  But see individual comments below.

> -----Original Message-----
> From: fuego-bounces@lists.linuxfoundation.org [mailto:fuego-
> bounces@lists.linuxfoundation.org] On Behalf Of Liu Wenlong
> Sent: Monday, November 06, 2017 7:43 AM
> To: fuego@lists.linuxfoundation.org
> Subject: [Fuego] [PATCH 4/6] commonAPI_SomeIp: using common script to
> generate someip test program
> 
> Signed-off-by: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
> ---
>  .../commonAPI_SomeIp.tar.gz                        | Bin 6533 -> 0 bytes
>  .../Functional.commonAPI_SomeIp/fuego_test.sh      |  44 +++++++++++++-
> -------
>  2 files changed, 27 insertions(+), 17 deletions(-)
>  delete mode 100755
> engine/tests/Functional.commonAPI_SomeIp/commonAPI_SomeIp.tar.gz
> 
> diff --git
> a/engine/tests/Functional.commonAPI_SomeIp/commonAPI_SomeIp.tar.gz
> b/engine/tests/Functional.commonAPI_SomeIp/commonAPI_SomeIp.tar.gz
> deleted file mode 100755
> index
> b00f19aaa659a2b9a4af3be583a980098db12d7b..0000000000000000000000000
> 000000000000000
> GIT binary patch
> literal 0
> HcmV?d00001
> 
> literal 6533
> zcmV;08G7a)iwFR)Z-
[omit binary stuff]
> rDpQ%tRHibOsZ3=mQ<=(CrZSbOOl2xlnaWhAzfSspNV{Xd0LTCU88!z)
> 
> diff --git a/engine/tests/Functional.commonAPI_SomeIp/fuego_test.sh
> b/engine/tests/Functional.commonAPI_SomeIp/fuego_test.sh
> index c47d56b..3b2c880 100755
> --- a/engine/tests/Functional.commonAPI_SomeIp/fuego_test.sh
> +++ b/engine/tests/Functional.commonAPI_SomeIp/fuego_test.sh
> @@ -1,28 +1,38 @@
> -tarball=commonAPI_SomeIp.tar.gz
> -
> -function test_build {
> -    $CXX -std=gnu++11 HelloWorldService.cpp HelloWorldStubImpl.cpp
> HelloWorldSomeIPStubAdapter.cpp HelloWorldStubDefault.cpp
> HelloWorldSomeIPDeployment.cpp -rdynamic -lCommonAPI -lCommonAPI-
> SomeIP -lvsomeip -I $SDKTARGETSYSROOT/usr/include/CommonAPI-3.1/ -o
> HelloWorldService
> -    $CXX -std=gnu++11 HelloWorldClient.cpp HelloWorldSomeIPProxy.cpp
> HelloWorldSomeIPDeployment.cpp -rdynamic -lCommonAPI -lCommonAPI-
> SomeIP -lvsomeip -I $SDKTARGETSYSROOT/usr/include/CommonAPI-3.1/ -o
> HelloWorldClient
> +#!/bin/bash
Should not use this.

> +
> +
> +tarball=../commonAPI/commonapi.tar.gz
> +source $FUEGO_CORE/engine/tests/commonAPI/common_api.sh
> +
> +function test_pre_check {
> +    if [ -z "test" ];then

This looks like debugging code that was left in.

> +        is_on_target libCommonAPI.so COMMON_API
> /lib:/usr/lib:/usr/local/lib
> +        assert_define COMMON_API
> +        is_on_target libCommonAPI-SomeIP.so COMMON_API_SOMEIP
> /lib:/usr/lib:/usr/local/lib
> +        assert_define COMMON_API_SOMEIP
> +        is_on_target libvsomeip.so VSOMEIP /lib:/usr/lib:/usr/local/lib
> +        assert_define VSOMEIP
> +        is_on_target libvsomeip-cfg.so VSOMEIP_CFG /lib:/usr/lib:/usr/local/lib
> +        assert_define VSOMEIP_CFG
> +    fi
>  }
> 
>  function test_deploy {
> -	put HelloWorldService $BOARD_TESTDIR/fuego.$TESTDIR/
> -	put HelloWorldClient $BOARD_TESTDIR/fuego.$TESTDIR/
> -	put ./fidl/vsomeip.json $BOARD_TESTDIR/fuego.$TESTDIR/
> +    put build/project/commonAPI_SomeIp/HelloWorldService
> $BOARD_TESTDIR/fuego.$TESTDIR/
> +    put build/project/commonAPI_SomeIp/HelloWorldClient
> $BOARD_TESTDIR/fuego.$TESTDIR/
> +    put build/project/fidl/vsomeip.json $BOARD_TESTDIR/fuego.$TESTDIR/
>  }
> 
>  function test_run {
> -	report "cd $BOARD_TESTDIR/fuego.$TESTDIR; export
> PATH=$BOARD_TESTDIR/fuego.$TESTDIR:$PATH; export
> VSOMEIP_CONFIGURATION_FILE=./fidl/vsomeip.json; chmod 777 *;
> -	if (VSOMEIP_APPLICATION_NAME=service-sample
> ./HelloWorldService &); then echo 'TEST-1 OK'; else echo 'TEST-1 FAILED'; fi;\
> -	if (VSOMEIP_APPLICATION_NAME=client-sample
> ./HelloWorldClient); then echo 'TEST-2 OK'; else echo 'TEST-2 FAILED'; fi;
> -        pkill HelloWorldServi"
> +    report "cd $BOARD_TESTDIR/fuego.$TESTDIR; export
> PATH=$BOARD_TESTDIR/fuego.$TESTDIR:\$PATH; \
> +        export VSOMEIP_CONFIGURATION_FILE=./vsomeip.json; chmod 777 *;
> \
> +        if (VSOMEIP_APPLICATION_NAME=service-sample ./HelloWorldService
> &); then echo 'TEST-1 OK'; else echo 'TEST-1 FAILED'; fi; \
> +            if (VSOMEIP_APPLICATION_NAME=client-sample
> ./HelloWorldClient); then echo 'TEST-2 OK'; else echo 'TEST-2 FAILED'; fi; \
> +                pkill HelloWorldServi"

Are these just whitespace changes?  I don't see any code difference.

>  }
> 
>  function test_processing {
> -	log_compare "$TESTDIR" "2" "^TEST.*OK" "p"
> -    	log_compare "$TESTDIR" "0" "^TEST.*FAILED" "n"
> +    log_compare "$TESTDIR" "2" "^TEST.*OK" "p"
> +    log_compare "$TESTDIR" "0" "^TEST.*FAILED" "n"
This log_compare is not needed.
>  }
> 
> -
> -
> -
> --
> 2.7.4

Thanks.  I'll hold off on this until we resolve the shared code/shared test_build
function issue.
 -- Tim


^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [Fuego] [PATCH 6/6] core: back to the original path once "test_build" done
  2017-11-06 15:42 ` [Fuego] [PATCH 6/6] core: back to the original path once "test_build" done Liu Wenlong
  2017-11-06 16:39   ` Liu, Wenlong
  2017-11-07  5:10   ` Bird, Timothy
@ 2017-11-08  0:21   ` Bird, Timothy
  2017-11-08  9:53     ` Liu, Wenlong
  2 siblings, 1 reply; 17+ messages in thread
From: Bird, Timothy @ 2017-11-08  0:21 UTC (permalink / raw)
  To: Liu Wenlong, fuego

OK - this looks like a good issue to fix.

> -----Original Message-----
> From: Liu Wenlong on Monday, November 06, 2017 7:43 AM
> It's possible that function "test_build" will change the path,
> such as Funcitonal.glib2.
> We need back to the current path before "touch
> fuego_test_succesfully_built"
> 
> Signed-off-by: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
> ---
>  engine/scripts/functions.sh | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/engine/scripts/functions.sh b/engine/scripts/functions.sh
> index da3415b..209b152 100755
> --- a/engine/scripts/functions.sh
> +++ b/engine/scripts/functions.sh
> @@ -284,12 +284,16 @@ function build {
>      if [ -e fuego_test_succesfully_built ]; then
>          build_duration=0
>      else
> +        # it's possible that function "test_build" will change the path, such as
> Funcitonal.glib2
> +        # we need back to the current path before "touch
> fuego_test_succesfully_built"
> +        ori_path=`pwd`
We already have the path we need to be in. $JOB_BUILD_DIR.

See my replacement patch below.

>          build_start_time=$(date +"%s.%N")
>          call_if_present test_build
>          ret=$?
>          build_end_time=$(date +"%s.%N")
>          build_duration=$(python -c "print $build_end_time -
> $build_start_time")
>          if [ $ret -eq 0 ]; then
> +            cd $ori_path
>              touch fuego_test_succesfully_built
>          else
>              abort_job "ERROR: test_build returned $ret"
> --
> 2.7.4

I replaced this patch with the one below.  Let me know
if you think this is OK.

commit 9a7f0aff700ee2173794bf2a389701a750076fb3
Author: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
Date:   Mon Nov 6 10:42:36 2017 -0500

    core: return to root of build directory after 'test_build'
    
    It's possible that the function "test_build" will change
    the current directory. (For example, Functional.glib2 does.)
    We need back to get back to the root of the job directrory
    before "touch fuego_test_succesfully_built".
    
    Signed-off-by: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
    Signed-off-by: Tim Bird <tim.bird@sony.com>

diff --git a/engine/scripts/functions.sh b/engine/scripts/functions.sh
index da3415b..afd2b32 100755
--- a/engine/scripts/functions.sh
+++ b/engine/scripts/functions.sh
@@ -289,6 +289,10 @@ function build {
         ret=$?
         build_end_time=$(date +"%s.%N")
         build_duration=$(python -c "print $build_end_time - $build_start_time")
+
+        # test_build may change the current dir
+        # get back to root of build dir, before 'touch'
+        cd $JOB_BUILD_DIR
         if [ $ret -eq 0 ]; then
             touch fuego_test_succesfully_built
         else


^ permalink raw reply related	[flat|nested] 17+ messages in thread

* Re: [Fuego] [PATCH 6/6] core: back to the original path once "test_build" done
  2017-11-08  0:21   ` Bird, Timothy
@ 2017-11-08  9:53     ` Liu, Wenlong
  0 siblings, 0 replies; 17+ messages in thread
From: Liu, Wenlong @ 2017-11-08  9:53 UTC (permalink / raw)
  To: Bird, Timothy, fuego

[-- Attachment #1: Type: text/plain, Size: 3492 bytes --]


> -----Original Message-----
> From: Bird, Timothy [mailto:Tim.Bird@sony.com]
> Sent: Wednesday, November 08, 2017 8:21 AM
> To: Liu, Wenlong/刘 文龙 <liuwl.fnst@cn.fujitsu.com>;
> fuego@lists.linuxfoundation.org
> Subject: RE: [Fuego] [PATCH 6/6] core: back to the original path once
> "test_build" done
> 
> OK - this looks like a good issue to fix.
> 
> > -----Original Message-----
> > From: Liu Wenlong on Monday, November 06, 2017 7:43 AM It's possible
> > that function "test_build" will change the path, such as
> > Funcitonal.glib2.
> > We need back to the current path before "touch
> > fuego_test_succesfully_built"
> >
> > Signed-off-by: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
> > ---
> >  engine/scripts/functions.sh | 4 ++++
> >  1 file changed, 4 insertions(+)
> >
> > diff --git a/engine/scripts/functions.sh b/engine/scripts/functions.sh
> > index da3415b..209b152 100755
> > --- a/engine/scripts/functions.sh
> > +++ b/engine/scripts/functions.sh
> > @@ -284,12 +284,16 @@ function build {
> >      if [ -e fuego_test_succesfully_built ]; then
> >          build_duration=0
> >      else
> > +        # it's possible that function "test_build" will change the
> > + path, such as
> > Funcitonal.glib2
> > +        # we need back to the current path before "touch
> > fuego_test_succesfully_built"
> > +        ori_path=`pwd`
> We already have the path we need to be in. $JOB_BUILD_DIR.
> 
> See my replacement patch below.
> 
> >          build_start_time=$(date +"%s.%N")
> >          call_if_present test_build
> >          ret=$?
> >          build_end_time=$(date +"%s.%N")
> >          build_duration=$(python -c "print $build_end_time -
> > $build_start_time")
> >          if [ $ret -eq 0 ]; then
> > +            cd $ori_path
> >              touch fuego_test_succesfully_built
> >          else
> >              abort_job "ERROR: test_build returned $ret"
> > --
> > 2.7.4
> 
> I replaced this patch with the one below.  Let me know if you think this
> is OK.
> 
> commit 9a7f0aff700ee2173794bf2a389701a750076fb3
> Author: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
> Date:   Mon Nov 6 10:42:36 2017 -0500
> 
>     core: return to root of build directory after 'test_build'
> 
>     It's possible that the function "test_build" will change
>     the current directory. (For example, Functional.glib2 does.)
>     We need back to get back to the root of the job directrory
>     before "touch fuego_test_succesfully_built".
> 
>     Signed-off-by: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
>     Signed-off-by: Tim Bird <tim.bird@sony.com>
> 
> diff --git a/engine/scripts/functions.sh b/engine/scripts/functions.sh
> index da3415b..afd2b32 100755
> --- a/engine/scripts/functions.sh
> +++ b/engine/scripts/functions.sh
> @@ -289,6 +289,10 @@ function build {
>          ret=$?
>          build_end_time=$(date +"%s.%N")
>          build_duration=$(python -c "print $build_end_time -
> $build_start_time")
> +
> +        # test_build may change the current dir
> +        # get back to root of build dir, before 'touch'
> +        cd $JOB_BUILD_DIR
>          if [ $ret -eq 0 ]; then
>              touch fuego_test_succesfully_built
>          else
Yes, I submitted another new patch for this issue before.
Attached is that patch file which seems to be same with you did.

Really thanks for your review.
I'll respond to your questions about other patches tomorrow.

> 




[-- Attachment #2: 0001-core-back-to-the-original-path-once-test_build-done.patch --]
[-- Type: application/octet-stream, Size: 1060 bytes --]

From cb0960773682c5cde1645c8c25587af80bdc41fb Mon Sep 17 00:00:00 2001
From: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
Date: Mon, 6 Nov 2017 11:17:23 -0500
Subject: [PATCH] core: back to the original path once "test_build" done

It's possible that function "test_build" will change the current path, such as Funcitonal.glib2.
We need back to current path before "touch fuego_test_succesfully_built"

Signed-off-by: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
---
 engine/scripts/functions.sh | 1 +
 1 file changed, 1 insertion(+)

diff --git a/engine/scripts/functions.sh b/engine/scripts/functions.sh
index da3415b..c7dba25 100755
--- a/engine/scripts/functions.sh
+++ b/engine/scripts/functions.sh
@@ -290,6 +290,7 @@ function build {
         build_end_time=$(date +"%s.%N")
         build_duration=$(python -c "print $build_end_time - $build_start_time")
         if [ $ret -eq 0 ]; then
+            cd "$WORKSPACE/$JOB_BUILD_DIR"
             touch fuego_test_succesfully_built
         else
             abort_job "ERROR: test_build returned $ret"
-- 
2.7.4


^ permalink raw reply related	[flat|nested] 17+ messages in thread

* Re: [Fuego] [PATCH 1/6] bsdiff: add test_pre_check for command "bsdiff bspatch"
  2017-11-07 19:04 ` [Fuego] [PATCH 1/6] bsdiff: add test_pre_check for command "bsdiff bspatch" Bird, Timothy
@ 2017-11-09  7:22   ` Liu, Wenlong
  2017-11-09 19:31     ` Bird, Timothy
  0 siblings, 1 reply; 17+ messages in thread
From: Liu, Wenlong @ 2017-11-09  7:22 UTC (permalink / raw)
  To: Bird, Timothy, fuego


> -----Original Message-----
> From: Bird, Timothy [mailto:Tim.Bird@sony.com]
> Sent: Wednesday, November 08, 2017 3:05 AM
> To: Liu, Wenlong/刘 文龙 <liuwl.fnst@cn.fujitsu.com>;
> fuego@lists.linuxfoundation.org
> Subject: RE: [Fuego] [PATCH 1/6] bsdiff: add test_pre_check for command
> "bsdiff bspatch"
> 
> > -----Original Message-----
> > From: Liu Wenlong on Monday, November 06, 2017 7:43 AM This test
> > depends on "bsdiff bspatch" being already installed on the target.
> >
> > Signed-off-by: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
> > ---
> >  engine/tests/Functional.bsdiff/fuego_test.sh | 7 +++++++
> >  1 file changed, 7 insertions(+)
> >
> > diff --git a/engine/tests/Functional.bsdiff/fuego_test.sh
> > b/engine/tests/Functional.bsdiff/fuego_test.sh
> > index 6ebe3c7..dfc3f80 100755
> > --- a/engine/tests/Functional.bsdiff/fuego_test.sh
> > +++ b/engine/tests/Functional.bsdiff/fuego_test.sh
> > @@ -1,5 +1,12 @@
> >  tarball=bsdiff.tar.gz
> >
> > +function test_pre_check {
> > +    is_on_target_path bsdiff PROGRAM_BSDIFF
> > +    assert_define PROGRAM_BSDIFF
> > +    is_on_target_path bspatch PROGRAM_BSPATCH
> > +    assert_define PROGRAM_BSPATCH
> > +}
> > +
> 
> Looks good.  Thanks.
> 
> I have a question about this interface.  It seems like this will be a common
> pattern:
> is_on_target_path, followed by assert_define.
> 
> Would it be useful to have an API that does both operations?
> I'm thinking of adding something to the dependency checker, so that the
> following declaration in fuego_test.sh would perform the same
> functionality:
> 
> NEED_PROGRAMS="bsdiff bspatch"
> 
> What do you think?
Yes, to add another way to perform the same functionality is useful.
But hope it will not let users misunderstand the relationship between "NEED_PROGRAMS" and "is_on_target".

I'll add this function for myself first, and then test on it.
Thanks for your review.

> The reason for adding this into the dependency checker, is to allow static
> evaluation of test dependencies, for (eventually) automatically building
> a testplan for a board, based on tests that will work on it.
> 
> >  function test_deploy {
> >      put data/*.modified data/*.original
> > $BOARD_TESTDIR/fuego.$TESTDIR/;  }
> > --
> > 2.7.4
> In the mean time, I have applied this patch.
>  -- Tim
> 




^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [Fuego] [PATCH 1/6] bsdiff: add test_pre_check for command "bsdiff bspatch"
  2017-11-09  7:22   ` Liu, Wenlong
@ 2017-11-09 19:31     ` Bird, Timothy
  0 siblings, 0 replies; 17+ messages in thread
From: Bird, Timothy @ 2017-11-09 19:31 UTC (permalink / raw)
  To: Liu, Wenlong, fuego



> -----Original Message-----
> From: Liu, Wenlong on Wednesday, November 08, 2017 11:22 PM
> > -----Original Message-----
> > From: Bird, Timothy [mailto:Tim.Bird@sony.com]
> > > -----Original Message-----
> > > From: Liu Wenlong on Monday, November 06, 2017 7:43 AM This test
> > > depends on "bsdiff bspatch" being already installed on the target.
> > >
> > > Signed-off-by: Liu Wenlong <liuwl.fnst@cn.fujitsu.com>
> > > ---
> > >  engine/tests/Functional.bsdiff/fuego_test.sh | 7 +++++++
> > >  1 file changed, 7 insertions(+)
> > >
> > > diff --git a/engine/tests/Functional.bsdiff/fuego_test.sh
> > > b/engine/tests/Functional.bsdiff/fuego_test.sh
> > > index 6ebe3c7..dfc3f80 100755
> > > --- a/engine/tests/Functional.bsdiff/fuego_test.sh
> > > +++ b/engine/tests/Functional.bsdiff/fuego_test.sh
> > > @@ -1,5 +1,12 @@
> > >  tarball=bsdiff.tar.gz
> > >
> > > +function test_pre_check {
> > > +    is_on_target_path bsdiff PROGRAM_BSDIFF
> > > +    assert_define PROGRAM_BSDIFF
> > > +    is_on_target_path bspatch PROGRAM_BSPATCH
> > > +    assert_define PROGRAM_BSPATCH
> > > +}
> > > +
> >
> > Looks good.  Thanks.
> >
> > I have a question about this interface.  It seems like this will be a common
> > pattern:
> > is_on_target_path, followed by assert_define.
> >
> > Would it be useful to have an API that does both operations?
> > I'm thinking of adding something to the dependency checker, so that the
> > following declaration in fuego_test.sh would perform the same
> > functionality:
> >
> > NEED_PROGRAMS="bsdiff bspatch"
> >
> > What do you think?
> Yes, to add another way to perform the same functionality is useful.
> But hope it will not let users misunderstand the relationship between
> "NEED_PROGRAMS" and "is_on_target".
> 
> I'll add this function for myself first, and then test on it.
> Thanks for your review.

I had some more thoughts on this, that I wanted to share, before you get
too far into an implementation.

I originally structured this with is_on_target, specifying a shell
environment variable name, in anticipation of being able to use
these types of "detected board features" as dynamic variables
for the board.   I haven't document the dynamic variables feature
of Fuego yet, but basically you can set an arbitrary variable on a
board, programmatically, using ftc.

Like so: ftc set_var myboard PROGRAM_FOO=/usr/weird/location/foo

Then, you can get the value of a variable using ftc:
$ ftc query_board myboard -n PROGRAM_FOO
/usr/weird/location/foo

These are stored in the /fuego-rw/boards directory.  The intent
of dynamic variables is to allow the system or another test to store
information that can be used by other tests.  This essentially
creates an information cache about the board, that is programmatically
generated and managed.

One use case for this to have a "board setup" test, that scans for lots of 
different items, and populates the dynamic variables with values that
are used by other tests.  Then, a function like is_on_target could check
the dynamic variables first, to determine if the variable is already known.
This avoids having to find the value at test runtime.  Some items take time
to discover (using e.g. 'find' on the target board), and this can help reduce
the time required to check these conditions.

I still need to add code to have the board dynamic variables appear automatically
in the environment for a test when it is run.  So essentially, this feature is
not quite complete yet.  But that's the direction I'm heading with it.

Anyway, getting back to the is_on_target/assert_define API.  I think it's
useful to have an environment variable name assigned to each item that
is scanned for on the target.  My simple example that I provided doesn't
do that:
NEED_PROGRAMS="bsdiff bspatch"

So I'm not sure this is how I'd like that API to look.  Maybe something more like:
NEED_PROGRAM_BSDIFF="bsdiff"
NEED_PROGRAM_BSPATCH="bspatch"

would be a better API.  These would, internally, set and use  PROGRAM_BSDIFF
and PROGRAM_BSPATCH variables, that could be cached from run to run, or manipulated
and/or read by other tests.

Let me know what you think.
 -- Tim


^ permalink raw reply	[flat|nested] 17+ messages in thread

end of thread, other threads:[~2017-11-09 19:31 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-06 15:42 [Fuego] [PATCH 1/6] bsdiff: add test_pre_check for command "bsdiff bspatch" Liu Wenlong
2017-11-06 15:42 ` [Fuego] [PATCH 2/6] curl: add test_pre_check for command "curl" Liu Wenlong
2017-11-07 19:09   ` Bird, Timothy
2017-11-06 15:42 ` [Fuego] [PATCH 3/6] commonAPI_Dbus: using common script to generate dbus test program Liu Wenlong
2017-11-07 23:54   ` Bird, Timothy
2017-11-06 15:42 ` [Fuego] [PATCH 4/6] commonAPI_SomeIp: using common script to generate someip " Liu Wenlong
2017-11-08  0:04   ` Bird, Timothy
2017-11-06 15:42 ` [Fuego] [PATCH 5/6] commonAPI_C++: using common script to generate commonapi c++ test Liu Wenlong
2017-11-07 23:56   ` Bird, Timothy
2017-11-06 15:42 ` [Fuego] [PATCH 6/6] core: back to the original path once "test_build" done Liu Wenlong
2017-11-06 16:39   ` Liu, Wenlong
2017-11-07  5:10   ` Bird, Timothy
2017-11-08  0:21   ` Bird, Timothy
2017-11-08  9:53     ` Liu, Wenlong
2017-11-07 19:04 ` [Fuego] [PATCH 1/6] bsdiff: add test_pre_check for command "bsdiff bspatch" Bird, Timothy
2017-11-09  7:22   ` Liu, Wenlong
2017-11-09 19:31     ` Bird, Timothy

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.