OO-4582: Fail if participant has to pass all course elements but a course element is failed

src/main/java/org/olat/course/nodes/st/assessment/STRootPassedEvaluator.java

@@ -90,10 +90,11 @@ public class STRootPassedEvaluator implements RootPassedEvaluator {
 		// All passed
 		if (config.getBooleanSafe(STCourseNode.CONFIG_PASSED_ALL)) {
 			Counts counts = passCounter.getCounts(courseNode, scoreAccounting);
-			if (counts.isAllAssessed() && counts.getPassable() > 0) {
-				if (counts.getPassable() == counts.getPassed()) {
+			if (counts.getPassable() > 0) {
+				if (counts.isAllAssessed() && counts.getPassable() == counts.getPassed()) {
 					return Boolean.TRUE;
-				} else if (getActivePassedConfigs(config) == 1) {
+				}
+				if (counts.getFailed() > 0 && getActivePassedConfigs(config) == 1) {
 					return Boolean.FALSE;
 				}
 			}

src/test/java/org/olat/course/nodes/st/assessment/STRootPassedEvaluatorTest.java

@@ -213,7 +213,7 @@ public class STRootPassedEvaluatorTest {
 		courseNode.getModuleConfiguration().setBooleanEntry(STCourseNode.CONFIG_PASSED_ALL, true);
 		ScoreAccounting scoreAccounting = new MappedScoreAccounting();
 		
-		Counts counts = new CountsImpl(3, 2, 1);
+		Counts counts = new CountsImpl(3, 1, 1);
 		PassCounter passCounter = mock(PassCounter.class);
 		when(passCounter.getCounts(any(), any())).thenReturn(counts);
 		STRootPassedEvaluator sut = new STRootPassedEvaluator(passCounter);