Francis Charpin reported that whitespace between the equals sign was leading to errors. I don't recall if there was a reason for not supporting that, so adding a test for this along with Francis' fix to trim the key/value in the custom properties file.

Author: hen

simple-jndi/src/java/org/osjava/sj/loader/util/CustomProperties.java

@@ -67,7 +67,7 @@ public synchronized void load(InputStream in) throws IOException {
  // split equals sign
  idx = line.indexOf('=');
  if(idx != -1) {
- this.setProperty(line.substring(0,idx), line.substring(idx+1));
+ this.setProperty(line.substring(0,idx).trim(), line.substring(idx+1).trim());
  } else {
  // blank line, or just a bad line
  // we ignore it

simple-jndi/src/test/config/default.properties

@@ -2,3 +2,4 @@ com.genjava=Foo
 name=Henri
 url=yandell.org
 name=Fred
+withspace = Check

simple-jndi/src/test/org/osjava/sj/loader/JndiLoaderTest.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2003-2005, Henri Yandell
+ * Copyright (c) 2003-2013, Henri Yandell
  * All rights reserved.
  * 
  * Redistribution and use in source and binary forms, with or 
@@ -132,6 +132,7 @@ public void testDefaultFile() {
  assertEquals( "Henri", list.get(0) );
  assertEquals( "Fred", list.get(1) );
  assertEquals( "Foo", ctxt.lookup("com.genjava") );
+ assertEquals( "Check", ctxt.lookup("withspace") );
  } catch(IOException ioe) {
  ioe.printStackTrace();
  fail("IOException: "+ioe.getMessage());