Added wait statements to make it wait for the test to pass.

The test doesn't always produce the same results on running. This commit belongs to issue #194.
vazexqi · May 18, 2011 · ec04c36 · ec04c36
1 parent 824c692
commit ec04c36
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 38 deletions.
diff --git a/...ms.logger/src/edu/illinois/codingspectator/refactoringproblems/logger/ProblemChanges.java b/...ms.logger/src/edu/illinois/codingspectator/refactoringproblems/logger/ProblemChanges.java
@@ -33,43 +33,6 @@ public class ProblemChanges {
 
 	Set<DefaultProblemWrapper> beforeMinusAfter;
 
-//	@Override
-//	public int hashCode() {
-//		final int prime= 31;
-//		int result= 1;
-//		result= prime * result + ((afterMinusBefore == null) ? 0 : afterMinusBefore.hashCode());
-//		result= prime * result + ((beforeMinusAfter == null) ? 0 : beforeMinusAfter.hashCode());
-//		return result;
-//	}
-
-//	@Override
-//	public boolean equals(Object obj) {
-//		if (this == obj) {
-//			return true;
-//		} else if (obj instanceof ProblemChanges == false) {
-//			return false;
-//		}
-//		ProblemChanges rhs= (ProblemChanges)obj;
-//		return isEqual(afterMinusBefore, rhs.afterMinusBefore) && isEqual(beforeMinusAfter, rhs.beforeMinusAfter);
-//
-//	}
-
-
-
-
-//	private boolean isEqual(Set<DefaultProblemWrapper> lhs, Set<DefaultProblemWrapper> rhs) {
-//
-//		if (lhs.size() != rhs.size()) {
-//			return false;
-//		}
-//		for (DefaultProblemWrapper problemEntry : rhs) {
-//			if (lhs.contains(problemEntry) == false) {
-//				return false;
-//			}
-//		}
-//		return true;
-//	}
-
 	@Override
 	public int hashCode() {
 		final int prime= 31;

diff --git a/...s.codingspectator.ui.tests/src/edu/illinois/codingspectator/ui/tests/RefactoringTest.java b/...s.codingspectator.ui.tests/src/edu/illinois/codingspectator/ui/tests/RefactoringTest.java
@@ -126,7 +126,7 @@ protected void doCleanRefactoringHistory() throws CoreException {
 	private void cleanUpWorkspace() throws CoreException {
 		bot.deleteProject(getProjectName());
 		EFS.getLocalFileSystem().getStore(getActualRefactoringProblemsLogPath()).delete(EFS.NONE, null);
-		bot.sleep();
+//		bot.sleep();
 		// Deleting a project is a refactoring that gets logged in .refactorings/.workspace.
 		// We need to delete the .refactoring folder after the deletion of the project,
 		// otherwise, the next test fails because it expects the refactoring history folder to be empty initially.
@@ -175,6 +175,7 @@ public final void cleanRefactoringHistory() throws CoreException {
 	@Test
 	public final void deleteCurrentProject() throws CoreException {
 		cleanUpWorkspace();
+		bot.sleep();
 		doRefactoringLogShouldBeEmpty();
 		bot.sleep();
 	}