Updated files and accounted for flake8 and isort issues

- Updated files with isort for consistent imports
- Fixed all flake8 issues
- Changed imports to ensure they are accepted by flake8, and fixed
  circular dependencies

jobfunnel/__main__.py

@@ -3,8 +3,9 @@
 """
 import os
 import sys
+
 from .backend.jobfunnel import JobFunnel
-from .config import parse_cli, build_config_dict, get_config_manager
+from .config import build_config_dict, get_config_manager, parse_cli
 
 
 def main():

jobfunnel/backend/__init__.py

@@ -1 +1,3 @@
 from jobfunnel.backend.job import Job, JobStatus
+
+__all__ = ["Job", "JobStatus"]

jobfunnel/backend/jobfunnel.py

@@ -3,27 +3,27 @@
 """
 
 import csv
+from datetime import date, datetime, timedelta
 import json
 import os
 import pickle
-from datetime import date, datetime, timedelta
 from time import time
-from typing import Dict, List
+from typing import Dict
 
 from requests import Session
 
 from jobfunnel import __version__
 from jobfunnel.backend import Job
 from jobfunnel.backend.tools import Logger
-from jobfunnel.backend.tools.filters import DuplicatedJob, JobFilter
+from jobfunnel.backend.tools.filters import JobFilter
 from jobfunnel.config import JobFunnelConfigManager
 from jobfunnel.resources import (
     CSV_HEADER,
     T_NOW,
-    Remoteness,
     DuplicateType,
     JobStatus,
     Locale,
+    Remoteness,
 )
 
 

jobfunnel/backend/scrapers/base.py

@@ -2,10 +2,10 @@
 Paul McInnis 2020
 """
 
-import random
 from abc import ABC, abstractmethod
 from concurrent.futures import ThreadPoolExecutor, as_completed
 from multiprocessing import Lock, Manager
+import random
 from time import sleep
 from typing import Any, Dict, List, Optional
 

jobfunnel/backend/scrapers/glassdoor.py

@@ -2,10 +2,10 @@
 FIXME: this is currently unable to get past page 1 of job results.
 """
 
-import re
 from abc import abstractmethod
 from concurrent.futures import ThreadPoolExecutor, wait
 from math import ceil
+import re
 from typing import Any, Dict, List, Tuple, Union
 
 from bs4 import BeautifulSoup
@@ -15,10 +15,9 @@
 from jobfunnel.backend.scrapers.base import (
     BaseCANEngScraper,
     BaseScraper,
-    BaseUSAEngScraper,
     BaseUKEngScraper,
+    BaseUSAEngScraper,
 )
-from jobfunnel.backend.tools import get_webdriver
 from jobfunnel.backend.tools.filters import JobFilter
 from jobfunnel.backend.tools.tools import calc_post_date_from_relative_str
 from jobfunnel.resources import MAX_CPU_WORKERS, JobField

jobfunnel/backend/scrapers/indeed.py

@@ -1,33 +1,33 @@
 """Scraper designed to get jobs from www.indeed.X
 """
 
-import re
 from concurrent.futures import ThreadPoolExecutor, wait
+import json
 from math import ceil
+import random
+import re
 from typing import Any, Dict, List, Optional
 from unicodedata import normalize
-import json
-import random
 
 from bs4 import BeautifulSoup
 from requests import Session
 
 from jobfunnel.backend import Job
 from jobfunnel.backend.scrapers.base import (
     BaseCANEngScraper,
+    BaseDEGerScraper,
+    BaseFRFreScraper,
     BaseScraper,
-    BaseUSAEngScraper,
     BaseUKEngScraper,
-    BaseFRFreScraper,
-    BaseDEGerScraper,
+    BaseUSAEngScraper,
 )
 from jobfunnel.backend.tools.filters import JobFilter
 from jobfunnel.backend.tools.tools import calc_post_date_from_relative_str
 from jobfunnel.resources import (
     MAX_CPU_WORKERS,
+    USER_AGENT_LIST_MOBILE,
     JobField,
     Remoteness,
-    USER_AGENT_LIST_MOBILE,
 )
 
 # pylint: disable=using-constant-test,unused-import

jobfunnel/backend/scrapers/monster.py

@@ -1,9 +1,9 @@
 """Scrapers for www.monster.X
 """
 
-import re
 from abc import abstractmethod
 from math import ceil
+import re
 from typing import Any, Dict, List, Optional
 
 from bs4 import BeautifulSoup
