Change filename according to Python best practices

mgm-hurry

@@ -20,7 +20,7 @@ from art import tprint
 from InquirerPy import prompt
 from InquirerPy.validator import NumberValidator
 
-from user_data.mgm_tools.mgm_hurry.FreqtradeCli import FreqtradeCli
+from user_data.mgm_tools.mgm_hurry.freqtrade_cli import FreqtradeCli
 from user_data.mgm_tools.mgm_hurry.MgmCli import MgmCli
 
 from user_data.mgm_tools.mgm_hurry.LeetLogger import get_logger

tests/test_mgm-hurry.py

@@ -1,9 +0,0 @@
-
-def inc(x):
-    return x + 1
-
-def test_answer_fails():
-    assert inc(3) == 5
-
-def test_answer_succeeds():
-    assert inc(3) == 4

tests/test_mgm_hurry.py

@@ -0,0 +1,34 @@
+import sys
+sys.path.append(".")
+sys.path.append("..")
+
+from user_data.mgm_tools.mgm_hurry.freqtrade_cli import FreqtradeCli
+
+def test_initialisation_without_logger():
+    fc = FreqtradeCli('.', None)
+    assert isinstance(fc, FreqtradeCli)
+
+def test_initialisation_with_logger():
+    '''
+    Todo:
+        - Mock logger object
+    '''
+    from user_data.mgm_tools.mgm_hurry.LeetLogger import get_logger
+    logger = get_logger()
+
+    fc = FreqtradeCli('.', logger)
+    assert isinstance(fc, FreqtradeCli)
+
+def test_initialisation_with_logger():
+    '''
+    Checks if instantiation succeeds without freqtrade installation
+
+    Todo:
+        - Mock logger object
+        - Probably mock freqtrade installation
+    '''
+    from user_data.mgm_tools.mgm_hurry.LeetLogger import get_logger
+    logger = get_logger()
+
+    fc = FreqtradeCli('.', logger)
+    assert isinstance(fc, FreqtradeCli)

user_data/mgm_tools/mgm_hurry/freqtrade_cli.py

@@ -1,7 +1,7 @@
 # -*- coding: utf-8 -*-
 # -* vim: syntax=python -*-
 
-import os
+import os, sys
 
 class FreqtradeCli:
     '''
@@ -17,9 +17,13 @@ class FreqtradeCli:
         '''
         self.basedir = basedir
         self.install_type = None
-        self.logger = logger
         self.freqtrade_binary = None
 
+        if logger is None:
+            return None
+
+        self.logger = logger
+
         if os.path.exists(f"{self.basedir}/.env/bin/freqtrade") is False:
             logger.warning('🤷‍♂️ No Freqtrade installation found.')
             return None