[kde-edu]: Review Request: Tests for Cantor backends

[Alexander Rieder]

> On 2010-07-13 15:34:53, Aleix Pol wrote:
> > I don't think there should be backend-dependent code/tests outside src/backends.
> 
> Miha Cancula wrote:
>     It was there already, but I suppose it can be moved to src/backends/, and backend-specific classes to their respective folders.

Yes, I agree. It was a bad idea to not keep the tests in the same location as the backend.
About the placing of the BackendTest class, I'm not sure. Normally things shared between backends go to the lib directory, but this is a special case as it won't be compiled most of the time, and really shoudlnt be in the cantorlib library. 
Basically I see 2 solutions:
- create a subdir in lib and new library called cantortest or something, that for now only contains this class but who knows the future, and have the backends tests all link against that
- put it inside src/backends/ as Miha proposed, allthough that looks a bit strange, having one single class there that doesn't really belong anywhere

I'm open for good suggestions on this one.


- Alexander


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4616/#review6532
-----------------------------------------------------------


On 2010-07-13 12:30:43, Miha Cancula wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/4616/
> -----------------------------------------------------------
> 
> (Updated 2010-07-13 12:30:43)
> 
> 
> Review request for KDE-Edu and Alexander Rieder.
> 
> 
> Summary
> -------
> 
> This patch introduces a BackendTest class which has the functionality previously shared by tests for Maxima and Sage backends. I also included a test for the new Octave backend, which also uses the same class. 
> 
> The only new functionality is skipping tests for backends which are not usable (either not installed or missing runtime requirements). Otherwise it's mostly copy/paste, so I kept Alexander's copyright in the new class. 
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdeedu/cantor/tests/testoctave.cpp PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/tests/testoctave.h PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/tests/testmaxima.h 1149413 
>   /trunk/KDE/kdeedu/cantor/tests/testmaxima.cpp 1149413 
>   /trunk/KDE/kdeedu/cantor/tests/backendtest.cpp PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/tests/CMakeLists.txt 1149413 
>   /trunk/KDE/kdeedu/cantor/tests/backendtest.h PRE-CREATION 
>   /trunk/KDE/kdeedu/cantor/tests/testsage.h 1149413 
>   /trunk/KDE/kdeedu/cantor/tests/testsage.cpp 1149413 
> 
> Diff: http://reviewboard.kde.org/r/4616/diff
> 
> 
> Testing
> -------
> 
> All three tests compile and run. Maxima's testPlot fails, but I think that happened before the change too. I don't have Sage installed, so its test is successfully skipped. 
> 
> 
> Thanks,
> 
> Miha
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-edu/attachments/20100714/d145620a/attachment.htm