@@ -12,10 +12,10 @@
 from jobfunnel.backend import Job
 from jobfunnel.backend.scrapers.base import (
     BaseCANEngScraper,
+    BaseFRFreScraper,
     BaseScraper,
-    BaseUSAEngScraper,
     BaseUKEngScraper,
-    BaseFRFreScraper,
+    BaseUSAEngScraper,
 )
 from jobfunnel.backend.tools.filters import JobFilter
 from jobfunnel.backend.tools.tools import calc_post_date_from_relative_str

jobfunnel/backend/scrapers/registry.py

@@ -4,26 +4,25 @@
 TODO: there must be a better way to do this by using class attrib of Provider
 """
 
-from jobfunnel.resources import Locale, Provider
-
+from jobfunnel.backend.scrapers.glassdoor import (
+    GlassDoorScraperCANEng,
+    GlassDoorScraperUKEng,
+    GlassDoorScraperUSAEng,
+)
 from jobfunnel.backend.scrapers.indeed import (
     IndeedScraperCANEng,
-    IndeedScraperUSAEng,
-    IndeedScraperUKEng,
-    IndeedScraperFRFre,
     IndeedScraperDEGer,
+    IndeedScraperFRFre,
+    IndeedScraperUKEng,
+    IndeedScraperUSAEng,
 )
 from jobfunnel.backend.scrapers.monster import (
     MonsterScraperCANEng,
-    MonsterScraperUSAEng,
-    MonsterScraperUKEng,
     MonsterScraperFRFre,
+    MonsterScraperUKEng,
+    MonsterScraperUSAEng,
 )
-from jobfunnel.backend.scrapers.glassdoor import (
-    GlassDoorScraperCANEng,
-    GlassDoorScraperUSAEng,
-    GlassDoorScraperUKEng,
-)
+from jobfunnel.resources import Locale, Provider
 
 SCRAPER_FROM_LOCALE = {
     # search terms which one to use

jobfunnel/backend/tools/__init__.py

@@ -1,3 +1,5 @@
-from jobfunnel.backend.tools.tools import get_webdriver, get_logger, Logger
+from jobfunnel.backend.tools.tools import Logger, get_logger, get_webdriver
+
+__all__ = ["get_webdriver", "get_logger", "Logger"]
 
 # NOTE: we can't import delays here or we cause circular import.

jobfunnel/backend/tools/delay.py

@@ -8,7 +8,7 @@
 from numpy import arange
 from scipy.special import expit  # pylint: disable=no-name-in-module
 
-from jobfunnel.config import DelayConfig
+from jobfunnel.config.delay import DelayConfig
 from jobfunnel.resources import DelayAlgorithm
 
 

jobfunnel/backend/tools/filters.py

@@ -3,10 +3,10 @@
 Paul McInnis 2020
 """
 
-import logging
 from collections import namedtuple
 from copy import deepcopy
 from datetime import datetime
+import logging
 from typing import Dict, List, Optional, Tuple
 
 import nltk

jobfunnel/backend/tools/tools.py

@@ -1,10 +1,10 @@
 """Assorted tools for all aspects of funnelin' that don't fit elsewhere
 """
 
+from datetime import date, datetime, timedelta
 import logging
 import re
 import sys
-from datetime import date, datetime, timedelta
 from typing import Optional
 
 from dateutil.relativedelta import relativedelta
@@ -14,8 +14,6 @@
 from webdriver_manager.microsoft import EdgeChromiumDriverManager, IEDriverManager
 from webdriver_manager.opera import OperaDriverManager
 
-from jobfunnel.backend import Job
-
 # Initialize list and store regex objects of date quantifiers
 HOUR_REGEX = re.compile(r"(\d+)(?:[ +]{1,3})?(?:hour|hr|heure)")
 DAY_REGEX = re.compile(r"(\d+)(?:[ +]{1,3})?(?:day|d|jour)")

jobfunnel/config/__init__.py

@@ -1,7 +1,20 @@
-from jobfunnel.config.settings import SettingsValidator, SETTINGS_YAML_SCHEMA
 from jobfunnel.config.base import BaseConfig
+from jobfunnel.config.cli import build_config_dict, get_config_manager, parse_cli
 from jobfunnel.config.delay import DelayConfig
+from jobfunnel.config.manager import JobFunnelConfigManager
 from jobfunnel.config.proxy import ProxyConfig
 from jobfunnel.config.search import SearchConfig
-from jobfunnel.config.manager import JobFunnelConfigManager
-from jobfunnel.config.cli import parse_cli, get_config_manager, build_config_dict
+from jobfunnel.config.settings import SETTINGS_YAML_SCHEMA, SettingsValidator
+
+__all__ = [
+    "SettingsValidator",
+    "SETTINGS_YAML_SCHEMA",
+    "BaseConfig",
+    "DelayConfig",
+    "ProxyConfig",
+    "SearchConfig",
+    "JobFunnelConfigManager",
+    "parse_cli",
+    "get_config_manager",
+    "build_config_dict",
+]

