Bug 26097 - bring regression tests up to decent coverage
Summary: bring regression tests up to decent coverage
Status: NEW
Alias: None
Product: Telepathy
Classification: Unclassified
Component: fargo (show other bugs)
Version: git master
Hardware: Other All
: high enhancement
Assignee: Simon McVittie
QA Contact: David Laban
Whiteboard: milestone4.7; metabug
Depends on: 26281 26278 26283
  Show dependency treegraph
Reported: 2010-01-18 07:25 UTC by Simon McVittie
Modified: 2010-01-28 11:15 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Description Simon McVittie 2010-01-18 07:25:58 UTC
I'm working on it.
Comment 1 Simon McVittie 2010-01-22 05:46:42 UTC
We have basic tests for registration and presence exchange.

Missing test coverage includes:

* presence probes - don't work right, Bug #26141. I consider that bug to include regression tests.
* decloaking and providing contacts' capabilities (as opposed to the gateway's capabilities) - I'm considering this to be part of Bug #26100
* accepting calls from XMPP to SIP - requires Bug #26100; I'm considering this to be part of Bug #26096
* accepting calls from SIP to XMPP - I'm considering this to be part of Bug #26096
* registering n times in one session
* invalid JIDs anywhere
* generally, the client doing anything outside the normal flow (perhaps maliciously)

So, the things left in this bug are basically "if the client does something unexpected, do we cope?" which is necessary for a production deployment, but not screamingly urgent for a working prototype.
Comment 2 Simon McVittie 2010-01-25 10:14:29 UTC
I'll consider this bug closed once we have basic regression tests for calls in both directions, which David is working on.
Comment 3 Simon McVittie 2010-01-27 07:44:06 UTC
Let's list obvious missing coverage on this bug.

missing coverage in calls (both ways): codecs with nonzero number of channels; codecs with parameters; needing to ignore video codecs
Comment 4 Simon McVittie 2010-01-27 11:04:10 UTC
Some other missing coverage etc.:

* should the initiator be the bare or full JID? Currently it's the bare JID and we assert that

* from-xmpp.py doesn't assert that the candidates have the right ports, components

* from-xmpp.py doesn't assert whether the session-accept has any candidates; should it?

* from-xmpp.py doesn't assert that the PTs in session-accept match the intended PTs
Comment 5 Simon McVittie 2010-01-28 10:29:45 UTC
Assigning to myself.
Comment 6 David Laban 2010-01-28 11:15:12 UTC
(cd tests
python-coverage -e
make check-TESTS PYTHON='python-coverage -x'
python-coverage -a ../telepathy_fargo/*.py)

combined with a bit of git grep suggests that there are quite a lot of dead functions that can probably be deleted.

Feel free to do some stomping/chopping/hacking/slashing.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.