You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@juddi.apache.org by al...@apache.org on 2014/02/19 03:10:17 UTC

svn commit: r1569609 - in /juddi/trunk: juddi-core/src/main/java/org/apache/juddi/api/impl/UDDIInquiryImpl.java uddi-tck/src/test/java/org/apache/juddi/v2/tck/UDDI_070_FindEntityIntegrationTest.java

Author: alexoree
Date: Wed Feb 19 02:10:16 2014
New Revision: 1569609

URL: http://svn.apache.org/r1569609
Log:
JUDDI-855 fixed
JUDDI-764 unignoring test case

Modified:
    juddi/trunk/juddi-core/src/main/java/org/apache/juddi/api/impl/UDDIInquiryImpl.java
    juddi/trunk/uddi-tck/src/test/java/org/apache/juddi/v2/tck/UDDI_070_FindEntityIntegrationTest.java

Modified: juddi/trunk/juddi-core/src/main/java/org/apache/juddi/api/impl/UDDIInquiryImpl.java
URL: http://svn.apache.org/viewvc/juddi/trunk/juddi-core/src/main/java/org/apache/juddi/api/impl/UDDIInquiryImpl.java?rev=1569609&r1=1569608&r2=1569609&view=diff
==============================================================================
--- juddi/trunk/juddi-core/src/main/java/org/apache/juddi/api/impl/UDDIInquiryImpl.java (original)
+++ juddi/trunk/juddi-core/src/main/java/org/apache/juddi/api/impl/UDDIInquiryImpl.java Wed Feb 19 02:10:16 2014
@@ -111,9 +111,10 @@ public class UDDIInquiryImpl extends Aut
         			String serviceKey = body.getServiceKey();
         			org.apache.juddi.model.BusinessService modelBusinessService = null;
         			try {
-        				em.find(org.apache.juddi.model.BusinessService.class, serviceKey);
+        				modelBusinessService=em.find(org.apache.juddi.model.BusinessService.class, serviceKey);
         			} catch (ClassCastException e) {
-                                logger.info("uh oh!", e);}
+                                logger.info("uh oh!", e);
+                                }
 	                if (modelBusinessService == null)
 	                    throw new InvalidKeyPassedException(new ErrorMessage("errors.invalidkey.ServiceNotFound", serviceKey));
 

Modified: juddi/trunk/uddi-tck/src/test/java/org/apache/juddi/v2/tck/UDDI_070_FindEntityIntegrationTest.java
URL: http://svn.apache.org/viewvc/juddi/trunk/uddi-tck/src/test/java/org/apache/juddi/v2/tck/UDDI_070_FindEntityIntegrationTest.java?rev=1569609&r1=1569608&r2=1569609&view=diff
==============================================================================
--- juddi/trunk/uddi-tck/src/test/java/org/apache/juddi/v2/tck/UDDI_070_FindEntityIntegrationTest.java (original)
+++ juddi/trunk/uddi-tck/src/test/java/org/apache/juddi/v2/tck/UDDI_070_FindEntityIntegrationTest.java Wed Feb 19 02:10:16 2014
@@ -350,13 +350,7 @@ public class UDDI_070_FindEntityIntegrat
 
         @Test
         public void sortByDateAsc_4() throws Exception {
-                try {
-                        findbinding(new String[]{UDDIv2Constants.sortByDateAsc});
-                        Assert.fail("unexpected success");
-                } catch (Exception ex) {
-                        logger.info("Expected failure: " + ex.getMessage());
-                        logger.debug("Expected failure: " + ex.getMessage(), ex);
-                }
+                findbinding(new String[]{UDDIv2Constants.sortByDateAsc});
         }
 
         @Test
@@ -381,13 +375,7 @@ public class UDDI_070_FindEntityIntegrat
 
         @Test
         public void sortByDateDesc_4() throws Exception {
-                try {
-                        findbinding(new String[]{UDDIv2Constants.sortByDateDesc});
-                        Assert.fail("unexpected success");
-                } catch (Exception ex) {
-                        logger.info("Expected failure: " + ex.getMessage());
-                        logger.debug("Expected failure: " + ex.getMessage(), ex);
-                }
+                findbinding(new String[]{UDDIv2Constants.sortByDateDesc});
         }
 
         @Test
@@ -448,11 +436,12 @@ public class UDDI_070_FindEntityIntegrat
         }
 
         /**
-         * This test should pass, but juddi fails for some reason with business service not found
-         * @throws Exception 
+         * This test should pass, but juddi fails for some reason with business
+         * service not found
+         *
+         * @throws Exception
          */
         @Test
-        @Ignore
         public void orAllKeys_4() throws Exception {
                 logger.info("Hello world!");
                 try {



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@juddi.apache.org
For additional commands, e-mail: commits-help@juddi.apache.org