svn commit: r763835 - in /tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src: main/java/org/apache/tuscany/sca/binding/atom/provider/AtomBindingInvoker.java test/java/org/apache/tuscany/sca/data/collection/AggregatorTestCase.java

[lresende]

Author: lresende
Date: Thu Apr  9 23:12:58 2009
New Revision: 763835

URL: http://svn.apache.org/viewvc?rev=763835&view=rev
Log:
TUSCANY-2871 - Providing better exception when feed entry is not supported

Modified:
    
tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/main/java/org/apache/tuscany/sca/binding/atom/provider/AtomBindingInvoker.java
    
tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/test/java/org/apache/tuscany/sca/data/collection/AggregatorTestCase.java

Modified: 
tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/main/java/org/apache/tuscany/sca/binding/atom/provider/AtomBindingInvoker.java
URL: 
http://svn.apache.org/viewvc/tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/main/java/org/apache/tuscany/sca/binding/atom/provider/AtomBindingInvoker.java?rev=763835&r1=763834&r2=763835&view=diff
==============================================================================
--- 
tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/main/java/org/apache/tuscany/sca/binding/atom/provider/AtomBindingInvoker.java
 (original)
+++ 
tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/main/java/org/apache/tuscany/sca/binding/atom/provider/AtomBindingInvoker.java
 Thu Apr  9 23:12:58 2009
@@ -357,7 +357,13 @@
                 if (status == 200) {
                     Document<Feed> doc = abderaParser.parse(new 
InputStreamReader(getMethod.getResponseBodyAsStream()));
                     parsing = true;
-                    Feed feed = doc.getRoot();
+                    
+                    Feed feed = null;
+                    try {
+                       feed = doc.getRoot();
+                    } catch(Exception e) {
+                        throw new IllegalArgumentException("Invalid feed 
format :" + uri);
+                    }
 
                     if (provider.supportsFeedEntries()) {
                         

Modified: 
tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/test/java/org/apache/tuscany/sca/data/collection/AggregatorTestCase.java
URL: 
http://svn.apache.org/viewvc/tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/test/java/org/apache/tuscany/sca/data/collection/AggregatorTestCase.java?rev=763835&r1=763834&r2=763835&view=diff
==============================================================================
--- 
tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/test/java/org/apache/tuscany/sca/data/collection/AggregatorTestCase.java
 (original)
+++ 
tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/test/java/org/apache/tuscany/sca/data/collection/AggregatorTestCase.java
 Thu Apr  9 23:12:58 2009
@@ -53,7 +53,7 @@
         new Socket("127.0.0.1", 8085);
     }
     
-    @Ignore
+    @Ignore("TUSCANY-2939")
     public void testAggregator() throws Exception {
         Entry<String, Item>[] entries = aggregatorService.getAll();