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 10 years ago. Updated over 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 10 years ago

Blocks Feature #2796: Tests for Ping and PingServer added

Actions

Copy link

Updated by Junxiao Shi almost 10 years ago

Status changed from New to In Progress

Actions

Copy link

Updated by Junxiao Shi almost 10 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 10 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 10 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