Hi Mihai,
Thanks for your patch.
I have skimmed all your test code and compared it with the
one found in
Gibberish,
as a base. You have done a good work and (of course) all
tests (6) passed.
This said, I have some remarks.
1 - In TestAccountInstallation, as you have used an
accountProperties, you might
use the following code :
rssProviderFactory.installAccount(
accountProperties.get(ProtocolProviderFactory.USER_ID),
accountProperties);
instead of
rssProviderFactory.installAccount("RSS",
accountProperties);
Another thing I noticed is that the rssaccregwizz
accepts to install
only one
RSS account. Is that limitation enforced by the protocol
provider ? In
other words,
if we try to install two (different) RSS accounts will it
always fails ?
2 - For which kind of tests do you plan to use the feed
server/factory (rather than using online
feeds) ?
3 - I noticed that a few method lacks of javadoc
comments, but since
you haven't finished,
I guess it is temporary ;)
As you stated that this patch is a preview version, I won't
commit it,
waiting for your latest
version, in case you plan to bring substantial modifications
soon. This
said, the accomplished work is
very good and if upcoming tests are written in the same way,
I think we
could use your test code verbatim.
Thanks for the time.
Regards
Sympho
Mihai Balan a écrit :
> Hello everybody,
> I'm working for some time on the test package for the
RSS protocol.
> Right now it's in a _very_ eraly stage, meaning that
there are only
> four, simple test cases implemented (very similar to
those in the
> Gibberish SLICK) and a very simple server that can/will
be used later
> on.
> Right now testing some of the features of the current
implementation
> isn't quite problems as the protocol still has some
holes in it
> (namely online/offline switching/detection for both
remote contacts
> (aka feeds) and user account).
> It's the first time I write unit tests, so any
suggestion is more than
> welcome. I won't be able to check my mail for a week or
so, but I look
> forward for your impressions.
>
> Thanks in advance for your time,
> Mihai
>
> PS: To actually get to run the test one should also
alter the
> testing.properties file and add
RssProtocolProviderServiceLick to the
> list of tests.
>
------------------------------------------------------------
------------
>
>
------------------------------------------------------------
---------
> To unsubscribe, e-mail: dev-unsubscribe sip-communicator.dev.java.net
> For additional commands, e-mail: dev-help sip-communicator.dev.java.net
------------------------------------------------------------
---------
To unsubscribe, e-mail: dev-unsubscribe sip-communicator.dev.java.net
For additional commands, e-mail: dev-help sip-communicator.dev.java.net
|