test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH 1/3] Support check testing environment for test case
@ 2015-09-23  6:40 Yong Liu
  2015-09-23  6:40 ` [dts] [PATCH 2/3] Add dpdk support case list execl file Yong Liu
  2015-09-23  6:40 ` [dts] [PATCH 3/3] Framework: enable test case support check Yong Liu
  0 siblings, 2 replies; 3+ messages in thread
From: Yong Liu @ 2015-09-23  6:40 UTC (permalink / raw)
  To: dts

From: Marvin Liu <yong.liu@intel.com>

This module will load supported cases information from dpdk_test_case_checklist.xls.
When checked environment not support this case, function case_support will return False.

Signed-off-by: Marvin Liu <yong.liu@intel.com>

diff --git a/framework/checkCase.py b/framework/checkCase.py
index b2e79a0..9e1dca8 100644
--- a/framework/checkCase.py
+++ b/framework/checkCase.py
@@ -5,6 +5,10 @@ from settings import nic_name_from_type
 filter_file = r'./conf/dpdk_test_case_checklist.xls'
 filter_case = []
 check_function_dict = {}
+support_file = r'./conf/dpdk_support_test_case.xls'
+support_case = []
+support_function_dict = {}
+
 
 class parse_file():
 
@@ -12,7 +16,10 @@ class parse_file():
         try:
             self.book = xlrd.open_workbook(filter_file)
             self.sheet = self.book.sheet_by_index(0)
+            self.support_book = xlrd.open_workbook(support_file)
+            self.support_sheet = self.support_book.sheet_by_index(0)
             self.init_check_function_dict()
+            self.init_support_function_dict()
         except:
             pass
 
@@ -20,22 +27,36 @@ class parse_file():
         '''
         init check case functio, and skip case message.
         '''
-        row_data  = self.sheet.row_values(0)
-        for i in range(1,len(row_data)):
-            if  row_data[i].lower() in ['wq number', 'comments']:
+        row_data = self.sheet.row_values(0)
+        for i in range(1, len(row_data)):
+            if row_data[i].lower() in ['wq number', 'comments']:
                 if 'message' not in check_function_dict:
                     check_function_dict['message'] = [i]
                 else:
                     check_function_dict['message'].append(i)
             else:
-                check_function_dict[row_data[i].lower()] = i       
+                check_function_dict[row_data[i].lower()] = i
+
+    def init_support_function_dict(self):
+        '''
+        init support case function, and skip case message.
+        '''
+        row_data = self.support_sheet.row_values(0)
+        for i in range(1, len(row_data)):
+            if row_data[i].lower() in ['wq number', 'comments']:
+                if 'message' not in support_function_dict:
+                    support_function_dict['message'] = [i]
+                else:
+                    support_function_dict['message'].append(i)
+            else:
+                support_function_dict[row_data[i].lower()] = i
 
     def set_filter_case(self):
-        for row in range(self.sheet.nrows):   
+        for row in range(self.sheet.nrows):
             row_data = self.sheet.row_values(row)
             # add case name
             tmp_filter = [row_data[0]]
-            for i in range(1,len(row_data) - 2):
+            for i in range(1, len(row_data) - 2):
                 tmp_filter.append(row_data[i].split(','))
 
             tmp_filter.append(row_data[-2])
@@ -43,12 +64,27 @@ class parse_file():
 
             filter_case.append(tmp_filter)
 
+    def set_support_case(self):
+        for row in range(self.support_sheet.nrows):
+            row_data = self.support_sheet.row_values(row)
+            # add case name
+            tmp_filter = [row_data[0]]
+            for i in range(1, len(row_data) - 2):
+                tmp_filter.append(row_data[i].split(','))
+
+            tmp_filter.append(row_data[-2])
+            tmp_filter.append(row_data[-1])
+
+            support_case.append(tmp_filter)
+
+
 class check_case_skip():
+
     def __init__(self, Dut):
         self.dut = Dut
         self.comments = ''
 
-    def check_os(self,os_type):
+    def check_os(self, os_type):
         if 'all' == os_type[0].lower():
             return True
         dut_os_type = self.dut.get_os_type()
@@ -66,7 +102,7 @@ class check_case_skip():
         else:
             return False
 
-    def check_target(self,target):
+    def check_target(self, target):
         if 'all' == target[0].lower():
             return True
         if self.dut.target in target:
@@ -82,7 +118,7 @@ class check_case_skip():
                 for key in check_function_dict.keys():
                     try:
                         if 'message' == key:
