-
Notifications
You must be signed in to change notification settings - Fork 265
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
Improve coverage framework #4607
Improve coverage framework #4607
Conversation
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.
make coverage_functional_test
(different from make functional_test
) needs that every .test file has a different name or some omissions may occur.
We have detected 3 name repeated in .test files. This PR fix it.
NTC (informative)
@@ -110,7 +110,6 @@ TEST(AppendContextElementRequest, release) | |||
AppendContextElementRequest acer; | |||
std::string out; | |||
ContextAttribute* caP = new ContextAttribute("caName", "caType", "121"); | |||
Metadata* mdP = new Metadata("mdName", "mdType", "122"); |
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.
During the preparation of this PR we detected some unused variables in unit tests.
/home/fermin/src/fiware-orion/test/unittests/convenience/AppendContextElementRequest_test.cpp:113:33: error: unused variable ‘mdP’ [-Werror=unused-variable]
113 | Metadata* mdP = new Metadata("mdName", "mdType", "122");
| ^~~
NTC (informative)
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.
LGTM
@fisuda this PR does some small modifications to doc/manuals/admin/build_source.md. It would be great if you could do a PR with the sync in the Japanese translation, pls. Thanks! |
I sent the PR #4608. |
(JP) Add documentation about coverage framework (#4607)
Issue #4418 (and additional minor fixes)