search

AppImageCommunity / libappimage

Implements functionality for dealing with AppImage files
https://appimage.org
Other
46 stars 29 forks source link

Generate test AppImage files #116

Open azubieta opened 5 years ago

azubieta commented 5 years ago

As we keep adding tests new AppImage files will be required to test each scenario. Having all of them in the sources repository will just make our repository bigger and slower to clone. As most of such files can be generated using appimagetool we must just include the recipes and generate them while building the whole project.

TheAssassin commented 5 years ago

I think this library should not rely on actual files for tests, but instead test things unit wise with the minimum relevant amount of data for this purpose.

azubieta commented 5 years ago

Indeed we can test a wide range of features with the approach you propose. But in the end we will also have to test if on real files. Otherwise it would remain incomplete.

TheAssassin commented 5 years ago

Sure. But not to just see whether parsing a broken desktop file will return in a specific error...