-                            continue 
+                            continue
                         check_function = getattr(self, 'check_%s' % key)
                     except:
                         print "can't check %s type" % key
@@ -91,10 +127,67 @@ class check_case_skip():
                     else:
                         skip_flage = False
                         break
-                if skip_flage: 
+
+                if skip_flage:
                     if 'message' in check_function_dict:
                         for i in check_function_dict['message']:
                             self.comments += '%s,' % rule[i]
                     return skip_flage
-                
+
         return skip_flage
+
+
+class check_case_support(check_case_skip):
+
+    def __init__(self, Dut):
+        self.dut = Dut
+        self.comments = ''
+
+    def case_support(self, case_name):
+        support_flag = True
+        for rule in support_case[1:]:
+            # check case name
+            if case_name == rule[0]:
+                for key in support_function_dict.keys():
+                    try:
+                        if 'message' == key:
+                            continue
+                        check_function = getattr(self, 'check_%s' % key)
+                    except:
+                        print "can't check %s type" % key
+                    if check_function(rule[support_function_dict[key]]):
+                        support_flag = True
+                    else:
+                        support_flag = False
+                        break
+
+                if support_flag is False:
+                    if 'message' in support_function_dict:
+                        for i in support_function_dict['message']:
+                            self.comments += '%s,' % rule[i]
+                    return support_flag
+
+        return support_flag
+
+
+class simple_dut(object):
+
+    def __init__(self, os='', target='', nic=''):
+        self.ports_info = [{}]
+        self.os = os
+        self.target = target
+        self.ports_info[0]['type'] = nic
+
+    def get_os_type(self):
+        return self.os
+
+if __name__ == "__main__":
+    dut = simple_dut(
+        os="linux", target='x86_64-native-linuxapp-gcc', nic='8086:1572')
+    check_case = parse_file()
+    check_case.set_filter_case()
+    check_case.set_support_case()
+    check_case_inst = check_case_skip(dut)
+    support_case_inst = check_case_support(dut)
+    print support_case_inst.case_support("l2pkt_detect")
+    print support_case_inst.comments
-- 
1.9.3

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

* [dts] [PATCH 2/3] Add dpdk support case list execl file
  2015-09-23  6:40 [dts] [PATCH 1/3] Support check testing environment for test case Yong Liu
@ 2015-09-23  6:40 ` Yong Liu
  2015-09-23  6:40 ` [dts] [PATCH 3/3] Framework: enable test case support check Yong Liu
  1 sibling, 0 replies; 3+ messages in thread
From: Yong Liu @ 2015-09-23  6:40 UTC (permalink / raw)
  To: dts

From: Marvin Liu <yong.liu@intel.com>

Tunnel packet type detect only support on FVL.
L2 packet type detect only support on FVL.
Ipv6 in ipv4 packet detect only support on 82599 and i350.

Signed-off-by: Marvin Liu <yong.liu@intel.com>

