Re: svn commit: r822534 - /continuum/branches/continuum-1.3.x/continuum-core/src/test/java/org/apache/maven/continuum/DefaultContinuumTest.java

[Brett Porter]

On 07/10/2009, at 10:16 AM, c...@apache.org wrote:
+        continuum.buildProject( project.getId() );
+
+        // wait for build to finish
+        Thread.sleep( 5000 );

Is there a way to do this without building the project? This is prone  
to sporadic failure if the build is > 5s, and otherwise is incredibly  
slow.

+
+        // delete project
+        continuum.removeProject( project.getId() );
+
+        try
+        {
+            project = continuum.getProject( project.getId() );
+
+            assertNull( project );
+
+            fail( "Project was not removed" );

Do we need to check the build results were removed too?

+        }
+        catch ( Exception e )
+        {
+            // successfully removed project
+        }

This needs to catch a more specific exception.

Cheers,
Brett