Merge pull request #56 from MarcCote/fix_empty_command

Fixed issue #54

Author: mgermain

smartdispatch/smartdispatch.py

@@ -95,7 +95,7 @@ def get_commands_from_file(fileobj):
     commands : list of str
         commands read from the file
     '''
-    return fileobj.read().split('\n')
+    return fileobj.read().strip().split('\n')
 
 
 def get_commands_from_arguments(arguments):

smartdispatch/tests/test_smartdispatch.py

@@ -72,6 +72,10 @@ def test_get_commands_from_file():
     fileobj = StringIO("\n".join(commands))
     assert_array_equal(smartdispatch.get_commands_from_file(fileobj), commands)
 
+    # Test stripping last line if empty
+    fileobj = StringIO("\n".join(commands) + "\n")
+    assert_array_equal(smartdispatch.get_commands_from_file(fileobj), commands)
+
 
 def test_get_commands_from_arguments():
     # Test single unfolded arguments