You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@archiva.apache.org by jz...@apache.org on 2010/07/04 11:28:04 UTC

svn commit: r960311 - in /archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test: ./ parent/

Author: jzurbano
Date: Sun Jul  4 09:28:02 2010
New Revision: 960311

URL: http://svn.apache.org/viewvc?rev=960311&view=rev
Log:
[MRM-1394] - Replace the use of clickLinkWithText() with getSelenium().open() for accessing navigation menu pages
* replaced clickLinkWithText() with getSelenium().open()
* added clickAddIcon() and clickDeleteIcon() to be used instead of clickLinkWithXPath() in RepositoryScanningTest
* added dependsOnMethods in testClickArtifactFromBrowse of BrowseTest


Modified:
    archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/AuditLogsReportTest.java
    archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/BrowseTest.java
    archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/RepositoryScanningTest.java
    archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/RepositoryTest.java
    archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/VirtualRepositoryTest.java
    archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractArchivaTest.java
    archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractArtifactManagementTest.java
    archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractArtifactReportsTest.java
    archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractBrowseTest.java
    archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractRepositoryTest.java

Modified: archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/AuditLogsReportTest.java
URL: http://svn.apache.org/viewvc/archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/AuditLogsReportTest.java?rev=960311&r1=960310&r2=960311&view=diff
==============================================================================
--- archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/AuditLogsReportTest.java (original)
+++ archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/AuditLogsReportTest.java Sun Jul  4 09:28:02 2010
@@ -28,7 +28,7 @@ public class AuditLogsReportTest
 {
     private void goToAuditLogReports()
     {
-        clickLinkWithText( "Audit Log Report" );        
+        getSelenium().open( "/archiva/report/queryAuditLogReport.action" );
     }
     
     private void assertAuditLogsReportPage()

Modified: archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/BrowseTest.java
URL: http://svn.apache.org/viewvc/archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/BrowseTest.java?rev=960311&r1=960310&r2=960311&view=diff
==============================================================================
--- archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/BrowseTest.java (original)
+++ archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/BrowseTest.java Sun Jul  4 09:28:02 2010
@@ -36,6 +36,7 @@ public class BrowseTest 
 		assertBrowsePage();
 	}
 	
