NFD

Junxiao Shi wrote:
An alternate to the suggested solution is to require sudo in README-dev.md "running unit tests" section.

Not really.

If I want to run unit tests when packaging on a Fedora(or any other) build-system, I won't have sudo. Running unit tests for packaging is required for sanity checking.

susmit shannigrahi wrote:

Junxiao Shi wrote:
An alternate to the suggested solution is to require sudo in README-dev.md "running unit tests" section.

Not really.

If I want to run unit tests when packaging on a Fedora(or any other) build-system, I won't have sudo.

+1. Requiring sudo to run unit tests is not a valid solution for packagers.

Could someone give me some idea about how to solve this bug? Am I supposed to modify the waf script?

Thanks a lot.

No. I think unit tests should be aware of whether they are running as root or normal users. Anything that requires root should be skipped with a message.

There is an example in tests/core/privilege-helper.t.cpp.

You also need to update jenkins script in .jenkins.d/20-tests.sh

@Alex, thanks for your reply.

Do you mean for each test case that needs to run as root, I need to add the following code to skip it:
if (::geteuid() != 0) {
BOOST_TEST_MESSAGE("This test case needs to be run as super user, skipping");
return;
}

If this is the case why do we still need to update .jenkins.d/20-tests.sh?

Besides, how am I supposed to reproduce this bug?

Thank you very much.

I tried to run the NFD daemon tests with ./build/unit-tests-daemon -l test_suite, but I didn't get the errors as showed in this bug, instead, for each failed test case, I'm getting the following error:

../tests/daemon/mgmt/fib-manager.t.cpp:158: Entering test case "UnknownFaceId"
1415684132.000000 INFO: [StrategyChoice] setDefaultStrategy /localhost/nfd/strategy/best-route/%FD%04
../src/security/sec-tpm-file.cpp:395: fatal error: in "virtual ndn::Block ndn::SecTpmFile::signInTpm(const uint8_t *, size_t, const ndn::Name &, ndn::DigestAlgorithm)": boost::exception_detail::clone_implboost::exception_detail::error_info_injector<ndn::SecTpmFile::Error >: FileStore: error opening file for reading: build/tmp-files/test-home/.ndn/ndnsec-tpm-file/zb5krffLzGeCok2Q5zuqJb0P8b5Fc2%cGMybeezt7WQ=.pri
../tests/daemon/mgmt/fib-manager.t.cpp:158: last checkpoint: "UnknownFaceId" entry.
../tests/daemon/mgmt/fib-manager.t.cpp:158: Leaving test case "UnknownFaceId"; testing time: 5192us

Does anyone know what's going wrong?

Weiwei Liu wrote:
I tried to run the NFD daemon tests with ./build/unit-tests-daemon -l test_suite, but I didn't get the errors as showed in this bug, instead, for each failed test case, I'm getting the following error:

[...]

Does anyone know what's going wrong?

Per our discussion yesterday, this happens in only Fedora. I haven't personally tested any other distribution.

Should we add a step to Jenkins build script that executes unit testing without root and without adding capabilities?