use native python logger

freqtrade/tests/optimize/test_hyperopt.py

@@ -64,7 +64,6 @@ def test_start(mocker, default_conf, caplog) -> None:
     Test start() function
     """
     start_mock = MagicMock()
-    mocker.patch('freqtrade.logger.Logger.set_format', MagicMock())
     mocker.patch('freqtrade.optimize.hyperopt.Hyperopt.start', start_mock)
     mocker.patch('freqtrade.optimize.hyperopt.hyperopt_optimize_conf',
                  MagicMock(return_value=default_conf))
@@ -184,7 +183,6 @@ def test_fmin_best_results(mocker, init_hyperopt, default_conf, caplog) -> None:
     mocker.patch('freqtrade.optimize.hyperopt.load_data', MagicMock())
     mocker.patch('freqtrade.optimize.hyperopt.fmin', return_value=fmin_result)
     mocker.patch('freqtrade.optimize.hyperopt.hyperopt_optimize_conf', return_value=conf)
-    mocker.patch('freqtrade.logger.Logger.set_format', MagicMock())
     mocker.patch('freqtrade.freqtradebot.exchange.validate_pairs', MagicMock())
 
     Strategy({'strategy': 'default_strategy'})
@@ -230,7 +228,6 @@ def test_fmin_throw_value_error(mocker, init_hyperopt, default_conf, caplog) ->
     conf.update({'timerange': None})
     conf.update({'spaces': 'all'})
     mocker.patch('freqtrade.optimize.hyperopt.hyperopt_optimize_conf', return_value=conf)
-    mocker.patch('freqtrade.logger.Logger.set_format', MagicMock())
     mocker.patch('freqtrade.freqtradebot.exchange.validate_pairs', MagicMock())
 
     Strategy({'strategy': 'default_strategy'})
@@ -274,7 +271,6 @@ def test_resuming_previous_hyperopt_results_succeeds(mocker, init_hyperopt, defa
     mocker.patch('freqtrade.optimize.hyperopt.load_data', MagicMock())
     mocker.patch('freqtrade.optimize.hyperopt.fmin', return_value={})
     mocker.patch('freqtrade.optimize.hyperopt.hyperopt_optimize_conf', return_value=conf)
-    mocker.patch('freqtrade.logger.Logger.set_format', MagicMock())
     mocker.patch('freqtrade.exchange.validate_pairs', MagicMock())
 
     Strategy({'strategy': 'default_strategy'})

freqtrade/tests/strategy/test_strategy.py

@@ -29,7 +29,6 @@ def test_strategy_structure():
 
 def test_load_strategy(result):
     strategy = Strategy()
-    strategy.logger = logging.getLogger(__name__)
 
     assert not hasattr(Strategy, 'custom_strategy')
     strategy._load_strategy('test_strategy')
@@ -42,14 +41,13 @@ def test_load_strategy(result):
 
 def test_load_not_found_strategy(caplog):
     strategy = Strategy()
-    strategy.logger = logging.getLogger(__name__)
 
     assert not hasattr(Strategy, 'custom_strategy')
     strategy._load_strategy('NotFoundStrategy')
 
     error_msg = "Impossible to load Strategy 'user_data/strategies/{}.py'. This file does not " \
                 "exist or contains Python code errors".format('NotFoundStrategy')
-    assert ('test_strategy', logging.ERROR, error_msg) in caplog.record_tuples
+    assert ('freqtrade.strategy.strategy', logging.ERROR, error_msg) in caplog.record_tuples
 
 
 def test_strategy(result):

freqtrade/tests/test_logger.py

@@ -1,97 +0,0 @@
-"""
-Unit test file for logger.py
-"""
-
-import logging
-
-from freqtrade.logger import Logger
-
-
-def test_logger_object() -> None:
-    """
-    Test the Constants object has the mandatory Constants
-    :return: None
-    """
-    logger = Logger()
-    assert logger.name == ''
-    assert logger.level == 20
-    assert logger.level is logging.INFO
-    assert hasattr(logger, 'get_logger')
-
-    logger = Logger(name='Foo', level=logging.WARNING)
-    assert logger.name == 'Foo'
-    assert logger.name != ''
-    assert logger.level == 30
-    assert logger.level is logging.WARNING
-
-
-def test_get_logger() -> None:
-    """
-    Test Logger.get_logger() and Logger._init_logger()
-    :return: None
-    """
-    logger = Logger(name='get_logger', level=logging.WARNING)
-    get_logger = logger.get_logger()
-    assert logger.logger is get_logger
-    assert get_logger is not None
-    assert hasattr(get_logger, 'debug')
-    assert hasattr(get_logger, 'info')
-    assert hasattr(get_logger, 'warning')
-    assert hasattr(get_logger, 'critical')
-    assert hasattr(get_logger, 'exception')
-
-
-def test_set_name() -> None:
-    """
-    Test Logger.set_name()
-    :return: None
-    """
-    logger = Logger(name='set_name')
-    assert logger.name == 'set_name'
-
-    logger.set_name('set_name_new')
-    assert logger.name == 'set_name_new'
-
-
-def test_set_level() -> None:
-    """
-    Test Logger.set_name()
-    :return: None
-    """
-    logger = Logger(name='Foo', level=logging.WARNING)
-    assert logger.level == logging.WARNING
-    assert logger.get_logger().level == logging.WARNING
-
-    logger.set_level(logging.INFO)
-    assert logger.level == logging.INFO
-    assert logger.get_logger().level == logging.INFO
-
-
-def test_sending_msg(caplog) -> None:
-    """
-    Test send a logging message
-    :return: None
-    """
-    logger = Logger(name='sending_msg', level=logging.WARNING).get_logger()
-
-    logger.info('I am an INFO message')
-    assert('sending_msg', logging.INFO, 'I am an INFO message') not in caplog.record_tuples
-
-    logger.warning('I am an WARNING message')
-    assert ('sending_msg', logging.WARNING, 'I am an WARNING message') in caplog.record_tuples
-
-
-def test_set_format(caplog) -> None:
-    """
-    Test Logger.set_format()
-    :return: None
-    """
-    log = Logger(name='set_format')
-    logger = log.get_logger()
-
-    logger.info('I am the first message')
-    assert ('set_format', logging.INFO, 'I am the first message') in caplog.record_tuples
-
-    log.set_format(log_format='%(message)s', propagate=True)
-    logger.info('I am the second message')
-    assert ('set_format', logging.INFO, 'I am the second message') in caplog.record_tuples