Skip to content

Commit 8a2de19

Browse files
authored
Merge pull request #220 from khermano/EAPQE-4810_from_master
[EAPQE-4810] - Fixing failing test in webconsole testsuite caused by missing attribute
2 parents f209aef + 1d1f5c2 commit 8a2de19

2 files changed

Lines changed: 28 additions & 12 deletions

File tree

tests-configuration-undertow/src/test/java/org/jboss/hal/testsuite/test/configuration/undertow/server/listener/HTTPListenerConfigurationTest.java

Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@
2323
import org.junit.BeforeClass;
2424
import org.junit.Test;
2525
import org.junit.runner.RunWith;
26+
import org.openqa.selenium.NoSuchElementException;
2627
import org.openqa.selenium.WebDriver;
2728
import org.wildfly.extras.creaper.core.CommandFailedException;
2829
import org.wildfly.extras.creaper.core.online.ModelNodeResult;
@@ -32,6 +33,9 @@
3233
import org.wildfly.extras.creaper.core.online.operations.Operations;
3334
import org.wildfly.extras.creaper.core.online.operations.Values;
3435

36+
import static org.junit.Assume.assumeNoException;
37+
import static org.junit.Assume.assumeTrue;
38+
3539
@RunWith(Arquillian.class)
3640
public class HTTPListenerConfigurationTest {
3741

@@ -355,12 +359,16 @@ public void editRequestParseTimeout() throws Exception {
355359

356360
@Test
357361
public void toggleRequireHostHttp11() throws Exception {
358-
boolean requireHostHttp11 = operations.readAttribute(
359-
HTTP_LISTENER_ADDRESS,
360-
"require-host-http11").booleanValue();
361-
crudOperations.update(
362-
HTTP_LISTENER_ADDRESS,
363-
page.getHttpListenerForm(), "require-host-http11", !requireHostHttp11);
362+
ModelNodeResult result = operations.readAttribute(HTTP_LISTENER_ADDRESS, "require-host-http11");
363+
assumeTrue(result.isDefined()); // if the attribute is not available in this configuration, skip the test
364+
boolean requireHostHttp11 = result.booleanValue();
365+
try {
366+
crudOperations.update(
367+
HTTP_LISTENER_ADDRESS,
368+
page.getHttpListenerForm(), "require-host-http11", !requireHostHttp11);
369+
} catch (NoSuchElementException e) {
370+
assumeNoException(e); // the UI element is not available in this configuration, skip the test
371+
}
364372
}
365373

366374
@Test

tests-configuration-undertow/src/test/java/org/jboss/hal/testsuite/test/configuration/undertow/server/listener/HTTPSListenerConfigurationTest.java

Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@
2424
import org.junit.BeforeClass;
2525
import org.junit.Test;
2626
import org.junit.runner.RunWith;
27+
import org.openqa.selenium.NoSuchElementException;
2728
import org.openqa.selenium.WebDriver;
2829
import org.wildfly.extras.creaper.core.CommandFailedException;
2930
import org.wildfly.extras.creaper.core.online.ModelNodeResult;
@@ -33,6 +34,9 @@
3334
import org.wildfly.extras.creaper.core.online.operations.Operations;
3435
import org.wildfly.extras.creaper.core.online.operations.Values;
3536

37+
import static org.junit.Assume.assumeNoException;
38+
import static org.junit.Assume.assumeTrue;
39+
3640
@RunWith(Arquillian.class)
3741
public class HTTPSListenerConfigurationTest {
3842

@@ -360,12 +364,16 @@ public void editRequestParseTimeout() throws Exception {
360364

361365
@Test
362366
public void toggleRequireHostHttp11() throws Exception {
363-
boolean requireHostHttp11 = operations.readAttribute(
364-
HTTPS_LISTENER_ADDRESS,
365-
"require-host-http11").booleanValue();
366-
crudOperations.update(
367-
HTTPS_LISTENER_ADDRESS,
368-
page.getHttpsListenerForm(), "require-host-http11", !requireHostHttp11);
367+
ModelNodeResult result = operations.readAttribute(HTTPS_LISTENER_ADDRESS, "require-host-http11");
368+
assumeTrue(result.isDefined()); // if the attribute is not available in this configuration, skip the test
369+
boolean requireHostHttp11 = result.booleanValue();
370+
try {
371+
crudOperations.update(
372+
HTTPS_LISTENER_ADDRESS,
373+
page.getHttpsListenerForm(), "require-host-http11", !requireHostHttp11);
374+
} catch (NoSuchElementException e) {
375+
assumeNoException(e); // the UI element is not available in this configuration, skip the test
376+
}
369377
}
370378

371379
@Test

0 commit comments

Comments
 (0)