jobfunnel/config/cli.py

@@ -2,24 +2,33 @@
 """
 
 import argparse
-from typing import Dict, Any, List
+from typing import Any, Dict, List
+
 import yaml
 
-from jobfunnel.config import (
-    DelayConfig,
-    JobFunnelConfigManager,
-    ProxyConfig,
-    SearchConfig,
-    SettingsValidator,
-)
+from jobfunnel.config.delay import DelayConfig
+from jobfunnel.config.manager import JobFunnelConfigManager
+from jobfunnel.config.proxy import ProxyConfig
+from jobfunnel.config.search import SearchConfig
+from jobfunnel.config.settings import SettingsValidator
 from jobfunnel.resources import (
     LOG_LEVEL_NAMES,
     DelayAlgorithm,
     Locale,
     Provider,
     Remoteness,
 )
-from jobfunnel.resources.defaults import *
+from jobfunnel.resources.defaults import (
+    DEFAULT_COMPANY_BLOCK_LIST,
+    DEFAULT_DELAY_ALGORITHM,
+    DEFAULT_DELAY_MAX_DURATION,
+    DEFAULT_DELAY_MIN_DURATION,
+    DEFAULT_LOG_LEVEL_NAME,
+    DEFAULT_MAX_LISTING_DAYS,
+    DEFAULT_PROVIDER_NAMES,
+    DEFAULT_REMOTENESS,
+    DEFAULT_SEARCH_RADIUS,
+)
 
 
 def parse_cli(args: List[str]) -> Dict[str, Any]:
@@ -153,7 +162,7 @@ def parse_cli(args: List[str]) -> Dict[str, Any]:
         "-l",
         dest="search.locale",
         type=str,
-        choices=[l.name for l in Locale],
+        choices=[locale.name for locale in Locale],
         help="Global location and language to use to scrape the job provider"
         " website (i.e. -l CANADA_ENGLISH -p indeed --> indeed.ca).",
         required=True,
@@ -305,6 +314,7 @@ def build_config_dict(args_dict: Dict[str, Any]) -> Dict[str, Any]:
     """Parse the JobFunnel configuration settings and combine CLI, YAML and
     defaults to build a valid config dictionary for initializing config objects.
     """
+
     # Build a config that respects CLI, defaults and YAML
     # NOTE: we a passed settings YAML first so we can inject CLI after if needed
     if "settings_yaml_file" in args_dict:

jobfunnel/config/manager.py

@@ -6,7 +6,10 @@
 from typing import List, Optional
 
 from jobfunnel.backend.scrapers.registry import SCRAPER_FROM_LOCALE
-from jobfunnel.config import BaseConfig, DelayConfig, ProxyConfig, SearchConfig
+from jobfunnel.config.base import BaseConfig
+from jobfunnel.config.delay import DelayConfig
+from jobfunnel.config.proxy import ProxyConfig
+from jobfunnel.config.search import SearchConfig
 from jobfunnel.resources import BS4_PARSER
 
 # pylint: disable=using-constant-test,unused-import

jobfunnel/config/proxy.py

@@ -25,7 +25,7 @@ def validate(self) -> None:
         try:
             # try to create an IPv4 address
             ipaddress.IPv4Address(self.ip_address)
-        except:
+        except Exception:
             raise ValueError(f"{self.ip_address} is not a valid IPv4 address")
         assert isinstance(self.port, int), "Port must be an integer"
         assert self.protocol, "Protocol is not set"

jobfunnel/config/search.py

@@ -2,12 +2,13 @@
 """
 
 from typing import List, Optional
+
 from jobfunnel.config import BaseConfig
 from jobfunnel.resources import Locale, Provider, Remoteness
 from jobfunnel.resources.defaults import (
-    DEFAULT_SEARCH_RADIUS,
-    DEFAULT_MAX_LISTING_DAYS,
     DEFAULT_DOMAIN_FROM_LOCALE,
+    DEFAULT_MAX_LISTING_DAYS,
+    DEFAULT_SEARCH_RADIUS,
 )
 
 
@@ -65,7 +66,7 @@ def __init__(
 
         # Try to infer the domain string based on the locale.
         if not domain:
-            if not self.locale in DEFAULT_DOMAIN_FROM_LOCALE:
+            if self.locale not in DEFAULT_DOMAIN_FROM_LOCALE:
                 raise ValueError(f"Unknown domain for locale: {self.locale}")
             self.domain = DEFAULT_DOMAIN_FROM_LOCALE[self.locale]
         else: