mijorus/gearlever

Appstream validation test failing.

Closed this issue · 2 comments

I built gearlever with meson and tested the files:

meson test -C build --print-error-logs

All tests succeed except for the appstream-util test:

1/3 Validate desktop file          OK              0.01s
2/3 Validate schema file           OK              0.00s
3/3 Validate appstream file        FAIL            4.11s   exit status 1
>>> MALLOC_PERTURB_=203 ASAN_OPTIONS=halt_on_error=1:abort_on_error=1:print_summary=1 UBSAN_OPTIONS=halt_on_error=1:abort_on_error=1:print_summary=1:print_stacktrace=1 /usr/bin/appstream-util validate data/it.mijorus.gearlever.appdata.xml
―――――――――――――――――――――――――――――――――――――――――――――――――――― ✀  ―――――――――――――――――――――――――――――――――――――――――――――――――――――
stdout:
data/it.mijorus.gearlever.appdata.xml: FAILED:
• attribute-invalid     : <screenshot> width too large [https://raw.githubusercontent.com/mijorus/gearlever/master/docs/gearlever3.png] maximum is 1600px
• attribute-invalid     : <screenshot> height too large [https://raw.githubusercontent.com/mijorus/gearlever/master/docs/gearlever3.png] maximum is 900px
• attribute-invalid     : <screenshot> width too large [https://raw.githubusercontent.com/mijorus/gearlever/master/docs/gearlever1.png] maximum is 1600px
• attribute-invalid     : <screenshot> height too large [https://raw.githubusercontent.com/mijorus/gearlever/master/docs/gearlever1.png] maximum is 900px
• attribute-invalid     : <screenshot> width too large [https://raw.githubusercontent.com/mijorus/gearlever/master/docs/gearlever4.png] maximum is 1600px
• attribute-invalid     : <screenshot> height too large [https://raw.githubusercontent.com/mijorus/gearlever/master/docs/gearlever4.png] maximum is 900px
• attribute-invalid     : <screenshot> width too large [https://raw.githubusercontent.com/mijorus/gearlever/master/docs/gearlever2.png] maximum is 1600px
• attribute-invalid     : <screenshot> height too large [https://raw.githubusercontent.com/mijorus/gearlever/master/docs/gearlever2.png] maximum is 900px
• style-invalid         : <p> is too short [- Bug fix]
stderr:
Validation of files failed
―――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――


Summary of Failures:

3/3 Validate appstream file FAIL            4.11s   exit status 1

Ok:                 2   
Expected Fail:      0   
Fail:               1   
Unexpected Pass:    0   
Skipped:            0   
Timeout:            0 

I understand building outside of flatpak isn't really supported but I was looking to install the package without flatpak so I created a PKGBUILD for arch linux. I apologize if I have done something wrong on my end.

I can continue by ignoring this fail with:

meson test -C build --print-errorlogs || :

This allows the other two tests to run while outputting whether or not they failed while allowing the script to continue upon the error above. This seems like a workaround but if this solution seems fine, feel free to close this issue.

Hi @TuxDeluxe ,
if the screenshot size becomes a real blocker, I feel free to reopen this issue. For now, I am closing this.