diff --git a/conf/dpdk_support_test_case.xls b/conf/dpdk_support_test_case.xls
new file mode 100644
index 0000000000000000000000000000000000000000..8f38c84ce8ff3338394944fa67c478d711ebe56a
GIT binary patch
literal 25600
zcmeHQeQ;FQbw6*lE3L$b5MRb%;34rLKwu$(48=x##Mn%Ppf)jNj7Kce6R1eLn$-%!
zj2YzC=^wS5I*{Vp&?au;W}3K3{n0v|x;38h*lkB{Gj15y4&yo<*ON4zw5E02c<gL{
zzkB!XzIXTS({7w(+R471_x9cM?){x}&pG$pkG)U&*55U~{rFQ0zbnSEO={%F*?Ot<
z&@J4zyXjei=h-Zi?)Qzj2dR4eKhi*b9EPlo$+3=C>;8Jel3E!2yAqK<j=qSi$Uosc
zETg=)f1bxqB}2lZ442_}w^+4efmS1zaBY(7DUr-p*E#C?4VCwKb-e{o{vGwWvgNfU
zn~_gY_E2QI%J-Y<+N`cKakb<*Rpw1a*&s{gD)PtWVY+)nj@DVbBq75xBu6EU>(494
zLisBQkUi_l5wmtE{*<MB8S1C3$J{A$Sh7)`r?N}x!jk$aazy01DY!&M%K=lA)Ob@U
zMw?~H=1@7JQ|7n|ZwSY@Dp8rP<Oq{=Td1V;uZ>gYxOs{kmUM^K_IN;TM{+)#$a#<j
zKM%L9NJB|EmPBum7%VHz%bQ!;T3XusR&PDr>Lw1aUJ_k~{^JJU^SkeG2JZ1^YL+@#
zT*$P(-H(Gbp;kfEW;2(8mdkpRtdS0pK8mu&b|uRt54S49;niDngywb4kh`ySdmp&0
zm(}>7sJGl2+goUj?JLnOZSqTgX1WG`eo8bC%kJ(1(=J&HdlZ<u<@0>q!iu%G<UIMZ
zAbvN0+8Jx!n!EO<nwLZu$`djV4-TZq9X!c>?M@7jIe0o({y-Y>bYBwB^kr}m^n2th
zsEu{EESD$kb<p8Gi2b&-fY^dhyR66AE`3nZR=Hd55Jv`7o>rv7%N$0Tx6#IVyq~qz
zx4vAqN#{7DO8@EUfx5L|yZjL`^!#k@N%RYT{paa_RD%B167;W^pkFLO|3(S=lO^cC
zTY~;z33@4cO5s1FaO$8I<%fAa{9XyVmZvzsmZzBh<q|kgm7qUcg8r8!=)X~d{%0lV
zmCExc3MXEqAMGdgF8%XOO^-}_WST@TsQe9i`rzOTOJ7(j(dQJs7BNrh`G1f>9j<?_
zE<tZAL0=O{p9V~Y|DQQ{(fHF#&}X>xyxp~*&vfbEmi@S0iqWW&AK_EuYdY9zI-;>l
zw{eSX2HlXS$<1%jXS?*gK6QJU<I=w^M{v8)#h>fa^YVX9$r*F$27W%?UTSKZKW#qj
z^_a?!PD9HNzS>^ej<p!$1=(x9o8Q34sKC`jLS$QBZwp+yksoAmu=%AA#H;(_bLBod
zT>jW8bH<<f&YAc+Ipd=<LqukvAF08e#bmCdnu!rgq__aHR8&9@fGglYK`P=9t+W75
zTM-A^yrKeil@_2;D&oMfQBeWLlZpy35L8rv5u&033>_5}U`(l~0E10vfgMJBU=*q;
z-5xmEsu5?9F6R#1&csa+hNdz==!T_17{|&0p=*`~VbCiBgzj1zWNvvNbm7t<43A|<
zLbomr!k9TF$Yf8rbr@jF013Aaqj4D^;numnJdkkfpd%|sl5p!RDi0*wI+(9|BtgR8
zm*}?B(HU+X%xB913AfIY@<76^gPyk>Ny4pTmj@DVon_^Lgj)wo08>{-ut*cu09(of
z3AfJj@<76^v!XnZaO<or4<y_=tI7ijw@zywgyW@4Qk(US!`OECAp6SSW((o*b4KU2
z8ae7z=f%7qoRMIVYuB#jLF#hAXV0F^x+@iN56#XU^&T3q)zJ_@I}<mBT7$e=C=Tq@
z;T>VRHGATTCyIpX?(QxUiZoLw(p+p;C@-ye6uTDLL&pW?IJvFT0_!R*u)fj)H|7h_
z+BBfgWr{1^o-aU4aCm1FFvl@iTFZqDa===?pMz_<m;cS{lqLw)6FqT9`GD8{bT$&h
z<4$KeAldG4kd5VlnEjLK2%7*>@SLLR6l=Z9SAe5%L+fUPxd}N6t+>1UWU$t=laCfP
zn4exNHm$fBLh12NU}$keMs~u=3spQW-r@tf<dpEI><=z^Kuq;)&8vsXFPgEsPMOV4
z=0V&b_w>_G7sV6O%y>c?u+`zl6ECee$lZ**UYj8vd7a!9Dk$>)?1J1a5USv+MVcyh
z)os24rf*wk;-4})oNg3|mYyw9?`GTfws*I~DVAh=UJ|++jN#X3ae*EA0)bv}J~A82
z{g`idw6Cz&**p6Con&Uix%gf=$Id->SkWcf8Cbk;mla~mI-EPCA0J1OlHm^xsEvgm
zvFf7E;MI5E%f@)G^eWjETrMG-CX-D^F`MQPHacD^HZf=L+STs_vzcwO=`3cmGK7te
zsESRkGx&q|-U?<j$7Hjyn9Z6HHag}iHg(S6JOB8v!EEN5Y&I3MSs%iNQ8{OuxHI_X
z5C0>W%{-G$S23H85H^hEIX3mq;78woBbd#6ve_8!X-mOoV;P<{+4IfGcY@hmXR-<K
zw3ZMyUQe6sdH0*o1+!UTvI+3CRUvG=o;KO@5AQr5%;tKNO@ODZ4PoQ;w8@^y%Wnp=
zS!l8e@U$C4*myl{vgf(Cz8B19k;x{&(>g=gcs*^h=ff+{2D7<=Y`Vfd%?6vUGCZxL
z^S7UQU9*9kb0<zdrY$zv1bEu=5H?;<>*##*!smn8EHT*xcv@=+8?UEzbiVZZ#b7o|
zO*R3Z))vCX>uDXGPkr%~U^ceNCcx9$L)dsdt)uhYt5<{BEHl{zc-qDgHeOHb==|V?
z-wI+g9XkjxmCpB@JB)sFakj2xUn#Qh{=WzYZ7G1lqfF3c2B_DI@axU~y}_W%3!v~H
z6Lf_E>h&3si+}#<V9*r>P<V?8y4nEsdWgux!0!ixt}KAUH%!oV2B_B`L_YcC%fX<l
z3ZU==6Lf<C>b1Vee_z-g1R9rCj^lPF2F}H|_$_ZO%GStEObrL}U1N;HHLO!`bi{iu
zX5=EtEbL{#ymMD#xPN>&kr_%Q&&BWe14ia&r-|M-)-U7Ov6aBkYe-TUf^mq=kpw|G
zl6Dzzp+hmK75r$0hnxmk$m~tUeW`k1>U^XT8(%Or+jZROKVk23hKJ9^&l+-Fmu-Mt
z$1xb}mlGHo?!w(LK3=J~h@l3B-T>-K#esfpUMih(rP3)^DxEqXiOSlEn(Vo_?H46h
z117tmk`a6oU?sx5Z4<S~+iK*k#a7fpUK@++?bvwYmRUCuMVXaGnP@hSGUM1ZmsVrn
zemR{lv3{ZsCDt1y>ax>|O89xrlN-6==GIfA!--@flS-eq4>+eX=i;~dp(2~Iv*cDe
zg`JbMeNx#!1wl?j=LaCpDfk2DHLmq(?3{Z=rG26v(tXl^tk2Gt-Ov}!m6Ce_G61mK
z0Xd5MG^j&hbRrL12Oph~2^Y2=Lw~F++?Gng|F`<Ft%cdB+y>N?(E_+A;);kmxL7kW
zy}(M~?M`A~D0$4r6vU{1%`6z~c8J9$MX`=SR6X@jo`rU}bk7+LM-Q)qWIDc{kIa-#
zFE~P~3EmU{uES~x@ND;2Uf|S#;2XW*2(2dg>;P~bUQ2-M5l2LUZ}Ng8#2Rs?W=;S&
z!>o3MI<H#T7`i6na0CM+5?3H4&8`+_^2446xd~1tdy=E$aMN+W-6M0Mlsz!dDB1}f
zvJjfMFymUDt2ZArG|j@kO88^XQ1XN`kc&;GgD*hL?8!N?cJ4xSs>!#Me4KP6BUfu`
zq5bKy1@cR;GoDE&5c-Tx!@2>TMlW2FHBG}Y8T9ltNX2?A*SPh)gf*RfV?-N*4Lbdb
zvji-A(Ol9hco8aRLJPO(Ida8kU~eF%=Dn%(NMhLTI3KCU&N=vcZz|)Qiy!raMjFrx
z37UbFf(pCc1XqFF_{$ZXLa(4xZv53LH~#9>`N%Z6S@0{wzVVE5fqM<6^8me1DW%v6
zy!{iyxz7nmC`h7Ht|U6;N}^MA=vyXYQ08*K2(cI}vtPw9ca?%VJRfP0ZS>s(Lz!V{
zYlFX3O(X1e0Pd23C8@9CHe?Z^TS25UbO~djq~>}BL4|8~h$=7hErw9<ZgkSMbyM`Y
zCfSazTn;AENd`jQ7j*jtjZ8zgau6KSXlQOM4C(_o;!qm?Z2XXX*0ZAXFlyI?&jElO
zRkghnBA-@wdr^0W*l{@G7HDxuPAU&7<Y1FJhAy9LKV5PJ`<6vMgIc}Ds0Pot^SI+=
zHjwf|E^WG$4pK5)r*tWukHr_rV;;G*uj}Q@FX2UkFL|D^+(U463!fLQI8Ko(n<Mg}
zyb|G$1pL()T}Xdo%(0V+5eHK3yA$!Rch4?3YbKFC=41fZpBfp#BG(w6?C9yi`t4vU
zojEx)JnZy2iDSb~TOmC*I+PyD6cPHyV3}bDybk`d-8(*V)Jel)!yTh1GJOM1#_7*2
zXVM-`^q+7twkFxB<nU>GY<zSS?2xc`-_?Wt5PSCb4JG^b>_54&FEgG@I>U2FJK1GW
zXN!!c?sd{i&bH)GBAFTLhqepFZC!k7rruIrWtZCN7puH-y;@E?ksPpxIybd@>)Vy9
zFErPC*KK#)noGivDp?)_=y0xEUaQfC#rR$03?=QM(Uav@6n$Mmnl_avTNc#iu2UGs
z<?4bTqEyl@8z776WL(bR2Pr_w`}lqU--|F3`6`^f0j7HtUOg>78>h%BbT&uu{QDo_
zBK`6CIC-a!`S?P=fFO1O4e5gC$<BYa<RWMnJv27Fhd_JCLn98me*H$zEZ-YF(;`rO
z?m$D3zrif*SJ2pJfFC=V2tr7sx^Kc&FvN|^hky9qYkQCG@9tCdHHyAgU4K<kCfKp%
zjui;)B5l_Fc=95)mkT$a%de^252))(<Q`PVyWQN%@7}lOQNGu`+H=2G-pXNroG&XX
zwS3#62F7I|`j>`Od;s5#cpHc{KNR5rpa&r|gAWJMAOG+MMe34iE<>5fkHdQ`=$&%$
z&89<<Ly=B({C&6kZOJ7}Yc3TOZGy<n@G^aT3K;N4<%=TASJIxM(!?KG%uQrE9pRI$
zh37S#7RL)|*wS+I3r5U5=r!yJkTHS@kh*WvRka!wjfq<LfO?ptQ38_&UPUx|kC_i0
ze-VxR&3xP&VbZ8DFYP1MGSqflJkcq70fp76i1L3oRf81e)X$<wk&8~nkfOXyu{>&@
zPZ5)H>t_)xCQ)LYB4*(kQK(Zsqm~+#ggWI#p{)uieE+S0!U0bs*Uc0P<!w~IhZBW<
zUC3Djlt$$l`k5DnBkM+dGzx3h$SJPEWcOV_iTY%T`efnAsSyJPU5@o^#DK+3c~S5K
zlcz>8A4*K2p!-zZsk=s*lTUe3YJDiRK9pL8!YIVr(sGR|9_f@9rOt;^=R>JeD2&T2
zM;kZtU8h{iixT&t#C<4ng~EMyugiSd3?qFJ!?@kP&o=zt_NT0@Z07~fcsz*s{-Cer
zM^HA3`ZVBmDflO$8D8S}#A1&^h>^v|;@8PBPay<3#-5r_g!K)&Ju9ZXV957>w_jk?
zWKW}`COy*1y3x}8#$sKCQPYibc^!zdiAruS5Cd(ZY@ze$*e9^S$F{<+<G1*#6@Hdh
z@R;=%`1Z#bm(;2$p<fAf<+nfcg=;_BcYM~@J{Om@tDboS16=tXoQ$Y^z8xp~i`#KB
z<{ZGu*nI>iN5rRavUhp_C!_eoIN8^|gF@Wf%CbDnd&Z-^L;dO0SZXk1-+HRw8P<{U
z@_jQ}-gw#Cj^`84I{)?3``@zI>s61c2C5pUYM`otss^eWsA{09fvN_o8mMZZs)4Eo
zsu~E_K-m00a^=-4m)0+ub^Z~||J#1_b<Y1^!pXTl=l`7db47GNPVDWFLpV8iKZ29<
z{o^>femaVi>i}F2;JU!$INfysJ%696%7a|;dkKJ<munCD2kXLZN8z?prXwT4xu`&)
z4yMv4#`p{Om?~dgFMv*}M^yt=4OBHy)j(APRSi@%P}M+H162)FHBi;SFPsKAkLPrm
z^J4xU&n-pVio|&~=lR^D#$BbHN^|bcc|3P{alaVn`rPBi-~2h>clUjA&d+s$1vt6Q
zn}6Gjf0~PbS&M%di&Jy{&j1_W%WyX1Y{9u4=L(!Faq@X9zE|U1gL5s;Hk|8lV%M>H
z{t@ypU-kIOHE;)B7U262Hr`g_pNHTZYLy)GC8l<mQ3Cpr9p_c#vbyHBvg!N$)%|}F
zBpd|yG+tUes^mD4&r|LV{sTasKV>Ml+z0$yCpcnqr+RgQuOPDSd>Lv`$wW%wolTv)
s_}U8oY(s26<?>%j7&aapG5G|z@g30->_4UjDwAjPPf-6=WSrywzkQe=82|tP

literal 0
HcmV?d00001

-- 
1.9.3

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

* [dts] [PATCH 3/3] Framework: enable test case support check
  2015-09-23  6:40 [dts] [PATCH 1/3] Support check testing environment for test case Yong Liu
  2015-09-23  6:40 ` [dts] [PATCH 2/3] Add dpdk support case list execl file Yong Liu
@ 2015-09-23  6:40 ` Yong Liu
  1 sibling, 0 replies; 3+ messages in thread
From: Yong Liu @ 2015-09-23  6:40 UTC (permalink / raw)
  To: dts

From: Marvin Liu <yong.liu@intel.com>

Before running each case, need to check whether this case should be skippped.
Then check whether testing enviornment support this case. Otherwise should be skipped.

Signed-off-by: Marvin Liu <yong.liu@intel.com>

diff --git a/framework/dts.py b/framework/dts.py
index d2ee015..30c39ab 100644
--- a/framework/dts.py
+++ b/framework/dts.py
@@ -291,8 +291,6 @@ def dts_run_target(crbInst, targets, test_suites, nic, scenario):
     """
     Run each target in execution targets.
     """
-    global skip_case_mode
-    skip_case_mode = check_case_skip(dut)
     if scenario != '':
         scene = VirtScene(dut, tester, scenario)
     else:
@@ -405,11 +403,13 @@ def run_all(config_file, pkgName, git, patch, skip_setup,
     global Patches
     global scenario
     global check_case_inst
+    global support_case_inst
     # save global variable
     Package = pkgName
     Patches = patch
     check_case = parse_file()
     check_case.set_filter_case()
+    check_case.set_support_case()
     # prepare the output folder
     if output_dir == '':
         output_dir = FOLDERS['Output']
@@ -487,6 +487,8 @@ def run_all(config_file, pkgName, git, patch, skip_setup,
         dts_crbs_init(crbInst, skip_setup, read_cache, project, base_dir, nics, virttype)
 
         check_case_inst = check_case_skip(dut)
+        support_case_inst = check_case_support(dut)
+
         # Run DUT prerequisites
         if dts_run_prerequisties(pkgName, patch) is False:
             dts_crbs_exit()
@@ -601,11 +603,18 @@ def execute_test_case(test_suite, test_case):
     result.test_case = test_case.__name__
     rst.write_title("Test Case: " + test_case.__name__)
     if check_case_inst.case_skip(test_case.__name__[len("test_"):]):
-       log_handler.info('Test Case %s Result SKIPED:' % test_case.__name__)
-       rst.write_result("N/A")
-       result.test_case_skip(skip_case_mode.comments)
-       save_all_results()
-       return
+        log_handler.info('Test Case %s Result SKIPED:' % test_case.__name__)
+        rst.write_result("N/A")
+        result.test_case_skip(check_case_inst.comments)
+        save_all_results()
+        return
+
+    if not support_case_inst.case_support(test_case.__name__[len("test_"):]):
+        log_handler.info('Test Case %s Result SKIPED:' % test_case.__name__)
+        rst.write_result("N/A")
+        result.test_case_skip(support_case_inst.comments)
+        save_all_results()
+        return
 
     if performance_only:
         rst.write_annex_title("Annex: " + test_case.__name__)
-- 
1.9.3

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

end of thread, other threads:[~2015-09-23  6:40 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-09-23  6:40 [dts] [PATCH 1/3] Support check testing environment for test case Yong Liu
2015-09-23  6:40 ` [dts] [PATCH 2/3] Add dpdk support case list execl file Yong Liu
2015-09-23  6:40 ` [dts] [PATCH 3/3] Framework: enable test case support check Yong Liu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).