Merge lp://staging/~dbarth/indicator-network/unit-test into lp://staging/~indicator-applet-developers/indicator-network/indicator-network

Proposed by David Barth
Status: Merged
Merged at revision: 44
Proposed branch: lp://staging/~dbarth/indicator-network/unit-test
Merge into: lp://staging/~indicator-applet-developers/indicator-network/indicator-network
Diff against target: 591 lines (+250/-98)
11 files modified
Makefile.am (+1/-0)
configure.ac (+1/-0)
src/Makefile.am (+2/-2)
src/indicator-network-service-dbus.c (+2/-2)
src/indicator-network-service-main.c (+127/-0)
src/indicator-network-service.c (+44/-89)
src/indicator-network-service.h (+8/-1)
src/network-menu.c (+5/-2)
src/services.c (+3/-0)
tests/Makefile.am (+21/-1)
tests/test-service.c (+36/-1)
To merge this branch: bzr merge lp://staging/~dbarth/indicator-network/unit-test
Reviewer Review Type Date Requested Status
Kalle Valo (community) Approve
Review via email: mp+30671@code.staging.launchpad.net

Description of the change

Adds a unit test infrastructure along with an example to illustrate how to have mock objects and test the indicator icon.

To post a comment you must log in.
Revision history for this message
Kalle Valo (kvalo) wrote :

Looks good to me.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches