Merge pull request #358 from roo-rb/raise_error_if_path_missing_from_…

…extractor Raise an error Roo::Excelx::Extractor document is missing
roo-rb · Dec 28, 2016 · b50c370 · b50c370
2 parents cb97486 + 27c81ed
commit b50c370
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/lib/roo/errors.rb b/lib/roo/errors.rb
@@ -6,4 +6,6 @@ class Error < StandardError; end
   # Raised when Roo cannot find a header row that matches the given column
   # name(s).
   class HeaderRowNotFoundError < Error; end
+
+  class FileNotFound < Error; end
 end
diff --git a/lib/roo/excelx/extractor.rb b/lib/roo/excelx/extractor.rb
@@ -8,9 +8,9 @@ def initialize(path)
       private
 
       def doc
-        if doc_exists?
-          ::Roo::Utils.load_xml(@path).remove_namespaces!
-        end
+        raise FileNotFound, "#{@path} file not found" unless doc_exists?
+
+        ::Roo::Utils.load_xml(@path).remove_namespaces!
       end
 
       def doc_exists?