Merge pull request #190 from jkowalleck/master

slight improv in ImageCrawler deduplication checks

Author: jkowalleck

nichtparasoup/core/__init__.py

@@ -142,7 +142,7 @@ def _add_image_to_blacklist(self, image: Image) -> None:
             self.blacklist.add(image.uri)
 
     def has_imagecrawler(self, imagecrawler: BaseImageCrawler) -> bool:
-        return imagecrawler in [crawler.imagecrawler for crawler in self.crawlers]
+        return imagecrawler in (crawler.imagecrawler for crawler in self.crawlers)
 
     def add_imagecrawler(self, imagecrawler: BaseImageCrawler, weight: _CrawlerWeight) -> None:
         self.crawlers.append(Crawler(

nichtparasoup/testing/config.py

@@ -13,7 +13,7 @@ class ConfigFileTest(TestCase):
     def validate(self, file: str) -> None:
         config = parse_yaml_file(file)
         self.assertIsInstance(config, dict)
-        imagecrawlers = []  # type: List[BaseImageCrawler]
+        imagecrawlers = list()  # type: List[BaseImageCrawler]
         for crawler_config in config['crawlers']:
             imagecrawler = get_imagecrawler(crawler_config)
             self.assertNotIn(imagecrawler, imagecrawlers, msg='Duplicate ImageCrawler')