Browse Source

626: make path configurable instead of hardcoded

Anna Antipova 4 years ago
parent
commit
7eef24b373

+ 2 - 2
kafka-ui-e2e-checks/src/test/java/com/provectus/kafka/ui/pages/Pages.java

@@ -24,8 +24,8 @@ public class Pages {
         return goTo(MainPage.path).mainPage;
     }
 
-    public TopicsList openTopicsList() {
-        return goTo(TopicsList.path).topicsList;
+    public TopicsList openTopicsList(String clusterName) {
+        return goTo(TopicsList.path.formatted(clusterName)).topicsList;
     }
 
     public TopicView openTopicView(String clusterName, String topicName) {

+ 1 - 1
kafka-ui-e2e-checks/src/test/java/com/provectus/kafka/ui/pages/TopicsList.java

@@ -10,7 +10,7 @@ import static com.codeborne.selenide.Selenide.$;
 import static com.provectus.kafka.ui.helpers.WaitUtils.refreshUntil;
 
 public class TopicsList {
-    public static final String path = "ui/clusters/secondLocal/topics";
+    public static final String path = "ui/clusters/%s/topics";
 
     @Step
     public TopicsList isOnPage() {

+ 3 - 3
kafka-ui-e2e-checks/src/test/java/com/provectus/kafka/ui/topics/TopicTests.java

@@ -57,7 +57,7 @@ public class TopicTests extends BaseTest {
     @DisplayName("should update a topic")
     @Test
     void updateTopic() {
-        pages.openTopicsList()
+        pages.openTopicsList(SECOND_LOCAL)
                 .isOnPage()
                 .openTopic(UPDATE_TOPIC);
         pages.openTopicView(SECOND_LOCAL, UPDATE_TOPIC)
@@ -84,11 +84,11 @@ public class TopicTests extends BaseTest {
     @Disabled
     void deleteTopic(){
 
-        pages.openTopicsList()
+        pages.openTopicsList(SECOND_LOCAL)
                 .isOnPage()
                 .openTopic(TOPIC_TO_DELETE);
         pages.openTopicView(SECOND_LOCAL, TOPIC_TO_DELETE).clickDeleteTopicButton();
-        pages.openTopicsList().isDeleted(TOPIC_TO_DELETE);
+        pages.openTopicsList(SECOND_LOCAL).isDeleted(TOPIC_TO_DELETE);
     }
 
 }