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

[Fabian Wiesel]

> On 2009-03-25 02:39:58, Robert Gruber wrote:
> > /trunk/KDE/kdevplatform/vcs/tests/vcsBlackBoxTest.cpp, line 375
> > <http://reviewboard.kde.org/r/384/diff/3/?file=3977#file3977line375>
> >
> >     Maybe you could use the interface here to import your test-files into the repo?
> >     
> >     ICentralizedVersionControl::import()

Will do so.


- Fabian


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


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
> 
>