|Summary:||bring regression tests up to decent coverage|
|Product:||Telepathy||Reporter:||Simon McVittie <smcv>|
|Component:||fargo||Assignee:||Simon McVittie <smcv>|
|Status:||NEW ---||QA Contact:||David Laban <david.laban>|
|i915 platform:||i915 features:|
|Bug Depends on:||26281, 26278, 26283|
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.