Review Request: Black-box test for (D)VCS-plugins

[Fabian Wiesel]

> On 2009-03-23 09:15:07, Andreas Pakulat wrote:
> > trunk/KDE/kdevelop/vcs/tests/vcsBlackBoxTest.cpp, line 167
> > <http://reviewboard.kde.org/r/384/diff/2/?file=3538#file3538line167>
> >
> >     You mean the svn plugin? If so please create a bugreport at bugs.kde.org with the relevant reproduction steps.

I wasn't sure, if this is a bug, as it is currently not clearly specified, how the interface should behave in that situation.
It just didn't match my expectations, so I coded around it.
This is a part I was hoping to get some feedback on.
But since you seem to consider it a bug, too, I've filed a bug report.


- Fabian


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


On 2009-03-25 00:44:15, Fabian Wiesel wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/384/
> -----------------------------------------------------------
> 
> (Updated 2009-03-25 00:44:15)
> 
> 
> Review request for KDevelop.
> 
> 
> Summary
> -------
> 
> Tests the various plug-ins on consistency. Currently works with Subversion and Mercurial. CVS needs to be changed to allow local repositories.
> Currently checks:
> All: isVersionControlled(), status(), add(), revert(), commit() 
> DVCS: init()
> 
> The test also codifies a proposal of the expected behaviour of the interfaces, which would be the common denominator of Subversion and Mercurial.
> Because of that, I thought, I'd rather let this be reviewed, before spending more time on implementing more test-cases, which codify an unwanted behaviour.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdevplatform/vcs/CMakeLists.txt 944152 
>   /trunk/KDE/kdevplatform/vcs/tests/CMakeLists.txt PRE-CREATION 
>   /trunk/KDE/kdevplatform/vcs/tests/vcsBlackBoxTest.h PRE-CREATION 
>   /trunk/KDE/kdevplatform/vcs/tests/vcsBlackBoxTest.cpp PRE-CREATION 
> 
> Diff: http://reviewboard.kde.org/r/384/diff
> 
> 
> Testing
> -------
> 
> Excluding CVS and Git:
> 
> 1/  1 Testing vcsBlackBoxTest                  Passed
> 
> 100% tests passed, 0 tests failed out of 1
> 
> 
> Thanks,
> 
> Fabian
> 
>