From 00565435de71925749ce897c40c14cf05a7a4fc8 Mon Sep 17 00:00:00 2001
From: "Guillaume (ioguix) de Rorthais" <ioguix@free.fr>
Date: Fri, 19 Dec 2008 22:46:17 -0500
Subject: [PATCH] Revert "Fix selenium login failed for user in 99-cleantests"
 This time I will fix setUser and moving User/Group methods in pg80 instead
 (hopefuly) This reverts commit 529655bda9cc83af0ed7e37ced38bbddc4d81082.

---
 tests/selenium/src/01-roles.php      | 79 ++++++++--------------------
 tests/selenium/src/99-cleantests.php |  2 +-
 2 files changed, 22 insertions(+), 59 deletions(-)

diff --git a/tests/selenium/src/01-roles.php b/tests/selenium/src/01-roles.php
index c5735cee..d39062fb 100644
--- a/tests/selenium/src/01-roles.php
+++ b/tests/selenium/src/01-roles.php
@@ -57,86 +57,49 @@
 
 
 	/* 5 */
-	$current_username="{$user}toalter";
 	$t->addComment('5. create user role/user with altered name, pass and props');
 	if ($data->hasRoles()) {
 		$t->clickAndWait("link={$lang['strroles']}");
 		$t->clickAndWait("link={$lang['strcreaterole']}");
 		$t->type('formRolename', "{$user}toalter");
-		$t->check('formCanLogin');
-		$t->check('formCreateRole'); // will be revert
-		$t->check('formInherits'); // will be revert
+		$t->click('formCanLogin');
+		$t->click('formCreateRole'); // will be revert
+		$t->click('formInherits'); // will be revert
 	} else {
 		$t->clickAndWait("link={$lang['strusers']}");
 		$t->clickAndWait("link={$lang['strcreateuser']}");
 		if ($data->hasUserRename()) $t->type('formUsername', "{$user}toalter");
-		else {
-			$t->type('formUsername', $user);
-			$current_username=$user;
-		}
+		else $t->type('formUsername', $user);
 	}
-	$t->check('formSuper'); // will be revert
-	$t->check('formCreateDB'); // will be revert
+	$t->click('formSuper'); // will be revert
+	$t->click('formCreateDB'); // will be revert
 	$t->type('formPassword', "{$user_pass}toalter");
 	$t->type('formConfirm', "{$user_pass}toalter");
 	$t->clickAndWait('create');
-	if ($data->hasRoles()) {
-		$t->assertText("//tr/td/a[text()='{$current_username}']", $current_username);
-		$t->assertText("//tr/td/a[text()='{$current_username}']/../../td[2]", $lang['stryes']);//super user ?
-		$t->assertText("//tr/td/a[text()='{$current_username}']/../../td[3]", $lang['stryes']);//create db ?
+	if ($data->hasRoles())
 		$t->assertText("//p[@class='message']", $lang['strrolecreated']);
-		$t->assertText("//tr/td/a[text()='{$current_username}']/../../td[4]", $lang['stryes']); //create role
-		$t->assertText("//tr/td/a[text()='{$current_username}']/../../td[5]", $lang['stryes']); //inherit
-		$t->assertText("//tr/td/a[text()='{$current_username}']/../../td[6]", $lang['stryes']); //can login
-	}
-	else {
-		$t->assertText("//tr/td[text()='{$current_username}']", $current_username);
-		$t->assertText("//tr/td[text()='{$current_username}']/../td[2]", $lang['stryes']);//super user ?
-		$t->assertText("//tr/td[text()='{$current_username}']/../td[3]", $lang['stryes']);//create db ?
+	else
 		$t->assertText("//p[@class='message']", $lang['strusercreated']);
-	}
-	
-	/* 6.1 */
-	$t->addComment('6. alter user back to the normal values');
+
+	/* 6 */
+	$t->addComment('6. alter user back to the normal value');
 	if ($data->hasRoles()) {
 		$t->clickAndWait("link={$lang['strroles']}");
-		$t->clickAndWait("link={$current_username}");
+		$t->clickAndWait("link={$user}toalter");
 		$t->clickAndWait("link={$lang['stralter']}");
 		$t->type('formNewRoleName', $user);
-		$t->uncheck('formCreateRole'); // revert
-		$t->uncheck('formInherits'); // revert
+		$t->click('formCreateRole'); // revert
+		$t->click('formInherits'); // revert
 	} else {
 		$t->clickAndWait("link={$lang['strusers']}");
-		$t->clickAndWait("//tr/td[text()='{$current_username}']/../td/a[text()='{$lang['stralter']}']");
-		if ($data->hasUserRename()) $t->type('newname', $user);
-	}
-	$t->uncheck('formSuper'); // revert
-	$t->uncheck('formCreateDB'); // revert
-
-	$t->clickAndWait('alter');
-	if ($data->hasRoles()) {
-		$t->assertText("//p[@class='message']", $lang['strrolealtered']);
-		$t->assertText("//tr/td/a[text()='{$user}']", $user);
-		$t->assertText("//tr/td/a[text()='{$user}']/../../td[2]", $lang['strno']);//super user ?
-		$t->assertText("//tr/td/a[text()='{$user}']/../../td[3]", $lang['strno']);//create db ?
-		$t->assertText("//tr/td/a[text()='{$user}']/../../td[4]", $lang['strno']); //create role
-		$t->assertText("//tr/td/a[text()='{$user}']/../../td[5]", $lang['strno']); //inherit
-		$t->assertText("//tr/td/a[text()='{$user}']/../../td[6]", $lang['stryes']); //can login
-	}
-	else {
-		$t->assertText("//tr/td[text()='{$user}']", $user);
-		$t->assertText("//tr/td[text()='{$user}']/../td[2]", $lang['strno']);//super user ?
-		$t->assertText("//tr/td[text()='{$user}']/../td[3]", $lang['strno']);//create db ?
-		$t->assertText("//p[@class='message']", $lang['struserupdated']);
-	}
-
-	/* 6.2 */
-
-	if ($data->hasRoles()) {
-		$t->clickAndWait("//tr/td/a[text()='{$user}']");
-		$t->clickAndWait("link={$lang['stralter']}");
+		if ($data->hasUserRename()) {
+			$t->clickAndWait("//tr/td[text()='{$user}toalter']/../td/a[text()='{$lang['stralter']}']");
+			$t->type('newname', $user);
+		} else
+			$t->clickAndWait("//tr/td[text()='{$user}']/../td/a[text()='{$lang['stralter']}']");
 	}
-	else $t->clickAndWait("//tr/td[text()='{$user}']/../td/a[text()='{$lang['stralter']}']");
+	$t->click('formSuper'); // revert
+	$t->click('formCreateDB'); // revert
 	$t->type('formPassword', $user_pass);
 	$t->type('formConfirm', $user_pass);
 	$t->clickAndWait('alter');
diff --git a/tests/selenium/src/99-cleantests.php b/tests/selenium/src/99-cleantests.php
index aaa99b71..fa5f3b33 100644
--- a/tests/selenium/src/99-cleantests.php
+++ b/tests/selenium/src/99-cleantests.php
@@ -28,7 +28,7 @@
 /** 2 **/
 	$t->addComment('2. logout / login as admin_user and drop user role/user');
 	$t->logout();
-	$t->login($admin_user, $admin_user_pass);
+	$t->login($admin_user, $admin_pass);
 	if ($data->hasRoles()) {
 		$t->clickAndWait("link={$lang['strroles']}");
 		$t->clickAndWait("//tr/td/a[text()='{$user}']/../../td/a[text()='{$lang['strdrop']}']");
-- 
2.39.5