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
DM-41131: Add pipetask report on output of a quantum graph #269
Conversation
Looks like you need to run |
383d32e
to
72ed035
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #269 +/- ##
==========================================
+ Coverage 87.07% 87.21% +0.13%
==========================================
Files 47 49 +2
Lines 4373 4419 +46
Branches 755 762 +7
==========================================
+ Hits 3808 3854 +46
Misses 407 407
Partials 158 158
☔ View full report in Codecov by Sentry. |
de611a7
to
11f1055
Compare
11f1055
to
7cfdde6
Compare
tests/test_cliCmdReport.py
Outdated
@@ -71,7 +71,7 @@ def test_report(self): | |||
|
|||
result = self.runner.invoke( | |||
pipetask_cli, | |||
["report", graph_uri, "-b", self.root, test_filename, "--no-logs"], | |||
["repo", self.root, "report", graph_uri, test_filename, "--no-logs"], |
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.
You seem to have swapped the arguments here? It should be
["report", self.root, graph_uri, test_filename, "--no-logs"]
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.
Thank you thank you thank you!
4947f88
to
09703a7
Compare
Checklist
doc/changes