+	@Test( dependsOnMethods = { "testAddArtifactValidValues" } )
 	public void testClickArtifactFromBrowse()
 	{
 		goToBrowsePage();

Modified: archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/RepositoryScanningTest.java
URL: http://svn.apache.org/viewvc/archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/RepositoryScanningTest.java?rev=960311&r1=960310&r2=960311&view=diff
==============================================================================
--- archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/RepositoryScanningTest.java (original)
+++ archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/RepositoryScanningTest.java Sun Jul  4 09:28:02 2010
@@ -30,7 +30,7 @@ public class RepositoryScanningTest 
 	public void testAddArtifactFileType_NullValue()
 	{
 		goToRepositoryScanningPage();
-		clickLinkWithXPath( "//div[@id='contentArea']/div/div[1]/table/tbody/tr[14]/td[2]/a/img" );
+		clickAddIcon( "newpattern_0" );
 		assertTextPresent( "Unable to process blank pattern." );
 	}
 	
@@ -38,7 +38,7 @@ public class RepositoryScanningTest 
 	public void testAddArtifactFileType()
 	{
 		setFieldValue( "newpattern_0" , "**/*.dll" );
-		clickLinkWithXPath( "//div[@id='contentArea']/div/div[1]/table/tbody/tr[14]/td[2]/a/img" );
+		clickAddIcon( "newpattern_0" );
 		Assert.assertEquals(getSelenium().getTable("//div[@id='contentArea']/div/div[1]/table.13.0"), "**/*.dll" );
 	}
 	
@@ -46,7 +46,7 @@ public class RepositoryScanningTest 
 	public void testAddArtifactFileType_ExistingValue()
 	{
 		setFieldValue( "newpattern_0" , "**/*.zip" );
-		clickLinkWithXPath( "//div[@id='contentArea']/div/div[1]/table/tbody/tr[15]/td[2]/a/img" );
+		clickAddIcon( "newpattern_0" );
 		Assert.assertEquals(getSelenium().getText("//span[@class='errorMessage']"), "Not adding pattern \"**/*.zip\" to filetype artifacts as it already exists." );
 	}
 	
@@ -54,7 +54,7 @@ public class RepositoryScanningTest 
 	public void testDeleteArtifactFileType()
 	{
 		Assert.assertEquals(getSelenium().getTable("//div[@id='contentArea']/div/div[1]/table.13.0"), "**/*.dll" );
-		clickLinkWithXPath( "//div[@id='contentArea']/div/div[1]/table/tbody/tr[14]/td[2]/a/img" );
+		clickDeleteIcon( "**/*.dll" );
 		Assert.assertEquals(getSelenium().getTable("//div[@id='contentArea']/div/div[1]/table.13.0"), "" );
 	}
 	
@@ -62,7 +62,7 @@ public class RepositoryScanningTest 
 	public void testAddAutoRemove_NullValue()
 	{
 		setFieldValue( "newpattern_1" , "" );
-		clickLinkWithXPath( "//div[@id='contentArea']/div/div[2]/table/tbody/tr[4]/td[2]/a/img" );
+		clickAddIcon( "newpattern_1" );
 		assertTextPresent( "Unable to process blank pattern." );
 	}
 	
@@ -70,7 +70,7 @@ public class RepositoryScanningTest 
 	public void testAddAutoRemove_ExistingValue()
 	{
 		setFieldValue( "newpattern_1" , "**/*-" );
-		clickLinkWithXPath( "//div[@id='contentArea']/div/div[2]/table/tbody/tr[4]/td[2]/a/img" );
+		clickAddIcon( "newpattern_1" );
 		Assert.assertEquals(getSelenium().getText("//span[@class='errorMessage']"), "Not adding pattern \"**/*-\" to filetype auto-remove as it already exists." );
 	}
 	
@@ -78,7 +78,7 @@ public class RepositoryScanningTest 
 	public void testAddAutoRemove()
 	{
 		setFieldValue( "newpattern_1" , "**/*.test" );
-		clickLinkWithXPath( "//div[@id='contentArea']/div/div[2]/table/tbody/tr[4]/td[2]/a/img" );
+		clickAddIcon( "newpattern_1" );
 		Assert.assertEquals(getSelenium().getTable("//div[@id='contentArea']/div/div[2]/table.3.0"), "**/*.test" );
 	}
 	
@@ -86,7 +86,7 @@ public class RepositoryScanningTest 
 	public void testDeleteAutoRemove()
 	{
 		Assert.assertEquals(getSelenium().getTable("//div[@id='contentArea']/div/div[2]/table.3.0"), "**/*.test" );
-		clickLinkWithXPath( "//div[@id='contentArea']/div/div[2]/table/tbody/tr[4]/td[2]/a/img" );
+		clickDeleteIcon( "**/*.test" );
 		Assert.assertEquals(getSelenium().getTable("//div[@id='contentArea']/div/div[2]/table.3.0"), "" );
 	}
 	
@@ -94,7 +94,7 @@ public class RepositoryScanningTest 
 	public void testAddIgnoredArtifacts_NullValue()
 	{
 		setFieldValue( "newpattern_2" , "" );
-		clickLinkWithXPath( "//div[@id='contentArea']/div/div[3]/table/tbody/tr[7]/td[2]/a/img" );
+		clickAddIcon( "newpattern_2" );
 		Assert.assertEquals(getSelenium().getText("//span[@class='errorMessage']"), "Unable to process blank pattern." );
 	}
 	
@@ -102,7 +102,7 @@ public class RepositoryScanningTest 
 	public void testAddIgnoredArtifacts_ExistingValue()
 	{
 		setFieldValue( "newpattern_2" , "**/*.sh" );
-		clickLinkWithXPath( "//div[@id='contentArea']/div/div[3]/table/tbody/tr[7]/td[2]/a/img" );
+		clickAddIcon( "newpattern_2" );
 		Assert.assertEquals(getSelenium().getText("//span[@class='errorMessage']"), "Not adding pattern \"**/*.sh\" to filetype ignored as it already exists." );
 	}
 	
@@ -110,7 +110,7 @@ public class RepositoryScanningTest 
 	public void testAddIgnoredArtifacts()
 	{
 		setFieldValue( "newpattern_2" , "**/*.log" );
-		clickLinkWithXPath( "//div[@id='contentArea']/div/div[3]/table/tbody/tr[7]/td[2]/a/img" );
+		clickAddIcon( "newpattern_2" );
 		Assert.assertEquals(getSelenium().getTable("//div[@id='contentArea']/div/div[3]/table.6.0"), "**/*.log" );
 	}
 	
@@ -118,7 +118,7 @@ public class RepositoryScanningTest 
 	public void testDeleteIgnoredArtifacts()
 	{
 		Assert.assertEquals(getSelenium().getTable("//div[@id='contentArea']/div/div[3]/table.6.0"), "**/*.log" );
-		clickLinkWithXPath( "//div[@id='contentArea']/div/div[3]/table/tbody/tr[7]/td[2]/a/img" );
+		clickDeleteIcon( "**/*.log" );
 		Assert.assertEquals(getSelenium().getTable("//div[@id='contentArea']/div/div[3]/table.6.0"), "" );
 	 }
 	
@@ -127,7 +127,7 @@ public class RepositoryScanningTest 
 	public void testAddIndexableContent_NullValue()
 	{
 		setFieldValue( "newpattern_3" , "" );
-		clickLinkWithXPath( "//div[@id='contentArea']/div/div[4]/table/tbody/tr[10]/td[2]/a/img" );
+		clickAddIcon( "newpattern_3" );
 		Assert.assertEquals(getSelenium().getText("//span[@class='errorMessage']"), "Unable to process blank pattern." );
 	}
 	
@@ -135,7 +135,7 @@ public class RepositoryScanningTest 
 	public void testAddIndexableContent_ExistingValue()
 	{
 		setFieldValue( "newpattern_3" , "**/*.xml" );
-		clickLinkWithXPath( "//div[@id='contentArea']/div/div[4]/table/tbody/tr[10]/td[2]/a/img" );
+		clickAddIcon( "newpattern_3" );
 		Assert.assertEquals(getSelenium().getText("//span[@class='errorMessage']"), "Not adding pattern \"**/*.xml\" to filetype indexable-content as it already exists." );
 	}
 	
@@ -143,7 +143,7 @@ public class RepositoryScanningTest 
 	public void testAddIndexableContent()
 	{
 		setFieldValue( "newpattern_3" , "**/*.html" );
-		clickLinkWithXPath( "//div[@id='contentArea']/div/div[4]/table/tbody/tr[10]/td[2]/a/img" );
+		clickAddIcon( "newpattern_3" );
 		Assert.assertEquals(getSelenium().getTable("//div[@id='contentArea']/div/div[4]/table.9.0"), "**/*.html" );
 	}
 	
@@ -151,7 +151,7 @@ public class RepositoryScanningTest 
 	public void testDeleteIndexableContent()
 	{
 		Assert.assertEquals(getSelenium().getTable("//div[@id='contentArea']/div/div[4]/table.9.0"), "**/*.html" );
-		clickLinkWithXPath( "//div[@id='contentArea']/div/div[4]/table/tbody/tr[10]/td[2]/a/img" );
+		clickDeleteIcon( "**/*.html" );
 		Assert.assertEquals(getSelenium().getTable("//div[@id='contentArea']/div/div[4]/table.9.0"), "" );
 	}
 	
@@ -180,4 +180,16 @@ public class RepositoryScanningTest 
 		assertPage( "Apache Archiva \\ Administration - Repository Scanning" );
 	}
 	
