From 0377dbbb3e3cea5bb89dd4731f6ebfa39ee94110 Mon Sep 17 00:00:00 2001
From: Dominik Stadler <centic@apache.org>
Date: Mon, 17 Aug 2015 20:54:32 +0000
Subject: [PATCH] Bug 54938: Throw InvalidFormatException as documented instead
 of IllegalArgumentException in WorkbookFactory.create()

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1696338 13f79535-47bb-0310-9956-ffa450edef68
---
 .../java/org/apache/poi/ss/usermodel/WorkbookFactory.java      | 2 +-
 src/ooxml/testcases/org/apache/poi/ss/TestWorkbookFactory.java | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/ooxml/java/org/apache/poi/ss/usermodel/WorkbookFactory.java b/src/ooxml/java/org/apache/poi/ss/usermodel/WorkbookFactory.java
index dfdf3f92e..8188b0287 100644
--- a/src/ooxml/java/org/apache/poi/ss/usermodel/WorkbookFactory.java
+++ b/src/ooxml/java/org/apache/poi/ss/usermodel/WorkbookFactory.java
@@ -208,7 +208,7 @@ public class WorkbookFactory {
         if (POIXMLDocument.hasOOXMLHeader(inp)) {
             return new XSSFWorkbook(OPCPackage.open(inp));
         }
-        throw new IllegalArgumentException("Your InputStream was neither an OLE2 stream, nor an OOXML stream");
+        throw new InvalidFormatException("Your InputStream was neither an OLE2 stream, nor an OOXML stream");
     }
 
     /**
diff --git a/src/ooxml/testcases/org/apache/poi/ss/TestWorkbookFactory.java b/src/ooxml/testcases/org/apache/poi/ss/TestWorkbookFactory.java
index f3fcbb889..965aa2c3a 100644
--- a/src/ooxml/testcases/org/apache/poi/ss/TestWorkbookFactory.java
+++ b/src/ooxml/testcases/org/apache/poi/ss/TestWorkbookFactory.java
@@ -30,6 +30,7 @@ import org.apache.poi.ss.usermodel.Workbook;
 import org.apache.poi.ss.usermodel.WorkbookFactory;
 import org.apache.poi.util.TempFile;
 import org.apache.poi.xssf.usermodel.XSSFWorkbook;
+import org.apache.poi.openxml4j.exceptions.InvalidFormatException;
 import org.apache.poi.openxml4j.opc.OPCPackage;
 
 import junit.framework.TestCase;
@@ -135,7 +136,7 @@ public final class TestWorkbookFactory extends TestCase {
                 stream.close();
             }
             fail();
-        } catch(IllegalArgumentException e) {
+        } catch(InvalidFormatException e) {
             // Good
         }
     }