diff --git a/tests/test_comicnames.py b/tests/test_comicnames.py index 8d3b11e5e9..a9d69c4a51 100644 --- a/tests/test_comicnames.py +++ b/tests/test_comicnames.py @@ -3,12 +3,15 @@ # Copyright (C) 2012-2014 Bastian Kleineidam # Copyright (C) 2015-2022 Tobias Gruetzmacher import re +from operator import attrgetter + +import pytest from dosagelib.scraper import scrapers from dosagelib.plugins import old -class TestComicNames(object): +class TestComicNames: def test_names(self): for scraperobj in scrapers.all(): @@ -20,11 +23,11 @@ def test_names(self): comicname = name assert re.sub("[^0-9a-zA-Z_]", "", comicname) == comicname - def test_renamed(self): - for scraperobj in scrapers.all(include_removed=True): - if not isinstance(scraperobj, old.Renamed): - continue - assert len(scraperobj.getDisabledReasons()) > 0 - # Renamed scraper should only point to an non-disabled scraper - newscraper = scrapers.find(scraperobj.newname) - assert len(newscraper.getDisabledReasons()) == 0 + @pytest.mark.parametrize(('scraperobj'), + [obj for obj in scrapers.all(include_removed=True) + if isinstance(obj, old.Renamed)], ids=attrgetter('name')) + def test_renamed(self, scraperobj): + assert len(scraperobj.getDisabledReasons()) > 0 + # Renamed scraper should only point to an non-disabled scraper + newscraper = scrapers.find(scraperobj.newname) + assert len(newscraper.getDisabledReasons()) == 0