+	private void clickAddIcon( String fieldId )
+	{
+		String xPath = "//preceding::td/input[@id='" + fieldId + "']//following::td/a/img";
+		clickLinkWithLocator( xPath );
+	}
+	
+	private void clickDeleteIcon( String pattern )
+	{
+		String xPath = "//preceding::td/code[contains(text(),'" + pattern + "')]//following::td/a/img";
+		clickLinkWithLocator( xPath );
+	}
+	
 }

Modified: archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/RepositoryTest.java
URL: http://svn.apache.org/viewvc/archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/RepositoryTest.java?rev=960311&r1=960310&r2=960311&view=diff
==============================================================================
--- archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/RepositoryTest.java (original)
+++ archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/RepositoryTest.java Sun Jul  4 09:28:02 2010
@@ -29,7 +29,7 @@ public class RepositoryTest
 	public void testAddManagedRepoValidValues()
 	{
 		goToRepositoriesPage();
-		clickLinkWithLocator( "//div[@id='contentArea']/div/div/a[@href='/archiva/admin/addRepository.action']" );
+		getSelenium().open( "/archiva/admin/addRepository.action" );
 		addManagedRepository( "managedrepo1", "Managed Repository Sample 1" , getRepositoryDir() + "repository/" , "", "Maven 2.x Repository", "0 0 * * * ?", "", "" );
 		clickButtonWithValue( "Save" );
 		assertTextPresent( "Managed Repository Sample 1" );		
@@ -40,7 +40,7 @@ public class RepositoryTest
 	public void testAddManagedRepoInvalidValues()
 	{				
 		goToRepositoriesPage();
-	    clickLinkWithLocator( "//div[@id='contentArea']/div/div/a[@href='/archiva/admin/addRepository.action']" );	
+		getSelenium().open( "/archiva/admin/addRepository.action" );	;	
 		addManagedRepository( "", "" , "" , "", "Maven 2.x Repository", "", "", "" );
 		assertTextPresent( "You must enter a repository identifier." );
 		assertTextPresent( "You must enter a repository name." );
@@ -105,7 +105,7 @@ public class RepositoryTest
 	@Test(dependsOnMethods = { "testAddRemoteRepoValidValues" } )
 	public void testAddRemoteRepoNullValues()
 	{		
-	    clickLinkWithLocator( "//div[@id='contentArea']/div/div/a[@href='/archiva/admin/addRemoteRepository.action']" );
+	    getSelenium().open( "/archiva/admin/addRemoteRepository.action" );
 		addRemoteRepository( "" , "" , "" , "" , "" , "" , "Maven 2.x Repository" );
 		assertTextPresent( "You must enter a repository identifier." );
 		assertTextPresent( "You must enter a repository name." );
@@ -136,7 +136,7 @@ public class RepositoryTest
 	@Test(dependsOnMethods = { "testDeleteManageRepo" } )
 	public void testAddRemoteRepoValidValues()
 	{		
-		clickLinkWithLocator( "//div[@id='contentArea']/div/div/a[@href='/archiva/admin/addRemoteRepository.action']" );
+		getSelenium().open( "/archiva/admin/addRemoteRepository.action" );
 		addRemoteRepository( "remoterepo" , "Remote Repository Sample" , "http://repository.codehaus.org/org/codehaus/mojo/" , "" , "" , "" , "Maven 2.x Repository" );
 		assertTextPresent( "Remote Repository Sample" );
 	}

Modified: archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/VirtualRepositoryTest.java
URL: http://svn.apache.org/viewvc/archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/VirtualRepositoryTest.java?rev=960311&r1=960310&r2=960311&view=diff
==============================================================================
--- archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/VirtualRepositoryTest.java (original)
+++ archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/VirtualRepositoryTest.java Sun Jul  4 09:28:02 2010
@@ -53,6 +53,7 @@ public class VirtualRepositoryTest 
 	@Test(dependsOnMethods = { "testAddRepositoryToRepositoryGroup" } )
 	public void testDeleteRepositoryOfRepositoryGroup()
 	{
+		goToRepositoryGroupsPage();
 		deleteRepositoryInRepositoryGroups();
 		assertTextPresent( "Repository Groups" );
 		assertTextNotPresent( "No Repository Groups Defined." );

Modified: archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractArchivaTest.java
URL: http://svn.apache.org/viewvc/archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractArchivaTest.java?rev=960311&r1=960310&r2=960311&view=diff
==============================================================================
--- archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractArchivaTest.java (original)
+++ archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractArchivaTest.java Sun Jul  4 09:28:02 2010
@@ -146,7 +146,7 @@ public abstract class AbstractArchivaTes
     //User Management
 	public void goToUserManagementPage()
 	{
-		clickLinkWithText( "User Management" );
+		getSelenium().open( "/archiva/security/userlist.action" );
 		assertUserManagementPage();
 	}
 	
@@ -216,7 +216,7 @@ public abstract class AbstractArchivaTes
 	private void createUser( String userName, String fullName, String emailAd, String password, String confirmPassword, boolean valid ) 
 	{
 		//login( getAdminUsername() , getAdminPassword() );
-		clickLinkWithText( "User Management" );
+		getSelenium().open( "/archiva/security/userlist.action" );
 		clickButtonWithValue( "Create New User" );
 		assertCreateUserPage();
         setFieldValue( "user.username", userName );
@@ -409,7 +409,7 @@ public abstract class AbstractArchivaTes
 	//Find Artifact
 	public void goToFindArtifactPage()
 	{
-		clickLinkWithText( "Find Artifact" );
+		getSelenium().open( "/archiva/findArtifact.action" );
 		assertFindArtifactPage();
 	}
 	
@@ -426,7 +426,7 @@ public abstract class AbstractArchivaTes
 	//Appearance
 	public void goToAppearancePage()
 	{
-		clickLinkWithText( "Appearance" );
+		getSelenium().open( "/archiva/admin/configureAppearance.action" );
 		assertAppearancePage();
 	}
 	
@@ -452,7 +452,7 @@ public abstract class AbstractArchivaTes
 	// Upload Artifact
     public void goToAddArtifactPage()
     {
-        clickLinkWithText( "Upload Artifact" );
+        getSelenium().open( "/archiva/upload.action" );
         assertAddArtifactPage();
     }
 
@@ -522,7 +522,7 @@ public abstract class AbstractArchivaTes
     {
         if( !getTitle().equals( "Apache Archiva \\ Administration - Repositories" ) )
         {
-            clickLinkWithText( "Repositories" );
+            getSelenium().open( "/archiva/admin/repositories.action" );
         }
     	assertRepositoriesPage();
     }

Modified: archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractArtifactManagementTest.java
URL: http://svn.apache.org/viewvc/archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractArtifactManagementTest.java?rev=960311&r1=960310&r2=960311&view=diff
==============================================================================
--- archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractArtifactManagementTest.java (original)
+++ archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractArtifactManagementTest.java Sun Jul  4 09:28:02 2010
@@ -42,7 +42,7 @@ public abstract class AbstractArtifactMa
 	public void goToDeleteArtifactPage()
 	{
         login( getProperty( "ADMIN_USERNAME" ), getProperty( "ADMIN_PASSWORD" ) );
-		clickLinkWithText( "Delete Artifact" );
+		getSelenium().open( "/archiva/deleteArtifact.action" );
 		assertDeleteArtifactPage();
 	}
 		
@@ -74,7 +74,7 @@ public abstract class AbstractArtifactMa
 	// Legacy Support
 	public void goToLegacySupportPage()
 	{
-		clickLinkWithText( "Legacy Support" );
+		getSelenium().open( "/archiva/admin/legacyArtifactPath.action" );
 		assertLegacySupportPage();
 	}
 	

Modified: archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractArtifactReportsTest.java
URL: http://svn.apache.org/viewvc/archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractArtifactReportsTest.java?rev=960311&r1=960310&r2=960311&view=diff
==============================================================================
--- archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractArtifactReportsTest.java (original)
+++ archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractArtifactReportsTest.java Sun Jul  4 09:28:02 2010
@@ -26,7 +26,7 @@ public abstract class AbstractArtifactRe
 	//Reports
 	public void goToReportsPage()
 	{
-		clickLinkWithText( "Reports" );
+		getSelenium().open( "/archiva/report/pickReport.action" );
 		assertReportsPage();
 	}
 	
@@ -77,4 +77,4 @@ public abstract class AbstractArtifactRe
 		clickButtonWithValue( "View Statistics" );
 	}
 
-}
\ No newline at end of file
+}

Modified: archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractBrowseTest.java
URL: http://svn.apache.org/viewvc/archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractBrowseTest.java?rev=960311&r1=960310&r2=960311&view=diff
==============================================================================
--- archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractBrowseTest.java (original)
+++ archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractBrowseTest.java Sun Jul  4 09:28:02 2010
@@ -26,7 +26,7 @@ public abstract class AbstractBrowseTest
 	//Browse
 	public void goToBrowsePage()
 	{
-		clickLinkWithText( "Browse" );
+		getSelenium().open( "/archiva/browse" );
 		assertBrowsePage();
 	}
 
@@ -37,4 +37,4 @@ public abstract class AbstractBrowseTest
 		assertTextPresent( "Groups" );
 	}
 
-}
\ No newline at end of file
+}

