fix selenium test add/delete user with webapp-js

git-svn-id: https://svn.apache.org/repos/asf/archiva/trunk@1225534 13f79535-47bb-0310-9956-ffa450edef68

3 files changed, 19 insertions, 9 deletions

diff --git a/archiva-modules/archiva-web/archiva-webapp-test-js/src/test/resources/testng.properties b/archiva-modules/archiva-web/archiva-webapp-test-js/src/test/resources/testng.properties
index c7ea281a2..1913b032f 100644
--- a/archiva-modules/archiva-web/archiva-webapp-test-js/src/test/resources/testng.properties
+++ b/archiva-modules/archiva-web/archiva-webapp-test-js/src/test/resources/testng.properties
@@ -59,7 +59,7 @@ NEW_USERROLE_PASSWORD=password123
 
 # Guest Role
 GUEST_USERNAME=guest_user
-GUEST_FULLNAME=Guest
+GUEST_FULLNAME=The Test Guest
 # Registered User Role
 REGISTERED_USERNAME=reg_user
 REGISTERED_FULLNAME=Registered User
diff --git a/archiva-modules/archiva-web/archiva-webapp-test-js/src/test/testng/org/apache/archiva/web/test/parent/AbstractArchivaTest.java b/archiva-modules/archiva-web/archiva-webapp-test-js/src/test/testng/org/apache/archiva/web/test/parent/AbstractArchivaTest.java
index b4adefc83..a785d221d 100644
--- a/archiva-modules/archiva-web/archiva-webapp-test-js/src/test/testng/org/apache/archiva/web/test/parent/AbstractArchivaTest.java
+++ b/archiva-modules/archiva-web/archiva-webapp-test-js/src/test/testng/org/apache/archiva/web/test/parent/AbstractArchivaTest.java
@@ -234,11 +234,7 @@ public abstract class AbstractArchivaTest
 
         clickLinkWithLocator( "user-create-form-register-button", true );
 
-        assertTextPresent( "user created:guest_user" );
-
-        //assertUserRolesPage();
-
-        //clickButtonWithName( "submitRolesButton", true );
+        assertTextPresent( "user created:"+userName );
 
         if ( valid )
         {
@@ -265,8 +261,12 @@ public abstract class AbstractArchivaTest
 
         clickLinkWithLocator( "users-grid-delete-" + userName );
 
+        clickLinkWithLocator( "dialog-confirm-modal-ok" );
+
         assertTextPresent( "user " + userName + " deleted" );
 
+        clickLinkWithLocator( "alert-message-success-close-a" );
+
         assertTextNotPresent( userName );
         assertTextNotPresent( fullName );
 
@@ -724,8 +724,8 @@ public abstract class AbstractArchivaTest
     {
         clickLinkWithText( "Logout" );
         assertTextNotPresent( "Current User:" );
-        assertLinkNotPresent( "Edit Details" );
-        assertLinkNotPresent( "Logout" );
-        assertLinkPresent( "Login" );
+        assertLinkNotVisible( "Edit Details" );
+        assertLinkNotVisible( "Logout" );
+        assertLinkVisible( "Login" );
     }
 }
\ No newline at end of file
diff --git a/archiva-modules/archiva-web/archiva-webapp-test-js/src/test/testng/org/apache/archiva/web/test/parent/AbstractSeleniumTest.java b/archiva-modules/archiva-web/archiva-webapp-test-js/src/test/testng/org/apache/archiva/web/test/parent/AbstractSeleniumTest.java
index 1619d33bf..03d88395d 100644
--- a/archiva-modules/archiva-web/archiva-webapp-test-js/src/test/testng/org/apache/archiva/web/test/parent/AbstractSeleniumTest.java
+++ b/archiva-modules/archiva-web/archiva-webapp-test-js/src/test/testng/org/apache/archiva/web/test/parent/AbstractSeleniumTest.java
@@ -212,6 +212,16 @@ public abstract class AbstractSeleniumTest
         Assert.assertFalse( isElementPresent( "link=" + text ), "The link('" + text + "' is present." );
     }
 
+    public void assertLinkNotVisible( String text )
+    {
+        Assert.assertFalse( isElementVisible( "link=" + text ), "The link('" + text + "' is visible." );
+    }
+
+    public void assertLinkVisible( String text )
+    {
+        Assert.assertTrue( isElementVisible( "link=" + text ), "The link('" + text + "' is not visible." );
+    }
+
     public void assertImgWithAlt( String alt )
     {
         assertElementPresent( "/¯img[@alt='" + alt + "']" );