From b78e1b8be4e6247c5c6604f1a6eb857a06c9822a Mon Sep 17 00:00:00 2001
From: Emmanuel Viennet <emmanuel.viennet@gmail.com>
Date: Mon, 23 Oct 2023 23:50:13 +0200
Subject: [PATCH] Add unit test for sco_find_etud.

---
 app/scodoc/sco_etud.py                     |  49 +++++++++++----------
 app/scodoc/sco_find_etud.py                |  46 ++++++++++---------
 sco_version.py                             |   2 +-
 tests/ressources/misc/ImportEtudiants.xlsx | Bin 10458 -> 10516 bytes
 tests/unit/test_etudiants.py               |  18 ++++++--
 5 files changed, 66 insertions(+), 49 deletions(-)

diff --git a/app/scodoc/sco_etud.py b/app/scodoc/sco_etud.py
index 21b5859922..a28728578f 100644
--- a/app/scodoc/sco_etud.py
+++ b/app/scodoc/sco_etud.py
@@ -219,26 +219,27 @@ _identiteEditor = ndb.EditableTable(
     "identite",
     "etudid",
     (
-        "etudid",
-        "nom",
-        "nom_usuel",
-        "prenom",
-        "prenom_etat_civil",
-        "cas_id",
+        "admission_id",
+        "boursier",
         "cas_allow_login",
         "cas_allow_scodoc_login",
-        "civilite",  # 'M", "F", or "X"
+        "cas_id",
         "civilite_etat_civil",
-        "date_naissance",
-        "lieu_naissance",
-        "dept_naissance",
-        "nationalite",
-        "statut",
-        "boursier",
-        "foto",
-        "photo_filename",
+        "civilite",  # 'M", "F", or "X"
         "code_ine",
         "code_nip",
+        "date_naissance",
+        "dept_naissance",
+        "etudid",
+        "foto",
+        "lieu_naissance",
+        "nationalite",
+        "nom_usuel",
+        "nom",
+        "photo_filename",
+        "prenom_etat_civil",
+        "prenom",
+        "statut",
     ),
     filter_dept=True,
     sortkey="nom",
@@ -300,7 +301,7 @@ def check_nom_prenom_homonyms(
         prenom = prenom.lower().strip()
     # Don't allow some special cars (eg used in sql regexps)
     if scu.FORBIDDEN_CHARS_EXP.search(nom) or scu.FORBIDDEN_CHARS_EXP.search(prenom):
-        return False, 0
+        return False, []
     # Liste homonymes (dans tous les départements):
     query = Identite.query.filter(
         Identite.nom.ilike(nom + "%"), Identite.prenom.ilike(prenom + "%")
@@ -351,7 +352,7 @@ def _check_duplicate_code(cnx, args, code_name, disable_notify=False, edit=True)
                     parameters = {}
             if not disable_notify:
                 err_page = f"""<h3><h3>Code étudiant ({code_name}) dupliqué !</h3>
-                <p class="help">Le {code_name} {args[code_name]} est déjà utilisé: un seul étudiant peut avoir 
+                <p class="help">Le {code_name} {args[code_name]} est déjà utilisé: un seul étudiant peut avoir
                 ce code. Vérifier votre valeur ou supprimer l'autre étudiant avec cette valeur.
                 </p>
                 <ul><li>
@@ -566,7 +567,7 @@ admission_edit = _admissionEditor.edit
 
 
 # Edition simultanee de identite et admission
-class EtudIdentEditor(object):
+class EtudIdentEditor:
     def create(self, cnx, args):
         admission_id = admission_create(cnx, args)
         args["admission_id"] = admission_id
@@ -979,12 +980,12 @@ def descr_situation_etud(etudid: int, ne="") -> str:
     cnx = ndb.GetDBConnexion()
     cursor = cnx.cursor(cursor_factory=ndb.ScoDocCursor)
     cursor.execute(
-        """SELECT I.formsemestre_id, I.etat 
-        FROM notes_formsemestre_inscription I,  notes_formsemestre S 
-        WHERE etudid=%(etudid)s 
-        and S.id = I.formsemestre_id 
-        and date_debut < now() 
-        and date_fin > now() 
+        """SELECT I.formsemestre_id, I.etat
+        FROM notes_formsemestre_inscription I,  notes_formsemestre S
+        WHERE etudid=%(etudid)s
+        and S.id = I.formsemestre_id
+        and date_debut < now()
+        and date_fin > now()
         ORDER BY S.date_debut DESC;""",
         {"etudid": etudid},
     )
diff --git a/app/scodoc/sco_find_etud.py b/app/scodoc/sco_find_etud.py
index 8601586f65..3b9cd1305a 100644
--- a/app/scodoc/sco_find_etud.py
+++ b/app/scodoc/sco_find_etud.py
@@ -99,6 +99,26 @@ def form_search_etud(
         return "\n".join(H)
 
 
+def search_etuds_infos_from_exp(expnom: str = "") -> list[dict]:
+    """Cherche étudiants, expnom peut être, dans cet ordre:
+    un etudid (int), un code NIP, ou le début d'un nom.
+    """
+    if not isinstance(expnom, int) and len(expnom) <= 1:
+        return []  # si expnom est trop court, n'affiche rien
+    try:
+        etudid = int(expnom)
+    except ValueError:
+        etudid = None
+    if etudid is not None:
+        etuds = sco_etud.get_etud_info(filled=True, etudid=expnom)
+        if len(etuds) == 1:
+            return etuds
+    expnom_str = str(expnom)
+    if scu.is_valid_code_nip(expnom_str):
+        return search_etuds_infos(code_nip=expnom_str)
+    return search_etuds_infos(expnom=expnom_str)
+
+
 def search_etud_in_dept(expnom=""):
     """Page recherche d'un etudiant.
 
@@ -111,21 +131,7 @@ def search_etud_in_dept(expnom=""):
     Args:
         expnom: string, regexp sur le nom ou un code_nip ou un etudid
     """
-    if isinstance(expnom, int) or len(expnom) > 1:
-        try:
-            etudid = int(expnom)
-        except ValueError:
-            etudid = None
-        if etudid is not None:
-            etuds = sco_etud.get_etud_info(filled=True, etudid=expnom)
-        if (etudid is None) or len(etuds) != 1:
-            expnom_str = str(expnom)
-            if scu.is_valid_code_nip(expnom_str):
-                etuds = search_etuds_infos(code_nip=expnom_str)
-            else:
-                etuds = search_etuds_infos(expnom=expnom_str)
-    else:
-        etuds = []  # si expnom est trop court, n'affiche rien
+    etuds = search_etuds_infos_from_exp(expnom)
 
     if request.method == "POST":
         vals = request.form
@@ -256,8 +262,8 @@ def search_etud_by_name(term: str) -> list:
                 """SELECT nom, prenom, code_nip
                 FROM identite
                 WHERE
-                dept_id = %(dept_id)s 
-                AND code_nip LIKE %(beginning)s 
+                dept_id = %(dept_id)s
+                AND code_nip LIKE %(beginning)s
                 ORDER BY nom
                 """,
                 {"beginning": term + "%", "dept_id": g.scodoc_dept_id},
@@ -274,8 +280,8 @@ def search_etud_by_name(term: str) -> list:
             r = ndb.SimpleDictFetch(
                 """SELECT id AS etudid, nom, prenom
                 FROM identite
-                WHERE 
-                dept_id = %(dept_id)s 
+                WHERE
+                dept_id = %(dept_id)s
                 AND nom LIKE %(beginning)s
                 ORDER BY nom
                 """,
@@ -350,7 +356,7 @@ def table_etud_in_accessible_depts(expnom=None):
     else:
         ss = ""
     H.append(
-        f"""<p>(recherche menée dans le{ss} département{ss}: 
+        f"""<p>(recherche menée dans le{ss} département{ss}:
         {", ".join(accessible_depts)})
         </p>
         <p>
diff --git a/sco_version.py b/sco_version.py
index 7c6b9fb4ba..9f9457694c 100644
--- a/sco_version.py
+++ b/sco_version.py
@@ -1,7 +1,7 @@
 # -*- mode: python -*-
 # -*- coding: utf-8 -*-
 
-SCOVERSION = "9.6.47"
+SCOVERSION = "9.6.48"
 
 SCONAME = "ScoDoc"
 
diff --git a/tests/ressources/misc/ImportEtudiants.xlsx b/tests/ressources/misc/ImportEtudiants.xlsx
index 142cab00bed28c40b4972317eae64aed0e4e7800..41deb06a51f6bbe50c5da9f9097055dc9dcf4b0f 100644
GIT binary patch
delta 4260
zcmZ8lbyO2jxZmjRZe)bChzukhFp<;|qKJTW45V}*Fr-0{*id4`Xz5OAr3D<|hNOUm
zATjBDdft2I#r@}t-*?Zs=iGCD-|xHMAE05?7gS^w4_TE_KxzS`$&5T(a8CUd(}_C4
zcxGIOOeYhnO%icDo1Jv;$+Kia`f|N)%vHx`{dKglDdhwA{tkTq+Wf06K0cYDo^)?(
zT2T?5BPyR}Lvyhe`v}iU)|WAu@ABz2kb44H=coDky*mpGtNNPi5(g`Q&~gzpuW2L~
z-gVT#V&_AgLA4<=RW|)KHsq6kp~~<tjISvNLF7}vC@YH$Go+n1D#KWJ$Uap`^t}^x
zUO+e9c$M#iDgMXl+awPaE+)-9ONVHL1ayQ)d}-q_3iKudg}5zjXf>zbqO{_9(6K2=
z0{%ByQALG*vYUC?hiOumkDzvTP0=)50-JQBZ#6GW&|L$`+8E_-@%;)Ug-rO_`T7CT
zQIQ7c*1!TGYQ!p8`u-XJ(QU?d{^GiIwE2{Z6ugjTg{@o(WyqDS>mnHCHVPK=Gfm)m
z5Sj{uL{SM={tUv}0U9)A>F_Ds8Wy<FNsGx&$uf>t7%8S2-$5i8;;XDGBuqv5_+8W%
zw!>EJB@fbChiYMxNidsQRursk;rmEJS^GHan9zv3(hiYs_-Y@67<4<iyb&l=+-v-d
z6RNr!!jc2Ys3Op%Fcf~Va3+3oYIrGU%8xta#2Ez6k8iehn5cmT4*EUi1sLW+jh8Nk
z#Ll`wh)COd!J@cZVm4C~kh>*}pJx=ZYIL2kAf$)RA#^#iTmXM4m@#_jM%oC*S1kTH
ztsuTwm)?#Tnf#cr-ty`wii`x)^iDN7T@4~Icl6Cyvl%h+T9!WW8}ka>Ro5ll_+tU8
zYx%oVI73K(al6p#hcl|R><6C|(`|8G2j7gF?m^!&{8l;z<-?iVQm;$lXnzwb{plq3
zNRXS|oU8RpaK=-XuIGweZ}sGMXG0*YO(vvEZ=oRz{GaZ5RhumDIV2HI391@TEI)0C
zCvGz}m8smMcbOk*=Z|hQvZ&h}-0*qEei4P~9VSH8KE9CampYDO54yXy^5Sm)-pTaA
z^xKYN)aF^6!`-#z;o)T)JSu07<s46DS3)a{`C6IMd`2gD2}C86vQew_t_i*)0RXO{
z?$8M$S#K~!?j$3kr{+;ckEzJvuw0lZD5)6`+m>3VY9t*1$7`As^#b00<U(!1cm$&}
z^-e~QM+o*Num#Uo=4#1*aO=7%nTEq*e(twF;NKAOV`pHn<B#z1tozW3yxC+qulH3p
zQ}fYbc(+aourMLf{O1h)L61qi=$D~z?{*~B`Ba6J7k})iR}7@8v~0ug=>g<dt&7p*
z>p!Ebj?4`~@qXZx4gZ$%_9x%$27lhqFRUo5E-w>#)m>PGcAs#*&l(I$rFmj0_A%82
zHV7Ei_S3!-vC2j&nm~KSii_=PW5&J0S`FlaJ`b02ygmc@6ndi|bgt^|&UvCaOQ}c~
z(-oNpg-z*tF*gN+=5}7?u`i>7Gig3peB2RciLyUB>4%;32WIQB;!nI$M57g2gUTNv
zT~LesG}*xT={Xg<3h&o>2AqCDLM>Yzw-^rbl0VcB3b5ool^Iz2=VImOR{eJn-Dmmv
zNok3472g}k{CC^U(dKWf26#Hee&r!C?8in12qkfs01qzZTNA?;l4`kY`*V>orW@~d
zq+VR6aOfq^?IQHu12_WfhR}Y|UW?Tx(>rWI!VRMdm`6Ovz5Iu>Vv8>pdjY@$RRUG?
zj+4uk9b@GEn7OTqJ>NVmI2_?-1<hiYh3#<@1rOD9V5dfh*BsUI>{QjVUnAj78$(s+
zsQg>T^z*CojosnC)`hYdS}s)pFE8o2Mjdw3ki`~5Mo1&GFAUDyBkRlVf||sq9$NRY
za=jwqRli~JJ>Y-bBW@GhEozVbPnND;3fWx*R>FF04K8E=L1J%Wn=igw_mGi$Z1TzW
zG@Gn=B1b6FnTdgFO<UYqNu!<<_W5e>9TJty%8m>UqC_J|0RS5s0Dv9<03bZ%eEkAE
z9DQXGaF22eE3ZsFwvej&E8rK^A1vB%+L9$@y;NK&4CZRnFHguHC8yWs*eShgcczdB
z8R5P?(o9S5Z3$je^-WvR?oa=tWu{%I_u%;tk?0u6JF_j*{R54c?+08JIYgacNo@T0
zkl`8*$aVvsV271$t(X^GkuEU5@z$rWo|^Je^?!*hj|~N<n(?_4O55?1af`J#pqK#c
zPQO6ES?+vq250ATg#AH9#T<ml<KCJd|119>lJ9A`=60kRqF-oA)^f3DBg`_=xVTTq
z;?$JSm{yec(FX1A=3r?a`H9M0Y7_A}@<ss3*jfP1wJ}CPyPe7OY>w<}jFaQhwFM~<
zu(7Ze)|5><uZV2Z3vwyq^M{5@puV=?1Ru~v2Lcjn3z7^a?~?l8SCL{)Lp@#Qx!Gzu
zSz?}M`eN%dmf5nAQr#GP2whKCJN2k_aIWvOyks-@{qp*d@4{s~f(B72;WQS7JYyYe
z->Fk;`h&^&kQlj#WOtDsC3=^7m7RWAdRLe|hh4&h%N8$CwmC7j#D=(;9&p!n$E`^_
ztg9H412vzn5qiLGnQrsUp1uNP>$WdrcMyVYe|XFl@pYuD4v2kIAZe$c&<_9J%;`ui
zl1kCRHYYe@I?QwV66UTv_y~rCeSA-HbK+h8=W2j5x!u;NVQpvG-ICTJW|^^;OUoiE
zr##nnZ)&Y5$DOK$(5v_HvfY;fB9!=Bsv({yZ%bww^m)JN8_5*U^)2NbK8)Wc7hlC_
z2Yy?8laj&qoxx`&cb?BW(VyaG6w9TGVR2bVEEvJP<1**$;e?ZiKOF)ggUYIG_f)sC
zY}NZ?_yNvIGvUEgw{MeX7qZ7-{CD!_)oZrXs8jQ-^do#jCrT7I6kk_`6*5_VZ$P_B
z(qUf+s5P8>JWffV7<axIF3wLejh^$1+OmRo^d9<U5lIC_KRh4azu0h$D>@K|;oeml
zKeky9pZuo#H?C0mb;kqQM<&z?U$i1MDK=Nsy_)lwuWAGbtjk_}!C5~^8L-yPtl*WY
z1kb=bh}u`DXIo0UQEj2^AG@y8aBN0(C)}a8rHpxrlGd#=%IV<uNjg+JGUqyb_+YW^
zq)@Q=4VT=UV-R;s09(WiqNXL5=<@EB-)dI(QCMzgaN0_KLlK5uUJ_afjar2XQ8!kF
z?n1>rZaAe~{Ck;ou`KnJ452e?f{qjbfE^bAzy(0bNbn#rDC@+0A-1R;@br~db5+Uj
z=86wlhEBMxS-4YYC39jb+b8nV)GwDu>V)q~t_rSw5)#j|kN5)8HMbkCSydZnH0Iu8
zGbO2zPxW|6esN;2k^x&%^OCAGq$O7m%};zYmb0pAkFNyol2KPkxhx02zqzVIn$b4k
zH@;*DFJDCpX#S3C86~MY6{@i0>F1t!(zvfT@LQeE6c6iq!^}%xp$U<e*uH_p4~)_)
z%4~PLdg3!ZLGjy65A;PZu+gwQ-OE<Dgr9{ytw)0qw6#r$;^TPA^pJDtJp;Rhn+=Ic
zBX3m}-f7z3?E9SVXLV(r=FlVU;YjGBmAtUjAM8b@r#`h^X6&iGDSa-R>E&QP-nqBI
zW0w;l)}h!ttJ#|NkmaSanXdFCd9XgQxCw;yxUs{(Z_qPp-Of5TypyYi&`OO+u^q>y
z`CA^xY2Q$=?i7-5+4Z3v%Qe$-b|Gqtr%UTq*YzEo<u+{@U3^JNQAH^IYAO#3&qE+2
zOHPo`li@|hjJFdAOabHCAh_;~#%Bsa*Zp-K65x*jijPmExuT=^A|6P@$b4167}bLo
z1PxR-f?J-wN%#Y0OvfxGCfAvQ4%>t$s3HxXhg{d@K!0S_xFfYc#5&tYD>NwP%~2XC
z*n6sQ`InZjuKsfJ)fL)u#&b!Y?VKIWh_u8gQ{M-sP*d-Rban`Hd@z|azcV-2X%k%#
z>l{>)p<$lA_j5$WIsWE+ho@58iCa^~+XdvsM!QLv{5l+2HQ#vHnXk+ci0nl)Xq=vS
zbxc=+z1r^OhUj$reUxn}0__wN_A2-<(n`Ir>~C*Zx%Z7YM+^)|LfQ5+Ze4R2L0;Ta
zRPaG2aB)X=Lc`)C$MlkG_PI$N_|D>m6^w)WsJM&6o`HU)YYCf=k<$zY3g6em8@z_}
zJf&uX9X`%Yst1!q*0-}ufVQ)=g#CAd^>uz}P_e;|9^@vqnvg_J26IRhY`^&nC5%Tf
z54uMv#p~Iag7fBT;5sY>stPV-a!-*b^d-L$3on-cma9iZ)Fh~u`7sPN1a;9jQsBkP
z+}e~@EON?<9Gxt$8_&q_=Lj^d8-@7x{f)A2{FNWhBe%g9(@pc9a&fDT)r}VyGd%~#
z5s+-4UX7D17b@46({N)&Wu!ozmVm0a>N31NPAp6BFO)!O?d)i)Bkv1mZIFqX-Zr+g
zDy^agOqT;4GG%wR5?u?fW%yeIh546@1d8$(zV+4*SgRXm&TBilL`S<Ze7Ll%ig>wX
zs;<A{Y?<S}V1{Z=fMZkjBlv2%-F7iW8A^KSCHCQ<??E#Q`U&1*9uUXJ>Ahlp6#m_9
z7^IU<Ki!>2pe^uwWVN*Fx2fX#WSr#Tv$A0k^H}l*H*+B8xb>5+6y7Wt&Xq!l8V%ID
zwgP~mBWcCpSS=<c<U*4Ic>1~(cc_P>ta5I~7hplub=zsBVJ01%D3yVy!j0Up9H5@-
z5THQuCjgc~!|6xE9j4btW;Y0DLLYI5PI7UFqI>zYhU}0(!i-gZFX$L>8Zs8Rj*CJJ
zJg@N)&uFwl9s>u!BBAKT9`Y=qQYyyKq+YxCR5C3INke~+lxj*L<fuuRRBO{Lr1}d-
z&iM2B<iEWl4xNCciyrd3TbzJ@*xlyP^z(85ZtWk@r4!L4!=7YZHF&gpgG45M-j-5K
z?VJhVS8S*+<G!0^NB6AUwW`As>~CUKXS|tQQ^9EscFSfxn8!OuPed}Ve712aqVySq
z%lho_8N(OqqAc3bVDs55ZC%HPh{31+`!jO-SHQYBK_V%c^W7=4x=;~CAd;fcO-2YN
z_~;g_5EtEAx0Ol8&O7`<|M-kYPvSw_I__m(|ILS^?<gNp+SqzbmGC7?K!FnTG5T=d
zeOP6WJDX)c+SWf`=q|ryy|k>HLgjTYI<<6YxHmhY3xzIxvgWdeDdMu~ARY8r;U9XF
z*{8R9{?((#=^O<IihBn|B1M(O((IclqXMtxiNJQLKO7Ya0K)7`?u7YppkfvUt4KwD
z=WtRuuA);n>v#O(Rh?-y;!tO{WrKu9_+*hbT~5y8(ZTYc`?t!?I;9v&2FQwM0lo>>
zPcK_lC%Lof&&a=xv^Fdp6p?t(urtCG8lrC_5){s@4hOK^1LVxds)l|Z&tdbpwlkl|
zIFK<e>67pybi}ydo#o8L%Y(YJ;687mMDEKwhPV`ImkiuWAXE()Cwz3$DbWE${T3zs
z=V!p6n9BbeerUy2c>ll7{7Y;9wTLJi@!P;uREGF{U@dA>T#WbsHogC@`)BW=q$PBL
jD<~fc5#E1SW())X9R3bi%>TOxP=czK08z+@{Y(A_n(`3W

delta 4210
zcmZ8kbx;(Jvp+cU=#Z3-13^F}4y2@{L6neg6bWg*)X|92cf^sB(w%b15dwnL(T7Mk
z$AOge^Y`AHH{W?XJ3F)S$IkA|XZHg)#5}LNAtCU@>}J65NZFwt7%}>g{H8nV3whuB
zoJvzo_z}%oZbqlxK@~syB#h~vc;Ek2ak=ErM)J@x%k(2E5xWQTK9HA+9T?{J-?T7M
z{!Wqtq>{1Ve{k>jBW3B?DK^f};yR_mCaKT15hmMzwy#=yqtW_QTJkag&Po!;emGaZ
zQR-2I5pl&jt+5kSi}44_KykHVDyPJ#zg^I4KgcSKjGxzxD=LtKTrxW@YT!XRnaBu8
z^*D$5leJ@Em01(_JaLW9Wx2?r>ODRsx%U>CTSW9SYP5NBS|#%;lnoihTf&mTz&|up
zEL|aV!dPSlb)0cp76KU7FNA(_85c`Buhs1(O?<`WTfYhQwtd@XTN0_?0FpB8W@S41
z8n2ErOE6K^_ywK_WOY}NmCcFES*oept80*Rj;?;0n3H4Vlm+UYBssM$V+ek$JiF1x
zAX7!V`VTNFNy_i*xL6n_$*Z^7Ax0lAVbd&mT8pXZ&T|0i>|6X^Y97BoN@+^6t~8vQ
zEdP@6hl3up`j})T8+5YLzHI0fY2TdO9`QJ836X@Ay;0mBpsi$&T)kI0ef;cHNiVkg
zn@GIh@&&I#w1joN3@@8jcFov&+=$DT6~8R+xo8nPR{u*lF0j{TH}SRqa$@K0Aa=Y>
zUob&1_p+8#o!GX})UCl+{-ICQ7XxH&$>SDRllP3!8Sapxqt$A~8myuO7E^i8)uYC|
zn{nEF13Uie7b&7bt!TSF<cye|#6m*IpIh=(R51En+FP+L;76DUv!5&H+yeT;I74oL
z8nNn&St)a$!zT(Ogrvx<UnvRYpG)-!VMH~pPG2;|y?kC<7wH?K>FXB}b#m?FiP$U@
z*U>e!AYobC+YC`k3?;RJ98mOIId6AZYLzYUsf@8DMb<2QeDir0eB^*BYEd{#q{?9M
zJo{es1U2G5szZ_2>gUm4;I<R!_gKbUw`$sBUIrOo4L?1A5?XabO-IkkOIw48^hU>O
z#?Ip9bFG$^CgdZZWn>gt&rarJ8LnpuY|F^Fawb|*+iqADE`hKVAU$cepL;I?9{`Ys
zJtgOWvOJ=W#tEIW83$Q2pd~abqni@w%MFE23)2Ey#XIQg)|=VxR1Y=eh(%|@7x}vf
zPKe@g@niAwXG$R*J4Q#UquNDV8?NFH$<Wn1#a1K3OI-&`i|}YFWq)O6-D5B9uSiQ|
z$crEjkX7Xk(jwMI(J0Mhc8W~#QM!Z>RsSiJ`_QDd0_n>u+r<y+tPm-ktgK)(%ust<
zNSWjBXUXt<ctFFS2Vs@|5i_jD#$Z7yX@>a%`uIh*&cwn+t6eIDNOIztVL2+=vB&&^
zH;sH*PF0#-^>yMS19aP=B3CX6l9j8I@k=zhmZ4m&3r(9ow+V~1U|;e~qZhx~Z$T^Q
z2Pw5*hC+>_%5Mw#8>1FFVva35ZVD0v83RiK#sov(1(Ptdx1_U7oCa$lZ3UK8zt*j@
z91y!Y$b8r-9LQYErJG@46!_xvbbvT@_`rP423k$~>xDjx8RvcjGX~(?Gc+tq*+ce=
z<9>8saPXVy7QV7bTy<fUSAg2gq2zOC=tdRF!>p%KMtRu27gdh1YB5~FlmCnxRlTpR
z2sW!f-D*<$u%9*X!fd#4JlnCf_(?GE%5vzocv-02$bJ0>8kO(fwI+qv3gjRnDN{tL
zLxQDMuFjtbL@Dn(<x}O;QFJo**)NFiR=rF4Aes#ATH{JZ;~E$R*JvmfR*uM_u8o=|
zp=<f~t36!~rDE$T*mL%{eOD|{b9i%eOv~p8iq0`!ynfBF1Whqx%y2wd?2JGv(5|Y9
zLp%>PM>n?}SC6)Glnbv1*G<sbmvoK>;M?ol6oLD-f?!7k{dp*L{rI#q`+hp6AbeK&
z@O)(8U_hZAd#LH8&QlVqV<I~4I|^=UJuuCnXe}=7<q=2MH)~h{&pl{c^E#I~5CG^T
z1pp`j06>7NsJG8Qu8!U!0dB5U#tZHXV&owlbx`V)gG)3-8-GD%atamZ&NNOrS7|pK
zCuT|h>L7>W>ioIa{SNaE?H0K{T3tgqIo^$RUoa%_(-&K<QV5bntgyj=57t2sZ5!((
zR8Qb);I+`qS_QftLihGJJ{CVB*2A5_SqMZNV5^;vPKVC3?eq!Xh`iM0LzcrC`cTHW
z>(UuGJVBE7XHtaC0a*|4fqa$f5m=94A|Hbhf|teFn)f4QAl6KAcBf9@G=6i+0>71b
z`ZBUDS>;@IN>6nDC{5LP#q+dwK-brb2gc+1y_T2S`Se9l@L_P?XR))$ZJIN^{GB(O
zZ!b~8ArS2ce7cdKX)V|)6MnOxX-MjB;}5Km&7ZO~z7fU~C5VRdzO!aQ8+s7nn=AuP
zQ60Bh%X-%`{iDK2RV;M-Tgp91OHyFO(}U{qR?m|Ohwn2k6wZWeuL4_jJ@beUoWHGw
zn`F`4u6shEYwhPD>05p*$8;B-Vz(v{G3&zYv4QS0mLFIK71pj_lBJZe5xm0JwJ%GT
zaUJ&zftH2)klrl9woXEeIw#ku)sOm6{>1o8S{Y*cF%OyuXFoPXO%5qJNiJZqlVvQo
ztwT)P?ZSf0^Maaf4?lIUwgDi78`q<wIZ4wzFD;{?4nkhGzu<x~JC52OrufNg4d=5S
zs5@`<44b<q8gAbaDx6t;iL5`*NbY8_RK}V+_Au#6Xw=WpU?GJ%?{PqFT0yiY&um(%
zqS~x|E-IcY8cIdyJMD24Gq%Q7b@d*50vV0o;*PtWHi2Bxo$1x(+4VK=GOwT6<Te-K
zrz%3r+n$~J*@5P7^^YDF(P9fCn+kFZ#JnvrT#XCqp1s9<-OXo7={Q&Bqi%qUz1^XT
zk&=X?^h{^a+x>@Iva9qQe+1sBnYkb6i@uPnSwg2a+x>$M9Ak%jiWth(c(?_{crReW
zwp+qADfOpZ(pPJx_e_RIblLy;Xw!M<lpz5<OFs%+`_{O3>I4&w6llAZYJYpTQtF#e
zWN@+KI{mIK>6B)lW_3|O`^aH37Dq%EC7%$&JQ7(6AxB<|lXTAK2qV8;i=X|5#7iB|
zU-FNxY3?5(q`j77ZK16T%WG-25nlYnHAFF`hMc4=U4RuToj+^-CTDB_z}+1XU;xVm
zQo!25)Qyd_#l)oFt23?ChyVaqm<<az)MJ5@J_ZNIDsmp!yMhx~g!GJ>hh{p;ufFvE
zfvDrH;a#Ac?!E#MxmqcuW$6iz-sPRkZ`K7vT-5$fSHe=Wk++0E*4Aa-P3!VzA<#=d
z?ig9wF3tu&AxKoHQV2C=k6--4fTH8|sVS~L59!XNd-(c*c2XtwaYB7o&~+1(bu*9*
znfTPKUQTyQ$9sO_aY0?Hn9igI-kAEngwHK`{in8PcQt99tgp>}Moj*6_rFnPTI4BA
z)w+BUWmu?o)UXr3k=BDunvW;*Y6{r;qeF~<gq9SS)b%6!!5U7^W3xM=a}l18QfrR8
zNMR$H(VgV10Z%t@jqt^Us%v~`_E$P74%-#~c{}i4>aE_LC0(#{fu`t)*m-PCdUGEe
z<trL8T}_V0)g#Ga0rQ9Ugg43W-g#X19q$SWP_rBG74g!)cKjhF9Nb+%j}Q?d(GfxZ
z?4=VhLhH-$?}e%%#-EW&xgfDrfqSM?*23j&k>H}xAxx{lhuF_CF*rE1i~JY(d{Qpg
z>0%Lb;X;}J)ZHwJ3&NsZR;E9;OfVxMt6NBHOAoTsZt81nN2RV0{81neB#kjyzBVIb
z;IrF8PDJJ=4>BqA-nYLdmtMmk+-&q=N8WJs79zU-BJJJXe;c^zd?H4P`ug5F6^VV4
zn+>6ceVr}tYKRret?>^=m5lrQk~8uQ^Fne2)KzUUEq_pPkSDo@9UF0yw#FJy9$nQc
zcAjfe6Ln&7(+e}VzO(~RaUM*yGhoHr5$T1~O*d)MsaAo^#96*s_<9ocRq)&VgU>EM
zm-Gj2d`AX@)!)az`u0A;3dW<{|8uM$CFM%S&VfEOk$@RWmY>3Hzw?-A5OT__;*fI@
zPa{4`_)TMqEL^$53PhiJs`V+kW)?p>s*^^bc)wbS>kW<o^d?BuK;tKIxbm*TCH(zV
z2(3WzL8B6vuQdUv{OW;$#v%?sdP=f~TAjC+reBOZL|6X(S)U)ZrVUTLohm3hkZDma
zi&v@#;(~_GkpRmz9eCo4`O^}_;8_k*J*F-*ub6a3WXWLVubgcqB=>}Wlse@`uf+fL
zWbOav#xi}~Hyb_tJ30lj7J6#1YaV3&1K^ilM4Z#I-rbo5xg9oI9(gLfxvx-&;zfM!
zo#=FHwY>B~9sC<lj@@w3PVOjY<~}6WzPzgj3Rrif;H=&GJ^D+Z=ic)vH^QhB7XMTG
zz|*VCM|}U3aZDB@EtBUSzWkQFJLb!eq_f`rtQs19{qw36wECze!T14<*Egb$?ga^>
zInL)F9;b(zbz$$cMpktKNYh%xV>8AHPYCn9E_U80GDoy%1T0yonEf)9&PS_jRnApb
zyF-&+@-a+R7!kIF9W)RMw@(Q2<%$C|KQV{#DpPVAy=t>TOM&QZZ=QsvwL5)A4guDU
zHY0$$^Y@ilGV#$ZA}TbR8Qj9{o_0zU`FI*qBLLoPFTnTG9>QGp7;dzPnyHsL!9^aP
zM&u)4vfK|mUMez1AI#2L_UoO99lUHJOX!29_0k?Ls5RbTzA@nU2ni>%Mcbzrszxh#
z)N=Rz%Ewa|K;?fsD$jKSq+ZE{t%t5eAfIQ51Pt~*&j3AauTDNE3`2@qyz9HdSNg>$
zGI&pfGgZ;Y^3dR-VE05N+))}oK42E!ZU)bwC`7tZZ}<Ys!1(>_nbURGL{DxRg=x+^
zocr`L+A+%WE=nqDPGh8Jp+HaX*bg0eg&}-mndD~5b+a@#XC}pWfW|(K-#~(qwThO;
zFFX=>P)gl2Go@mM2A5*35-MNkd1=xIQt4Ru;kjh8L80G72`)a#xh6-1ssZ#BHP>Lw
z%>6tb-vG%_f^pg|F^wjpHywc67*%tsJTz_kAu4KEBUnozhcJuu<eNjm;3XE$ueWka
znasZezq_up4SmeUXl!0jxYj#e!`$%3R{yX-wcqK84ZI^JKh;ouR=mt(h|`@05)2rG
z6Kn|c({w9Z5vyYqMNfwM1hr>fmUfu@hZusXpPhH77uCPXD$~4VQc&#1?G0>wbuq1Q
zH@R<+2hwZD+jN2X-}nZV8OkyHkyT~-zU|u6C0%V9;V5=+p)xvTDf!d7pAr%oPR4!s
z1hGB7T#V%NKmP{WXbeb?;-&nK<c#9?W?2gMsm))+iBkgq`EkgxTKJw3ZZQEtQ0s>w
z$3kvJe|rF6VE(!PVf8?;%)eNS8d!$)|9^{r5$u2E6ovrH;s?>f>S_64m*59@nJ_*9
ze%AlHXiNaWU;USe!kh%u@G4+c0z9n$*_$370O0VKF46p}1^{q}?F#S`9rOJs_+L|B
B`f~sP

diff --git a/tests/unit/test_etudiants.py b/tests/unit/test_etudiants.py
index 15739a1879..2884980a52 100644
--- a/tests/unit/test_etudiants.py
+++ b/tests/unit/test_etudiants.py
@@ -3,7 +3,7 @@
 """Test modèle étudiant (identite)
 
 
-Utiliser comme: 
+Utiliser comme:
     pytest tests/unit/test_etudiants.py
 
 """
@@ -16,7 +16,7 @@ import app
 from app import db
 from app.models import Admission, Adresse, Departement, FormSemestre, Identite
 from app.scodoc import sco_etud
-from app.scodoc import sco_import_etuds
+from app.scodoc import sco_find_etud, sco_import_etuds
 from config import TestConfig
 
 from tests.unit import setup, dict_include
@@ -267,7 +267,7 @@ def test_import_etuds_xlsx(test_client):
             "civilite_etat_civil": "F",
             "civilite": "X",
             "code_ine": "ine10",
-            "code_nip": "nip10",
+            "code_nip": "1000010",
             "date_naissance": "",
             "dept_acronym": "TEST_",
             "dept_naissance": "",
@@ -302,7 +302,7 @@ def test_import_etuds_xlsx(test_client):
             "civilite_etat_civil": "F",
             "civilite": "X",
             "code_ine": "ine10",
-            "code_nip": "nip10",
+            "code_nip": "1000010",
             "date_naissance": "",
             "dept_acronym": "TEST_",
             "dept_naissance": "",
@@ -367,3 +367,13 @@ def test_import_etuds_xlsx(test_client):
             "telephonemobilestr": "",
         },
     )
+    # Test de search_etud_in_dept
+    etuds = sco_find_etud.search_etuds_infos_from_exp("NOM10")
+    assert len(etuds) == 1
+    assert etuds[0]["code_ine"] == "ine10"
+    etuds = sco_find_etud.search_etuds_infos_from_exp("NOM")
+    assert len(etuds) > 1
+    assert all(e["nom"].startswith("NOM") for e in etuds)
+    etuds = sco_find_etud.search_etuds_infos_from_exp("1000010")
+    assert len(etuds) == 1
+    assert etuds[0]["code_ine"] == "ine10"