Author: olamy
Date: Thu Nov  3 23:45:14 2011
New Revision: 1197354

URL: http://svn.apache.org/viewvc?rev=1197354&view=rev
Log:
add unit tests to ensure javadoc is respected regarding usage of repo Ids in 
SearchRequest

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

Modified: 
archiva/trunk/archiva-modules/archiva-web/archiva-rest/archiva-rest-services/src/test/java/org/apache/archiva/rest/services/SearchServiceTest.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/SearchServiceTest.java?rev=1197354&r1=1197353&r2=1197354&view=diff
==============================================================================
--- 
archiva/trunk/archiva-modules/archiva-web/archiva-rest/archiva-rest-services/src/test/java/org/apache/archiva/rest/services/SearchServiceTest.java
 (original)
+++ 
archiva/trunk/archiva-modules/archiva-web/archiva-rest/archiva-rest-services/src/test/java/org/apache/archiva/rest/services/SearchServiceTest.java
 Thu Nov  3 23:45:14 2011
@@ -24,7 +24,6 @@ import org.apache.archiva.rest.api.model
 import org.apache.archiva.rest.api.services.ManagedRepositoriesService;
 import org.apache.archiva.rest.api.services.SearchService;
 import org.apache.archiva.security.common.ArchivaRoleConstants;
-import org.apache.commons.io.FileUtils;
 import org.junit.Test;
 
 import java.io.File;
@@ -224,6 +223,70 @@ public class SearchServiceTest
     }
 
     @Test
+    /**
+     * ensure we don't return response for an unknown repo
+     */
+    public void searchWithSearchUnknwownRepoId()
+        throws Exception
+    {
+
+        String testRepoId = "test-repo";
+        // force guest user creation if not exists
+        if ( getUserService( authorizationHeader ).getGuestUser() == null )
+        {
+            assertNotNull( getUserService( authorizationHeader 
).createGuestUser() );
+        }
+
+        createAndIndexRepo( testRepoId );
+
+        SearchService searchService = getSearchService( authorizationHeader );
+
+        SearchRequest searchRequest = new SearchRequest();
+        searchRequest.setBundleExportPackage( 
"org.apache.karaf.features.command.completers" );
+        searchRequest.setRepositories( Arrays.asList( "tototititata" ) );
+
+        List<Artifact> artifacts = searchService.searchArtifacts( 
searchRequest );
+
+        assertNotNull( artifacts );
+        assertTrue( " not 0 results for Bundle ExportPackage 
org.apache.karaf.features.command.completers but "
+                        + artifacts.size() + ":" + artifacts, artifacts.size() 
== 0 );
+
+        deleteTestRepo( testRepoId );
+    }
+
+    @Test
+    /**
+     * ensure we revert to all observable repos in case of no repo in the 
request
+     */
+    public void searchWithSearchNoRepos()
+        throws Exception
+    {
+
+        String testRepoId = "test-repo";
+        // force guest user creation if not exists
+        if ( getUserService( authorizationHeader ).getGuestUser() == null )
+        {
+            assertNotNull( getUserService( authorizationHeader 
).createGuestUser() );
+        }
+
+        createAndIndexRepo( testRepoId );
+
+        SearchService searchService = getSearchService( authorizationHeader );
+
+        SearchRequest searchRequest = new SearchRequest();
+        searchRequest.setBundleExportPackage( 
"org.apache.karaf.features.command.completers" );
+
+        List<Artifact> artifacts = searchService.searchArtifacts( 
searchRequest );
+
+        assertNotNull( artifacts );
+        assertTrue( " not 0 results for Bundle ExportPackage 
org.apache.karaf.features.command.completers but "
+                        + artifacts.size() + ":" + artifacts, artifacts.size() 
== 1 );
+
+        log.info( "artifact url " + artifacts.get( 0 ).getUrl() );
+        deleteTestRepo( testRepoId );
+    }
+
+    @Test
     public void getAllGroupIds()
         throws Exception
     {


Reply via email to