Added tests for BlockUniTensor and Bond #347
Conversation
Codecov Report
@@ Coverage Diff @@
## dev-master #347 +/- ##
==============================================
+ Coverage 17.79% 17.81% +0.01%
==============================================
Files 220 220
Lines 48736 48736
Branches 19932 19934 +2
==============================================
+ Hits 8672 8681 +9
+ Misses 35868 35852 -16
- Partials 4196 4203 +7 see 2 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
CMakeLists.txt
Outdated
| @@ -417,3 +417,8 @@ if(BUILD_PACKAGES) | |||
| include("Package.cmake") | |||
| include(CPack) | |||
| endif() | |||
|
|
|||
There was a problem hiding this comment.
This might not be neccessary?
There was a problem hiding this comment.
Do you mean include(CTest) is not necessary?
There was a problem hiding this comment.
Here is a commit explicitly including CTest to suppress the warning. a66fbaa
There was a problem hiding this comment.
Oh! Thanks for the clarification!
Should we remove this block from this commit?
There was a problem hiding this comment.
I guess it will not break the cmake script.
There was a problem hiding this comment.
But current cmakelist already contains include(CTest)?
There was a problem hiding this comment.
Bump up test coverage