Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 17 additions & 0 deletions tests/test_common.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
import unittest

from wdlgen import ParameterMeta


class TestParamMeta(unittest.TestCase):
def test_quote_sanitise(self):
meta = ParameterMeta(foo='"bar"').get_string()
self.assertEqual('foo: "\\"bar\\""', meta)

def test_nl_sanitise(self):
meta = ParameterMeta(foo="bar\nbaz").get_string()
self.assertEqual('foo: "bar\\nbaz"', meta)

def test_backslackquote_sanitise(self):
meta = ParameterMeta(foo='bar\\"').get_string()
self.assertEqual('foo: "bar\\\\\\""', meta)
7 changes: 6 additions & 1 deletion wdlgen/util.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,12 @@ def convert_python_value_to_wdl_literal(val) -> str:
if isinstance(val, bool):
return "true" if val else "false"
if isinstance(val, str):
return f'"{val}"'
# sanitise string here
sanitised = val\
.replace("\\", "\\\\")\
.replace("\n", "\\n")\
.replace('"', '\\"')
return f'"{sanitised}"'

return str(val)

Expand Down