From 334ac2a3aebcc77dec96f153d367ce7e12f96093 Mon Sep 17 00:00:00 2001
From: srosse <none@none>
Date: Tue, 11 Apr 2017 21:38:53 +0200
Subject: [PATCH] OO-2687: fix selenium tests

---
 src/test/java/org/olat/selenium/page/qti/QTI21Page.java     | 6 ++++--
 .../org/olat/selenium/page/repository/AuthoringEnvPage.java | 1 +
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/test/java/org/olat/selenium/page/qti/QTI21Page.java b/src/test/java/org/olat/selenium/page/qti/QTI21Page.java
index 4c1bc1f5a85..35705df9128 100644
--- a/src/test/java/org/olat/selenium/page/qti/QTI21Page.java
+++ b/src/test/java/org/olat/selenium/page/qti/QTI21Page.java
@@ -100,14 +100,14 @@ public class QTI21Page {
 	}
 	
 	public QTI21Page answerSingleChoice(String answer) {
-		By choiceBy = By.xpath("//tr[contains(@class,'choiceinteraction')][td[contains(@class,'choiceInteraction')][p[contains(normalize-space(text()),'" + answer + "')]]]/td[contains(@class,'control')]/input[@type='radio']");
+		By choiceBy = By.xpath("//tr[contains(@class,'choiceinteraction')][td[contains(@class,'choiceInteraction')][label/p[contains(normalize-space(text()),'" + answer + "')]]]/td[contains(@class,'control')]/input[@type='radio']");
 		browser.findElement(choiceBy).click();
 		return this;
 	}
 
 	public QTI21Page answerMultipleChoice(String... answers) {
 		for(String answer:answers) {
-			By choiceBy = By.xpath("//tr[contains(@class,'choiceinteraction')][td[contains(@class,'choiceInteraction')][p[contains(text(),'" + answer + "')]]]/td[contains(@class,'control')]/input[@type='checkbox']");
+			By choiceBy = By.xpath("//tr[contains(@class,'choiceinteraction')][td[contains(@class,'choiceInteraction')][label/p[contains(text(),'" + answer + "')]]]/td[contains(@class,'control')]/input[@type='checkbox']");
 			browser.findElement(choiceBy).click();
 		}
 		return this;
@@ -337,6 +337,7 @@ public class QTI21Page {
 		}
 		
 		By optionsBy = By.cssSelector("ul.o_sel_course_settings a.o_sel_qti_resource_options");
+		OOGraphene.waitElement(optionsBy, browser);
 		browser.findElement(optionsBy).click();
 		OOGraphene.waitBusy(browser);
 		return new QTI21OptionsPage(browser);
@@ -373,6 +374,7 @@ public class QTI21Page {
 	
 	public QTI21Page openSettingsMenu() {
 		By settingsMenuCaret = By.cssSelector("a.o_sel_course_settings");
+		OOGraphene.waitElement(settingsMenuCaret, browser);
 		browser.findElement(settingsMenuCaret).click();
 		OOGraphene.waitElement(settingsMenu, browser);
 		return this;
diff --git a/src/test/java/org/olat/selenium/page/repository/AuthoringEnvPage.java b/src/test/java/org/olat/selenium/page/repository/AuthoringEnvPage.java
index 97e65ea8b6b..d8f124d1a2f 100644
--- a/src/test/java/org/olat/selenium/page/repository/AuthoringEnvPage.java
+++ b/src/test/java/org/olat/selenium/page/repository/AuthoringEnvPage.java
@@ -197,6 +197,7 @@ public class AuthoringEnvPage {
 		if(saveButton.isEnabled()) {
 			saveButton.click();
 			OOGraphene.waitBusy(browser);
+			OOGraphene.waitElement(RepositoryEditDescriptionPage.generaltabBy, browser);
 		}
 		return this;
 	}
-- 
GitLab