-
Notifications
You must be signed in to change notification settings - Fork 14
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
add test for workchain #472
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #472 +/- ##
==========================================
+ Coverage 64.03% 70.45% +6.42%
==========================================
Files 40 42 +2
Lines 2736 2955 +219
==========================================
+ Hits 1752 2082 +330
+ Misses 984 873 -111
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
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.
@superstar54 thanks a lot! These are very useful tests. I only have minor requests good to me overall.
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.
Thanks for the work! @superstar54
The chrome test failed with an error, I retrigger it and let's see if it is because of timeout.
🥂 |
This PR adds tests for the QeAppWorkChain, and also includes the plugins:
bands
andpdos
.The test generates the
projection
data andbands
data, then mounts them to the QeAppWorkChain. The final result is then checked.