From e9a4957c6ca0a0830f56a30c049be73e29cad0f2 Mon Sep 17 00:00:00 2001 From: devketanpro Date: Tue, 31 Dec 2024 10:34:23 +0530 Subject: [PATCH] address comment --- superdesk/io/feed_parsers/__init__.py | 2 +- ...fp_newsml_2_0.py => afp_newsml_1_2_new.py} | 2 +- tests/io/feed_parsers/afpnewsml20_test.py | 23 ++++++++++++++----- tests/io/fixtures/afp20.xml | 15 +++--------- 4 files changed, 22 insertions(+), 20 deletions(-) rename superdesk/io/feed_parsers/{afp_newsml_2_0.py => afp_newsml_1_2_new.py} (96%) diff --git a/superdesk/io/feed_parsers/__init__.py b/superdesk/io/feed_parsers/__init__.py index b1189e5a1..90f489bfd 100644 --- a/superdesk/io/feed_parsers/__init__.py +++ b/superdesk/io/feed_parsers/__init__.py @@ -352,4 +352,4 @@ class EmailFeedParser(FeedParser, metaclass=ABCMeta): from superdesk.io.feed_parsers.ana_mpe_newsml import ANANewsMLOneFeedParser # NOQA from superdesk.io.feed_parsers.bbc_ninjs import BBCNINJSFeedParser # NOQA from superdesk.io.feed_parsers.ap_media import APMediaFeedParser # NOQA -from superdesk.io.feed_parsers.afp_newsml_2_0 import AFPNewsMLFeedParser # NOQA +from superdesk.io.feed_parsers.afp_newsml_1_2_new import AFPNewsMLFeedParser # NOQA diff --git a/superdesk/io/feed_parsers/afp_newsml_2_0.py b/superdesk/io/feed_parsers/afp_newsml_1_2_new.py similarity index 96% rename from superdesk/io/feed_parsers/afp_newsml_2_0.py rename to superdesk/io/feed_parsers/afp_newsml_1_2_new.py index 86ad87f04..2e9300756 100644 --- a/superdesk/io/feed_parsers/afp_newsml_2_0.py +++ b/superdesk/io/feed_parsers/afp_newsml_1_2_new.py @@ -33,9 +33,9 @@ def parse_content(self, item, xml): encoding="unicode", ) .replace("", "") + .replace("", "") .strip() ) - pass register_feed_parser(AFPNewsMLFeedParser.NAME, AFPNewsMLFeedParser()) diff --git a/tests/io/feed_parsers/afpnewsml20_test.py b/tests/io/feed_parsers/afpnewsml20_test.py index 9e69eb87d..e8cb48ce8 100644 --- a/tests/io/feed_parsers/afpnewsml20_test.py +++ b/tests/io/feed_parsers/afpnewsml20_test.py @@ -8,13 +8,15 @@ # AUTHORS and LICENSE files distributed with this source code, or # at https://www.sourcefabric.org/superdesk/license -import datetime + import os +import re +import datetime import unittest from pytz import utc from superdesk.etree import etree -from superdesk.io.feed_parsers.afp_newsml_2_0 import AFPNewsMLFeedParser +from superdesk.io.feed_parsers.afp_newsml_1_2_new import AFPNewsMLFeedParser class TestCase(unittest.TestCase): @@ -27,14 +29,14 @@ def setUp(self): def test_headline(self): self.assertEqual( - self.item.get("headline"), "Décès de Naomi Musenga: l'opératrice du Samu définitivement condamnée" + self.item.get("headline"), "Décès de Naomi Musenga: l'opératrice du Samu définitivement condamnée" ) def test_dateline(self): self.assertEqual(self.item.get("dateline", {}).get("text"), "Strasbourg, 10 juil 2024 (AFP)") def test_slugline(self): - self.assertEqual(self.item.get("slugline"), "procès-santé-secours") + self.assertEqual(self.item.get("slugline"), "procès-santé-secours") def test_byline(self): self.assertEqual(self.item.get("byline"), "") @@ -53,5 +55,14 @@ def test_coreitemvalues(self): self.assertEqual(self.item.get("firstcreated"), datetime.datetime(2024, 7, 10, 12, 59, 2, tzinfo=utc)) self.assertEqual(self.item.get("pubstatus"), "usable") - def test_content_is_text(self): - self.assertIsInstance(self.item.get("body_html"), type("")) + def test_body_content(self): + expected_output = ( + "

