-
Notifications
You must be signed in to change notification settings - Fork 59
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Gui improvements 28 06 2024 #3762
base: develop
Are you sure you want to change the base?
Conversation
…aging for clusters tab
if (!clusterMenu().context().$$(node()).stream() | ||
.filter(not(master())) | ||
.collect(toList()).isEmpty()) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we simplify this to clusterMenu().context().$$(node()).stream().anyMatch(not(master()))
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
@@ -300,4 +320,19 @@ private static Stream<String> prepareExpectedLogMessages(final String pipelineNa | |||
private static LogAO onRunPage() { | |||
return new LogAO(); | |||
} | |||
|
|||
private void checkNodes() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe it will be better to use just one forEach() where we check the condition for each node() instead of the same two stream
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
@@ -55,7 +55,8 @@ protected StorageRulesTabAO open() { | |||
|
|||
public StorageRulesTabAO deleteStorageRule(String fileMaskString) { | |||
$(byText(fileMaskString)).closest("tr").findAll(tagName("td")).get(3).find(tagName("a")).shouldHave(text("Delete")).click(); | |||
$(className("ant-confirm-title")).shouldHave(text("Do you want to delete rule \"" + fileMaskString + "\"?")); | |||
$$(className("ant-confirm-title")).filter(text("Do you want to delete rule")).first() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we add line breaks?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
public ToolSettings clickCustomParameter() { | ||
return clickAddParameter("String parameter"); | ||
} | ||
|
||
private ToolSettings clickAddParameter(String parameterType) { | ||
$(byId("add-parameter-dropdown-button")).shouldBe(visible).hover(); | ||
$(byText(parameterType)).shouldBe(visible).click(); | ||
return this; | ||
} | ||
|
||
public ToolSettings setName(String name, int parameterIndex) { | ||
SelenideElement nameField = $(byId(String.format("parameters.params.param_%d.name", parameterIndex))); | ||
nameField.click(); | ||
setValue(nameField, name); | ||
return this; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we reuse parameter logic from RunParameterAO?
@@ -15,6 +15,7 @@ | |||
*/ | |||
package com.epam.pipeline.autotests.mixins; | |||
|
|||
import static com.codeborne.selenide.Condition.exist; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Clean up?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
@@ -127,6 +127,7 @@ public class C { | |||
DEFAULT_CLUSTER_AWS_EBS_TYPE = conf.getProperty("e2e.ui.cluster.aws.ebs.type"); | |||
TEST_RUN_TAG = conf.getProperty("e2e.ui.test.run.tag"); | |||
DEFAULT_CLOUD_REGION = conf.getProperty("e2e.ui.default.cloud.region"); | |||
SSH_CLOUD_REGION = conf.getProperty("e2e.ssh.cloud.region"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe we should add this config to the default.conf
too
Signed-off-by: Yulia Kamyshova <[email protected]>
# Conflicts: # e2e/gui/src/test/java/com/epam/pipeline/autotests/LaunchParametersTest.java
return new ToolParameterAO(this, parameterIndex); | ||
} | ||
|
||
public ToolParameterAO getParameterByIndex(int parameterIndex) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This method is not used
Signed-off-by: Yulia Kamyshova <[email protected]>
Signed-off-by: Yulia Kamyshova <[email protected]>
This PR provides following updates:
umont
#2736 test was changed