Testing of object, map, and array literals#8
Testing of object, map, and array literals#8jggatter wants to merge 4 commits intoopenwdl:masterfrom
Conversation
| 2 11 | ||
| MIXED | ||
| a 1 | ||
| b 2 No newline at end of file |
There was a problem hiding this comment.
I thought you decided that you wanted to assert that there would be a newline added here?
There was a problem hiding this comment.
I put the newline in that WDL command so that Cromwell output would match this output file. If you're referring to lack of new line at the end of file, I don't know what's right, I just wanted write_map() to give a similar format to array's write_lines()
| @@ -0,0 +1,3 @@ | |||
| { | |||
There was a problem hiding this comment.
Do you need this if the file is empty?
There was a problem hiding this comment.
FileNotFoundError: Could not find any of test_data.json,tests/test_data.json starting from /Users/jggatter/Desktop/Projects/Testathon-2020/pytest-wdl-tests/tests/whitespace
I deleted the jsons and the input dictionaries from every line in the python script and it gave me that error message. Any way around supplying inputs to py-test wdl?
|
|
||
| workflow mixed { | ||
| input { | ||
| String message = "I am a document. I am mixed. Let me fail!" |
There was a problem hiding this comment.
Really? This looks fine to me?
If I'm reading this right: The engine will strip the common spacing from the start of the command section, and then the python heredoc will have a single tab indentation in it?
There was a problem hiding this comment.
Failed to read task definition at line 17 column 6 (reason 1 of 1): Cannot mix leading whitespace characters in command: [" ", "\t"]
I get this error message, I believe it fails because of the tab in the heredoc
I did also try a version where I put a tab on line 6 or 7 and that failed similarly
cjllanwarne
left a comment
There was a problem hiding this comment.
Also, could you please add docker images for all of your tasks?
No description provided.