fix(jira): Check test plan only when the campaign is linked with jira…

… issue (#218)
chutney-testing · Nov 7, 2024 · d3db2ec · d3db2ec
1 parent 8f141fc
commit d3db2ec
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 0 deletions.
diff --git a/chutney/jira/src/main/java/com/chutneytesting/jira/infra/HttpJiraXrayImpl.java b/chutney/jira/src/main/java/com/chutneytesting/jira/infra/HttpJiraXrayImpl.java
@@ -171,6 +171,9 @@ public String createTestExecution(String testPlanId) {
 
     @Override
     public boolean isTestPlan(String issueId) {
+        if (issueId.isEmpty()) {
+            return false;
+        }
         return getIssue(issueId).getIssueType().getId().equals(getIssueTypeByName("Test Plan").getId());
     }
 

diff --git a/chutney/jira/src/test/java/com/chutneytesting/jira/infra/HttpJiraXrayImplTest.java b/chutney/jira/src/test/java/com/chutneytesting/jira/infra/HttpJiraXrayImplTest.java
@@ -90,6 +90,28 @@ void test_issue_as_test_plan() {
                 .hasMessage("Unable to get issue [PRJ-666] : ")
                 .hasRootCauseExactlyInstanceOf(UnknownHostException.class);
         }
+
+        @Test
+        void test_empty_issue_as_test_plan() {
+            // Given
+            String issueId = "";
+
+            var config = new JiraServerConfiguration(
+                "http://fake-server-jira",
+                "user",
+                "password",
+                null,
+                "",
+                ""
+            );
+            var sut = new HttpJiraXrayImpl(config);
+
+            // When
+            boolean result = sut.isTestPlan(issueId);
+
+            // Then
+            assertThat(result).isFalse();
+        }
     }
 
     @Nested