Add unit test for formatting

.github/actions/security-issues/action.yml

@@ -65,6 +65,7 @@ runs:
       shell: bash
       run: |
         tbx security cve create --project "${{ inputs.project }}" < issues.jsonl | tee created.jsonl
+
     - name: Define Output Parameter
       id: get-created-issues
       shell: bash

exasol/toolbox/tools/security.py

@@ -250,17 +250,15 @@ def create(
     Output:
     Links to the created issue(s)
     """
-    def _format_jsonl(issue_url: str, issue: Issue) -> str:
-        issue = asdict(issue)
-        issue["url"] = issue_url
-        return json.dumps(issue)
-
     for issue in _issues(input_file):
         std_err, issue_url = create_security_issue(issue, project)
         stderr(std_err)
-        stdout(_format_jsonl(issue_url, issue))
-
+        stdout(format_jsonl(issue_url, issue))
 
+def format_jsonl(issue_url: str, issue: Issue) -> str:
+    issue = asdict(issue)
+    issue["url"] = issue_url.strip()
+    return json.dumps(issue)
 
 if __name__ == "__main__":
     CLI()

test/unit/security_test.py

@@ -359,3 +359,14 @@ def test_convert_maven_input(maven_report):  # pylint: disable=redefined-outer-n
 def test_convert_maven_input_no_vulnerable():  # pylint: disable=redefined-outer-name
     actual = set(security.from_maven("{}"))
     assert len(actual) == 0
+
+def test_format_jsonl():
+    issue = security.Issue(coordinates="coordinates", cve="cve", cwe="cwe", description="description", references=())
+    actual = security.format_jsonl("issue_url", issue)
+    assert actual == '{"cve": "cve", "cwe": "cwe", "description": "description", "coordinates": "coordinates", "references": [], "url": "issue_url"}'
+
+
+def test_format_jsonl_removes_newline():
+    issue = security.Issue(coordinates="coordinates", cve="cve", cwe="cwe", description="description", references=())
+    actual = security.format_jsonl("issue_url\n", issue)
+    assert actual == '{"cve": "cve", "cwe": "cwe", "description": "description", "coordinates": "coordinates", "references": [], "url": "issue_url"}'