From 5fc1ecc858cf8ffc8d3c03de052fe97492889d51 Mon Sep 17 00:00:00 2001 From: Vladimir Druzhinin <128374224+StateOf-Vlado@users.noreply.github.com> Date: Thu, 10 Aug 2023 08:58:50 +0200 Subject: [PATCH] test(Onboarding) Skip closing test app (#11837) - Closing all app instances by port for local run and by process name for CI - Threshold for emoji hash decreased in image comparison - Verification point for cropped user icon added --- qwe.sh | 0 test/ui-pytest/configs/__init__.py | 4 ++ test/ui-pytest/configs/_local.py.ci | 1 - test/ui-pytest/configs/_local.py.default | 1 - test/ui-pytest/conftest.py | 2 + test/ui-pytest/driver/aut.py | 7 +- test/ui-pytest/driver/server.py | 22 +++++-- .../jpeg_onboarding.png | Bin 0 -> 1814 bytes .../test_generate_new_keys/jpeg_profile.png | Bin 0 -> 1973 bytes .../mac/user_image_profile.png | Bin 2782 -> 0 bytes .../test_generate_new_keys/png_onboarding.png | Bin 0 -> 2612 bytes .../test_generate_new_keys/png_profile.png | Bin 0 -> 3218 bytes .../user_image_onboarding.png | Bin 2451 -> 0 bytes .../user_image_profile.png | Bin 2506 -> 0 bytes test/ui-pytest/scripts/utils/local_system.py | 60 ++++++++++++++++-- test/ui-pytest/tests/test_onboarding.py | 39 +++++++----- 16 files changed, 107 insertions(+), 29 deletions(-) create mode 100644 qwe.sh create mode 100644 test/ui-pytest/ext/vp/test_onboarding/test_generate_new_keys/jpeg_onboarding.png create mode 100644 test/ui-pytest/ext/vp/test_onboarding/test_generate_new_keys/jpeg_profile.png delete mode 100644 test/ui-pytest/ext/vp/test_onboarding/test_generate_new_keys/mac/user_image_profile.png create mode 100644 test/ui-pytest/ext/vp/test_onboarding/test_generate_new_keys/png_onboarding.png create mode 100644 test/ui-pytest/ext/vp/test_onboarding/test_generate_new_keys/png_profile.png delete mode 100644 test/ui-pytest/ext/vp/test_onboarding/test_generate_new_keys/user_image_onboarding.png delete mode 100644 test/ui-pytest/ext/vp/test_onboarding/test_generate_new_keys/user_image_profile.png diff --git a/qwe.sh b/qwe.sh new file mode 100644 index 000000000..e69de29bb diff --git a/test/ui-pytest/configs/__init__.py b/test/ui-pytest/configs/__init__.py index f68fc5504..df885a504 100644 --- a/test/ui-pytest/configs/__init__.py +++ b/test/ui-pytest/configs/__init__.py @@ -19,3 +19,7 @@ if APP_DIR is None: if system.IS_WIN and 'bin' not in APP_DIR: exit('Please use launcher from "bin" folder in "APP_DIR"') APP_DIR = SystemPath(APP_DIR) + +# Application will be stuck after the first test execution if set to False +# We need to investigate more time on it. +ATTACH_MODE = True diff --git a/test/ui-pytest/configs/_local.py.ci b/test/ui-pytest/configs/_local.py.ci index d20174297..af3374a68 100644 --- a/test/ui-pytest/configs/_local.py.ci +++ b/test/ui-pytest/configs/_local.py.ci @@ -1,5 +1,4 @@ import os LOCAL_RUN = False -ATTACH_MODE = False APP_DIR = os.getenv('APP_DIR') diff --git a/test/ui-pytest/configs/_local.py.default b/test/ui-pytest/configs/_local.py.default index 04950e900..13b3a3ec2 100644 --- a/test/ui-pytest/configs/_local.py.default +++ b/test/ui-pytest/configs/_local.py.default @@ -1,3 +1,2 @@ LOCAL_RUN = True -ATTACH_MODE = True APP_DIR = None diff --git a/test/ui-pytest/conftest.py b/test/ui-pytest/conftest.py index 17bb52680..14fa7af63 100644 --- a/test/ui-pytest/conftest.py +++ b/test/ui-pytest/conftest.py @@ -6,6 +6,7 @@ import pytest from PIL import ImageGrab import configs +import driver from driver.aut import AUT from scripts.utils.system_path import SystemPath from tests.fixtures.path import generate_test_info @@ -58,6 +59,7 @@ def pytest_exception_interact(node): name='Screenshot on fail', body=screenshot.read_bytes(), attachment_type=allure.attachment_type.PNG) + driver.context.detach() AUT().stop() except Exception as ex: _logger.debug(ex) diff --git a/test/ui-pytest/driver/aut.py b/test/ui-pytest/driver/aut.py index 96c19d329..521d58b67 100644 --- a/test/ui-pytest/driver/aut.py +++ b/test/ui-pytest/driver/aut.py @@ -49,8 +49,11 @@ class AUT: return self @allure.step('Close application by process name') - def stop(self, verify: bool = True): - local_system.kill_process_by_name(self.process_name, verify=verify) + def stop(self): + if configs.LOCAL_RUN: + local_system.kill_process_by_port(self.port) + else: + local_system.kill_process_by_name(self.process_name) @allure.step("Start application") def launch(self, *args) -> 'AUT': diff --git a/test/ui-pytest/driver/server.py b/test/ui-pytest/driver/server.py index 9d8dbc9da..a5eb84cde 100644 --- a/test/ui-pytest/driver/server.py +++ b/test/ui-pytest/driver/server.py @@ -1,3 +1,5 @@ +import logging +import time import typing import configs.testpath @@ -5,6 +7,8 @@ from scripts.utils import local_system _PROCESS_NAME = '_squishserver' +_logger = logging.getLogger(__name__) + class SquishServer: @@ -22,19 +26,27 @@ class SquishServer: cmd = [ f'"{self.path}"', '--configfile', str(self.config), - '--verbose', f'--host={self.host}', f'--port={self.port}', ] local_system.execute(cmd) try: local_system.wait_for_started(_PROCESS_NAME) - except AssertionError: + except AssertionError as err: + _logger.info(err) local_system.execute(cmd, check=True) - @classmethod - def stop(cls, attempt: int = 2): - local_system.kill_process_by_name(_PROCESS_NAME, verify=False) + def stop(self, attempt: int = 2): + local_system.run([self.path, '--stop']) + try: + local_system.wait_for_close(_PROCESS_NAME, 2) + except AssertionError as err: + _logger.debug(err) + if attempt: + time.sleep(1) + self.stop(attempt-1) + else: + raise err # https://doc-snapshots.qt.io/squish/cli-squishserver.html def configuring(self, action: str, options: typing.Union[int, str, list]): diff --git a/test/ui-pytest/ext/vp/test_onboarding/test_generate_new_keys/jpeg_onboarding.png b/test/ui-pytest/ext/vp/test_onboarding/test_generate_new_keys/jpeg_onboarding.png new file mode 100644 index 0000000000000000000000000000000000000000..156de157d14e20ec6319805053530c5059fa6f59 GIT binary patch literal 1814 zcmeAS@N?(olHy`uVBq!ia0vp^dLYcf1SD_V-jl$<`X= zpW~?YovWNB>`PeL^p}zM8yvU&J-hyGX1;~&gLB;X*q3w9O1Pl$(Z7b@McHuDkG5rR z4}JT8eC|1xGWVFM`R+`;Ygz8HBs;xdt88o-_CU=f)y89C zO?gd*3%`d7x#!-^;7yVb`qk>17yB}2*4e`=L^e)xNY-{VKRDyUZ>`S7+w!)sZRFSd zdq`>fx)QUq&k{qrB0LvtoVs9kg3pWEsYe2%&7+!cv`?u^oERN$H9PIIftKqUl?yk# zF8CU#ec9*rXvJFTwT^F`z4mW7u~vJo@8(L0sY+o&89Ayi)Fgzf;#CW;T& zyEu=v+za=V*;=~Tw4>JJrMTG6(xsa-&G;7w9q~d0!xI5h@w1)Sd=YD}Y_jnC)sJ8W z$Ko);sqAmh1)vSbov*~0s7qB(M0dh%USlWWU#|{E*6iP#vE^9Zs@c;QZAgv0Wis~{ zB%sR<0}a?$RnYNnduj2ajnbjpbzBoqZ`zV+^|d)@htiFg-(K7QeG$ID?)BOE^;da+ z)XVm_m*21bu45Pdfw%JgpIjSXg$ThTlUW|y?5}zqmi1O${(st^R}(c_MIFIX{)&cr zpED(9RX;1A|3~bzlGB0I{(S;L4`j6%RYjPJddNc{%45 z+u8!xed`M*Kal#U!nI5C*}c1`eydnzzxw9V93g1VsV8VIa7h;wozs`!yweYi?@GtS z#jY358a&H;S}QbvYw2pyZ%R)tf)ickEZx6Isq*qEV2p1sU2M`ZSL5ywPL)G0bI!Cb zH{1BxLQ4@_?!Bmg;`0_8h=PNnSaRpzsFa!dpbIVcifY_gJv&fjC9r&W zW7FqV;5siHSb&`8I2`3pK`L614} z_C2gE?RjTk^J6Jf$6Yst)922eKmBEf@8v~>zIOk9D6g5KvBu*Oo6zdagZ%MTAMfp* z|M%_Y^V14NSd&zl96so%Fmj~Kc_#n&$=2(2e{Mam|KC1AAZFtC`~OwLxSFz$bX71p zCd`^PJ%6u|{3HMR|1WQpE@C*C%yiJJ<9{!Mhvc%C-|kwkzrXkMb^E`P##0Z89;_F< z<5KhZ+uZB+e_kBke(%rG#qPFd!ZFHvb7lG8@A-bt&~DO?S$h@~xOSvRyU6MC95(;c z{7ORpR`)YYUd4TG4ua>mX?)Fy`*dg8`zbdgclkUnR_Hjca$$$kk@AUoi5Y%-zaKxm zaA$^}NuI!?n;b92#CGbQKmBG#kre9@uYJou&Umn!b7{Q9tl;;5KNeI}TC5AKmFOrdYO#7hS2mUmA%JVW|tU)ir7mQS+CqO1*Wq^W0Y{PLN-&z6n`OW(l-lB cUGr7GmS^ji#gpX4f%P4Or>mdKI;Vst0C(AOV*mgE literal 0 HcmV?d00001 diff --git a/test/ui-pytest/ext/vp/test_onboarding/test_generate_new_keys/jpeg_profile.png b/test/ui-pytest/ext/vp/test_onboarding/test_generate_new_keys/jpeg_profile.png new file mode 100644 index 0000000000000000000000000000000000000000..6b8b411ffca8eede1c711e4c03f3aa5dfc309038 GIT binary patch literal 1973 zcmeAS@N?(olHy`uVBq!ia0vp^Mj*_=1SBWM%0FXZV6X9XaSVyzoO>{Swd&2-;}6fj z-@Ym=^!cIf3p+Xt?;Vd8T(A1x;pEgO%fhd}To<=C-yuZc4|DOWlC$eB?_O196vx^u zz|!0_y(@u(L-K>{oVhpCYi(^WUtG8G?VC@PI!)|lnuk_>xOnz#;jC#tOCO)O;aC>_ zMd^EK@bcr~{Y~A6rn!DNpqKtIRHE&ph3vD`Tp3C06Ysj>_Ef}OFZGI*mf?yPJIK6y zPHT9g#IeUEGp27XOnIZQTf6GxqsrLpnx(k~8aa_H{BMfH;#w0W`Y$KOT{||h(DJ08 ztL1mYV^emSS%_={Il_{A{UL*%<1Y>Lvmc+>F(+mDqB*}Ml5toN}Kc0 z(@GTy(>H(6;yGL~hwt+yv%a3wo@H96cJ>=CyR!A*!VRf{HXBYyR*1GG+W56sMoPOM zd#YkB`t;6miQrea9t7Oj#36U%sm>m*!v_0i@tww^-?Zg`-uFYR3v7F~Z*Q-RyL~`! z(%i11cga1_;Nex=;H?=Ux-S3N?c0ZK*5@XOqXf^IHh~*UMb>a#H-G&0t;uzzyJ5k`w|^@=h^Me znpIKywdecB`#;~_=W09Pkf2~7FheqaLJE);G5Bu(<6QZ_f9&=@9@_uePiXZfk-z#B&Grz-V3+0$x{ z?oB+{K0EQ(j}KLUO{|3Y&-~y53Lmppsnz@SXHDnX$yZ#>1){^6t)~m$;a;D!Lo)8} zzNlyZi#B&3@jv?c&hZF)&+y*Q`y*QfZaj_oaB6)#-|@q5H`QDMCWqxouy9S+pE~WaWgQZE%k9@XzERlGzVCgK@NV_mqt!Ro|9`c9hjqLF z6VKyjuA@$WAMyWxk-z`@-u=J-eSdvQ<#FT3X)H5l9;xg?|Sf299^(#*fN_y5TjWl{s@nYxGB{Wccy0!H@??iuuydoZya;`lec{%`naoBB2kKmMPxr;y*i_4)bvp@wm`JG&m#Om*dGESFTUdl0i>*RL086{XeI=d)j@ zsns20_{iv5rvCKKbwfMPyUOOyz=Y3o+|xjSFXLA@+h6Bhkw#t zV3`M1IrGLQHjX`~e^_=Yv^;$FbaiIbG*c_1f3rUpd!Uq@yb%f0s}1-MpT$wKg5@^| n5H4A9%Z;?gqzRj_udtF$I~uLnEx=^rlEvVYe; zn~zcIEVz7gJg5F$7^(~25vW8NcsLf|u{el+)h~WGTA9<7v+F*3YKpC!Pwn-SNxEF( zHlgYnDWD0)r=~(#>Y*gv(u^Y;qig!dD1KrTqj-N4LS1i>W8mX=OFOpk4O&&Wet}k>E zq%rIPJ^}Um7O&k+fKL(U`;t-#5-$UE?6jR+T?Mb<1ALblsS=l}^5=x-@aUDn2e~5h z@+bB=zq(-uBCte+a0z%Y+wd=C()u9Nw<6%rb=wIE0u3x;V#+WF7K3vax zAE}Av57|}fS{pIWU$X&amUcEfbWb?ty}U6GBT$U+RL^a7na>YH=U2Z^dJCL)m5QSc_ajg!UiKSK$#-h7<9V6aYP>UbIA_~8UPZGCC zt15CVm6t$1fV(FH+oCiQqYBuxrbM=$o;Nkz=gx*C1%TnOFHozklW8H8xU6S|@X-uHBNzu6G>fe$Ad;x-}jwDT)TLMC!rd@}yMQ zxyDabyu7fl%LGuSvA=%+M*==?X4!T!c82BOGM>=1F<`CjD_E<^8U8j3fQpIwS&b3B z_K~-7pWDC84o6^za*8|UW{Xl^9peOWS<{*dL)o1#%EMldOfP9VAFR7bg9E#wh{X3- zD_@qss;FBMQXMS?{w+p$lTJcloCZZozq<`22m;LZjp9J zb@-Zba0bf|3Bb_YsGd0!59`CYEN5iG=%!z88gpU8r4%`JLZo@jKtWGM&e1Xbx=_p5 zk!>k`H2>3i|IJGicJ9d!5f#W`09-^^MVjpoT6p+?eYFD~hZ=JK^}7u&fkW5ENX8e? ziRT)7(|8|iHfTmLh7h}5OB2w+RLXS;u0J>IBRobKk!#o&cxH9-hW^Gsf~}uD-WoDL z%zX{FswtT^!V7y`1M@IYDJjvJo1GCfk;@j}QmNlh0)8Cs%Uice{Fw@u+{lzE%?$D04uzT9i z?!L3|S-aGwdUN&Y>6Hn!;X4A?$mQo?QErcA?a18WrI#h+F5Ab4pV&?_EiMiY4!L1L zw!XXI!B2k}%C^p3?+c6w_gv_#sMy@3-#ZZTDZcjh3{x!wUS?CgxVABjE1f=pC5iVq z9hc-J@5q2T=tCc0f zlr7E>CzJ`#|FO+)`a9hsCJDnTtO@u>RNb1T@_RkiwO}Z>x_>{s-*smIRa3T6zHbe) zfBg}^*{)oNzExi#-R&z2f?JVgp+P}) zG#FqX*JM=pp0LpJ_li2S!__v!K_K(RS!$p|n`oSzG^t7(*%==DP-diB_rRZLqw6HR z4(?IPQjXPTK3MSewW=Aeuiui24FY&HXR4Hxn}o#OZT$45y=wQFRV}ktRnY2_Gwbe& z+UWqO6@>4H2iJaE?A|?G>+IX9K({b8p-{mZ;2s?fas!5aYI*X>$YxSt6*dHdApg`3 zYZHl)idPMj)=X(D=2mQl8%S$I;M~=|#`8RPNhxR(lzP_VT;Y8mjB*NthQl2HBfbfB zUndQ6@QPgwO-sSf7xcxz?$`V_1cNaM=YeI=GzaYI!q@-cWu|xtBd3#lHLg4a*Q#+97k3#=E+2}65p9!O zE90m{IGNOHCzFhbM%}lRj;B9kk}d^HRo6fANfU;F!$W#vJT3nsJ*J@C^C$n#>Q&kwBpvHV_fev`Z^N%bUnr4z zt?d{n5~G0#QZQ=|QM?a98Nee-C zjCl-9dxyHJ2kRiGrJw>RA-Kms7g$^hKq;WL%)Kr**l2;tkk-OoEvY6#{TQfT@riK! z@pF>2hKuK}sz`WWv^AiEoTLulEhldCPf1MyF)hr<7pS)43hi`RjUjX#7#Q=^DaaZK z#@SQApzgx|8vvwaF338V;pAc)b#i;OEZ-5|OlX1;FogYH+yujGuNg4porOX&MDh~2 sAi;t{0ha&%>J?CEnv_}>b^m|@W+D7TpZ0cZ1T2LJ#7 diff --git a/test/ui-pytest/ext/vp/test_onboarding/test_generate_new_keys/png_onboarding.png b/test/ui-pytest/ext/vp/test_onboarding/test_generate_new_keys/png_onboarding.png new file mode 100644 index 0000000000000000000000000000000000000000..6f75463cdcc5feecd307b5caa0715069679c92b4 GIT binary patch literal 2612 zcmbtWcTm&G9uF}H1d*7RAnhcC4hj-l=mA4S6bvo$0HrCt7|9$U)d&(Ipx_{nPJ)Vn z5jYep1f+NtN+^OTL_kbXIHZ?A@?yD}$NlkU-o4-KZ+B*AXZO?gJDY)bIxGd22Lk{A zsU!AS7vZS;?H30M`{8#_eE|Ro-Vv;gn}+$d+PNLY?xz#vqQ^K?w1?c8g4>(y9 z$Ov{=tj|TcQKPw-6Eq+x@cjJ8aIsaW?``v|Bdszuq8|FWM9J{Bg9Yc8D`)-S=6818 zHim21eTG;@sfqpmtE{mS+3XjT&(l`-;ASQp``k-2PfIiyGloJr-FV43{uZ3p+~U2eVr(U7I}-h)VW?3C&Dk)uk1L8E8SCjy`6fH3-X zx8l4WfaANKq<2!`I2`K&koHe+&)scxZn_&gZHs+VZToEvVG17TG}+&+07f~=gK7&tYM7-G7s&E*kSL7Yh519p^T;lq8$H}h!2<(OWn z${~&3Eax3zg%9f8hr`A!4GDJ+qNuxJSi?Nu4$~9bdlPpkI~}y8g(%BYrS=yyQ3KYR z+GrWsQVL6}-}#K8+p<=m4u!a4|MmR|!IK^v!cD6UYQe1y(3Srwz9}Th}?X zb&*bngvmv(FnKEoO1!nF8ait%AnfE8F)zi zj4=rX)wLeDmKDbp4{ciK zq#(&vZiJPU6)Jnb+J5)E6uKncQAG1Ga-KrZLW=^ypw^dTMNwmGZ=@u}%K@x3dR7YE zy!p4Yzi3?LViN3GI14&5bfftyf1-!I^(cN(W9vE{V3VSn?b=b>c|zvyB-)*X%#hX^ zp&WJ_*c?lnHlJ(e^_wL3v4gO1RhtAucRR7%SPD9uYs(rtm|%6$_PL9YVobU*rDAXo;S2hQUz^uG7@B+`DW@>5>_Gr}q|QPYun0rIEf z+ozxan=~Le#jqNz*`iWSzCw5V5U3_zqB9n26iGv03UcKt#RgB*-Na41_dxNaa&ig;Oit*99>s_N!2w$ISPCHBYF!?EAQ0PD6()b4{V-;3Y?8S$EL_}EY#Kes{TykhOuiIdAJTdn>kq$oLvm+Hn|>Vh{v%R!vW5BC4vn7*P>r zW zaoLAIwy}{WIXxmB>e^!!U&k!7C?dsYn@5GKK{JHkmmzRNLqnCDcK$2jrp*TzH?1R^ zo9pZ62j7~ig027#P+nOVjl)a=#uzC9S~>gvkl@m`$nGkrc<62V^8P*?v{ z+a+k7dMn(`s!ULlld~69fD4GOI3);;)Qz;cr{$Cm2HIkQJ&S}Pf==tX75_$0#x1=Y zVfRM{J`V8tHmCwpH!&vY-Rjo-{QR@!^`rSgJ=r%tEktu%R16AU2UgTO*ZOo#P0HYC zDbPyWpE}lrh24V3O1POBZivN92$IAZuo)PEHefAX) z{ys2Yt)i~&z@((5%{dRBG|ck&YiqIN8=E&@H3S`YiC%vcU6oUCEUl|scnejI4!GsegL{vJzDAxmQ~2BCkN#2^$E z7n=<;jVvr?iYNc*{uP-ctDvCcUBq1K3lAl!mc1%Dn$>+8Rg~3O@gC&9 zxDlQG#Qa?b);~P_&e;fl-_eFDR!)Mh&`Y!b=H*{mo)7!s<(9M`@?ln!=vg!2YY=e6 L&Ix2Ra>XnW_QG5SS1yuVhF=LKO|g8Bq>iRXK0wwJe`IjPiK`RspHT= z#b}4rwtrsix0SfdyE~kI z2kUa1ii-nRGMi8S2AH$Y9NKNoz(3sP{}bLs=jSpN8ACbqH?&*D?H#dARMZ2$@=Z(| zUeu&_rP5cSySqhNYV?Iuw(m!HPoVRYAbbAvqbWLp&wjqHO;M8O)lt2$AS<>~X6D5W z>8D~VNFkG=5Dj}YG+uq8;9lM>m#Xm@^3$Knqy|(_Q2!sS)AgcGP0vQn+m%2t7#Nl$ zS<08Mi2~JHfiEW-Vz+MI9JWia*j?{d-jIxaY6T|v$Rtgo?u1qPnvb&n@zq05nh7IPc_rk>fdVxz8Rq15q*08U0GE|xOOIRm}}-^VD6qah|}zP zaR5$)uVHtYzVdZAwz-bdYnZ3;xX=pRxMg{5*yhcC*Zb;7A!n*e9abdAm~J-@kQ*e` z{>7FO_A*%zDwah2+}d|Jv2~yJa~vybvbQ_w>4h5DCq63XWTu5&AW)j zqdvj%Pz7GJp2_b~&#%=w|JX1~;}IzHUaCa+hmkcDO{vzC*XKL=_2fBw5>g5TPx`N> zN6o%k#hb{vJ=PCcbIp;pxY;3NHw`!(j|V^?hzZ=3jOL5V7dc~d`TmP;#aT!hiE&kT zSrXCo;13Ek+>_fsy@@Hj9u@j2mm@BY);O$6r2=3u6dDi2W0p!5HW;~`s7jk@KHtzS z6bj1E@%E`YXMtB7#|}lc5?3I`u%-2#*5S_Fs9R_+)-s;i-oq}0!$~teOS5#Y2_6GN zpa_mOAdOS4#59(uMNDVEl~soC-{I`|Clg%8TN+Hac4RG0%gK6^2P^eI??M6~>*E}^ zML32Q1Hnqf(EuN#FBtKZx0s&wOR^Lg^$EdsaMZ9ICwe|8IE^`>yvEWhS(*rLU0A#+ zx~=Chuor3tPETM`NzWsF)%_xMd5j96K-_6B zI~Z|-uJQ>X16_weW@K8Nu-Oc-L69ohZ&ArtZKS4MHU%zD3%SGnH1*&jXxm}*a+24o zrqR5l(`al`%02YsRb7KL$w$9_=RLI(^n^mB{tnYxcy-U$Wo0XGY)C3+t8H~CY<%;S zWh50yAChI8(U!`k9W}d1wO)$_h9D>_VH(=Jof|!pJq^r6J7*3mv*tN0$~ZkonfKw@ z6_gA-f7?O)K?2-J6&q;k1Xi5Ow*liPRIn2mC5yp~?350sR;(~V#^`gpH&1WmiVN2S znXl?YpN0+k=3$xVmyC=+@Uh;sYuER-n5Ee3^ z2F5#bw^QOjU(wG?!~#lEv~-w^ijw{3N|d@7tqBgU65%Y$*uD z5e~`ZHu*N{{K%HK?s-K-z9Fr@z|DZR?t4G>1cbgoo!*ReQh6}2u%LBAD~ES6!wG#H zg4lt-CSy=61>Hjhd|y~+s30G~kycjy&Z7$*`Npi88uDOsa$H=TZg1q!qFBT^D0n_z z&f?dcJ#>gO*;Ab#<=5{P!V-rp#kTJKybFZfzI}<$AK@LnBzNfK#q2>qjb;Uk^A?Zg zix)blZWx^#%ZupM3mbXf(vnwnV_~qRrKQ-ps5fSwr$re^j}`Uw)s(f_I%m20`?D73 z;yiUC?vi#+Cm2NV>na~0A+kK?M8H57OPrf}X`xh6_pGOuR=8dyHWR=O9KMKAMUc2p zbQIcOhR7X@99YAdS5>s<*vX}uhfpX{!lFKfFYj1Y9`*J06#f)zpRTTmQN()3<9XeE z3-0UVaw)=LHwvfCuflg|NX_iLxI%8+)>e1KtY^5ND6!bg5es+*)L>JrJg6xX&SvHRE)cqolf3)%V*P`H&hoooONp7DWu$ReI3z+x$IZ=6 z+{p0@8}jv4YckW)36j(=qZewfgCsWXji(W~h}{H6OKM?ZlAK0Pa&mGg^WeUH`;5=? z=EXq_eH@M(#2||nA_GH1KM7BZrKGxotB#dN--w9t4I9pROYEECcSI%KPgQH0vj__$ z-gmDm1I17~sbBqWYOB9liJC@=rVx}0;PU)bEo83W`(X5~Zo zT}L~qcpG#If`A1#2B?%AxR%5YxZ?ziZ}c!QGz^Pr_#ld9cx*wVgBc$6F)?-f59k?0 zl%7A1$!&i(evTvHpA9and!Wx3G5OcM2 zr7jvKaQD+iMgC6Ywt5>XG?Sk z{KNE?x@J<%%;x-j1Bu1oO-(0dtlh6*k4LdfJm%iJ4m*x#zScs6cd4sDNV)q~zo)Zx zIW41Q8#iqDJ(THDIug~(yVRaTcdLgP=e7wsY2Ijs5?w{xRNKc7&TWL=ZZ42FA*ceC zhh0K+zL~*~{0q+eV+_lqRC;8Ug7P;C!=Vh1_Eid6@(qVyPtU3V;L5O_bV%1{?21!-aiLdy?c!_8dB|1KiV<#>dl*p_Y<$hVjhpT zqPKb~xS_H}+X!*8;s|RGiFoq%%Ya?GcmJd9-2u#HNJT?sWTAVS z=QqNINx!}e*tEVd$7U-Oe0uWZ$LDjxTcc5JZEb(VLTn7P^mt)@eDRIsUjwid$+i-z)+mZ&_z~xwC874N}NHl-6$-I zCBdG+*J7pl0KkYzsB4T$UL`7RD4dKz+MIJtdu};pJy{%IguAarez16tm7FQjzNp~2%jeeI6eDc(42 z`lV(tX10O{P3ouz3dU&6oUN4+bew7ud4J@p;_EqweFQl)ca*xJncO!0u|y6S3`tk? zCV()Nr~u;38Wm8pO@5vA{f z_t;von@1luHzv5E$9npmGnEuhI2S-2c*C)o${lkK?g|)f&=|ncIi`MjXE~>MEX^wQ z%9A5vSQiBvC9X+_BQ-bOek_UFWhUgPL68ko6T_x`o^PSYKCgrSfN*XooCzTXZd;x1 zIEfxT9sI1M$7^M7DuY%-PCq=^Jxxi0(~qaXtZ_9y6AHFn^Clm$T<&)~Mw__iqbUib zx3saO4^(2*J%}sbV`qaZhv!28C}Cj!oW=emGe0}q+T%ghQ7Y#7>;jRj;fGRSObKk% zV=nvu1)|Z_Rapqt7!Hl5JLfO?uFXD{F!U9XaqqK;ElVMlRkzw`2Br`;0--7e*xmi` z%9_?mjtR6QZAm2fXn9>-UA@1PuVSYNjnZ?yCEd}+L0LGs8f9Jro4%wq-Ml#SSg7rN?uhr+>D}FO-Wl3mA6@|2ILL}x?BkFXg-z|$ zAxs(bo($={ZXt$8ocCck3LE@7K>50o-`tkzsBIoQntofzi#X;PEVRH}Q{uqvy5S9~UY3^eF*$Txugm`mx5}5C#IVspCHVh4Ub*ceAAg-` z`QQSvq%=<@>8umex$-D%<49KNOsf^*!OL>0+e*tDH-o!65<-?LbXB9o)ZM7Z{V#w& zk?W102lIX_I~(S{~7 zV(!mwtHlbV^4owVQn(8{VN$4g1VI z6YEsGC>W&=I5_)4QcC)|LH~IkcL%LeVQV9&HN=e{G&3b-q0vkC-goKsm1w7FaiTbN_a{+IL|y3H z*ge(E+iZwqO@@rwAkO-gF2)kkxnXy$LL$I>Zs-M&?YGkApsbaw7!~SW6{AF=)7no5 zFrg`mhX1aS5M?#`-AUh-17;g@myP^4%HLs?wkf>hA{Q{vePbP zefU?2j>c<)HkpncWkkvL?{6g}itSf8k?*c}I zhP_66GdC%R0=@+kcQ+R^cg7nYf$wqtLw%*xw!?CO#4qV_L*!S?A(;$w6XfN*i@SRI zCBHUka6r0!=}6{Av*#yo+x3q#aljUjSElXnN09~wI=#$M@2at<-M?DIT%AMJ{-r2e z5Y$Pgj7oZ5t!rS=d!9)v{o0;sestl#y!?;7KEnPBo9gd;v88yYCs+Ntl%7tKL~INF zq-5d|QMMN(Q&lJ)f#7uRH+wpqt_^{-)YW`OhD!X$@4RBy(Iqb;3@ifMH@2E{W~ArqPiRF1&iN zC@hfQ<2Wv99E^>1*Nj6#$)=j}eKX=$@L_J3GOF*^Lwc>upUoPu>>! z^*8clh&bwO!UKGo-nh$IwCW#u5G{GYP^7+I1R7F2b1E+u0&H{BOL8&y98c&oYVcb) zDQl^0r5grJKZq>^tRhjm=sfW&uw1Y_@T*(5R^0eOqc&B3AmL4Ri(YgofEyKp78a}F zds!C&*xZV$n2 yZu~&*P*I4M5l$om4}}Hcr+s2CwR6KL3^t%(jd*e~<0bR6#ezX$)k=}(5B>&9sl$Q* diff --git a/test/ui-pytest/ext/vp/test_onboarding/test_generate_new_keys/user_image_profile.png b/test/ui-pytest/ext/vp/test_onboarding/test_generate_new_keys/user_image_profile.png deleted file mode 100644 index 7be1907cfd95820905b6c1b4b7e60bc97f4b5dc5..0000000000000000000000000000000000000000 GIT binary patch literal 0 HcmV?d00001 literal 2506 zcmZuzc|4Ts9=8-@NsOY9GT!XRzLhY!6Elq6Sdv86!&p-%ODM|NXQ+(nL^|$aCN!2| zsL`>+$i8G4OqM8=BW2gzH|Kory`Rs$f4uMe$Majhzwh_^JkN`Bu(LWOEF;Xr!*d9Q zL^y$8-5(Ex4}9ajPR{f22;otP^Dgc}b${4oUcTq?dX>AzSa?LFrm{{;4GWeugjB?mCHEA5FJ4E4|v0?M09Br__d+kMgA1PIP4Ypw!Y7ZV< zL_O5<9@$!3!0O=-umWIzM<F>AFkP=a{9pWJvBZf!0yQ?dS+~&f$^>oOu zrg2~qmH?o$UxYO;?np`8>hA3=6|Z+Y8oN1tuOl+w&qR+GmC+8<>xi7o?XLC0&^jWI ze6#0l-2<6WzQeXVeC|(pfOT~b01M?OG~ltXBppdwyR#-Mmpb3khVC3TgCNOjVu%!F zxe_5kz?}Oj{L`fgCb%?K6L`h~6NyATuL|`UF)G?2oowEgG3Sl_!c_jRg*^NCy>HxG zwllOLdi26(il4@_*xitvm)lR>fH0Gaii%a#{hyv~&zf)U-I)3%uZYKVwM{bK$zOyb z&mF&NBCR<-e;w@A97sYTcxXhB*pA0!-=cJ;Uf;R3skoJ3Il7~)Pg56K%D1RZo_IJ- z{0fOb_Uix#B!$KcilyV>an>+fakRYPk$K|x!BQKI`H9YeoVFc6iAE>(8_c@P^%FS> zE>KBS5>|;&kmuvo zNkXA)a9l_YS$`U$aJcnVh0Y^S&W9bdZu#RGt*{nJxD#g5NE8Yk1%vW(B)@2jPRO*< zzhyC2T9@%uV>AJrjJ(M3$<1xBV|$V$riH8^H;)N~Wlc|PTXHsh);rzfld$kSa%1>0 z0H_>Vf2(u~$4{d>cD8CqgorM@HQHv_`;l}N%m%edDCDKzKULVW3t9647V>wKJ+B(q zGX^#iVC;OMRs7vCw8hL>Y&dpY02mVhBA@2o^LZB_I&+6X{eMAk+YF6<;FWXFMDVDU zkfB!M3T(9NsNptaL7bfggas7z0Kk+{N#e~&3R&OBmn55dPLyBpH6F?MGnErNFH5l< z+T9JjwO&!r31Y7=j!^z5d)+`7#PcQc2Uvay>anGn_8TjWJ2f5ICw7Y5F3VFyZL7m$ zuuoUw%n$@DoS=;UEGr;{OHiLA(CQ7hjDIlBOQNzSUw#Saem}V}5#6UXwI*V|IeqwH zqg)4`xi}7z7@r1!c-eBt0`$OKgA$M;GrJ%B6+t3Clkz1Ac{cCvQ>K~Cn zUIHaVHh~kZM>C-QgkvsG#hnJ$>+s`K46tR3&zkEL7jnO8XbjR^&;dBsCD#FRFMTJ6g^W)^_C4~V7gv*c?}?0ej}^!}l@ zSn$RoM4!nRsRLa>_iqhVQlR!(f@b(}nEfZCkk-@P?yFOz)mS5bn$ zJK%2mOJ84aHVwr2j9lPybYH!R>S)T6vi;*sNLbIm4f5CYr@DPko4uP@+-Bw}E8h0h z>ka^I`mw0KxuzB(?k*fbB%HsFQc$JH-c{(9CtlE;d!Cz&)Dq322rUL>R*4Gu^d_4l1zrEK;rV9bOhFXHeaA4dQ{e#r?jO; zvjP$l>Q;7I&_FWJO7iOmLW<>WkB2rMyOy*h3kzSNiSM-z?}RfiNaa7 zL3C7^SYf8?NUDV^LfyaL(JQaqp&%mNAMN34?ctZwl^?U%Mr4h>52Bqu?r`uTE@)13 zI5qpiZ=FiIM`cL;QF#t0^p97NMN*%oxpoylZB%l8qCI`&O+odu{OV4~ebuF5oF292 z%|NJ)Zn1DJbfD9#OyopgJ|$Ss`3be>g9X7PNcSQ}TG&IPBLBQqjIBU=%HQ2}&MHE0 zYW#XOQtA5jGlf``fJ?F~OolEJ^JJD7Wq=J()2AKxLPiPNT2dA*6x2)Z>wYM!WWw>{ zcGOJOH`c0_fe2Vmsk3r#Td$^7%}>lEwSH@a;*BSo*UmXt{<8<2`r7cR2b1Y77e~qcnPN(4$sNDzW_JH){OuF diff --git a/test/ui-pytest/scripts/utils/local_system.py b/test/ui-pytest/scripts/utils/local_system.py index ff0f40260..3b50efa56 100644 --- a/test/ui-pytest/scripts/utils/local_system.py +++ b/test/ui-pytest/scripts/utils/local_system.py @@ -33,6 +33,35 @@ def find_process_by_name(process_name: str): return processes +@allure.step('Find process by pid') +def find_process_by_pid(pid): + for proc in psutil.process_iter(): + try: + if proc.pid == pid: + return process_info( + proc.pid, + proc.name(), + datetime.fromtimestamp(proc.create_time()).strftime("%H:%M:%S.%f") + ) + except (psutil.NoSuchProcess, psutil.AccessDenied, psutil.ZombieProcess): + pass + + +@allure.step('Find process by port') +def find_process_by_port(port: int): + for proc in psutil.process_iter(): + try: + for conns in proc.connections(kind='inet'): + if conns.laddr.port == port: + return process_info( + proc.pid, + proc.name(), + datetime.fromtimestamp(proc.create_time()).strftime("%H:%M:%S.%f") + ) + except (psutil.NoSuchProcess, psutil.AccessDenied, psutil.ZombieProcess): + pass + + @allure.step('Kill process by name') def kill_process_by_name(process_name: str, verify: bool = True, timeout_sec: int = 10): _logger.info(f'Closing process: {process_name}') @@ -46,8 +75,22 @@ def kill_process_by_name(process_name: str, verify: bool = True, timeout_sec: in wait_for_close(process_name, timeout_sec) +@allure.step('Kill process by PID') +def kill_process_by_pid(pid, verify: bool = True, timeout_sec: int = 10): + os.kill(pid, signal.SIGILL if IS_WIN else signal.SIGKILL) + if verify: + wait_for_close(pid=pid, timeout_sec=timeout_sec) + + +@allure.step('Kill process by port') +def kill_process_by_port(port: int): + proc = find_process_by_port(port) + if proc is not None and proc.pid: + kill_process_by_pid(proc.pid) + + @allure.step('Wait for process start') -def wait_for_started(process_name: str, timeout_sec: int = configs.timeouts.PROCESS_TIMEOUT_SEC): +def wait_for_started(process_name: str = None, timeout_sec: int = configs.timeouts.PROCESS_TIMEOUT_SEC): started_at = time.monotonic() while True: process = find_process_by_name(process_name) @@ -60,11 +103,19 @@ def wait_for_started(process_name: str, timeout_sec: int = configs.timeouts.PROC @allure.step('Wait for process close') -def wait_for_close(process_name: str, timeout_sec: int = configs.timeouts.PROCESS_TIMEOUT_SEC): +def wait_for_close(process_name: str = None, timeout_sec: int = configs.timeouts.PROCESS_TIMEOUT_SEC, pid=None): started_at = time.monotonic() while True: - if not find_process_by_name(process_name): - break + if process_name is not None: + process = find_process_by_name(process_name) + if not process: + break + elif pid is not None: + process = find_process_by_pid(pid) + if process is None: + break + else: + raise RuntimeError('Set process name or PID to find process') time.sleep(1) assert time.monotonic() - started_at < timeout_sec, f'Close process error: {process_name}' _logger.info(f'Process closed: {process_name}') @@ -112,3 +163,4 @@ def run( process = subprocess.run(command, shell=shell, stderr=stderr, stdout=stdout, timeout=timeout_sec) if check and process.returncode != 0: raise subprocess.CalledProcessError(process.returncode, command, process.stdout, process.stderr) + _logger.debug(f'stdout: {process.stdout}') diff --git a/test/ui-pytest/tests/test_onboarding.py b/test/ui-pytest/tests/test_onboarding.py index 0fe0ab806..6850efa18 100755 --- a/test/ui-pytest/tests/test_onboarding.py +++ b/test/ui-pytest/tests/test_onboarding.py @@ -1,4 +1,5 @@ import logging + import allure import pytest from allure import step @@ -18,13 +19,15 @@ pytestmark = allure.suite("Onboarding") @allure.testcase('https://ethstatus.testrail.net/index.php?/cases/view/703421', 'Generate new keys') @pytest.mark.case(703421) -@pytest.mark.parametrize('user_name, password, user_image', [ - pytest.param('Test-User _1', '*P@ssw0rd*', None), - pytest.param('_1Test-User', '*P@ssw0rd*', 'tv_signal.jpeg', marks=pytest.mark.smoke), - pytest.param('Test-User', '*P@ssw0rd*', 'tv_signal.png'), +@pytest.mark.parametrize('user_name, password, user_image, zoom, shift', [ + pytest.param('Test-User _1', '*P@ssw0rd*', None, None, None), + pytest.param('Test-User', '*P@ssw0rd*', 'tv_signal.png', 5, shift_image(0, 0, 0, 0)), + pytest.param('_1Test-User', '*P@ssw0rd*', 'tv_signal.jpeg', 5, shift_image(0, 1000, 1000, 0), + marks=pytest.mark.smoke), ]) -def test_generate_new_keys(main_window, user_name, password, user_image: str): +def test_generate_new_keys(main_window, user_name, password, user_image: str, zoom, shift): with step('Open Generate new keys view'): + if configs.system.IS_MAC: AllowNotificationsView().wait_until_appears().allow() BeforeStartedPopUp().get_started() @@ -32,17 +35,17 @@ def test_generate_new_keys(main_window, user_name, password, user_image: str): keys_screen = wellcome_screen.get_keys() with step(f'Setup profile with name: {user_name} and image: {user_image}'): + profile_view = keys_screen.generate_new_keys() profile_view.set_display_name(user_name) if user_image is not None: profile_picture_popup = profile_view.set_user_image(configs.testpath.TEST_FILES / user_image) - profile_picture_popup.make_profile_picture(zoom=5, shift=shift_image(0, 200, 200, 0)) + profile_picture_popup.make_profile_picture(zoom=zoom, shift=shift) assert not profile_view.error_message - with step('Open Profile details view'): - details_view = profile_view.next() + with step('Open Profile details view and verify user info'): - with step('Verify Profile details'): + details_view = profile_view.next() if user_image is None: assert not details_view.is_user_image_background_white() assert driver.waitFor( @@ -52,14 +55,15 @@ def test_generate_new_keys(main_window, user_name, password, user_image: str): else: image.compare( details_view.cropped_profile_image, - configs.testpath.TEST_VP / f'user_image_onboarding.png', + f'{user_image.split(".")[1]}_onboarding.png', + threshold=0.9 ) - chat_key = details_view.chat_key emoji_hash = details_view.emoji_hash assert details_view.is_identicon_ring_visible - with step('Finalize onboarding and prepare main screen'): + with step('Finalize onboarding and open main screen'): + create_password_view = details_view.next() assert not create_password_view.is_create_password_button_enabled confirm_password_view = create_password_view.create_password(password) @@ -69,7 +73,8 @@ def test_generate_new_keys(main_window, user_name, password, user_image: str): SplashScreen().wait_until_appears().wait_until_hidden() WelcomeStatusPopup().confirm() - with step('Open User Canvas and verify profile'): + with step('Open User Canvas and verify user info'): + user_canvas = main_window.left_panel.open_user_canvas() assert user_canvas.user_name == user_name if user_image is None: @@ -78,11 +83,12 @@ def test_generate_new_keys(main_window, user_name, password, user_image: str): configs.timeouts.UI_LOAD_TIMEOUT_MSEC ) - with step('Open Profile popup and verify profile'): + with step('Open Profile popup and verify user info'): + profile_popup = user_canvas.open_profile_popup() assert profile_popup.user_name == user_name assert profile_popup.chat_key == chat_key - assert profile_popup.emoji_hash.compare(emoji_hash.view) + assert profile_popup.emoji_hash.compare(emoji_hash.view, threshold=0.9) if user_image is None: assert driver.waitFor( lambda: profile_popup.is_user_image_contains(user_name[:2]), @@ -91,5 +97,6 @@ def test_generate_new_keys(main_window, user_name, password, user_image: str): else: image.compare( profile_popup.cropped_profile_image, - 'user_image_profile.png', + f'{user_image.split(".")[1]}_profile.png', + threshold=0.9 )