Merge pull request #146 from chezou/zero-file

fix: adds zero size file check
chezou · May 3, 2019 · baacafe · baacafe
2 parents e61d46e + f57022d
commit baacafe
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/tabula/wrapper.py b/tabula/wrapper.py
@@ -132,6 +132,9 @@ def read_pdf(input_path,
  if not os.path.exists(path):
  raise FileNotFoundError(errno.ENOENT, os.strerror(errno.ENOENT), path)
 
+ if os.path.getsize(path) == 0:
+ raise ValueError("{} is empty. Check the file, or download it manually.".format(path))
+
  try:
  output = _run(java_options, kwargs, path, encoding)
  finally:
@@ -246,6 +249,9 @@ def convert_into(input_path, output_path, output_format='csv', java_options=None
  if not os.path.exists(path):
  raise FileNotFoundError(errno.ENOENT, os.strerror(errno.ENOENT), path)
 
+ if os.path.getsize(path) == 0:
+ raise ValueError("{} is empty. Check the file, or download it manually.".format(path))
+
  try:
  _run(java_options, kwargs, path)
  finally: