mirror of
https://pagure.io/fedora-qa/os-autoinst-distri-fedora.git
synced 2024-11-29 09:03:08 +00:00
check_default_fonts: move reference files, use diff -u
Let's not have the reference files in one person's fedorapeople space, in case that person leaves. Let's upload the text.txt for checking (it's easier to be able to just read what's in it than try and figure it out from the diff output), and let's use diff -u because non-unified diff output is awful. Signed-off-by: Adam Williamson <awilliam@redhat.com>
This commit is contained in:
parent
4ee583ad43
commit
e047a88ffc
@ -49,9 +49,11 @@ sub run {
|
|||||||
my $username = get_var("USER_LOGIN") // "test";
|
my $username = get_var("USER_LOGIN") // "test";
|
||||||
script_run("cd /home/$username/");
|
script_run("cd /home/$username/");
|
||||||
# Download the language reference file.
|
# Download the language reference file.
|
||||||
script_run("wget https://lruzicka.fedorapeople.org/$language-reference.txt");
|
script_run("wget https://fedorapeople.org/groups/qa/openqa-fonts/$language-reference.txt");
|
||||||
|
# upload the log for debugging.
|
||||||
|
upload_logs "test.txt", failok=>1;
|
||||||
# Compare the test file and the reference file.
|
# Compare the test file and the reference file.
|
||||||
assert_script_run("diff test.txt $language-reference.txt");
|
assert_script_run("diff -u test.txt $language-reference.txt");
|
||||||
}
|
}
|
||||||
|
|
||||||
# For the rest of languages that are not currently defined, do nothing.
|
# For the rest of languages that are not currently defined, do nothing.
|
||||||
|
Loading…
Reference in New Issue
Block a user