Re: Test Data for DigiKam

Steven Robbins <[email protected]> Wed, 28 Sep 2022 08:13:44 -0500
Newsgroups gmane.comp.kde.digikam.devel
Message-ID <2848490.e9J7NaK4W3@riemann>
On Wednesday, September 28, 2022 12:43:44 A.M. CDT Gilles Caulier wrote:

> If we choose this solution, i propose to check in cmake script if the
> test data is present, and in this case to enable unit tests which use
> this data.
> 
> The advantage is to not run all the unit tests in the CI which is
> limited in all cases to one hour. The disadvantage is to run unit
> tests in developers computers for these unit-tests using data files.
> 
> What do you think about this proposal ?

I realise that digikam is under massively active development -- with multiple 
commits per day often.  So there is some disadvantage to running a massive 
amount of testing on every commit.

Some may quibble about what "massive" means -- for my paid work, we have a 
code base that takes about 30 hours to run all the tests.  So I see the 
current 30 minutes as small.  :-)

But I hear you -- the CI is a shared resource with all of KDE, after all.  

When I work on the code, I for sure like to be able to run the tests locally.  
So your arrangement fulfils that need.  However, I suggest there is great value 
in ALSO running them -- at some frequency -- on the CI machines for wider 
testing.  At home, I'm personally testing just one configuration: Debian linux 
with FFMPEG 5.  Sometimes I additionally test FFMPEG 4 in a chroot.  But the 
CI provides coverage on windows, for example, which I am never going to do 
myself.

So: can we devise a mechanism where we run lower-frequency (once per day?) 
test jobs to get this coverage?  That would require the CI "checkout" 
mechanism to pull down this second repo in some fashion.  Is that do-able?

Cheers,
-Steve
signature.asc (application/pgp-signature, 833 B)
-----BEGIN PGP SIGNATURE-----

iQIzBAABCAAdFiEEy89k8fa3rclNjyokyeVeL63I9LkFAmM0SIgACgkQyeVeL63I
9Lmv0g/9G84fLnvOoUPctquctSYgMp2+u4WrHG7/GICrAEwuJkE90jtzxy9sRdXv
8HQYgifVeQOQfC37bkgdzxIS509+1sCMHwiGEH2g3m5MM9owOIeD0eVCOuQVEou9
dYqEpgJhOGJITbTeqpVB/kSBqLjKv8bTTR9+RtESfcMUaDBBXs4qlk4iT6bdqlrB
+rKMSDaMg15cecfZ+3Oke96hWdsg8wJ/wG9R5EgoEhPq/FLRbJFJRd8y1VNWf6Kh
8XMhV6Vcod1lfEZ9x5ZntaJeyvzQKLhzomtDmd/nVOMusId5HRJrPxIfXt3PJCs2
qJv73vI8MuPfDjOIvoP/g7/Sr8dqCplj2JS1xY7y5DZqOc0y9DUpc7dzuWiB3hYn
4WM372/d6gHthMNyBvLpbn61O6cHy4pBI3Ufpw9BzxMWvl2QuwP9xBD1PO9QVhTI
2hmcH02YliAnO0taQKEduL3R4sCZIrcDKSkXIER8NRYfaeRboW0Aj4Lm+pT0RLMc
t182nZxFO5ZzVBp/hu0fy3uJ9fLfyiIhSlQ/EW+oA2H0vZood6EIxA3LIXOPEoQg
6Re995ju3Qi4Lofg0XIJU87IuWyQn3BaR6g1DQAtzHSHOs/GxJ8wTlYWZBXIpnoH
Zjztosb/w9N9L4EQX29xF906F7R1jiLpawRtbC1MZuHYCLvEyyg=
=jHbW
-----END PGP SIGNATURE-----