From: Olivier Lamy <olamy@apache.org>
Date: Thu, 29 Dec 2011 13:54:33 +0000 (+0000)
Subject: fix selenium test add/delete user with webapp-js
X-Git-Tag: archiva-1.4-M3~1675
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=cfa7d6107211a683e791baa8748d4a5aadace66a;p=archiva.git

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
---

diff --git a/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/templates/message.html b/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/templates/message.html
index 21a987216..09b5bdaff 100644
--- a/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/templates/message.html
+++ b/archiva-modules/archiva-web/archiva-webapp-js/src/main/webapp/js/archiva/templates/message.html
@@ -1,18 +1,18 @@
 <script id='alert-message-success' type='text/x-jquery-tmpl'>
   <div class="alert-message fade in success" data-alert="alert">
-    <a class="close" href="#">×</a>
+    <a class="close" href="#" id='alert-message-success-close-a'>×</a>
     <p>${message}</p>
   </div>
 </script>
 <script id='alert-message-error' type='text/x-jquery-tmpl'>
   <div class="alert-message fade in error" data-alert="alert">
-    <a class="close" href="#">×</a>
+    <a class="close" href="#" id='alert-message-error-close-a'>×</a>
     <p>${message}</p>
   </div>
 </script>
 <script id='alert-message-warning' type='text/x-jquery-tmpl'>
   <div class="alert-message fade in warning" data-alert="alert">
-    <a class="close" href="#">×</a>
+    <a class="close" href="#" id='alert-message-warning-close-a'>×</a>
     <p>${message}</p>
   </div>
 </script>
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 + "']" );