Merge pull request #27 from myoung34/file_logging

Add ability to log to a file

README.md

@@ -40,6 +40,7 @@ $ cat <<EOF >config.ini
 [general]
 sleep_interval = 2 # defaults to 1
 logging_level = DEBUG # defaults to INFO
+logfile = /var/log/foo.log # defaults to stdout
 
 # stdout example
 [stdout]

tests/test_cli.py

@@ -90,4 +90,4 @@ def test_cli_no_params_success(
     runner = CliRunner()
     result = runner.invoke(cli.run, [])
     assert result.exit_code == 0
-    assert "Scanning for Tilt data...\n{'color': 'Black', 'gravity': 1.053, 'temp': 60, 'mac': '00:0a:95:9d:68:16', 'timestamp'" in result.output# noqa
+    assert result.output == 'Scanning for Tilt data...\n'

tilty/cli.py

@@ -48,7 +48,7 @@ def scan_and_emit(device: tilt_device.TiltDevice, emitters: List[dict]):
     tilt_data = device.scan_for_tilt_data()
     if tilt_data:
         LOGGER.debug('tilt data retrieved')
-        click.echo(tilt_data)
+        LOGGER.info(tilt_data)
         emit(emitters=emitters, tilt_data=tilt_data)
     else:
         LOGGER.debug('No tilt data')
@@ -107,12 +107,18 @@ def run(
 
     CONFIG.read(config_file)
 
+    handler = logging.StreamHandler(sys.stdout)
     logging_level = 'INFO'
     try:
         logging_level = CONFIG['general'].get('logging_level', 'INFO')
+        logfile = CONFIG['general'].get('logfile', None)
+        if logfile:
+            handler = logging.FileHandler(filename=logfile)
     except KeyError:
         pass
     LOGGER.setLevel(logging.getLevelName(logging_level))
+    handler.setLevel(logging_level)
+    LOGGER.addHandler(handler)
 
     device = tilt_device.TiltDevice()
     signal.signal(signal.SIGINT, partial(terminate_process, device))

tilty/tilty.py

@@ -2,16 +2,11 @@
 """ Class to encapsulate all the emitter logic """
 import configparser
 import logging
-import sys
 from typing import Any, List
 
 from tilty.exceptions import ConfigurationFileEmptyException
 
-LOGGER = logging.getLogger(__name__)
-LOGGER.setLevel(logging.DEBUG)
-handler = logging.StreamHandler(sys.stdout)
-handler.setLevel(logging.DEBUG)
-LOGGER.addHandler(handler)
+LOGGER = logging.getLogger()
 
 
 def parse_config(config: configparser.ConfigParser) -> List[dict]: