Skip to content

Commit

Permalink
Fixed tests.
Browse files Browse the repository at this point in the history 
Damn Python 2 unicode strings again
  • Loading branch information
@PabloCastellano
PabloCastellano committed Jun 7, 2016
1 parent 7942c0c commit 13e2da3
Show file tree
Hide file tree
Showing 2 changed files with 36 additions and 35 deletions.
4 changes: 2 additions & 2 deletions bormeparser/borme.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -343,11 +343,11 @@ def _get_url_borme_a(self, seccion=None, provincia=None):
urls = {}

if seccion and provincia:
xpath1 = '//sumario/diario/seccion[@num="{}"]/emisor/item/titulo[text()="{}"]'.format(seccion, provincia)
xpath1 = u'//sumario/diario/seccion[@num="{}"]/emisor/item/titulo[text()="{}"]'.format(seccion, provincia)
elif seccion:
xpath1 = '//sumario/diario/seccion[@num="{}"]/emisor/item'.format(seccion)
else:
xpath1 = '//sumario/diario/seccion/emisor/item/titulo[text()="{}"]'.format(provincia)
xpath1 = u'//sumario/diario/seccion/emisor/item/titulo[text()="{}"]'.format(provincia)

for item in self.xml.xpath(xpath1):
if seccion and provincia:
Expand Down
67 changes: 34 additions & 33 deletions bormeparser/tests/test_borme.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -255,51 +255,52 @@ def setUpClass(cls):
cls.bxml = BormeXML.from_file(path)

def test_get_url_pdfs(self):
urls_a = {'CEUTA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-51.pdf',
'ÍNDICE ALFABÉTICO DE SOCIEDADES': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-99.pdf',
'MÁLAGA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-29.pdf',
'BURGOS': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-09.pdf',
'SEGOVIA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-40.pdf',
'PONTEVEDRA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-36.pdf',
'CASTELLÓN': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-12.pdf',
'LAS PALMAS': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-35.pdf',
urls_a = {u'A CORU\xd1A': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-15.pdf',
'ALICANTE': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-03.pdf',
u'ALMER\xcdA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-04.pdf',
u'ARABA/\xc1LAVA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-01.pdf',
'ASTURIAS': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-33.pdf',
'BADAJOZ': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-06.pdf',
'BARCELONA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-08.pdf',
'BURGOS': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-09.pdf',
'CANTABRIA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-39.pdf',
u'CASTELL\xd3N': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-12.pdf',
'CEUTA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-51.pdf',
'CUENCA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-16.pdf',
u'C\xc1CERES': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-10.pdf',
u'C\xc1DIZ': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-11.pdf',
u'C\xd3RDOBA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-14.pdf',
'HUESCA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-22.pdf',
'ILLES BALEARS': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-07.pdf',
'CÓRDOBA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-14.pdf',
'LA RIOJA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-26.pdf',
'SANTA CRUZ DE TENERIFE': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-38.pdf',
'PALENCIA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-34.pdf',
'ZAMORA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-49.pdf',
'A CORUÑA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-15.pdf',
'OURENSE': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-32.pdf',
'LAS PALMAS': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-35.pdf',
'LLEIDA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-25.pdf',
'MADRID': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-28.pdf',
'CUENCA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-16.pdf',
'SEVILLA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-41.pdf',
'MURCIA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-30.pdf',
'ARABA/ÁLAVA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-01.pdf',
'ZARAGOZA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-50.pdf',
u'M\xc1LAGA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-29.pdf',
'NAVARRA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-31.pdf',
'OURENSE': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-32.pdf',
'PALENCIA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-34.pdf',
'PONTEVEDRA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-36.pdf',
'SANTA CRUZ DE TENERIFE': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-38.pdf',
'SEGOVIA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-40.pdf',
'SEVILLA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-41.pdf',
'TARRAGONA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-43.pdf',
'ALMERÍA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-04.pdf',
'VALENCIA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-46.pdf',
'CÁCERES': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-10.pdf',
'CÁDIZ': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-11.pdf',
'BARCELONA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-08.pdf',
'LLEIDA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-25.pdf',
'VALLADOLID': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-47.pdf',
'HUESCA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-22.pdf',
'NAVARRA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-31.pdf',
'CANTABRIA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-39.pdf',
'BADAJOZ': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-06.pdf',
'ALICANTE': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-03.pdf'}
'ZAMORA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-49.pdf',
'ZARAGOZA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-50.pdf',
u'\xcdNDICE ALFAB\xc9TICO DE SOCIEDADES': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-99.pdf'}


url_cve_a = {'BORME-A-2015-183-06': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-A-2015-183-06.pdf'}

urls_b = {'MÁLAGA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-B-2015-183-29.pdf',
'ALMERÍA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-B-2015-183-04.pdf',
'A CORUÑA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-B-2015-183-15.pdf',
urls_b = {u'A CORU\xd1A': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-B-2015-183-15.pdf',
u'ALMER\xcdA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-B-2015-183-04.pdf',
'ILLES BALEARS': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-B-2015-183-07.pdf',
u'JA\xc9N': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-B-2015-183-23.pdf',
'MADRID': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-B-2015-183-28.pdf',
'JAÉN': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-B-2015-183-23.pdf'}
u'M\xc1LAGA': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-B-2015-183-29.pdf'}

url_cve_b = {'BORME-B-2015-183-04': 'https://www.boe.es/borme/dias/2015/09/24/pdfs/BORME-B-2015-183-04.pdf'}

Expand All @@ -310,7 +311,7 @@ def test_get_url_pdfs(self):
self.assertEqual(self.bxml.get_url_pdfs(seccion=SECCION.A), urls_a)
self.assertEqual(self.bxml.get_url_pdfs(seccion=SECCION.A, provincia='BADAJOZ'), url_cve_a)
self.assertEqual(self.bxml.get_url_pdfs(seccion=SECCION.B), urls_b)
self.assertEqual(self.bxml.get_url_pdfs(seccion=SECCION.B, provincia='ALMERÍA'), url_cve_b)
self.assertEqual(self.bxml.get_url_pdfs(seccion=SECCION.B, provincia=u'ALMER\xcdA'), url_cve_b)
self.assertEqual(self.bxml.get_url_pdfs(seccion=SECCION.C), urls_c)
self.assertRaises(AttributeError, self.bxml.get_url_pdfs)

Expand Down

0 comments on commit 13e2da3

Please sign in to comment.