Skip to content

Commit e65e939

Browse files
committed
🐛 💄 better error handling. fix #70
1 parent c21d36f commit e65e939

File tree

2 files changed

+11
-1
lines changed

2 files changed

+11
-1
lines changed

pyexcel_io/io.py

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -190,7 +190,12 @@ def load_data(
190190
reader = READERS.get_a_plugin(file_type, library)
191191
except NoSupportingPluginFound:
192192
if file_name:
193-
if not os.path.exists(file_name):
193+
if os.path.exists(file_name):
194+
if os.path.isfile(file_name):
195+
raise
196+
else:
197+
raise IOError("%s is not a file" % file_name)
198+
else:
194199
raise IOError("%s does not exist" % file_name)
195200
else:
196201
raise

tests/test_io.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,11 @@
1515
PY2 = sys.version_info[0] == 2
1616

1717

18+
@raises(IOError)
19+
def test_directory_name_as_file():
20+
get_data('/')
21+
22+
1823
def test_force_file_type():
1924
test_file = "force_file_type.txt"
2025
data = get_data(

0 commit comments

Comments
 (0)