[jboss-svn-commits] JBL Code SVN: r23466 - in labs/jbossrules/trunk/drools-repository/src: test/java/org/drools/repository/remoteapi and 1 other directory.

[jboss-svn-commits at lists.jboss.org]

Author: michael.neale at jboss.com
Date: 2008-10-15 22:06:52 -0400 (Wed, 15 Oct 2008)
New Revision: 23466

Modified:
   labs/jbossrules/trunk/drools-repository/src/main/java/org/drools/repository/remoteapi/RestAPI.java
   labs/jbossrules/trunk/drools-repository/src/test/java/org/drools/repository/remoteapi/RestAPITest.java
Log:
spaced in dir listing properties file returned

Modified: labs/jbossrules/trunk/drools-repository/src/main/java/org/drools/repository/remoteapi/RestAPI.java
===================================================================
--- labs/jbossrules/trunk/drools-repository/src/main/java/org/drools/repository/remoteapi/RestAPI.java	2008-10-16 02:05:04 UTC (rev 23465)
+++ labs/jbossrules/trunk/drools-repository/src/main/java/org/drools/repository/remoteapi/RestAPI.java	2008-10-16 02:06:52 UTC (rev 23466)
@@ -187,7 +187,7 @@
 				if (lastMod == null) {
 					lastMod = a.getCreatedDate();
 				}
-				sb.append(a.getName() + "." + a.getFormat() + "=" + sdf.format(lastMod.getTime()) + "," + a.getVersionNumber());
+				sb.append(a.getName().replaceAll("\\s", "\\\\ ") + "." + a.getFormat() + "=" + sdf.format(lastMod.getTime()) + "," + a.getVersionNumber());
 				sb.append('\n');
 			}
 		}
@@ -198,6 +198,10 @@
 		return r;
 	}
 
+	private String escapeSpacesForProps(String s) {
+		return s.replaceAll("\\s", "\\ ");
+	}
+
 	/**
 	 * This is the format used to sent dates as text, always.
 	 */

Modified: labs/jbossrules/trunk/drools-repository/src/test/java/org/drools/repository/remoteapi/RestAPITest.java
===================================================================
--- labs/jbossrules/trunk/drools-repository/src/test/java/org/drools/repository/remoteapi/RestAPITest.java	2008-10-16 02:05:04 UTC (rev 23465)
+++ labs/jbossrules/trunk/drools-repository/src/test/java/org/drools/repository/remoteapi/RestAPITest.java	2008-10-16 02:06:52 UTC (rev 23466)
@@ -21,10 +21,32 @@
 import org.drools.repository.remoteapi.Response.Text;
 
 public class RestAPITest extends TestCase {
-	String someAsset = "packages/SomeName/SomeFile.drl";
-	String getAList = "packages/SomeName"; //will show a list
-	String getPackageConfig = "packages/SomeName/.package"; //should load package config
+	//String someAsset = "packages/SomeName/SomeFile.drl";
+	//String getAList = "packages/SomeName"; //will show a list
+	//String getPackageConfig = "packages/SomeName/.package"; //should load package config
 
+	
+	public void testGetWithSpaces() throws Exception {
+		RulesRepository repo = RepositorySessionUtil.getRepository();
+		PackageItem pkg = repo.createPackage("testRestGetSpaces", "");
+		AssetItem ass = pkg.addAsset("some space", "");
+		ass.updateFormat("drl");
+		ass.checkin("hey");
+		
+		RestAPI api = new RestAPI(repo);
+		String url = "packages/testRestGetSpaces";
+		Response res = api.get(url);
+		
+		ByteArrayOutputStream out = new ByteArrayOutputStream();
+		res.writeData(out);
+		
+		assertTrue(new String(out.toByteArray()).indexOf("\\ ") > -1);
+		
+		url = "packages/testRestGetSpaces/some space.drl";
+		res = api.get(url);
+		assertNotNull(res.lastModified);
+		
+	}
 
 	public void testGetBasics() throws Exception {