Task #2795: Introduce unit testing - ndn-tools - NDN project issue tracking system

Actions

Copy link

Task #2795

closed

Introduce unit testing

Added by Junxiao Shi almost 9 years ago. Updated almost 9 years ago.

Status:

Closed

Priority:

High

Assignee:

Junxiao Shi

Start date:

Due date:

% Done:

100%

Estimated time:

3.00 h

Description

Introduce unit testing infrastructure using Boost.Test.

Related issues 2 (0 open — 2 closed)

Actions

Copy link

Updated by Junxiao Shi almost 9 years ago

Blocks Feature #2796: Tests for Ping and PingServer added

Actions

Copy link

Updated by Junxiao Shi almost 9 years ago

Status changed from New to In Progress

Actions

Copy link

Updated by Junxiao Shi almost 9 years ago

% Done changed from 0 to 10

http://gerrit.named-data.net/2052

I gave this a shot but there are major difficulties with wscript:

tools/foo compiles into one or more executables, not objects. This means it contains main function(s).
tools/foo directory structure is only understood by tools/foo/wscript.
tools/foo can declare extra dependencies.

tests/wscript needs to:

Avoid linking the main function(s) in tools/foo, or skip the file(s) that contains a main function.
Understand the directory structure inside tools/foo.
Make the unit testing binary depend on the extra dependencies declared in tools/foo.

I'm unsure how to achieve those.

Actions

Copy link

Updated by Alex Afanasyev almost 9 years ago

I'm a little bit confused.

Is the following logic correct:

a single unit-tests binary file
test cases are .cpp files placed under tests/foo, tests/bar, etc.
there could be other .cpp files placed in tests/foo
a single unit-tests program is linked to all .o files from all tools (with exception of files containing main function) and all relevant dependencies

Actions

Copy link

Updated by Junxiao Shi almost 9 years ago

Answer to note-4:

I haven't decided whether the unit tests are in a single binary, or one binary per tool; whichever way is easier shall be the way to go.

The other three points are accurate.

Actions

Copy link