Project

General

Profile

Actions

Task #2449

closed

Remove `tests/utility/test-face-controller.cpp` test case

Added by Alex Afanasyev almost 11 years ago. Updated over 10 years ago.

Status:
Closed
Priority:
High
Assignee:
Target version:
-
Start date:
Due date:
% Done:

0%

Estimated time:

Description

The existing TestFaceController suite has two test cases: FaceCreateCanonize and FaceCreateFailure.

I don't know what suppose to be done in the second one, but if it is just about canonization failures than the following comment applies to it as well.

It is not NLSR's job to test code in ndn-cxx library. The code is already covered by library's unit tests. If you think something is missing from tests, please submit code to ndn-cxx directly, but not implement tests inside NLSR.


As of right now, master branch does not pass unit tests on some platforms. This is the reason I'm marking this issue as high priority.


Related issues 1 (0 open1 closed)

Has duplicate NLSR - Bug #2498: FaceCreateCanonize in test-face-controller.cpp fails due to IPv4/IPv6 support assumptionClosedVince Lehman02/09/2015

Actions
Actions

Also available in: Atom PDF