Author: olamy
Date: Thu Oct 11 19:26:31 2012
New Revision: 1397249

URL: http://svn.apache.org/viewvc?rev=1397249&view=rev
Log:
add unit test to ensure we delete artifact without setting the packaging

Modified:
    
archiva/trunk/archiva-modules/archiva-web/archiva-rest/archiva-rest-services/src/test/java/org/apache/archiva/rest/services/RepositoriesServiceTest.java

Modified: 
archiva/trunk/archiva-modules/archiva-web/archiva-rest/archiva-rest-services/src/test/java/org/apache/archiva/rest/services/RepositoriesServiceTest.java
URL: 
http://svn.apache.org/viewvc/archiva/trunk/archiva-modules/archiva-web/archiva-rest/archiva-rest-services/src/test/java/org/apache/archiva/rest/services/RepositoriesServiceTest.java?rev=1397249&r1=1397248&r2=1397249&view=diff
==============================================================================
--- 
archiva/trunk/archiva-modules/archiva-web/archiva-rest/archiva-rest-services/src/test/java/org/apache/archiva/rest/services/RepositoriesServiceTest.java
 (original)
+++ 
archiva/trunk/archiva-modules/archiva-web/archiva-rest/archiva-rest-services/src/test/java/org/apache/archiva/rest/services/RepositoriesServiceTest.java
 Thu Oct 11 19:26:31 2012
@@ -43,7 +43,7 @@ public class RepositoriesServiceTest
     extends AbstractArchivaRestTest
 {
 
-    @Test( expected = ServerWebApplicationException.class )
+    @Test ( expected = ServerWebApplicationException.class )
     public void scanRepoKarmaFailed()
         throws Exception
     {
@@ -79,7 +79,7 @@ public class RepositoriesServiceTest
         assertTrue( service.scanRepository( repoId, true ) );
     }
 
-    @Test( expected = ServerWebApplicationException.class )
+    @Test ( expected = ServerWebApplicationException.class )
     public void deleteArtifactKarmaFailed()
         throws Exception
     {
@@ -104,7 +104,7 @@ public class RepositoriesServiceTest
         }
     }
 
-    @Test( expected = ServerWebApplicationException.class )
+    @Test ( expected = ServerWebApplicationException.class )
     public void deleteWithRepoNull()
         throws Exception
     {
@@ -129,6 +129,72 @@ public class RepositoriesServiceTest
     }
 
 
+    /**
+     * delete a version of an artifact without packaging
+     *
+     * @throws Exception
+     */
+    @Test
+    public void deleteArtifactVersion()
+        throws Exception
+    {
+        initSourceTargetRepo();
+
+        BrowseService browseService = getBrowseService( authorizationHeader, 
false );
+
+        List<Artifact> artifacts =
+            browseService.getArtifactDownloadInfos( 
"org.apache.karaf.features", "org.apache.karaf.features.core",
+                                                    "2.2.2", SOURCE_REPO_ID );
+
+        log.info( "artifacts: {}", artifacts );
+
+        Assertions.assertThat( artifacts ).isNotNull().isNotEmpty().hasSize( 2 
);
+
+        VersionsList versionsList =
+            browseService.getVersionsList( "org.apache.karaf.features", 
"org.apache.karaf.features.core",
+                                           SOURCE_REPO_ID );
+        Assertions.assertThat( versionsList.getVersions() 
).isNotNull().isNotEmpty().hasSize( 2 );
+
+        log.info( "artifacts.size: {}", artifacts.size() );
+
+        try
+        {
+            File artifactFile = new File(
+                
"target/test-origin-repo/org/apache/karaf/features/org.apache.karaf.features.core/2.2.2/org.apache.karaf.features.core-2.2.2.jar"
 );
+
+            assertTrue( "artifact not exists:" + artifactFile.getPath(), 
artifactFile.exists() );
+
+            Artifact artifact = new Artifact();
+            artifact.setGroupId( "org.apache.karaf.features" );
+            artifact.setArtifactId( "org.apache.karaf.features.core" );
+            artifact.setVersion( "2.2.2" );
+            artifact.setContext( SOURCE_REPO_ID );
+
+            RepositoriesService repositoriesService = getRepositoriesService( 
authorizationHeader );
+
+            repositoriesService.deleteArtifact( artifact );
+
+            assertFalse( "artifact not deleted exists:" + 
artifactFile.getPath(), artifactFile.exists() );
+
+            artifacts =
+                browseService.getArtifactDownloadInfos( 
"org.apache.karaf.features", "org.apache.karaf.features.core",
+                                                        "2.2.2", 
SOURCE_REPO_ID );
+
+            Assertions.assertThat( artifacts ).isNotNull().isEmpty();
+
+            versionsList = browseService.getVersionsList( 
"org.apache.karaf.features", "org.apache.karaf.features.core",
+                                                          SOURCE_REPO_ID );
+
+            Assertions.assertThat( versionsList.getVersions() 
).isNotNull().isNotEmpty().hasSize( 1 );
+
+        }
+        finally
+        {
+            cleanRepos();
+        }
+    }
+
+
     @Test
     public void deleteArtifact()
         throws Exception


Reply via email to