From c8c0adf65ebe22bac6e2f675be7ee874cad32f6d Mon Sep 17 00:00:00 2001
From: Aron Roberts <aron@socrates.berkeley.edu>
Date: Wed, 4 Apr 2012 13:35:23 -0700
Subject: [PATCH] CSPACE-4814: More minor changes to improve reporting of
 import results, so as to facilitate automated testing.

---
 .../services/imports/nuxeo/ImportCommand.java      | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/services/imports/service/src/main/java/org/collectionspace/services/imports/nuxeo/ImportCommand.java b/services/imports/service/src/main/java/org/collectionspace/services/imports/nuxeo/ImportCommand.java
index 9204102fe..b68ef11a3 100644
--- a/services/imports/service/src/main/java/org/collectionspace/services/imports/nuxeo/ImportCommand.java
+++ b/services/imports/service/src/main/java/org/collectionspace/services/imports/nuxeo/ImportCommand.java
@@ -44,7 +44,8 @@ public class ImportCommand {
         DocumentReader reader = null;
         DocumentWriter writer = null;
         DocumentModel docModel = null;
-        ServiceContext<PoxPayloadIn, PoxPayloadOut> ctx = null;
+        DocumentRef keyDocRef, valueDocRef;
+        String docType;
         StringBuffer dump = new StringBuffer("NO RESULTS");
         try {
             System.out.println("importTree reading file: "+file+(file!=null ? " exists? "+file.exists() : " file param is null"));
@@ -54,12 +55,17 @@ public class ImportCommand {
             // pipe.addTransformer(transformer);
             pipe.setReader(reader);
             pipe.setWriter(writer);
+            // FIXME: pipe.run() appears to return at least one type
+            // of Exception that is logged but not thrown; this is a
+            // potential workaround
             DocumentTranslationMap dtm = pipe.run();
-            DocumentRef keyDocRef, valueDocRef;
-            String docType;
             Map<DocumentRef,DocumentRef> documentRefs = dtm.getDocRefMap();
             if (documentRefs.size() > 0) {
                 dump.setLength(0);
+                // Assumes that every import request must necessarily
+                // be attempting to import at least one record.
+            } else {
+                throw new Exception("No records were successfully imported");
             }
             dump.append("<importedRecords>");
             for (Map.Entry entry: documentRefs.entrySet()) {
@@ -89,7 +95,7 @@ public class ImportCommand {
             throw e;
         } finally {
             if (reader != null) {
-                dump.append("<fullReport>"+(((LoggedXMLDirectoryReader)reader).report())+"</fullReport>");
+                dump.append("<report>"+(((LoggedXMLDirectoryReader)reader).report())+"</report>");
                 reader.close();
             }
             if (writer != null) {
-- 
2.47.3