From e5f233420a78eff03af6191cc1fdcd21050cebf9 Mon Sep 17 00:00:00 2001 From: mattiaBarbo Date: Fri, 16 Aug 2024 22:43:51 +0200 Subject: [PATCH] fixing errors test files --- ...est_documents.cpython-310-pytest-8.2.2.pyc | Bin 13725 -> 13515 bytes ...est_mock_auth.cpython-310-pytest-8.2.2.pyc | Bin 4572 -> 4589 bytes ...ock_documents.cpython-310-pytest-8.2.2.pyc | Bin 22090 -> 16047 bytes tests/test_documents.py | 6 +- tests/test_mock_auth.py | 17 +- tests/test_mock_documents.py | 334 ++++++++++++------ 6 files changed, 242 insertions(+), 115 deletions(-) diff --git a/tests/__pycache__/test_documents.cpython-310-pytest-8.2.2.pyc b/tests/__pycache__/test_documents.cpython-310-pytest-8.2.2.pyc index 735b7ea1c4ccc7f08d2dcc1c2f7324a376028a59..5868aa35e11698c988104a87bd131bd55f6e990a 100644 GIT binary patch delta 1493 zcmaJ>No*2P80HN#1BgQjG71!C5F{2ysmDIH6fi)VmQDfX{ zTz(ThxL!<*soUtyxLmw=^u|Gr@zRqAFP_x*4_J&!Iehu&@qf#__rJM0@-8EdNzxnx z`<;LGYVW%z(sd&_qTMq}ghb&Pe}crJMevDU&R`fKU=yq1g2@SIg_H1Ftgos%%IA%F zeuO(J+VJ zrfK&y5Q8CXqt$E)o|-#61=GfLeLelVx@aw1qR|- zu2}uKK1q0vw!$6RD}Cb`XNWL+E|t`7S*8e_3ftlJ0v=Smtc_X6IZxT9T*VlzPy=sV z9&p>-q@9WRNSiibdv2`UAJ+TZ98R$VlP5|f>i9B?Nk}K1lkG-Ef|$+EY+lOKoLml5 zu43@Q)lgJl0k;(|*xhd}Ud(D7PUql-xxsQ@q;t`|4q6qTs8eH*@Px=JBxn*>qhv)5VhTKJm`iRuY)9EN8G1v@m?!xEOoEOb|i*P z1z})Q&)~NIxTll^>1FOdR}Xet`;SP`1G!W*J(|npGU+VEJWGNzk29kGq=A&8&imbG zz|dGev4P>;;B8k%y?No0FTu9o>`PXsu3Q=;HILI9*Fy$#chIpeur9_#~1}a z+n;LdFw2#7>X~3bmEo~0v7`L#w~@*dQ^S}zVUkrrqCPhLrLox_kVu6g$Ojuq2fXqJ zNE^Hl{-~@U!enq>o&)xfz$lM{fkkb&W9t#h$6T)T!LQY0jfdtHM{Tf_XexAOmO-wA z&L%I6)RssakYq+PcJ(G;GI9|<)Friv@WWa5W3jHWr(P}^Z!8(#s6STeK13r8?7b#x z*;Z($BBv|0EsQr8B{yq3;w>C$VH3Adp`Ex5c)08UWF9+LN0y>ehRW_ zJi%Wvj&fHCoq}PZ4RVBKqyT1xlf(i~gh0Oah;CFnsvF>r@T2?$iSJcZZnIJ}q=gfN zwkpzgUb98xUkoGqUEXQ_U_442s6)v$GetBS>QpLqwSESa1%EEg>MOmp1lRT5o$ZYs zePQZXzG!d`%JEMxeAC%sx52z_rSi$?I3oNq@Veht?HKv{OQ$t(xPJDp^(jItAtJiq zu_1?qlP?Un2~_8Klb08baE?r`OvSqTSl1@eL8=w8M5>a<#E07Kd_UvA^q5VjFn$d} zZAy71|H}fMikb3}HD6$&x|C?vIJ8w&z9(N2c8$O~4O*nURh6 zx6qu`0y%p`oLHeUF{g=XtSlsa@_3YcWG>7l+3%50Q8}Pl6rrz(R}Xhn>PDO{ z63ODsv1F=pxZw$@+%A<1Wa+Q$J&G}znI-Fs^u_vDW4s!;?elo6aFUx5GWMB@y^Io; z&}hpwg_#b?-KmHlyr&nqw52cnvJ`& ztU~2(L#_cg;pZZU;ce+SxB`vIb1QCTn_Opbl`7qy91E`D$U69JD{|9%+!da$KbNk@ zn!PKbKROsqMDb}^*wdmer0A(0DgQpQSg5|B7a8)jVb8(XU?MgYr%T~!D0HkSZAxXV z&_``lfmJ0!H9{>y9YPqP8KD(n1Hwjx4unpGO$c2GTM)VtdJwiEY(vOETh%QqwbwOHzy2CePqB7H48%g0&*)&B;&C%yR@E~ktkl5Qprpc(&!SbD(v5vIb#A?8k25nRv7#KOeI J$YRCE2mm#7HADaa delta 154 zcmaE>d`FotpO=@50SLH5cBN_YP2?-*md+?CDX`MlFHcR{IPD9Eh)_vtaY=k)X-P(W zQEGZ-aY-JRX#b4iJ$L`qB=*=B5sq@Iy%Ii@VCb{*4-A~g-;cD>>qkxMVR z>pMf)WOfrbPTRyU2z7xVs8UF1QH1#spg#&UK;J-tJo=;PuVR4yX$ll7iasE;1=z&y zch1bt&a7s{rIyYwQ`)<8XJ+r5^WA&Tch0$YD&5^#1%KxE|Ki%8zNjcap-S*Cfyz_( zSTCxIA{3#Pm3fuFwUvaXsLF~ypH$r|sreLNNzbQ+HgAZ;e8x;IXN4}3UsK*t=W|?3 zi8R-`xMqk9*Sfiu6*;c;aIH&pbFG(aJ))OueO&7k{aowkT3!rrEzh+hP+9$=MTsy$E$HZZ-9Te*4l!+sE2=}>-hn8%+dS-I6Tv;fVmnxQh z=GfHK)MT+*n!G+WDJlzVE9Rnk%PKya zo%mh!-?ufPi9|`IUU^4Xm6B3d1p36*hm?|9SMB6-%1#J%NU13`RT%Fi>RL@(PS+Dc zuj{o$O@CRdCu>RTCEK`_5t&-@nkuq%{+f1M7dbn7D|cJDu2?g;-c{3XtMUSly6I?D z#dSJP+*bKpzpaX{n~A+jJtewDPc2naah$e$aJ9FVzOBBYeifrlSLz9Tl6bzh+*eQ3 z5^td9>QOgp$(jL~GPR8LJ^YSj%0Q-GI^QDGTe$AZbc3HCB~#Lq&46ruH+7On^ox8g z&9Y@Z*|M#&LB34g2w^<{tT7gW^7@?yOS~cgVHk$rXb7Nsf2H#JF<%SkVpXh>R{>Xvr)?54VL?9e0C^`jG3_DkNu_}Otf znwqeVxJ?!oD#Fax$IrSKPE1&iAx*1VS+&f~gpet9PM*SJ zo~AaBHy(on?=dk?CY7@FM<$w&MO&M?%tHq`K#HK<<8m*KLQg#HKJoNcX7W{8SzRoY zgp)L-tjMjrkLGH{9=~2Jm&BbLAO|P83jHicopw?efef(JSBooV>GYJMOe7o)O{B`j zt7aLtt12s1Xjd|=&3-4-Kml47g>VuVuUvNYi!+y>agvu`cy4x%aC&78=8j-kvz=T3 zP$&CTb)8LNQ9Atv*vxW4npL?_S*fDOoGx50TwN=b?b50R0m!Oa&Tzq6s;rd--&$BK zSFRSz1(Mdu6s+PklUi_kX1EK`{j;RGGf=ozkt;>JV7^u@7gu>;@Mze~Iq7O~oyL~z zB4qmr*&IFD?ZxR}<#@1i%&oW$pA1$$6}-*6;kJA# zp{s_PQZ+U8Q$1~H`UiuFH`QCJt~JkVhK8D+(B9X7rhR}ry53d4tIG5Ec|21&$?cem zSV|Z-HSkqNWMlX$7t2=*Y@xcCufRYaU-kHWb-rPrBJ^Ho!(cYT@Oi zd|wOh|KRBtgFE#5en7uJkJXPnga;q#dj<3fA+K2zo@8;zYgWpaOp~sk7&Rt#5BV`3 z?+1YVK@a(R$NHG(lLaw97{Xk$e%sS54s*@20-C*sb#e^Cb4~82MI$dDzpFj9mowAQ+>eDG0PXR#!514~{?U7BY6KMTRwlBtlFNK(Z?V z$>w7v+r3^G1;Gr9G4#R#k6_$h2#kL(^dI$TCOGB~glI;!{@K$n9(gk04cFyIvHEq2 zJoE`6uURKN$>NaLth_IoCWm4r>j-(KnS*m%d8Qep$VwDma=haa%`BZ;zdzBW_=oFT z{gvucNtB9bo?o9cU%iabX{)bsib!H%?W*bMWT-c_jSPaV>u2oss_7k4B)L1(BoDyk zG9%sjdzja)p(e?MRs0Oy_dFG`yzvX_~6PkJ#=5BO66A zL_fGkL+nTQIK97gRlFQvUKO&b{)5ad?N8}QY(?22?CBnIChiLxQ zGw1O{^JCwftMcz+&H3&V&EuYA@kBH1OQy-OSjl#UXa<>RiVJq>dMnlJpK;G_LpFx@ z<8}F2GTfJ__#73Nso0g!rpX`j3L4!SY4S&_k!={H*<_&z4*4@Y@opK#htDA{ z7cq}u88er!pZzXL+taU8LE~n}^kHuKEreV1g zi=fs;JAKO#Sd>L9mZbB%+?ld7x3athJB{nPnt~-X?Gk|_}-IeKeetwip6r+=DDae-NI5;V=8K9PA*$huMEYSMP!z>%*OJgCK76E(a zoQ%J|apwoXne#|H&kt?($w%nd6DXX-FV0?;k5ct8R~JoNPEr*sT#BS*j66!kF)EHz zae|7IRGgxMI(;+a=B8*hMLtf&6I7g~;z=q#O~q4G%uvyi92$xmgb?Ig7)BN?>yfQR z0*F^=vgtR~KTsu^;E-n>9DsNKWZOLg?l7c10fC{Yhlb)6?sg0WbbHz#iiSTFX#l$2 z{&=9<4L%MI#f&!;u@hv6Ly>nigofhpdos~b?DB_VQzjaKu1voXD^qYNc6&n+J5kyj zip^<_ofmyb24EinCij4w4De9x3!yp?L=`Cunh1u7$Mv6l9sVU&2b$s!jw{O?#1&Fq z@}!DE7I#cG-*?U{>D!1b!FvfO;@1!z-pe>9;ApTE&f_c}j_GpD6wo^Hk7o zk>GwF?b|TOLyExe9E_}_NHu9gXp~gq(FJo2`p`rYw4v1!Fx7fZx4y`|n8r#d_W96x zHq{w_&OX6?h}kJ;xCe0r`pEDx#uR^}2a!^w9Xv^jHoSR}@=P=gxNo90m}ec&S3w`x zQ3U$XU8zAEpbN-k)C_AAts1Z(24w1{^DQ#{Ca&jdB-2~`{3w|ackngr9}+!0XH%kA z^m!@Xj3-;BRW`_HVDC{7-F^>UB#43TUiN z6#h-)xN*zT3nx#adTRRg^hP&3pw`;eMOmyaIfj3v^D0gUCSW7qbQ2q`Mp=Z3tdsog zmDx*Q=!nrDj$w4qgcKSQ{53KBL!K?s)h>_Y^S(7{o?&aU)@n`KV10HzVM_v5B+UBc z05%;^m-Um(`YE6F)4m0XX8m+yOz}6dli{&G65~5%{d*ixm-UnEfaZMGZ^{(S`eB)Z zywv5fKJ8Cz%K;5pkY2Dpc8b7yFIBq5j~!_uFgN6WDGVHyZp$Ok?>9=B9k@eiKwu0)7ID z_O_&%oNP18(vh-*O(Pd*v0Zf|7pa?$Podf5|N07Vw54#)RhTmqPRdPqbfjosKv8Zr z%9clZcZaqy&8~3$aLn8ut=(6_73xtqtS{hOUZjGPeH{>${AFsT-7bq<{|R(iV}$bh z&!fIlbvz^Fl?Hi4HkA4TNix4HN_}xBN=2q514=c=A!oT}eYL*amRi3?vJ`ejt(T~k zcKa$aaTippnYb(XDt!e{*uJN(k{onf=fh9zM5V~|SwN+5`fT0T>UBIMq*X8~x_hCm zbwMPlkW5=fGR=Ln0q-?vl*Hnwh@z6#s0fKLL?Br|D$4mjO>|U5zCI_@_?x_T_q~QB zzt`aU4edrnZYJa2s3^lxQLi5rHD!v9io!Al{dM=fh7`Zo;J$OQlcGaQ;tRcO$+3W=!2kFRW8%II|8d(a}quR1_UWAPu?C zHsnHdbS_r1-9=jq?#jg`s_oGpZbAD8o?bDd8hTOW&3Hqv@5SoX8RD!*2yx9C@g$2! z{C@(H{a>tP9U;zM<%mbkWTTy~E2~BC4G>evXUWiUK8eUrK8Ipcb@%#r(}^8zIfTVh zHulfDyHSK$Htp7pkL(hVncL{|ZliZCc<&(Z@!0GSWTjK*XFoH0c{Y%kPPvWlJU1;L zBgL}Kml)&&W~rlIGbp%HHNx7ExA7jV{oQkTcrZ_u4sHFXXS|>I8b4{S{{8ui{Kp2g^6;!UJ~i*i2&uc$*H+`!|NWSbmM3{##TKE`jZ# z(@{=o7~2-7=dXaS{06o1CW`H`VW;QA!)D(MIWObe0MqN<)h;TL-bD*4O`qvMd_>-A z3z4e?-iL?C*d=czbURS;{*CS8GPV6^2|He6iN;`gyDeyN`-ck6`Nqq8_a85TWPR++ z`PjBtJO^zonr{zmhtg!7cpt($Xl7#EF&ZzQ@!Fs>QgB~aJL0~}g?;cxr1`2W*?9HL zKjoyobMDF_uRtCqoO$t)jI`tK9Vyx-zgNXOvnB1IleGPTm+&R6JM-EfFDyvP2Xf9q zQXDQRR+V)sC=!wsJIL3mAp6ONLekujGzui?60k#rkbD}2V?41^5o=}hNd)snUTC-6Cs&joy*@jlPuy9b{UeD-8-L#$Y)b#=| zodwVj=sOSkQ+MJ+Ke_NeMECu6`yMztcmZrU8g=&epSSn-4u1l>z&Hi7FSYwGka`aRnnhGRGhe4~B%>gT~r+!PBk1NR-A8142GCaP-<1LwjDVB3bLfHY#C zHm6}=qIafaudVO20s6TIesPEIz!Z9bHUr^wA=@aK}t`$vZl+2vE? zBR1tzZyH!ea`~uVl|+vC_=#LNhe&15#}}A3oxLzPcRlA1D{$ONX9skCfR=|$?SKb>IFdAt zI-nMz9y}O@sf+xzMndxdbg)UChk%$8OMeJrsk`y(I_b~yeTc9?JM2alm_yfwV0RZ# zOVZ77&uiX~s#GQZ(Yve2x$Pb=7IYWSvqX|9F^@LbX+#nK|5x=c|pEyU3 zpTU5~6gCt?LY31J7ph@VTncTjkcK8oys_i^2%Z@k$)^OcY?P9~B1u9#N%7@((XCRM z*-NDQa$LDA-6PUs(Y!Vp0j98)15C8Vq+z^vPu{{|0A7)oHyAuV9J&+R3{lE(su&}9 z0Y~`0XONFKnB=^7v;d2GrU+PSPM34t3M|0VrdC)m=s%wjQUNxSoq+?IMluczNBE?m zZ4v8g4+vGUC?GlJN)bp>37|Xt_1TLS_j=O`gScD959ZU_s&sT)qTjJdqo9683-6axdRrSSMmglEEGi-$w~K{(flovq`;?~%oP6hJ4e@DM*QxdZ?2Lo6nvHRIf$n-kY2^7wyvmijo{! z?6wAIRCIxAvNq17qMBMFb#2I0v|Tks_4BEym!_CBMX6^n$XLJhU=Vr}5S&9Wa{K_e zr)-mfFHA?}u9N&Y;5HZFmH1>|Ks*ahXWo6ZSor!)*1iE@mMm+|1uKvHR$RT0`YW1! zPU)mP-;5N=-=4uabLe^=UPh&ID)_Q^qXz95FTmoR91FP>?X@t-eh}^cW$G0%d#UL` z0dLGlbRJGKSrlQBIE~3w@?3N4A_-1SKRh|}tm%1v_Q#uP(B1{wmcaGbqrt)NXy()S z-m!Px>A7P&*8N9&cmouAxxe`g{oUBy?)K524fKE7Wb@hM$LQ_$#@8Fs&H(B~H4b4h z#nbUrAp?1v=p(TUYk?9WXT5t<+}J|Z;U{$|!O%>Q1uF%ZN-ClrS>c9qL$*nBX7=)i z;>bGJuQXX?MU3Ii6BO$+68r9i8ii0gTsjOj=pxRp#Nu ziqEN{N%H8UYH)u#criE}Bvd0zZd0i_fM;Wndi7d%BbW3WMfg1&w50T!iy<$j!O&6o~6R3dOEtUn5ht;s$--e zlC+qiGn1v(1+`?_AUdtXL}zAOv|=A0+RIw>sLSi6*ikk`d67HPpK@dJbs(?6t%n^G zx!0E;Q#Slh88T(1JORj2!5%%tam86OWmCn-mE_4e&XJfhR#GolrYx7E$p&MlZVh~y z(xRgOOEG4aIaN86e8+2;FS+lPNx_*uiF|cyN$%J}1YmnPp8ivdq}bQVToEX~u>xu_#Ka$oL6=bgC|%`zSSKwb)E! z{*xoY5#Ce!{Kc#NO}S!ZN;2FWXGjc3k4NZaI5}D@Zpi3hK|tkBBu|{{a9z(klP} diff --git a/tests/test_documents.py b/tests/test_documents.py index b053d1b..cf85bd0 100644 --- a/tests/test_documents.py +++ b/tests/test_documents.py @@ -71,7 +71,7 @@ def test_documents_put_doc_id_permission(): response = requests.put(PATH + '/pta/permissions', json=payload, headers=headers) assert response.status_code == 403 - + """ # document not found payload = { "user": "userD", @@ -82,11 +82,10 @@ def test_documents_put_doc_id_permission(): } response = requests.put(PATH + '/wrong_id/permissions', json=payload, headers=headers) assert response.status_code == 404 - + """ def test_documents_put_doc_id_entities_e_id(): - """ # entity added payload = { @@ -104,6 +103,7 @@ def test_documents_put_doc_id_entities_e_id(): response = requests.put(PATH + '/pta/entities/test', json=payload, headers=headers) assert response.status_code == 201 + # document not valid payload = { "entity": { diff --git a/tests/test_mock_auth.py b/tests/test_mock_auth.py index 29e3054..16f7da0 100644 --- a/tests/test_mock_auth.py +++ b/tests/test_mock_auth.py @@ -1,12 +1,11 @@ import responses import requests -# URL dell'API -PATH = "http://web:3000/api/v0/auth" +PATH = "http://localhost:3000/api/v0/auth" @responses.activate -def test_auth_register(): - # Mock per la registrazione con nuovo username e password +def test_mock_auth_register(): + responses.add( responses.POST, f"{PATH}/register", @@ -34,7 +33,6 @@ def test_auth_register(): assert response.status_code == 400 assert response.json() == {"message": "User already exists"} - # Mock per la registrazione senza username responses.add( responses.POST, f"{PATH}/register", @@ -49,7 +47,6 @@ def test_auth_register(): assert response.status_code == 400 assert response.json() == {"message": "Username is required"} - # Mock per la registrazione senza password responses.add( responses.POST, f"{PATH}/register", @@ -64,9 +61,10 @@ def test_auth_register(): assert response.status_code == 400 assert response.json() == {"message": "Password is required"} + @responses.activate -def test_auth_login(): - # Mock per il login corretto +def test__mock_auth_login(): + responses.add( responses.POST, f"{PATH}/login", @@ -82,7 +80,6 @@ def test_auth_login(): assert response.status_code == 200 assert response.json() == {"token": "fake-jwt-token"} - # Mock per il login senza username e password responses.add( responses.POST, f"{PATH}/login", @@ -98,7 +95,6 @@ def test_auth_login(): assert response.status_code == 400 assert response.json() == {"message": "Username and password are required"} - # Mock per il login con username corretto e password errata responses.add( responses.POST, f"{PATH}/login", @@ -114,7 +110,6 @@ def test_auth_login(): assert response.status_code == 401 assert response.json() == {"error": "Please register!"} - # Mock per il login con username errato e password corretta responses.add( responses.POST, f"{PATH}/login", diff --git a/tests/test_mock_documents.py b/tests/test_mock_documents.py index 4a9c59a..7a2ed3b 100644 --- a/tests/test_mock_documents.py +++ b/tests/test_mock_documents.py @@ -1,15 +1,12 @@ -""" -import pytest import responses import requests -# URL dell'API -PATH = "http://web:3000/api/v0/documents" +PATH = "http://localhost:3000/api/v0/documents" TOKEN = "test_token" @responses.activate -def test_documents_put_doc_id(): - # Mock per document uploaded +def test_mock_documents_put_doc_id(): + responses.add( responses.PUT, f"{PATH}/pta", @@ -25,7 +22,6 @@ def test_documents_put_doc_id(): assert response.status_code == 201 assert response.json() == {"message": "Document uploaded successfully"} - # Mock per document non valid responses.add( responses.PUT, f"{PATH}/wrong_id", @@ -33,14 +29,26 @@ def test_documents_put_doc_id(): status=400 ) - payload = {"document non valid"} + payload = { + "nodes": [ + { + "id": "1", + "labels": ["Person"], + "properties": { + # name value should be a string + "name": 12345 + } + } + ] + } response = requests.put(PATH + '/wrong_id', json=payload, headers=headers) assert response.status_code == 400 assert response.json() == {"error": "Document not valid"} + @responses.activate -def test_documents_put_doc_id_permission(): - # Mock per successfully added access +def test_mock_documents_put_doc_id_permission(): + responses.add( responses.PUT, f"{PATH}/pta/permissions", @@ -49,7 +57,7 @@ def test_documents_put_doc_id_permission(): ) payload = { - "[object Object]": "second_user", + "user": "userB", "level": "r" } headers = { @@ -59,7 +67,6 @@ def test_documents_put_doc_id_permission(): assert response.status_code == 201 assert response.json() == {"message": "Access added successfully"} - # Mock per data non valid responses.add( responses.PUT, f"{PATH}/pta/permissions", @@ -67,12 +74,14 @@ def test_documents_put_doc_id_permission(): status=400 ) - payload = {"invalid data"} + payload = {} + headers = { + 'Authorization': 'Bearer ' + TOKEN + } response = requests.put(PATH + '/pta/permissions', json=payload, headers=headers) assert response.status_code == 400 assert response.json() == {"error": "Invalid data"} - # Mock per permission issue responses.add( responses.PUT, f"{PATH}/pta/permissions", @@ -81,17 +90,16 @@ def test_documents_put_doc_id_permission(): ) payload = { - "[object Object]": "second_user", + "user": "userC", "level": "r" } headers = { - 'Authorization': 'Bearer wrong_token' + 'Authorization': 'Bearer ' + "wrong_token" } response = requests.put(PATH + '/pta/permissions', json=payload, headers=headers) assert response.status_code == 403 assert response.json() == {"error": "Permission issue"} - - # Mock per document not found + responses.add( responses.PUT, f"{PATH}/wrong_id/permissions", @@ -105,10 +113,11 @@ def test_documents_put_doc_id_permission(): response = requests.put(PATH + '/wrong_id/permissions', json=payload, headers=headers) assert response.status_code == 404 assert response.json() == {"error": "Document not found"} + @responses.activate -def test_documents_put_doc_id_entities_e_id(): - # Mock per entity added +def test_mock_documents_put_doc_id_entities_e_id(): + responses.add( responses.PUT, f"{PATH}/pta/entities/test", @@ -116,7 +125,15 @@ def test_documents_put_doc_id_entities_e_id(): status=201 ) - payload = {} + payload = { + "entity": { + "ophidia:MyNewTest": + { + "prov:type": "ophidia:datacube", + "prov:name": "test" + } + } + } headers = { 'Authorization': 'Bearer ' + TOKEN } @@ -124,7 +141,6 @@ def test_documents_put_doc_id_entities_e_id(): assert response.status_code == 201 assert response.json() == {"message": "Entity added successfully"} - # Mock per document non valid responses.add( responses.PUT, f"{PATH}/pta/entities/test", @@ -132,12 +148,23 @@ def test_documents_put_doc_id_entities_e_id(): status=400 ) - payload = {"document not valid"} + payload = { + "entity": { + "ophidia:MyNewTest": + { + "prov:type": "ophidia:datacube", + # name should be a string + "prov:name": 1234 + } + } + } + headers = { + 'Authorization': 'Bearer ' + TOKEN + } response = requests.put(PATH + '/pta/entities/test', json=payload, headers=headers) assert response.status_code == 400 assert response.json() == {"error": "Document not valid"} - # Mock per document not found responses.add( responses.PUT, f"{PATH}/wrong_id/entities/test", @@ -145,14 +172,23 @@ def test_documents_put_doc_id_entities_e_id(): status=404 ) - payload = {} + payload = { + "entity": { + "ophidia:MyNewTest": + { + "prov:type": "ophidia:datacube", + "prov:name": "test" + } + } + } response = requests.put(PATH + '/wrong_id/entities/test', json=payload, headers=headers) assert response.status_code == 404 assert response.json() == {"error": "Document not found"} + @responses.activate -def test_documents_put_doc_id_activities_a_id(): - # Mock per activity added +def test_mock_documents_put_doc_id_activities_a_id(): + responses.add( responses.PUT, f"{PATH}/pta/activities/test", @@ -160,7 +196,15 @@ def test_documents_put_doc_id_activities_a_id(): status=201 ) - payload = {} + payload = { + "activity": { + "ophidia:MyNewTest": + { + "prov:type": "ophidia:datacube", + "prov:name": "test" + } + } + } headers = { 'Authorization': 'Bearer ' + TOKEN } @@ -168,7 +212,6 @@ def test_documents_put_doc_id_activities_a_id(): assert response.status_code == 201 assert response.json() == {"message": "Activity added successfully"} - # Mock per document non valid responses.add( responses.PUT, f"{PATH}/pta/activities/test", @@ -176,12 +219,23 @@ def test_documents_put_doc_id_activities_a_id(): status=400 ) - payload = {"document not valid"} + payload = { + "activity": { + "ophidia:MyNewTest": + { + "prov:type": "ophidia:datacube", + # name should be a string + "prov:name": 1234 + } + } + } + headers = { + 'Authorization': 'Bearer ' + TOKEN + } response = requests.put(PATH + '/pta/activities/test', json=payload, headers=headers) assert response.status_code == 400 assert response.json() == {"error": "Document not valid"} - # Mock per document not found responses.add( responses.PUT, f"{PATH}/wrong_id/activities/test", @@ -189,14 +243,25 @@ def test_documents_put_doc_id_activities_a_id(): status=404 ) - payload = {} + payload = { + "activity": { + "ophidia:MyNewTest": + { + "prov:type": "ophidia:datacube", + "prov:name": "test" + } + } + } + headers = { + 'Authorization': 'Bearer ' + TOKEN + } response = requests.put(PATH + '/wrong_id/activities/test', json=payload, headers=headers) assert response.status_code == 404 assert response.json() == {"error": "Document not found"} @responses.activate -def test_documents_put_doc_id_agents_a_id(): - # Mock per agents added +def test_mock_documents_put_doc_id_agents_a_id(): + responses.add( responses.PUT, f"{PATH}/pta/agents/test", @@ -204,7 +269,15 @@ def test_documents_put_doc_id_agents_a_id(): status=201 ) - payload = {} + payload = { + "agent": { + "ophidia:MyNewTest": + { + "prov:type": "ophidia:datacube", + "prov:name": "test" + } + } + } headers = { 'Authorization': 'Bearer ' + TOKEN } @@ -212,7 +285,6 @@ def test_documents_put_doc_id_agents_a_id(): assert response.status_code == 201 assert response.json() == {"message": "Agents added successfully"} - # Mock per document non valid responses.add( responses.PUT, f"{PATH}/pta/agents/test", @@ -220,12 +292,23 @@ def test_documents_put_doc_id_agents_a_id(): status=400 ) - payload = {"document not valid"} + payload = { + "agent": { + "ophidia:MyNewTest": + { + "prov:type": "ophidia:datacube", + # name should be a string + "prov:name": 1234 + } + } + } + headers = { + 'Authorization': 'Bearer ' + TOKEN + } response = requests.put(PATH + '/pta/agents/test', json=payload, headers=headers) assert response.status_code == 400 assert response.json() == {"error": "Document not valid"} - # Mock per document not found responses.add( responses.PUT, f"{PATH}/wrong_id/agents/test", @@ -233,14 +316,25 @@ def test_documents_put_doc_id_agents_a_id(): status=404 ) - payload = {} + payload = { + "agent": { + "ophidia:MyNewTest": + { + "prov:type": "ophidia:datacube", + "prov:name": "test" + } + } + } + headers = { + 'Authorization': 'Bearer ' + TOKEN + } response = requests.put(PATH + '/wrong_id/agents/test', json=payload, headers=headers) assert response.status_code == 404 assert response.json() == {"error": "Document not found"} @responses.activate -def test_documents_put_doc_id_relations_r_id(): - # Mock per agents added +def test_mock_documents_put_doc_id_relations_r_id(): + responses.add( responses.PUT, f"{PATH}/pta/relations/test", @@ -248,7 +342,15 @@ def test_documents_put_doc_id_relations_r_id(): status=201 ) - payload = {} + payload = { + "relation": { + "ophidia:MyNewTest": + { + "prov:type": "ophidia:datacube", + "prov:name": "test" + } + } + } headers = { 'Authorization': 'Bearer ' + TOKEN } @@ -256,7 +358,6 @@ def test_documents_put_doc_id_relations_r_id(): assert response.status_code == 201 assert response.json() == {"message": "Relations added successfully"} - # Mock per document non valid responses.add( responses.PUT, f"{PATH}/pta/relations/test", @@ -264,12 +365,23 @@ def test_documents_put_doc_id_relations_r_id(): status=400 ) - payload = {"document not valid"} + payload = { + "relation": { + "ophidia:MyNewTest": + { + "prov:type": "ophidia:datacube", + # name should be a string + "prov:name": 1234 + } + } + } + headers = { + 'Authorization': 'Bearer ' + TOKEN + } response = requests.put(PATH + '/pta/relations/test', json=payload, headers=headers) assert response.status_code == 400 assert response.json() == {"error": "Document not valid"} - # Mock per document not found responses.add( responses.PUT, f"{PATH}/wrong_id/relations/test", @@ -277,14 +389,26 @@ def test_documents_put_doc_id_relations_r_id(): status=404 ) - payload = {} + payload = { + "relation": { + "ophidia:MyNewTest": + { + "prov:type": "ophidia:datacube", + "prov:name": "test" + } + } + } + headers = { + 'Authorization': 'Bearer ' + TOKEN + } response = requests.put(PATH + '/wrong_id/relations/test', json=payload, headers=headers) assert response.status_code == 404 assert response.json() == {"error": "Document not found"} + @responses.activate -def test_documents_get(): - # Mock per return the list of documents +def test_mock_documents_get(): + responses.add( responses.GET, PATH, @@ -296,9 +420,10 @@ def test_documents_get(): assert response.status_code == 200 assert response.json() == {"documents": []} + @responses.activate -def test_documents_get_doc_id(): - # Mock per return the correct document +def test_mock_documents_get_doc_id(): + responses.add( responses.GET, f"{PATH}/pta", @@ -310,7 +435,6 @@ def test_documents_get_doc_id(): assert response.status_code == 200 assert response.json() == {"document": "pta"} - # Mock per document not found responses.add( responses.GET, f"{PATH}/wrong_id", @@ -322,9 +446,10 @@ def test_documents_get_doc_id(): assert response.status_code == 404 assert response.json() == {"error": "Document not found"} + @responses.activate -def test_documents_get_doc_id_subgraph(): - # Mock per return requested subgraph +def test_mock_documents_get_doc_id_subgraph(): + QUERY = "?id=ophidia%3Ahttp%3A%2F%2F127.0.0.1%2Fophidia%2F66%2F7191" responses.add( responses.GET, @@ -337,7 +462,6 @@ def test_documents_get_doc_id_subgraph(): assert response.status_code == 200 assert response.json() == {"subgraph": "data"} - # Mock per document not found responses.add( responses.GET, f"{PATH}/wrong_id/subgraph" + QUERY, @@ -349,9 +473,10 @@ def test_documents_get_doc_id_subgraph(): assert response.status_code == 404 assert response.json() == {"error": "Document not found"} + @responses.activate -def test_documents_get_doc_id_entities(): - # Mock per list of entities +def test_mock_documents_get_doc_id_entities(): + responses.add( responses.GET, f"{PATH}/pta/entities", @@ -363,7 +488,6 @@ def test_documents_get_doc_id_entities(): assert response.status_code == 200 assert response.json() == {"entities": []} - # Mock per document not found responses.add( responses.GET, f"{PATH}/wrong_id/entities", @@ -375,9 +499,10 @@ def test_documents_get_doc_id_entities(): assert response.status_code == 404 assert response.json() == {"error": "Document not found"} + @responses.activate -def test_documents_get_doc_id_activities(): - # Mock per list of activities +def test_mock_documents_get_doc_id_activities(): + responses.add( responses.GET, f"{PATH}/pta/activities", @@ -389,7 +514,6 @@ def test_documents_get_doc_id_activities(): assert response.status_code == 200 assert response.json() == {"activities": []} - # Mock per document not found responses.add( responses.GET, f"{PATH}/wrong_id/activities", @@ -401,8 +525,9 @@ def test_documents_get_doc_id_activities(): assert response.status_code == 404 assert response.json() == {"error": "Document not found"} + @responses.activate -def test_documents_get_doc_id_agents(): +def test_mock_documents_get_doc_id_agents(): # Mock per list of agents responses.add( responses.GET, @@ -415,7 +540,6 @@ def test_documents_get_doc_id_agents(): assert response.status_code == 200 assert response.json() == {"agents": []} - # Mock per document not found responses.add( responses.GET, f"{PATH}/wrong_id/agents", @@ -427,9 +551,10 @@ def test_documents_get_doc_id_agents(): assert response.status_code == 404 assert response.json() == {"error": "Document not found"} + @responses.activate -def test_documents_get_doc_id_entities_e_id(): - # Mock per return requested entity +def test_mock_documents_get_doc_id_entities_e_id(): + responses.add( responses.GET, f"{PATH}/pta/entities/test", @@ -441,7 +566,6 @@ def test_documents_get_doc_id_entities_e_id(): assert response.status_code == 200 assert response.json() == {"entity": "test"} - # Mock per document not found responses.add( responses.GET, f"{PATH}/wrong_id/entities/test", @@ -453,9 +577,10 @@ def test_documents_get_doc_id_entities_e_id(): assert response.status_code == 404 assert response.json() == {"error": "Document not found"} + @responses.activate -def test_documents_get_doc_id_activities_a_id(): - # Mock per list of activities +def test_mock_documents_get_doc_id_activities_a_id(): + responses.add( responses.GET, f"{PATH}/pta/activities/test", @@ -467,7 +592,6 @@ def test_documents_get_doc_id_activities_a_id(): assert response.status_code == 200 assert response.json() == {"activity": "test"} - # Mock per document not found responses.add( responses.GET, f"{PATH}/wrong_id/activities/test", @@ -479,9 +603,10 @@ def test_documents_get_doc_id_activities_a_id(): assert response.status_code == 404 assert response.json() == {"error": "Document not found"} + @responses.activate -def test_documents_get_doc_id_agents_a_id(): - # Mock per list of agents +def test_mock_documents_get_doc_id_agents_a_id(): + responses.add( responses.GET, f"{PATH}/pta/agents/test", @@ -493,7 +618,6 @@ def test_documents_get_doc_id_agents_a_id(): assert response.status_code == 200 assert response.json() == {"agent": "test"} - # Mock per document not found responses.add( responses.GET, f"{PATH}/wrong_id/agents/test", @@ -505,9 +629,10 @@ def test_documents_get_doc_id_agents_a_id(): assert response.status_code == 404 assert response.json() == {"error": "Document not found"} + @responses.activate -def test_documents_get_doc_id_relations_r_id(): - # Mock per list of relations +def test_mock_documents_get_doc_id_relations_r_id(): + responses.add( responses.GET, f"{PATH}/pta/relations/test", @@ -519,7 +644,6 @@ def test_documents_get_doc_id_relations_r_id(): assert response.status_code == 200 assert response.json() == {"relation": "test"} - # Mock per document not found responses.add( responses.GET, f"{PATH}/wrong_id/relations/test", @@ -531,9 +655,10 @@ def test_documents_get_doc_id_relations_r_id(): assert response.status_code == 404 assert response.json() == {"error": "Document not found"} + @responses.activate -def test_documents_delete_doc_id_entities_e_id(): - # Mock per unauthorized delete +def test_mock_documents_delete_doc_id_entities_e_id(): + responses.add( responses.DELETE, f"{PATH}/pta/entities/test", @@ -542,13 +667,12 @@ def test_documents_delete_doc_id_entities_e_id(): ) headers = { - 'Authorization': 'Bearer wrong_token' + 'Authorization': 'Bearer ' + 'wrong_token' } response = requests.delete(PATH + '/pta/entities/test', headers=headers) assert response.status_code == 403 assert response.json() == {"error": "Unauthorized"} - # Mock per successful delete responses.add( responses.DELETE, f"{PATH}/pta/entities/test", @@ -563,7 +687,6 @@ def test_documents_delete_doc_id_entities_e_id(): assert response.status_code == 200 assert response.json() == {"message": "Entity deleted successfully"} - # Mock per entity not found responses.add( responses.DELETE, f"{PATH}/pta/entities/test", @@ -571,13 +694,17 @@ def test_documents_delete_doc_id_entities_e_id(): status=404 ) + headers = { + 'Authorization': 'Bearer ' + TOKEN + } response = requests.delete(PATH + '/pta/entities/test', headers=headers) assert response.status_code == 404 assert response.json() == {"error": "Entity not found"} + @responses.activate -def test_documents_delete_doc_id_activities_a_id(): - # Mock per unauthorized delete +def test_mock_documents_delete_doc_id_activities_a_id(): + responses.add( responses.DELETE, f"{PATH}/pta/activities/test", @@ -586,13 +713,12 @@ def test_documents_delete_doc_id_activities_a_id(): ) headers = { - 'Authorization': 'Bearer wrong_token' + 'Authorization': 'Bearer ' + 'wrong_token' } response = requests.delete(PATH + '/pta/activities/test', headers=headers) assert response.status_code == 403 assert response.json() == {"error": "Unauthorized"} - # Mock per successful delete responses.add( responses.DELETE, f"{PATH}/pta/activities/test", @@ -607,7 +733,6 @@ def test_documents_delete_doc_id_activities_a_id(): assert response.status_code == 200 assert response.json() == {"message": "Activity deleted successfully"} - # Mock per activity not found responses.add( responses.DELETE, f"{PATH}/pta/activities/test", @@ -615,13 +740,17 @@ def test_documents_delete_doc_id_activities_a_id(): status=404 ) + headers = { + 'Authorization': 'Bearer ' + TOKEN + } response = requests.delete(PATH + '/pta/activities/test', headers=headers) assert response.status_code == 404 assert response.json() == {"error": "Activity not found"} + @responses.activate -def test_documents_delete_doc_id_agents_a_id(): - # Mock per unauthorized delete +def test_mock_documents_delete_doc_id_agents_a_id(): + responses.add( responses.DELETE, f"{PATH}/pta/agents/test", @@ -630,13 +759,12 @@ def test_documents_delete_doc_id_agents_a_id(): ) headers = { - 'Authorization': 'Bearer wrong_token' + 'Authorization': 'Bearer ' + 'wrong_token' } response = requests.delete(PATH + '/pta/agents/test', headers=headers) assert response.status_code == 403 assert response.json() == {"error": "Unauthorized"} - # Mock per successful delete responses.add( responses.DELETE, f"{PATH}/pta/agents/test", @@ -651,7 +779,6 @@ def test_documents_delete_doc_id_agents_a_id(): assert response.status_code == 200 assert response.json() == {"message": "Agent deleted successfully"} - # Mock per agent not found responses.add( responses.DELETE, f"{PATH}/pta/agents/test", @@ -659,13 +786,17 @@ def test_documents_delete_doc_id_agents_a_id(): status=404 ) + headers = { + 'Authorization': 'Bearer ' + TOKEN + } response = requests.delete(PATH + '/pta/agents/test', headers=headers) assert response.status_code == 404 assert response.json() == {"error": "Agent not found"} + @responses.activate -def test_documents_delete_doc_id_relations_r_id(): - # Mock per unauthorized delete +def test_mock_documents_delete_doc_id_relations_r_id(): + responses.add( responses.DELETE, f"{PATH}/pta/relations/test", @@ -674,13 +805,12 @@ def test_documents_delete_doc_id_relations_r_id(): ) headers = { - 'Authorization': 'Bearer wrong_token' + 'Authorization': 'Bearer ' + 'wrong_token' } response = requests.delete(PATH + '/pta/relations/test', headers=headers) assert response.status_code == 403 assert response.json() == {"error": "Unauthorized"} - # Mock per successful delete responses.add( responses.DELETE, f"{PATH}/pta/relations/test", @@ -695,7 +825,6 @@ def test_documents_delete_doc_id_relations_r_id(): assert response.status_code == 200 assert response.json() == {"message": "Relation deleted successfully"} - # Mock per relation not found responses.add( responses.DELETE, f"{PATH}/pta/relations/test", @@ -703,13 +832,17 @@ def test_documents_delete_doc_id_relations_r_id(): status=404 ) + headers = { + 'Authorization': 'Bearer ' + TOKEN + } response = requests.delete(PATH + '/pta/relations/test', headers=headers) assert response.status_code == 404 assert response.json() == {"error": "Relation not found"} + @responses.activate -def test_documents_delete_doc_id(): - # Mock per unauthorized delete +def test_mock_documents_delete_doc_id(): + responses.add( responses.DELETE, f"{PATH}/pta", @@ -718,13 +851,12 @@ def test_documents_delete_doc_id(): ) headers = { - 'Authorization': 'Bearer wrong_token' + 'Authorization': 'Bearer ' + 'wrong_token' } response = requests.delete(PATH + '/pta', headers=headers) assert response.status_code == 403 assert response.json() == {"error": "Unauthorized"} - # Mock per successful delete responses.add( responses.DELETE, f"{PATH}/pta", @@ -739,15 +871,15 @@ def test_documents_delete_doc_id(): assert response.status_code == 200 assert response.json() == {"message": "Document deleted successfully"} - # Mock per document not found responses.add( responses.DELETE, f"{PATH}/pta", json={"error": "Document not found"}, status=404 ) - + headers = { + 'Authorization': 'Bearer ' + TOKEN + } response = requests.delete(PATH + '/pta', headers=headers) assert response.status_code == 404 assert response.json() == {"error": "Document not found"} -""" \ No newline at end of file