L'opératrice du Samu condamnée pour non-assistance à personne en danger après avoir raillé " + "au téléphone Naomi Musenga, jeune femme de 22 ans décédée peu après à l'hôpital, a décidé de ne pas faire appel, " + "s'est félicité mercredi l'avocat de la famille Musenga., avait été condamnée à un an de prison avec sursis, le" + " 4 juillet, par le tribunal correctionnel de Strasbourg. Contacté par l'AFP, le cabinet d'avocats de Me Thomas Callen," + " qui la défendait, a confirmé qu'elle n'interjetterait pas appel de cette condamnation.

L'opératrice, Corinne M.

" + ) + + actual_output = re.sub(r"\s+", " ", self.item.get("body_html")).strip() + self.assertIn(expected_output, actual_output) diff --git a/tests/io/fixtures/afp20.xml b/tests/io/fixtures/afp20.xml index a41223ba0..400b2164f 100644 --- a/tests/io/fixtures/afp20.xml +++ b/tests/io/fixtures/afp20.xml @@ -1,4 +1,4 @@ - + @@ -44,17 +44,8 @@ -

L'opératrice du Samu condamnée pour non-assistance à personne en danger après avoir raillé au téléphone Naomi Musenga, jeune femme de 22 ans décédée peu après à l'hôpital, a décidé de ne pas faire appel, s'est félicité mercredi l'avocat de la famille Musenga.

-

"La famille de Naomi Musenga a pris connaissance de la volonté de Mme M., opératrice du Samu 67, de ne pas relever appel des condamnations prononcées à son encontre", a indiqué dans un communiqué le cabinet d'avocats Coubris & Associés.

-

L'opératrice, Corinne M., avait été condamnée à un an de prison avec sursis, le 4 juillet, par le tribunal correctionnel de Strasbourg. Contacté par l'AFP, le cabinet d'avocats de Me Thomas Callen, qui la défendait, a confirmé qu'elle n'interjetterait pas appel de cette condamnation.

-

"Cette décision est accueillie avec un grand soulagement par les proches de la victime", a indiqué le cabinet Coubris & Associés. Le volet pénal est clos mais "la procédure continuera à la rentrée prochaine, cette fois devant la justice administrative et à l'encontre des Hôpitaux universitaires de Strasbourg, afin d'envisager l'indemnisation de leurs préjudices", a-t-il précisé.

-

Mère d'une enfant de 18 mois, Naomi Musenga est décédée le 29 décembre 2017 à l'hôpital de Strasbourg après avoir été prise en charge avec "un retard global de près de 02H20", selon un rapport de l'Inspection générale des affaires sociales (Igas).

-

Se plaignant d'importantes douleurs au ventre, elle avait pris contact avec les pompiers qui avaient transféré l'appel au Samu.

-

"J'ai très mal au ventre", "je vais mourir...", avait expliqué Naomi Musenga, peinant à s'exprimer. "Oui vous allez mourir, certainement un jour comme tout le monde", avait rétorqué la régulatrice, avant de raccrocher.

-

Naomi Musenga avait finalement été dirigée vers SOS Médecins et hospitalisée mais n'avait pu être sauvée.

-

Selon l'enquête, son décès n'est pas lié au retard de prise en charge.

-

Après la mort de Naomi Musenga, une première expertise, dénoncée par sa famille, avait conclu à un décès consécutif à une "intoxication au paracétamol absorbé par automédication sur plusieurs jours".

-

Mais une deuxième expertise avait réfuté ces conclusions, évoquant un accident vasculaire digestif ayant entraîné une hémorragie.

+

L'opératrice du Samu condamnée pour non-assistance à personne en danger après avoir raillé au téléphone Naomi Musenga, jeune femme de 22 ans décédée peu après à l'hôpital, a décidé de ne pas faire appel, s'est félicité mercredi l'avocat de la famille Musenga., avait été condamnée à un an de prison avec sursis, le 4 juillet, par le tribunal correctionnel de Strasbourg. Contacté par l'AFP, le cabinet d'avocats de Me Thomas Callen, qui la défendait, a confirmé qu'elle n'interjetterait pas appel de cette condamnation.

+

L'opératrice, Corinne M.