From 1c37798c7ab831bb5e239c6de5ff5b0593475139 Mon Sep 17 00:00:00 2001 From: Hani Date: Mon, 4 Sep 2023 19:24:00 +0530 Subject: [PATCH 1/4] write unit test cases for code coverage for 'IterableInAppFragmentHTMLNotification.java' and 'IterableApi.java' --- build.gradle | 2 +- iterableapi/build.gradle | 3 +- iterableapi/jacoco.exec | Bin 0 -> 149715 bytes ...IterableInAppFragmentHTMLNotification.java | 4 +- .../iterable/iterableapi/IterableApiTest.java | 42 ++++ ...ableInAppFragmentHTMLNotificationTest.java | 185 ++++++++++++++++++ 6 files changed, 233 insertions(+), 3 deletions(-) create mode 100644 iterableapi/jacoco.exec create mode 100644 iterableapi/src/test/java/com/iterable/iterableapi/IterableInAppFragmentHTMLNotificationTest.java diff --git a/build.gradle b/build.gradle index e6da3a6e1..214635186 100644 --- a/build.gradle +++ b/build.gradle @@ -1,6 +1,6 @@ // Top-level build file where you can add configuration options common to all sub-projects/modules. buildscript { - ext.kotlin_version = '1.4.32' + ext.kotlin_version = '1.5.31' ext.mavenPublishEnabled = true repositories { google() diff --git a/iterableapi/build.gradle b/iterableapi/build.gradle index 6ae7e8953..3e419066d 100644 --- a/iterableapi/build.gradle +++ b/iterableapi/build.gradle @@ -51,6 +51,7 @@ dependencies { testImplementation 'org.khronos:opengl-api:gl1.1-android-2.1_r1' testImplementation 'com.squareup.okhttp3:mockwebserver:4.2.2' testImplementation 'org.skyscreamer:jsonassert:1.5.0' + testImplementation project(path: ':iterableapi') androidTestImplementation 'androidx.test:runner:1.3.0' androidTestImplementation 'androidx.test:rules:1.3.0' @@ -83,7 +84,6 @@ ext { allLicenses = ["Apache-2.0"] } -apply from: 'https://raw.githubusercontent.com/nuuneoi/JCenter/master/installv1.gradle' if(hasProperty("mavenPublishEnabled")) { apply from: '../maven-push.gradle' } @@ -97,6 +97,7 @@ task javadoc(type: Javadoc) { tasks.withType(Test) { jacoco.includeNoLocationClasses = true + jacoco.excludes = ['jdk.internal.*'] } task checkstyle(type: Checkstyle) { diff --git a/iterableapi/jacoco.exec b/iterableapi/jacoco.exec new file mode 100644 index 0000000000000000000000000000000000000000..290c39d0dbfd83deb0c2d57c422f3511b5001bb5 GIT binary patch literal 149715 zcmeHQ349dg*`G}?;Sw@{h$pm-P&s7ho&htzXysD`ErM7hiipLc>FoRNCY!Kfm)(ikNuKSm%m#nE zY-XST|9zhSbGfRkGtx7tUZbul96u>9e^B`zJm@&}h|RkDKZ@Ro+ z>f*zX>#L}~O}_agwfv?+eHJn*yr`^aD&?l`l@S~DG zuz%{#4I8Iyj=@^+@005z?_&KPsr0a=LoWLFB^Ruw21NZ7)(xD9ZmvM({tGUmDEieg zYj)keeA;ak7xk5a&rj~(M5H6n98}icMSnIqC9Bhp$=Q?`_4N_IhV*%~H{+J+^S=D( z3Tj}~w+MdnfCata+IOn}$ zHn^2(SJNFoK5=SscDo%hS|V#9_0+x3XS?=%w&*NhT%KuqJtZjk>GR8cIw~*smGu}w zwWmL<>{0dk)@1`?`04^I5dgZc#QLPqY+LNjd?TmoEdOlC{u`O@z}j1%aQ)DgKKgF% zrf*9NHd9%#-KGiFhql-$b5fb7*dnRWH2|h|o$0yjZ$Ey|tv}pD4Qle;DlCH^JPOV6 z=xA(-=?mC@dAE;iQ|T$k-|`O5x_@2_Mu@+#9IvlD2<1%1GXEYMgl|Ti`uuGFcIaN< z*9U)-dtB*MY1&60_RjlmI7Z>TwJv)7MO*GWmU^&n4AzK=%;Ey;rab4`bJRtk+~Cg( zDSFVyd)k|O`Yxrio4hfRK3pb&i>JKQtIjP%e!r)rxZLHUzmZ;~zcJrG7SFv3P~PzV zhwm7+;ks>S`JU=Oj^?|i(E8Rbi))uk-_^4QRWI6dEj1*LUnILY$LIBxS-(K`FY=xo zcg5aYC?>YI3H!1wvRJmY`{X5gLwZuVQD5Eo_zS=%v+-XNLObtL`io^JcMUnOPi-uC z^83p?nq|$tl7Jm|KG2o!a9{QI8#}TGsVk#?tbo}EPU0Jis}JO55Nn~)BPY%MFZ};s zsliRYKCdJahjd(+LKl|z5&0#(tEm1>zBu%iTI`?W^`H{}b16l;M_`Nnm$HVUUR<*$ zZ@~|TYEPf4J@svEcZz!a>zBVix^TGbw2K}s7X5tWm_0qHp|QMukmNG!Sg!7{JvD6T zOYe_<{x>mv@r}Qw)HM2U3s&>n<*FfPxkI5}GRIe9dWti%D?DBUm0i}avbHz9Jb%;V zzi)Zv))>y|aEb_#+y%cMMbYKO8*jU!cQ-c1!$IgcKZWkQa=Wl(;Fw)8+`Q|M|5v`U zo2*Y?fKFCB z$j$6pyO4g}{GV+P9lEqQZjUQK)vgUsSAAQ3|5MZzQUB!tP`V9FO`#uL@Rx_CKhSMs z3`b;uk|$PJH?T~qC45YOn>#)I_012w5QmS)bZ}AhqaUy8IeyH&AH;AVWL5ZQ1zq^T zuFjPd)q(!{RN8mle>2QDSG%)ryWfIoXX^u8`Q@XV?!MuEA9Z!qTNdz+LnyGG2R@I5 z!#N%c1|6+)w$ph24gT(vU&QcX8Y3`3o+&Bx$=j}ds=9sY^cW$)S>O1(gPxu~%r&(~ z48)3&SLfIU%w^T-MZX?@^7{*7vbp-xD$^p^GIDTPC(imcQ+wVws%BFx-(bKyE-&?! zl%v#i+I7A7M^UNW6a#o0kjB^&>!VCSe$}w<+C?cA$I}~Lxo!N^HF;m1<;IOcs@Jj~ z5?^#p9ZB!)*X^l6f6Sg1!#`i|)i)yEz)jpRt*q+}-~a2?7|@>U3%GVa@5XK)Rz&B@ zn{o@~eJ@c;EQS}6_$^1ZYV{HS$@~$oQ+(8S8HG%>!t2NPa*SGHSn;9du}t5-uk_?l z|J-;y9?p9Gb~UeX;^$i5m~1Fgq-ec~tBK%WH#OeS3>63c{%2?4YsJ_XAoy=W@qpkT zg8w=V27-SG{vr4eYQT*=h`>Pv!9N85UZh%y0to&g_=n)X37;|)@j~zq!M~-+Lhx_p zmaysy!GE&`|GX&~oQT{w=$|v|2)QnOCT__vkPPS$r<{LT;KflNm9FNjEU-zb+-@aR z6UY^`JLd|A6iTLC)JQ7{c)N@0Lb*m#l@wK7TRG-Um-U3A0t#O6(LI4gL~0-r5r-H@ zQJ2n0Ttt3IG4&D`K5}l7LZAa1y*c=i_+x>)UN>UUVQ&l7lVK%v) zTSczQggAL2?Bj-PIO5f=G=nQ8!HC|KNXFpYtD7R;jOz4Af1Qyi_7lzBzjyCGqiXk_ zFR0;BrF5(&Tx*pk%(k*fV}=b{dhU$9sB@0myqk}Gk7N>t~Mi{KKgM))x+ zjpAmxrig}NI91|I4HL}$H$6nIH-8O86IA2^!b>Q&#CQrzO#;z`%a%YiO%(RlS5LHI zqTvxI>XJz+q#r2yGbWk`jDpR zBqGSH9Bn!AR!hdp@jnBi0ipq-0irqAE{i6KMl&UokvSd{%$n0n@pY}GUBy?54H`|8 zopDxS>c zYV!);-CWq<6{nCMX~FKtux(4Ja^HUwNpEHJ*8gw)=)MCn&7InS`faIz+BJ zKXp#Tk=XP?Xm<#lrDv4C1+vxl#k6S$L4kn`SJbANSZWMK)=&h4G=1jji=W4QrT=a@ z@D=!1Hu_ho?E}gXD8|V!bJo!rd4;ob@`ixhN%PRUJ(#c1<3F|Wbfpe{kMkShS|;Tj zJ_PM8`Ls9Is<0KzU{p{%=TdC7#q-|)s|D>Xd9^pjEonN-OT=1F(fB&6-4W+-A;hvx z2)oJCx(~6eHQXJlHKP^(#}doRn&uXG0at70%|87y=BtwShv1ZjQ#OfDSyn{6Tg5ed zc_lNgEk;%~LCK9P!D_*3lhUR7npk~q9%`!-^=Z#@FOA?SuS>V}Z#0N>D_Jcr_T zo4SB!Dda7>@k-2Bi^?j1uYj+Bub_@3+3H9XRgpE-Abhp-;s3*YwRH1hTcd&o84mJ; zbNCSQ9LbmGP&h$Hj7h4WRy;fmTkVR+9(ImZfnp4baj3L5avNr|EOm>kO9%STg1&3y zFUne07_DEHQ#3cnF{CHr!A(EGeD&a#MR3Z3_LjWb8z&>hFeN$+rEoP~L6^>@qhbxpf#oWoZMEPYPy$?A6YBt%jZ4(y-0bHm0d zo6mV$wk_lRTGFd_bd{}S3P^YBI!5ZBM%0k&SQQW(@Cw5hDg5XIJpr*nC$D}NflYC+ zqY=xQiYg(E)M~DO=_p3*`Zqsz4q`!h1LX~rH_HYhs#%RYN&~y*hZ=j=)SP!Qbyd_1 zd4ti(Nzo_GA*9GGBAJ4xT-lhfo;v>%;49EDK*InHBiS?z-V}5};E0B?t!y;rt8EK> zz*oRmz*oRm$;MZb&Z>&Sk{r>8^9N(T`tbMF&fzQ2T!V9Dn%+hk+H}xBd2Qjfx1hO3 zI1pp$USmarQ4vcbqC@@qV#FRA_=R&23l(|ESCPk9!^l)!BR!FaT_dqo9ZnwzRu$5k z!qxnGO&KO2`%2Q9+%uqPEGG&g=~+DT-OsR9ecB0jZAUVy2I5;LV@-lfX5)g%03?_h z1$CZYUpdOs{T^S5)755PWfhiT$SHV~n}zx6=m-Z*xr1m3qM-QZ~3mdf0+YT1y&WmRf&ut>AZn!#qwwFNW)e&^Nqc> zc9oOKXbm&}h2?!jeo5~ts(&mlWscW_O8n2IxEj1k(?8Hxe);I8yKlJPM_nDu+h>jq zM!%N#npNSSm0#wY16}3G+f^>QB{#B?9@tvd&7H{g*>Sd7E!hLoz{Y@UfotRETEu9E zB)IW@$b4?T_2>?OpfpS^MwAhuOyHzGwji_QTo%uY@E*{Re)*ky`$nP&eY z@5ymj?7f9zqE->&W}%$c(}vC7H6|_rs3rH%9Y;eA z$&AdY5@~l>+rKM@+S*}#!F_Hjdml;(z%Zyr@o>|lF2Q{^J#F26<^{xYteYgSHnb-- zm>VvB2F`Q`H)X+@o&<_mug1lbx+XJ%rjg2yjYC%8GkxQ@4*+TawWzYqkG>Pl(eAUZ zB2$(aqB?KtJPt!`Q?KCwHGmpGtw+`8TbB)>21LCRv5b3=IN5Er1&|8kNALPy47Epp z|BiDa)mG~*!CK3pSWrI@2-aE(gH%Owi`JevjMT>_?k896FU~pu+kwnHs1%>}O2<(9R6Yby1E>MiY^~>344y$sP!Rm|fo}=a zsy6`C0BQg=TcM^SN#snOM5>=%vkOD*vzey>Y5+BWnypadRkzAZlAA#6lpkOJdD$Hx z3V|rpe4NWd)m9JP znU4(mZwprQ+vTbu)ZnPrL8j_wW> zFw}ZXhQghYEUcC`+RGYp$y)uwT075ll6inQ{TFREPB%4~*Cd%#6I}R2@cMV5InVUK z5f5j&+E~6trdr|kW0`x5T4H!nS-eYfTI~upXIB$g4eA1;;)S&ZyY@P!sBZ?K%v#{=)duoBSQA!b`dCk!o5&!~19_fg z%kwCzY4T)^N5Q~M4~E*noMzR80BVtvNRfA($F-vzr(-005yP`tx)3-FY>%5V2CmQ zwUb<*%LS+b)BtL>LQNBNBq$<_A;yj-kYkNs0@MI%05w~oW(aP{6hy;*@F$7L|h ztq?-0qh67PREKYNT@JtQNq929fyR*;=(nj_eZ+qrK4Ajo9YlnSut4|N2)XK=>BMcXlC3o=UyNh=!dtlA0Be5ZM}ES;0*(52KH!X zYE>B~53^+`1W0d~`L2+LKhXN@8%TLHR;_(=;4iXfG>@W-SxOo6u%FdH-kv zwb!0+ZU+w}F_6UePu;m;ZD}1q#!X!G|%>Z6%);~(^w!H zkis>I8zb^k-_vDO3XpYxcj%^R2fzyS-~A|?83~n zqqYYyQ@9HR%+zRt5BBB5V1P>kY#-MZ-kJ%_6ibOl*jM6oW)fs;ETEv`5u7t#DflyH zrdMuywe2tyxIn??sxYgzooIrRS7?4emfRD{^g>}&NVIR-fl8`wIo9gn5g3I8XS?&h z!$h;&f2VVZ27;^56mdIUka{sZ_^icUkhofO9i0kd3{emj6*nNv*s~7r)|j#9jei3+ z1LMaUDC$W+4H&Q)3@OQEw63)aX-8v|H<7Ay9FYz_*}Vv#n}_@Da}G9Jt%KdHP_rs6 zTh>92>ll1>%$i+yFQ0Z>4DAN_73H&tx;GK&$TJ6(wRbrisv^rG)nv#qdc;?Np?c)G z>zxDDM7R5biyU&1A^$)Un@QA_lqff~;jSi=7ez(m@c_3OH3dbO*=m00p!@|mA;CcA z+A_6xHY)LtnKU8yI-lQTdUQ3AVq5Q`4&n&B(FAI6LW1#xLv%I;@q)mKSkNyT_z~W} zQ8Xy~HE=>8)ku!yrVY6{GXPKofjbt#35jTaqheV#F7v!(7 zfZ2fAN=xf^JhjNs2Vl0inawm5191%AZC=EV`zdBNwz8XZm<_@!2(Q9Vq2Tk@r}0|j zwFMa=Ja(ao%iE2i%H6jOpxSs-(~sWxY-c(K?8&g`~GG}qwhig2VUJj(|q zNM&?SjBVwvN1Wq>0JZ_!jauF7yEWPpwtJ4crf~eEy!=7g6SF2WqA&=tnqcA{$)aIX zhht<9TPXpup%1IU$qlD=EMppys{dKnOvzwaB5jVicnQY#h=M-O!FH>S{B3uzt^Jf? zj~12~p^qzkvQ#XEhP2njA)^{{tg+q;L|D+WL^XJOI+V2O!m9NR<@T&5;o!MI&#$oFH}cL_limLn_t1 z4yM)RE%T7j3zg8q)M|`d~E}F8rE+ z%FESal!<@HF3$0JePu3(j|ehFL($b>q3f!vc3{M=;+DhN+KSFrKx|-=dn-Y#qD!hM zNaT!tXk|4<>_cnc2E+nlJ%h)FA6TJ$Bc7g}@PO*L@-}eB3WEgBnrUwYC*Z76CSt^{ z8Geb=*Y(=BII(4H^Racu)~|hi{)DqFCkn#pNADf&`0vK z4=a0AeZJMU zH{J49%NmN?lo9ENdGf0ZG1oqM@>Z}~u-brsNXj~s&1xk%H^^fJ4$? zZ`XIrbFIK2gBJ}_?C|t!9kBI2z0*PCr9tflwKwSDhu7q_TD4a<6izS=2_yBH?<&dl zhrys7fp!GiQIHEms#!}URL`h=U<6GwHC4k%JutE(x#p|_!~$Xgu}Of~GfE&-w<^mV zIbuHuZ`>$aP`N` zK80^%!1mE@u`wfAd#oY1l=vWPXrnf<=K z8-JHS@4pzxw-u3Z?XgJteFp|Oj)!3qR6%h(s|Z#j7DjrGyqR1*_uI(c)WJyh2K6Kn z(#iFs Date: Tue, 5 Sep 2023 16:55:43 +0530 Subject: [PATCH 2/4] added more steps --- iterableapi/jacoco.exec | Bin 149715 -> 171058 bytes ...ableInAppFragmentHTMLNotificationTest.java | 34 +++++++++--------- 2 files changed, 18 insertions(+), 16 deletions(-) diff --git a/iterableapi/jacoco.exec b/iterableapi/jacoco.exec index 290c39d0dbfd83deb0c2d57c422f3511b5001bb5..04acee5d0f727598b0b0465f14d9f3beb60a891a 100644 GIT binary patch delta 1107 zcmaizO-L0{6vyXI5%l~@Bj2yVn7sTSGJii~W z1?|`q9!;VYLK_m(&X%yLTn&+?6N^y;`x5CY{smg`7SQ02t=0xpbLe@HVrp}uEY_mw z*s*<+Q%QF3zYBU{DdTvJ!gn9hOQd<6I@EWFJbX}e1GKUOu8k}&Ef*<@(!N;r2_Xig z?LFTw9li0{6?b=|#MER-DHb~QAZPE_^}znPxPnd3(6eo9+QOE^6TA)7YX)-du$gJP zRB$FR^geiLQF;fTw&e0=28B)6i3vwq#_A7&PAIvoWwChFVU#=BIV2l~q&4|sJe%$R ztp}qYP&`U}fz23} Date: Wed, 6 Sep 2023 12:38:39 +0530 Subject: [PATCH 3/4] add assert check --- build.gradle | 2 +- iterableapi/build.gradle | 2 +- iterableapi/jacoco.exec | Bin 171058 -> 178710 bytes .../iterable/iterableapi/IterableApiTest.java | 27 ++++++++++-------- ...ableInAppFragmentHTMLNotificationTest.java | 10 +++++++ 5 files changed, 27 insertions(+), 14 deletions(-) diff --git a/build.gradle b/build.gradle index 214635186..e6da3a6e1 100644 --- a/build.gradle +++ b/build.gradle @@ -1,6 +1,6 @@ // Top-level build file where you can add configuration options common to all sub-projects/modules. buildscript { - ext.kotlin_version = '1.5.31' + ext.kotlin_version = '1.4.32' ext.mavenPublishEnabled = true repositories { google() diff --git a/iterableapi/build.gradle b/iterableapi/build.gradle index 3e419066d..b8f3c5004 100644 --- a/iterableapi/build.gradle +++ b/iterableapi/build.gradle @@ -51,7 +51,6 @@ dependencies { testImplementation 'org.khronos:opengl-api:gl1.1-android-2.1_r1' testImplementation 'com.squareup.okhttp3:mockwebserver:4.2.2' testImplementation 'org.skyscreamer:jsonassert:1.5.0' - testImplementation project(path: ':iterableapi') androidTestImplementation 'androidx.test:runner:1.3.0' androidTestImplementation 'androidx.test:rules:1.3.0' @@ -84,6 +83,7 @@ ext { allLicenses = ["Apache-2.0"] } +apply from: 'https://raw.githubusercontent.com/nuuneoi/JCenter/master/installv1.gradle' if(hasProperty("mavenPublishEnabled")) { apply from: '../maven-push.gradle' } diff --git a/iterableapi/jacoco.exec b/iterableapi/jacoco.exec index 04acee5d0f727598b0b0465f14d9f3beb60a891a..3ad62fd04c1b90d92bf4f555a887cd2f95690f7b 100644 GIT binary patch delta 339 zcmdnAgKOFsu7(!IElfspCQrC6A(3KiYG#?3X3D_8*p;cgW(t^A;VInCn8kFJk%@s} z`UgM8uE__KO1ICiVPavFFfunyGOV@Fe4T#_0i`Od{JILYZbT0swrSZP)++ delta 13 UcmbRCg=^Ceu7(!IElfsp04ul!k^lez diff --git a/iterableapi/src/test/java/com/iterable/iterableapi/IterableApiTest.java b/iterableapi/src/test/java/com/iterable/iterableapi/IterableApiTest.java index 86a32f90b..ef049ab5c 100644 --- a/iterableapi/src/test/java/com/iterable/iterableapi/IterableApiTest.java +++ b/iterableapi/src/test/java/com/iterable/iterableapi/IterableApiTest.java @@ -720,7 +720,6 @@ public void testTrackInAppDeleteWithNullParameters() throws Exception { assertNull(requestJson.optString(IterableConstants.KEY_INBOX_SESSION_ID, null)); } - @Test public void testFetchRemoteConfigurationCalledWhenInForeground() throws Exception { @@ -748,34 +747,38 @@ public void testFetchRemoteConfigurationCalledWhenInForeground() throws Exceptio @Test public void testSetEmailWithAuthToken() { - IterableApi.getInstance().setEmail("test@example.com", "authToken", mockSuccessHandler, mockFailureHandler); + String email = "test@example.com"; + String auth_token = "authToken"; + + assertNotNull(mockSuccessHandler); + assertNotNull(mockFailureHandler); + assertEquals(email,"test@example.com"); + assertEquals(auth_token,"authToken"); - // Verify that the method was called with the correct arguments + IterableApi.getInstance().setEmail(email, auth_token, mockSuccessHandler, mockFailureHandler); verifyNoMoreInteractions(mockSuccessHandler); - // Add more assertions as needed } @Test public void testSetEmailWithoutAuthToken() { - IterableApi.getInstance().setEmail("test@example.com", null, mockSuccessHandler, mockFailureHandler); + String email = "test@example.com"; + assertEquals(email,"test@example.com"); + assertNotNull(mockSuccessHandler); + assertNotNull(mockFailureHandler); - // Verify that the method was called with the correct arguments + IterableApi.getInstance().setEmail(email, null, mockSuccessHandler, mockFailureHandler); verifyNoMoreInteractions(mockFailureHandler); - // Add more assertions as needed } - // Write similar test cases for other method variants - @Test public void testSetEmailWithSameEmailAndAuthToken() { IterableApi.getInstance().setEmail("test@example.com"); - // Mock the _email field when(IterableApi.getInstance().getEmail()).thenReturn("test@example.com"); IterableApi.getInstance().setEmail("test@example.com", "authToken"); - // Verify that the method was called with the correct arguments + assertNotNull(mockSuccessHandler); + assertNotNull(mockFailureHandler); verifyNoMoreInteractions(mockSuccessHandler, mockFailureHandler); - // Add more assertions as needed } } diff --git a/iterableapi/src/test/java/com/iterable/iterableapi/IterableInAppFragmentHTMLNotificationTest.java b/iterableapi/src/test/java/com/iterable/iterableapi/IterableInAppFragmentHTMLNotificationTest.java index e47e28a74..95d9b4e20 100644 --- a/iterableapi/src/test/java/com/iterable/iterableapi/IterableInAppFragmentHTMLNotificationTest.java +++ b/iterableapi/src/test/java/com/iterable/iterableapi/IterableInAppFragmentHTMLNotificationTest.java @@ -116,6 +116,15 @@ public void testOnCreate() { when(mockBundle.getString(eq("IN_APP_BG_COLOR"), eq("Color"))).thenReturn("Color"); when(mockBundle.getBoolean(eq("IN_APP_SHOULD_ANIMATE"), anyBoolean())).thenReturn(true); + mockBundle.putString("HTML_STRING", "Your HTML String"); + mockBundle.putBoolean("CALLBACK_ON_CANCEL", true); + mockBundle.putString("MESSAGE_ID", "Your Message ID"); + mockBundle.putDouble("BACKGROUND_ALPHA", 0.5); + mockBundle.putParcelable("INSET_PADDING", new Rect()); + mockBundle.putDouble("IN_APP_BG_ALPHA", 0.7); + mockBundle.putString("IN_APP_BG_COLOR", "Color"); + mockBundle.putBoolean("IN_APP_SHOULD_ANIMATE", true); + htmlNotification.setArguments(mockBundle); htmlNotification.onCreate(mockBundle); } @@ -154,6 +163,7 @@ public void testSetLoaded() { @Test public void testOnStop() { + assertNotNull(orientationListener); Mockito.doNothing().when(orientationListener).disable(); htmlNotification.onStop(); } From 0ff6e57236149da97d25217c0ea7b3df6746ee54 Mon Sep 17 00:00:00 2001 From: Hani Date: Thu, 30 Nov 2023 19:02:44 +0530 Subject: [PATCH 4/4] resolve code style build fail issue --- .../java/com/iterable/iterableapi/IterableApiTest.java | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/iterableapi/src/test/java/com/iterable/iterableapi/IterableApiTest.java b/iterableapi/src/test/java/com/iterable/iterableapi/IterableApiTest.java index ef049ab5c..064e57d23 100644 --- a/iterableapi/src/test/java/com/iterable/iterableapi/IterableApiTest.java +++ b/iterableapi/src/test/java/com/iterable/iterableapi/IterableApiTest.java @@ -748,21 +748,21 @@ public void testFetchRemoteConfigurationCalledWhenInForeground() throws Exceptio @Test public void testSetEmailWithAuthToken() { String email = "test@example.com"; - String auth_token = "authToken"; + String authToken = "authToken"; assertNotNull(mockSuccessHandler); assertNotNull(mockFailureHandler); - assertEquals(email,"test@example.com"); - assertEquals(auth_token,"authToken"); + assertEquals(email, "test@example.com"); + assertEquals(authToken, "authToken"); - IterableApi.getInstance().setEmail(email, auth_token, mockSuccessHandler, mockFailureHandler); + IterableApi.getInstance().setEmail(email, authToken, mockSuccessHandler, mockFailureHandler); verifyNoMoreInteractions(mockSuccessHandler); } @Test public void testSetEmailWithoutAuthToken() { String email = "test@example.com"; - assertEquals(email,"test@example.com"); + assertEquals(email, "test@example.com"); assertNotNull(mockSuccessHandler); assertNotNull(mockFailureHandler);