Merge pull request #91 from RickMeasham/fix-error-template

fix: Template on empty/unreadable file exception is a string, not a float

Author: eigenmannmartin

icalevents/icaldownload.py

@@ -82,7 +82,7 @@ def data_from_file(self, file, apple_fix=False):
             content = f.read()
 
         if not content:
-            raise IOError("File %f is not readable or is empty!" % file)
+            raise IOError("File %s is not readable or is empty!" % file)
 
         return self.decode(content, apple_fix=apple_fix)
 

test/test_data/empty.ics

@@ -93,3 +93,14 @@ def test_read_only_directory(self):
             # Delete tmp dir
             os.chdir("..")
             shutil.rmtree("tmp")
+
+    def test_empty_file(self):
+        empty_ical = "test/test_data/empty.ics"
+
+        with self.assertRaises(IOError) as cm:
+            icalevents.icaldownload.ICalDownload().data_from_file(empty_ical)
+
+        self.assertEqual(
+            str(cm.exception),
+            "File test/test_data/empty.ics is not readable or is empty!",
+        )