Skip to content

Commit

Permalink
see if the reason that multi database test fails is because of the in…
Browse files Browse the repository at this point in the history
…terleaved test methods on the database tests
  • Loading branch information
chenson42 committed Apr 25, 2008
1 parent 8591f34 commit 451c182
Showing 1 changed file with 3 additions and 2 deletions.
Expand Up @@ -73,7 +73,8 @@ public Object[] createTests() throws Exception {
List<Object> tests2Run = new ArrayList<Object>();
Object[][] clientAndRootCombos = getClientAndRootCombos();
for (Object[] objects : clientAndRootCombos) {
tests2Run.addAll(createDatabaseTests(objects[1].toString()));
// temporarily disable to see if the mismash of test methods causes tests to fail.
//tests2Run.addAll(createDatabaseTests(objects[1].toString()));
tests2Run.addAll(createIntegrationTests(objects[0].toString(), objects[1].toString()));
}

Expand All @@ -94,7 +95,7 @@ public List<? extends AbstractTest> createIntegrationTests(String clientDatabase
/**
* If you want to add any additional node 2 node tests, subclass from AbstractIntegrationTest and add the test here. The reason
* for the anonymous inner class trick is so TestNG actually runs tests sequentially (I found that if the same test is added
* more than once each of the same test methods are run once for each instance of the class instead of running once test to completetion, then
* more than once each of the same test methods are run once for each instance of the class instead of running once test to completion, then
* another.)
*/
protected void addAbstractIntegrationTests(final File clientFile, final File rootFile,
Expand Down

0 comments on commit 451c182

Please sign in to comment.