-
-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added parsers for CWL input files #199
Conversation
028c895
to
47ae327
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #199 +/- ##
==========================================
- Coverage 40.99% 33.26% -7.73%
==========================================
Files 30 30
Lines 23563 31014 +7451
Branches 6406 8997 +2591
==========================================
+ Hits 9659 10316 +657
- Misses 11827 18284 +6457
- Partials 2077 2414 +337 ☔ View full report in Codecov by Sentry. |
47ae327
to
5f1886d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Huzzah! Can we get some test cases?
Added tests for the most critical parts:
As usual, complete coverage would require to run conformance tests suite using this parser. |
c923e1c
to
90f4b86
Compare
28f3137
to
90f4b86
Compare
if isinstance(doc, str): | ||
url = loadingOptions.fetcher.urljoin(baseuri, doc) | ||
if url in loadingOptions.idx: | ||
return loadingOptions.idx[url] | ||
doc_url, frg = urldefrag(url) | ||
text = loadingOptions.fetcher.fetch_text(doc_url) | ||
textIO = StringIO(text) | ||
textIO.name = str(doc_url) | ||
yaml = yaml_no_ts() | ||
result = yaml.load(textIO) | ||
add_lc_filename(result, doc_url) | ||
loadingOptions = cwl.LoadingOptions(copyfrom=loadingOptions, fileuri=doc_url) | ||
_inputfile_load( | ||
result, | ||
doc_url, | ||
loadingOptions, | ||
) | ||
return loadingOptions.idx[url] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we get a test for this code branch?
1b9dc32
to
84d2023
Compare
578dfc7
to
8ffe613
Compare
294dd35
to
8386913
Compare
117fa25
to
5f06479
Compare
6a9e041
to
08cde99
Compare
This commit adds support for parsing CWL input files with autogenerated Schema SALAD parsers, in order to obtain a coherent representation of CWL object between processfile and jobfile.
08cde99
to
76016cd
Compare
This commit adds support for parsing CWL input files with autogenerated Schema SALAD parsers, in order to obtain a coherent representation of CWL object between processfile and jobfile.
This PR depends on common-workflow-language/common-workflow-language#953, common-workflow-language/cwl-v1.1#83, and common-workflow-language/cwl-v1.2#220.