Re: Unit tests
- From: Ruben Vermeersch <ruben savanne be>
- To: Anton Keks <anton azib net>
- Cc: f-spot-list gnome org
- Subject: Re: Unit tests
- Date: Tue, 08 Jun 2010 12:27:52 +0200
If someone could set this up, it'd be massively cool, but I don't have
the resources to do so.
There is another important factor that prevents breakage though: me. If
your patch breaks unit tests, it won't go in! :-)
So please write unit tests when possible, it'll also make it so much
easier to make sure stuff is correct and to review it.
On Mon, 2010-06-07 at 22:45 +0300, Anton Keks wrote:
> The only way to force that is to make running them compulsory at the
> end of each build.
> Otherwise, sooner or later, people will start skipping them and they
> will be made useless again.
>
> Maybe we need continuous integration server somewhere? My experience
> shows that this is required in order to keep a hand on the tests
> properly.
>
> Anton
>
>
> On Mon, Jun 7, 2010 at 8:00 PM, Daniel Köb <daniel koeb peony at> wrote:
> > Hi,
> >
> > For some code changes I'm planning to make I would like to write some unit
> > tests first. It's always nice to know that you didn't break anything
> > unexpectedly in some tricky function...
> > The HACKING file states in a single paragraph that NUnit is used for unit
> > testing and explains a little bit how such tests should be implemented.
> > Unfortunately, this paragraph is rather outdated (2006) and it seems to be
> > wrong at least for some parts.
> >
> > So my first question is: Is anybody writing unit tests when hacking f-spot?
> >
> > Furthermore, I managed to build and run the existing tests (2, is this
> > it?!?). I also managed to write, build, and run my own tests with some
> > problems I had to work around.
> > Before I'm going into more detail about these problems and asking further
> > questions I would like to know if f-spot should have unit tests at all (I'd
> > say yes) and if they would be used and maintained by others as well.
> >
> > Regards,
> > Daniel
> > _______________________________________________
> > f-spot-list mailing list
> > f-spot-list gnome org
> > http://mail.gnome.org/mailman/listinfo/f-spot-list
> >
> _______________________________________________
> f-spot-list mailing list
> f-spot-list gnome org
> http://mail.gnome.org/mailman/listinfo/f-spot-list
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]