-
Notifications
You must be signed in to change notification settings - Fork 86
Create prototype of unit-test #1210
base: master
Are you sure you want to change the base?
Conversation
change pop_count.h and CMakeLists.txt to use converter implementation |
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.
RA for C++
@lm-kajihara is this PR still WIP ? |
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.
what editor do you use? At your convenience, please update your editor config so that all text files end with a newline.
include(third/DownloadProject/DownloadProject.cmake) | ||
download_project(PROJ googletest | ||
GIT_REPOSITORY https://github.com/google/googletest.git | ||
GIT_TAG master |
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.
Don't fetch master
. Tests can break over time.
Use the tag consistent with mist/
This PR needs Approvals as follows.
Please choose reviewers and requet reviews! Click to see how to approve each reviewsYou can approve this PR by triggered comments as follows.
See all trigger commentsPlease replace [Target] to review target
|
Remove "[WIP]" from this PR name. |
Related to #743
Using googletest ([https://github.com/google/googletest]) and DownloadProject ([https://github.com/Crascit/DownloadProject])
All .cpp files in "tests/converter/unit-test/src" are unit test file of converter C++ runtime.
And all header files in "tests/converter/unit-test/include" are dummy of files in "blueoil/converter/templates/include".
This PR include only template function test.
(I forgot to remove pragma once in "tests/converter/unit-test/include")