Remove unused imports and variables in tests (#38)

tests/conftest.py

@@ -2,12 +2,9 @@
 import logging
 import itertools
 import pytest
-import py
 import os
 import subprocess
-import datetime
 import time
-import calendar
 
 default_levels = loguru._logger.Logger._levels.copy()
 

tests/test_add_options.py

@@ -1,12 +1,9 @@
 import json
 import pytest
-import sys
-import multiprocessing
 from unittest.mock import MagicMock
 from loguru import logger
 import ansimarkup
 import io
-import os
 import colorama
 
 

tests/test_catch_wrapper.py

@@ -1,7 +1,5 @@
 import pytest
-import traceback
 import textwrap
-import re
 from loguru import logger
 
 zero_division_error = "ZeroDivisionError: division by zero\n"

tests/test_defaults.py

@@ -1,6 +1,5 @@
 import pytest
 from loguru._defaults import env
-import os
 
 
 @pytest.mark.parametrize("value", ["test", ""])

tests/test_exceptions.py

@@ -1,7 +1,5 @@
 import pytest
 import textwrap
-import py
-import os
 import sys
 from itertools import zip_longest, dropwhile
 import re

tests/test_filesink_rotation.py

@@ -127,7 +127,6 @@ def test_time_rotation(monkeypatch_date, tmpdir, when, hours):
 
     i = logger.add(str(tmpdir.join("test_{time}.log")), format="{message}", rotation=when, mode="w")
 
-    from loguru._datetime import now as nownow
 
     for h, m in zip(hours, ["a", "b", "c", "d", "e"]):
         now += datetime.timedelta(hours=h)

tests/test_interception.py

@@ -1,6 +1,4 @@
-import sys
 import logging
-import pytest
 
 from loguru import logger
 
@@ -12,7 +10,7 @@ def emit(self, record):
 
 def test_formatting(writer, make_logging_logger):
     fmt = "{name} - {file.name} - {function} - {level.name} - {level.no} - {line} - {module} - {message}"
-    expected = "tests.test_interception - test_interception.py - test_formatting - Level 10 - 10 - 21 - test_interception - This is the message\n"
+    expected = "tests.test_interception - test_interception.py - test_formatting - Level 10 - 10 - 19 - test_interception - This is the message\n"
 
     logging_logger = make_logging_logger("tests", InterceptHandler())
 
@@ -60,7 +58,7 @@ def test_remove_interception(writer, make_logging_logger):
 
 
 def test_intercept_too_low(writer, make_logging_logger):
-    logging_logger = make_logging_logger("tests.test_interception", InterceptHandler())
+    make_logging_logger("tests.test_interception", InterceptHandler())
     logger.add(writer, format="{message}")
     logging.getLogger("tests").error("Nope 1")
     logging.getLogger("foobar").error("Nope 2")

tests/test_parse.py

@@ -1,5 +1,4 @@
 import pytest
-import random
 import re
 import pathlib
 import io

tests/test_propagation.py

@@ -13,7 +13,7 @@ def emit(self, record):
 def test_formatting(make_logging_logger, capsys):
     fmt = "%(name)s - %(filename)s - %(funcName)s - %(levelname)s - %(levelno)s - %(lineno)d - %(module)s - %(message)s"
     expected = "tests.test_propagation - test_propagation.py - test_formatting - DEBUG - 10 - 18 - test_propagation - This is my message\n"
-    logging_logger = make_logging_logger("tests.test_propagation", StreamHandler(sys.stderr), fmt)
+    make_logging_logger("tests.test_propagation", StreamHandler(sys.stderr), fmt)
     logger.add(PropagateHandler(), format="{message}")
     logger.debug("This {verb} my {}", "message", verb="is")
 
@@ -71,7 +71,7 @@ def test_propagate_too_high(make_logging_logger, capsys):
 
 @pytest.mark.parametrize("use_opt", [False, True])
 def test_exception(make_logging_logger, capsys, use_opt):
-    logging_logger = make_logging_logger("tests", StreamHandler(sys.stderr))
+    make_logging_logger("tests", StreamHandler(sys.stderr))
     logger.add(PropagateHandler(), format="{message}")
 
     try: