Correct test-commands unit test failures.

[ossama-othman]

The test-commands unit test fails due to a number of issues:

• It incorrectly assumes the add_addr_accepted and subflows (see ip-mptcp(8)) MPTCP resource limits are zero at test start, which causes the get_limits command test case to fail when either one of those MPTCP limits is non-zero.
• The set_flags command test case was performed after the remove_addr test case. This caused the kernel to return an invalid argument (EINVAL) error since the network address for which flags were to be set was no longer tracked by the kernel.
• The main event loop exited before all of the underlying generic netlink send and receive operations were completed, which then prevented the mptcpd command test cases from completing successfully.

Correct all of these issues.

[ossama-othman]

The set_flags test case also need the change in PR #259 before it'll succeed as well.

[coveralls]

Pull Request Test Coverage Report for Build 3056927280

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

• For more information on this, see Tracking coverage changes with pull request builds.
• To avoid this issue with future PRs, see these Recommended CI Configurations.
• For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.2%) to 64.923%

---

Totals
Change from base Build 3040164997:	0.2%
Covered Lines:	1390
Relevant Lines:	2141

---

💛 - Coveralls