Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: SUSE AI sanity test suite #20817

Merged

Conversation

rqueiroz-suse
Copy link
Contributor

Adding wrapper script to call SUSE AI sanity test suite

Related ticket: https://jira.suse.com/browse/SUSEAI-8
Needles: N/A
Verification run: https://openqa.suse.de/tests/overview?distri=sle-micro&version=6.0&build=0153&groupid=592

Copy link

Great PR! Please pay attention to the following items before merging:

Files matching lib/**.pm:

  • Consider adding or extending unit tests in t/

This is an automatically generated QA checklist based on modified files.

my $test_folder = $test_archive;
$test_folder =~ s/\.tar(\.gz)?$//;

assert_script_run("curl -O " . $sanity_tests_url);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
assert_script_run("curl -O " . $sanity_tests_url);
assert_script_run("curl -OLq " . $sanity_tests_url);

assert_script_run("pip3 install -r ./" . $test_folder . "/requirements.txt");
assert_script_run("cp " . $test_folder . "/env.example " . $test_folder . "/.env");
assert_script_run("pytest $test_folder/tests/");
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The virtualenv is never left, perhaps a exit here would be desirable?

@asmorodskyi
Copy link
Member

this is just a clone of #20799 closed due to technical reasons . so no review needed ( see #20799 ) auto-merging

@asmorodskyi asmorodskyi merged commit 4fa856b into os-autoinst:master Dec 17, 2024
10 checks passed
@rqueiroz-suse rqueiroz-suse deleted the feature/suse-ai-sanity-test branch December 17, 2024 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants