[ https://issues.apache.org/jira/browse/CONNECTORS-341?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13177135#comment-13177135 ]
Martin Goldhahn commented on CONNECTORS-341: -------------------------------------------- Sounds good to me. I just found another issue. org.apache.manifoldcf.core.tests.Base deletes the config files. So there should be a pair of functions concerned with creation/deletion of the config files. I suggest {code} /** * This function can be used to provide configuration file and set the system property {@code org.apache.manifoldcf.configfile}. * If this function doesn't provide any config file (returns null), the system property has to point to a valid config file. */ protected String createConfigFiles() { } /** * This function can be used to cleaup the resources created with {@link #createConfigFiles} */ protected void deleteConfigFiles() { } {code} With this you can choose to write your config files > override config for test cases > ------------------------------ > > Key: CONNECTORS-341 > URL: https://issues.apache.org/jira/browse/CONNECTORS-341 > Project: ManifoldCF > Issue Type: Improvement > Components: Tests > Reporter: Martin Goldhahn > Attachments: CONNECTORS-341.patch > > > It is hard to write a test case for a repository connector using the existing > *.Base classes that uses a different config file. > The config files are actually written in > org.apache.manifoldcf.core.tests.Base.localSetup(). > The localSetup method is called by all sub-classes in their overridden > localSetup function. each sub-class adds something specific for the test > case. If I wanted to put something different in the config files I had to > override localSetup() and copy all content of the superclasses' localSetup to > it. This can be avoided by moving the code that writes the config file in the > initialize() method. It makes also more sense in that the creation of the > config files and the provision of their content are now in the same function. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira