Changeset 32524 in osm for applications/editors/josm/plugins/pdfimport/src

Timestamp:

2016-07-02T02:48:15+02:00 (9 years ago)

Author:

donvip

Message:

add non-regression unit tests

File:

• applications/editors/josm/plugins/pdfimport/src/pdfimport/pdfbox/PdfBoxParser.java (modified) (2 diffs)

applications/editors/josm/plugins/pdfimport/src/pdfimport/pdfbox/PdfBoxParser.java

@@ -5 +5 @@
 import java.awt.geom.Rectangle2D;
 import java.io.File;
+import java.io.IOException;
 import java.util.List;
 
@@ -22 +23 @@
         }
 
-        public void parse(File file, int maxPaths, ProgressMonitor monitor) throws Exception
-        {
-                monitor.beginTask(tr("Parsing PDF", 1));
+    public void parse(File file, int maxPaths, ProgressMonitor monitor) throws IOException {
+        monitor.beginTask(tr("Parsing PDF", 1));
 
-                PDDocument document = PDDocument.load( file);
+        try (PDDocument document = PDDocument.load(file)) {
 
-                if( document.isEncrypted() ){
-                        throw new Exception(tr("Encrypted documents not supported."));
-                }
+            if (document.isEncrypted()) {
+                throw new IllegalArgumentException(tr("Encrypted documents not supported."));
+            }
 
-                List<?> allPages = document.getDocumentCatalog().getAllPages();
+            List<?> allPages = document.getDocumentCatalog().getAllPages();
 
-                if (allPages.size() != 1) {
-                        throw new Exception(tr("The PDF file must have exactly one page."));
-                }
+            if (allPages.size() != 1) {
+                throw new IllegalArgumentException(tr("The PDF file must have exactly one page."));
+            }
+            
+            PDPage page = (PDPage) allPages.get(0);
+            PDRectangle pageSize = page.findMediaBox();
+            Integer rotationVal = page.getRotation();
+            int rotation = 0;
+            if (rotationVal != null) {
+                rotation = rotationVal.intValue();
+            }
+    
+            new PageDrawer().drawPage(new GraphicsProcessor(target, rotation, maxPaths, monitor), page);
+            this.target.bounds = new Rectangle2D.Double(
+                    pageSize.getLowerLeftX(),
+                    pageSize.getLowerLeftY(), 
+                    pageSize.getWidth(), 
+                    pageSize.getHeight());
+        }
 
-                PDPage page = (PDPage)allPages.get(0);
-                PDRectangle pageSize = page.findMediaBox();
-                Integer rotationVal = page.getRotation();
-                int rotation = 0;
-                if (rotationVal != null){
-                        rotation = rotationVal.intValue();
-                }
-
-                GraphicsProcessor p = new GraphicsProcessor(target, rotation, maxPaths, monitor);
-                PageDrawer drawer = new PageDrawer();
-                drawer.drawPage(p, page);
-                this.target.bounds = new Rectangle2D.Double(pageSize.getLowerLeftX(), pageSize.getLowerLeftY(), pageSize.getWidth(), pageSize.getHeight());
-
-                monitor.finishTask();
-        }
+        monitor.finishTask();
+    }
 }