and one other thing:
can this:
protected String[] getConfigLocations() {
return new String[] {
"classpath*:/applicationContext-resources.xml",
"classpath*:/applicationContext-dao.xml",
"classpath*:/applicationContext-service.xml",
"/WEB-INF/applicationContext*.xml",
"/WEB-INF/security.xml" };
}
be changed to a default convention:
a getConfigLocations in the
AbstractTransactionalDataSourceSpringContextTests which reads the web.xml?
tibi
tibi wrote:
hi all,
i'm testing and trying your appfuse with most of the time a smile on
my face :-D
here is what i found.....(or not found ;) )
1) i miss a lot off appfuse (test) 2 docs (but i know its not there
officially so i will wait)
2a) cobertura is cool but the latest maven plugin (2.1) gives me all
100% covarege so i needed to add:
<version>2.0</version>
2b)convention over configuration i would add (especcially because the
docs online have a bug (exludes tag is missing):
<plugin>
<groupId>org.codehaus.mojo</groupId>
<artifactId>cobertura-maven-plugin</artifactId>
<version>2.0</version>
<configuration>
<instrumentation>
<excludes>
<exclude>mypackage/**/*Test.class</exclude>
</excludes>
</instrumentation>
</configuration>
</plugin> (but i'm not sure if the
excluding is needed. sometimes tests come up in my reports)
3) i like jmock to easy and fast testing. but i found it difficult to
use a test case which extends 'BaseActionTestCase' and use Jmock
(which likes to extend 'BaseManagerMockTestCase').
to pass this little problem i did this:
=============================================================
package nl.tibi.janneke;
import org.appfuse.service.impl.BaseManagerMockTestCase;
/**
* just for extending baseamangerMocktestCase which is abstract.
*/
public class MyBaseManagerMockTestCaseDummy extends
BaseManagerMockTestCase {
}
===============================================================
package nl.tibi.janneke.action;
import nl.tibi.janneke.MyBaseManagerMockTestCaseDummy;
import nl.tibi.janneke.model.Person;
import org.appfuse.service.GenericManager;
import org.appfuse.service.impl.BaseManagerMockTestCase;
import org.appfuse.webapp.action.BaseActionTestCase;
import org.jmock.Mock;
public class PersonActionTest extends BaseActionTestCase {
private PersonAction personAction;
public void setPersonAction(PersonAction personAction) {
this.personAction = personAction;
}
public void testSave() throws Exception {
BaseManagerMockTestCase testManager = new
MyBaseManagerMockTestCaseDummy();
Mock mockManager = testManager.mock(GenericManager.class);
personAction.setPersonManager((GenericManager<Person, Long>)
mockManager.proxy());
Person person = new Person();
person.setFirstName("testFirstName");
person.setLastName("testLastName");
mockManager.expects(testManager.once()).method("save").with(testManager.eq(person));
personAction.setPerson(person);
String result = personAction.save();
assertEquals(result, PersonAction.FORM);
}
}
==========================================================
mayby add someclass in appfuse to overcome this??
4) to be able to run jetty on exploded code i needed to remove
struts.xml as well (besides the lib):
rm src/main/webapp/WEB-INF/classes/struts.xml
i think thats it for now ;)
ciao,
tibi
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
