Skip to content

Commit

Permalink
fix(api): preserve order when merging semantic and fulltext results
Browse files Browse the repository at this point in the history 
fix #2095
  • Loading branch information
@lionelB
lionelB committed Dec 30, 2019
1 parent af4790a commit 7ad8413
Show file tree
Hide file tree
Showing 15 changed files with 5,929 additions and 6,929 deletions.
1,068 changes: 532 additions & 536 deletions packages/code-du-travail-api/src/server/routes/__tests__/__snapshots__/items.test.js.snap
View file

Large diffs are not rendered by default.

166 changes: 83 additions & 83 deletions packages/code-du-travail-api/src/server/routes/__tests__/__snapshots__/search.test.js.snap
View file
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ exports[`returns article results when searching article R1225-18 1`] = `
Object {
"articles": Array [
Object {
"_score": 20.642752,
"_score": 16.519085,
"description": "Le salarié informe son employeur de sa démission, en application de l'article L. 1225-66 , par lettre recommandée avec avis de réception ou remise contre récépissé. Il adresse à l'employeur sa demande de réembauche, en application de l'article L. 1225-67 , par lettre recommandée avec avis de réception ou remise contre récépissé",
"slug": "r1225-18",
"source": "code_du_travail",
Expand All @@ -14,42 +14,51 @@ Object {
],
"documents": Array [
Object {
"_score": 1.2591128,
"_score": 1.1191393,
"action": "Calculer",
"algo": "semantic",
"description": "Le Code du travail ne prévoit aucune forme particulière pour présenter sa démission : elle peut être verbale, écrite ou résulter d’un comportement sans ambiguïté du salarié (ce qui n’est pas le cas, par exemple, de la seule absence du salarié à son poste de travail ou de l’absence de reprise du travail à l’issue des congés payés).",
"slug": "la-demission#Comment-presenter-une-demission",
"source": "fiches_ministere_travail",
"title": "La démission : comment présenter une démission ?",
"description": "Calculez la durée de préavis de démission",
"slug": "preavis-demission",
"source": "outils",
"title": "Préavis de démission",
},
Object {
"_score": 1.1894765,
"_score": 1.1074626,
"action": "Estimer",
"algo": "semantic",
"breadcrumbs": Array [
Object {
"slug": "8-depart-de-lentreprise",
"title": "Départ de l'entreprise",
},
Object {
"slug": "81-demission",
"title": "Démission",
},
],
"description": "La démission permet au salarié, à son initiative, de rompre son contrat de travail, sous conditions.",
"slug": "demission-dun-salarie",
"source": "fiches_service_public",
"title": "Démission d'un salarié",
"url": "https://www.service-public.fr/particuliers/vosdroits/F2883",
"description": "Estimez le montant de l’indemnité de licenciement",
"slug": "indemnite-licenciement",
"source": "outils",
"title": "Indemnité de licenciement",
},
Object {
"_score": 1.1783916,
"_score": 1.0945752,
"algo": "semantic",
"description": "En cas d’absence prolongée de l’entreprise sans justification valable (par exemple, un arrêt de travail), ou d’absence de reprise du travail après un arrêt de travail, il n’est pas possible pour l’employeur de considérer le salarié comme démissionnaire. Dans ces hypothèses (souvent qualifiées d’abandon de poste), l’employeur doit demander à l’intéressé les raisons de son absence. Si le salarié ne répond pas dans un délai raisonnable, l’employeur peut engager une procédure de licenciement pour absence injustifiée Ainsi, une salariée qui, après une observation, avait quitté son travail et n’avait pas réintégré l’entreprise les jours suivants, n’a pas été considérée par la Cour de cassation comme démissionnaire.",
"slug": "la-demission#L-absence-prolongee-du-salarie-est-elle-une-demission",
"source": "fiches_ministere_travail",
"title": "La démission : l’absence prolongée du salarié est-elle une démission ?",
},
Object {
"_score": 1.1642654,
"_score": 1.0863602,
"action": "Consulter",
"algo": "semantic",
"description": "Consulter et utiliser ses droits pour la formation",
"slug": "mon-compte-formation",
"source": "external",
"title": "Mon compte formation",
"url": "https://www.moncompteformation.gouv.fr",
},
Object {
"_score": 1.0595243,
"algo": "semantic",
"description": "Le Code du travail ne prévoit aucune forme particulière pour présenter sa démission : elle peut être verbale, écrite ou résulter d’un comportement sans ambiguïté du salarié (ce qui n’est pas le cas, par exemple, de la seule absence du salarié à son poste de travail ou de l’absence de reprise du travail à l’issue des congés payés).",
"slug": "la-demission#Comment-presenter-une-demission",
"source": "fiches_ministere_travail",
"title": "La démission : comment présenter une démission ?",
},
Object {
"_score": 1.0449069,
"algo": "semantic",
"breadcrumbs": Array [
Object {
Expand All @@ -68,40 +77,26 @@ Object {
"url": "https://www.service-public.fr/particuliers/vosdroits/F33164",
},
Object {
"_score": 1.1540345,
"algo": "semantic",
"description": "Connaître la durée du préavis de démission",
"slug": "preavis-demission",
"source": "outils",
"title": "Simulateur de durée de préavis de démission",
},
Object {
"_score": 1.1081983,
"_score": 1.0381742,
"action": "Calculer",
"algo": "semantic",
"description": "Simulez simplement le montant d'une indemnité de licenciement en fonction de votre situation",
"slug": "indemnite-licenciement",
"source": "outils",
"title": "Simulateur d'indemnité de licenciement",
},
Object {
"_score": 1.0708303,
"algo": "semantic",
"description": "La rupture anticipée du contrat à déterminée n’est possible que dans certains cas. La rupture d’un commun accord est l’un de ces cas : ce modèle permet à l’une des parties de demander à l’autre, la rupture du CDD.",
"slug": "modele-rupture-dun-commun-accord-dun-contrat-a-duree-determinee",
"source": "modeles_de_courriers",
"title": "Modèle : Rupture d’un commun accord d’un contrat à durée déterminée",
"description": "L’outil de calcul de votre index égalité professionnelle Femmes- Hommes",
"slug": "index-egapro",
"source": "external",
"title": "Index Egapro",
"url": "https://index-egapro.travail.gouv.fr",
},
],
"themes": Array [
Object {
"_score": 1.1876618,
"_score": 1.123209,
"algo": "semantic",
"slug": "81-demission",
"source": "themes",
"title": "Démission",
},
Object {
"_score": 1.1013293,
"_score": 1.1099768,
"algo": "semantic",
"slug": "1-embauche-et-contrat-de-travail",
"source": "themes",
Expand All @@ -115,15 +110,16 @@ exports[`returns search results for demission from datafiller 1`] = `
Object {
"articles": Array [
Object {
"_score": 2.667768,
"_score": 0,
"algo": "pre-qualified",
"description": "En cas de démission, l'existence et la durée du préavis sont fixées par la loi, ou par convention ou accord collectif de travail. En l'absence de dispositions légales, de convention ou accord collectif de travail relatifs au préavis, son existence et sa durée résultent des usages pratiqués dans la localité et dans la profession. Un décret en Conseil d'Etat détermine les modalités d'application du présent article",
"slug": "l1237-1",
"source": "code_du_travail",
"title": "L1237-1",
"url": "https://www.legifrance.gouv.fr/affichCodeArticle.do;?idArticle=LEGIARTI000006901174&cidTexte=LEGITEXT000006072050",
},
Object {
"_score": 2.7447872,
"_score": 1.582805,
"description": "Le salarié informe son employeur de sa démission, en application de l'article L. 1225-66 , par lettre recommandée avec avis de réception ou remise contre récépissé. Il adresse à l'employeur sa demande de réembauche, en application de l'article L. 1225-67 , par lettre recommandée avec avis de réception ou remise contre récépissé",
"slug": "r1225-18",
"source": "code_du_travail",
Expand Down Expand Up @@ -181,14 +177,14 @@ Object {
],
"themes": Array [
Object {
"_score": 1.1876618,
"_score": 1.123209,
"algo": "semantic",
"slug": "81-demission",
"source": "themes",
"title": "Démission",
},
Object {
"_score": 1.1013293,
"_score": 1.1099768,
"algo": "semantic",
"slug": "1-embauche-et-contrat-de-travail",
"source": "themes",
Expand All @@ -202,15 +198,15 @@ exports[`returns search results for demission from elastic 1`] = `
Object {
"articles": Array [
Object {
"_score": 2.7447872,
"_score": 1.582805,
"description": "Le salarié informe son employeur de sa démission, en application de l'article L. 1225-66 , par lettre recommandée avec avis de réception ou remise contre récépissé. Il adresse à l'employeur sa demande de réembauche, en application de l'article L. 1225-67 , par lettre recommandée avec avis de réception ou remise contre récépissé",
"slug": "r1225-18",
"source": "code_du_travail",
"title": "R1225-18",
"url": "https://www.legifrance.gouv.fr/affichCodeArticle.do;?idArticle=LEGIARTI000018537778&cidTexte=LEGITEXT000006072050",
},
Object {
"_score": 2.667768,
"_score": 1.5555403,
"description": "En cas de démission, l'existence et la durée du préavis sont fixées par la loi, ou par convention ou accord collectif de travail. En l'absence de dispositions légales, de convention ou accord collectif de travail relatifs au préavis, son existence et sa durée résultent des usages pratiqués dans la localité et dans la profession. Un décret en Conseil d'Etat détermine les modalités d'application du présent article",
"slug": "l1237-1",
"source": "code_du_travail",
Expand All @@ -220,16 +216,26 @@ Object {
],
"documents": Array [
Object {
"_score": 1.2591128,
"_score": 1.1191393,
"action": "Calculer",
"algo": "both",
"description": "Le Code du travail ne prévoit aucune forme particulière pour présenter sa démission : elle peut être verbale, écrite ou résulter d’un comportement sans ambiguïté du salarié (ce qui n’est pas le cas, par exemple, de la seule absence du salarié à son poste de travail ou de l’absence de reprise du travail à l’issue des congés payés).",
"slug": "la-demission#Comment-presenter-une-demission",
"source": "fiches_ministere_travail",
"title": "La démission : comment présenter une démission ?",
"description": "Calculez la durée de préavis de démission",
"slug": "preavis-demission",
"source": "outils",
"title": "Préavis de démission",
},
Object {
"_score": 7.8231583,
"algo": "both",
"_score": 1.1074626,
"action": "Estimer",
"algo": "semantic",
"description": "Estimez le montant de l’indemnité de licenciement",
"slug": "indemnite-licenciement",
"source": "outils",
"title": "Indemnité de licenciement",
},
Object {
"_score": 5.339034,
"algo": "fulltext",
"breadcrumbs": Array [
Object {
"slug": "8-depart-de-lentreprise",
Expand All @@ -247,23 +253,15 @@ Object {
"url": "https://www.service-public.fr/particuliers/vosdroits/F2883",
},
Object {
"_score": 7.438704,
"algo": "both",
"description": "Connaître la durée du préavis de démission",
"slug": "preavis-demission",
"source": "outils",
"title": "Simulateur de durée de préavis de démission",
},
Object {
"_score": 1.1783916,
"_score": 1.0945752,
"algo": "both",
"description": "En cas d’absence prolongée de l’entreprise sans justification valable (par exemple, un arrêt de travail), ou d’absence de reprise du travail après un arrêt de travail, il n’est pas possible pour l’employeur de considérer le salarié comme démissionnaire. Dans ces hypothèses (souvent qualifiées d’abandon de poste), l’employeur doit demander à l’intéressé les raisons de son absence. Si le salarié ne répond pas dans un délai raisonnable, l’employeur peut engager une procédure de licenciement pour absence injustifiée Ainsi, une salariée qui, après une observation, avait quitté son travail et n’avait pas réintégré l’entreprise les jours suivants, n’a pas été considérée par la Cour de cassation comme démissionnaire.",
"slug": "la-demission#L-absence-prolongee-du-salarie-est-elle-une-demission",
"source": "fiches_ministere_travail",
"title": "La démission : l’absence prolongée du salarié est-elle une démission ?",
},
Object {
"_score": 1.1642654,
"_score": 4.8279843,
"algo": "both",
"breadcrumbs": Array [
Object {
Expand All @@ -282,32 +280,34 @@ Object {
"url": "https://www.service-public.fr/particuliers/vosdroits/F33164",
},
Object {
"_score": 1.1081983,
"_score": 1.0863602,
"action": "Consulter",
"algo": "semantic",
"description": "Simulez simplement le montant d'une indemnité de licenciement en fonction de votre situation",
"slug": "indemnite-licenciement",
"source": "outils",
"title": "Simulateur d'indemnité de licenciement",
"description": "Consulter et utiliser ses droits pour la formation",
"slug": "mon-compte-formation",
"source": "external",
"title": "Mon compte formation",
"url": "https://www.moncompteformation.gouv.fr",
},
Object {
"_score": 1.0708303,
"algo": "semantic",
"description": "La rupture anticipée du contrat à déterminée n’est possible que dans certains cas. La rupture d’un commun accord est l’un de ces cas : ce modèle permet à l’une des parties de demander à l’autre, la rupture du CDD.",
"slug": "modele-rupture-dun-commun-accord-dun-contrat-a-duree-determinee",
"source": "modeles_de_courriers",
"title": "Modèle : Rupture d’un commun accord d’un contrat à durée déterminée",
"_score": 4.7920084,
"algo": "both",
"description": "Le Code du travail ne prévoit aucune forme particulière pour présenter sa démission : elle peut être verbale, écrite ou résulter d’un comportement sans ambiguïté du salarié (ce qui n’est pas le cas, par exemple, de la seule absence du salarié à son poste de travail ou de l’absence de reprise du travail à l’issue des congés payés).",
"slug": "la-demission#Comment-presenter-une-demission",
"source": "fiches_ministere_travail",
"title": "La démission : comment présenter une démission ?",
},
],
"themes": Array [
Object {
"_score": 1.1876618,
"_score": 1.123209,
"algo": "semantic",
"slug": "81-demission",
"source": "themes",
"title": "Démission",
},
Object {
"_score": 1.1013293,
"_score": 1.1099768,
"algo": "semantic",
"slug": "1-embauche-et-contrat-de-travail",
"source": "themes",
Expand Down
15 changes: 10 additions & 5 deletions ...ages/code-du-travail-api/src/server/routes/__tests__/__snapshots__/sheets-mt.test.js.snap
View file
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
// Jest Snapshot v1, https://goo.gl/fbAQLP

exports[`return theme data for slug 5-questions-reponses-sur-la-sante-au-travail 1`] = `
exports[`return content data for slug 5-questions-reponses-sur-la-sante-au-travail 1`] = `
Object {
"_id": "0",
"_index": "cdtn_test_mt-sheets",
Expand Down Expand Up @@ -79,6 +79,7 @@ Si l’avis d’inaptitude mentionne expressément que tout maintien du salarié
"slug": "demission-dun-salarie",
"source": "fiches_service_public",
"title": "Démission d'un salarié",
"url": "https://www.service-public.fr/particuliers/vosdroits/F2883",
},
Object {
"description": "En cas d’absence prolongée de l’entreprise sans justification valable (par exemple, un arrêt de travail), ou d’absence de reprise du travail après un arrêt de travail, il n’est pas possible pour l’employeur de considérer le salarié comme démissionnaire. Dans ces hypothèses (souvent qualifiées d’abandon de poste), l’employeur doit demander à l’intéressé les raisons de son absence. Si le salarié ne répond pas dans un délai raisonnable, l’employeur peut engager une procédure de licenciement pour absence injustifiée Ainsi, une salariée qui, après une observation, avait quitté son travail et n’avait pas réintégré l’entreprise les jours suivants, n’a pas été considérée par la Cour de cassation comme démissionnaire.",
Expand All @@ -91,12 +92,16 @@ Si l’avis d’inaptitude mentionne expressément que tout maintien du salarié
"slug": "demission-dune-assistante-maternelle",
"source": "fiches_service_public",
"title": "Démission d'une assistante maternelle",
"url": "https://www.service-public.fr/particuliers/vosdroits/F33164",
},
Object {
"description": "Connaître la durée du préavis de démission",
"slug": "preavis-demission",
"source": "outils",
"title": "Simulateur de durée de préavis de démission",
"action": "Consulter",
"description": "Consulter et utiliser ses droits pour la formation",
"icon": "Formation",
"slug": "mon-compte-formation",
"source": "external",
"title": "Mon compte formation",
"url": "https://www.moncompteformation.gouv.fr",
},
],
}
Expand Down
2 changes: 1 addition & 1 deletion packages/code-du-travail-api/src/server/routes/__tests__/sheets-mt.test.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ const router = require("../sheets-mt");
const app = new Koa();
app.use(router.routes());

test("return theme data for slug 5-questions-reponses-sur-la-sante-au-travail", async () => {
test("return content data for slug 5-questions-reponses-sur-la-sante-au-travail", async () => {
const response = await request(app.callback()).get(
`/api/v1/sheets-mt/5-questions-reponses-sur-la-sante-au-travail`
);
Expand Down
Loading

0 comments on commit 7ad8413

Please sign in to comment.