Modified: archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractRepositoryTest.java
URL: http://svn.apache.org/viewvc/archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractRepositoryTest.java?rev=960311&r1=960310&r2=960311&view=diff
==============================================================================
--- archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractRepositoryTest.java (original)
+++ archiva/branches/archiva-1.3.x/archiva-modules/archiva-web/archiva-webapp-test/src/test/testng/org/apache/archiva/web/test/parent/AbstractRepositoryTest.java Sun Jul  4 09:28:02 2010
@@ -30,7 +30,7 @@ public abstract class AbstractRepository
 	{
 	    if( !getTitle().equals( "Apache Archiva \\ Administration - Repository Groups" ) )
 	    {
-	        clickLinkWithText( "Repository Groups" );
+	        getSelenium().open( "/archiva/admin/repositoryGroups.action" );
 	    }
 		assertRepositoryGroupsPage();
 	}
@@ -67,7 +67,7 @@ public abstract class AbstractRepository
 		assertPage( "Apache Archiva \\ Admin: Delete Repository Group" );
 		assertTextPresent( "WARNING: This operation can not be undone." );
 		assertTextPresent( "Are you sure you want to delete the following repository group?" );
-		assertTextPresent( "ID: " + repositoryName );
+		assertElementPresent( "//preceding::td[text()='ID:']//following::td/code[text()='" + repositoryName + "']" );
 		assertButtonWithValuePresent( "Confirm" );
 		assertButtonWithValuePresent( "Cancel" );
 	}