Unit tests under Windows

Bug #208821 reported by Jaspervdg on 2008-03-29
Affects Status Importance Assigned to Milestone

Bug Description

Basically this is some preliminary work I did for the GSoC 2008 test suite proposal. It makes unit tests work on Windows, but it's not very pretty (at least part of it). It shouldn't break anything (I also tried the change I made to xml/repr-action-test.cpp on Linux, so it should be okay), but part of it will probably be undone again, so I'm not sure whether committing it to SVN at this point would be a good idea. In any case, for those who want to have a go at running the unit tests on Windows, here's your chance.

Jaspervdg (jaspervdg) wrote :
nightrow (jb-benoit) wrote :

Thanks for you work, but are you sure that posting this kind of work here is appropriated ?
What do you want us to do with it ?

Jaspervdg (jaspervdg) wrote :

This is a cleaned up version of the change, which does not include any of the non-CxxTest unit tests. It only affects Windows and should be relatively safe to commit. I haven't done so though since it also affects the normal build process. Which I consider a good thing, as it will prevent people from breaking tests, but I wouldn't want to commit it without at least some community response.

Jaspervdg (jaspervdg) wrote :

This fixes the buildtool part of the patch to work with the newer version of buildtool (the old patch failed to expand variables in options).

nightrow (jb-benoit) wrote :

As i didn't got any answer on what to do with it, i'm invalidating this bug.
Feel free to re-open it if there is anything for us to do with it

Changed in inkscape:
status: New → Invalid
theAdib (theadib) wrote :

windows currently does not use makefile and so no "make check" is supported. If this patch can bring a "*buildsystem* check" it should be more than welcome. This was one goal for the 0.47.

I will check this tomorrow ...


Jaspervdg (jaspervdg) wrote :

This bug is outdated. I did the work for GSoC and currently you CAN do the equivalent of a "make check" on Windows. (Only it's "btool check".)

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers