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

Fix evolution smoke test #18644

Merged
merged 1 commit into from
Jul 31, 2024
Merged

Conversation

DeepthiYV
Copy link
Contributor

@DeepthiYV DeepthiYV commented Feb 14, 2024

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.

@foursixnine foursixnine added the qe-core Tag used to filter PR's created by QE-Core's members or are assigned to them label Feb 17, 2024
@rfan1
Copy link
Contributor

rfan1 commented Feb 19, 2024

Can you please run VRs on sle as well ? osd-job-search

@dzedro
Copy link
Contributor

dzedro commented Feb 19, 2024

Can you please run VRs on sle as well ? osd-job-search

https://openqa.suse.de/tests/overview?distri=sle&build=DeepthiYV%2Fos-autoinst-distri-opensuse%2318644

@DeepthiYV can you fix url to your TW VR and make sure you instance is accessible from outside ?

@DeepthiYV DeepthiYV marked this pull request as draft February 22, 2024 08:35
@DeepthiYV DeepthiYV force-pushed the evolution_test branch 3 times, most recently from c10d469 to 62878d2 Compare March 7, 2024 15:30
@DeepthiYV DeepthiYV force-pushed the evolution_test branch 2 times, most recently from a91e9a2 to 3d78c47 Compare July 26, 2024 13:43
@DeepthiYV
Copy link
Contributor Author

Can you please run VRs on sle as well ? osd-job-search

I am seeing needles matching vary across different SLE versions. Let's merge it for Tumbleweed for now. Later, I will add this test development job for OSD

@DeepthiYV DeepthiYV marked this pull request as ready for review July 29, 2024 08:49
if (match_has_tag "evolution_mail-auth") {
send_key "alt-a"; #disable keyring option
#send_key "alt-a"; #disable keyring option, in SP2 or tumbleweed
Copy link
Contributor

Choose a reason for hiding this comment

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

This can be removed.

lib/x11test.pm Outdated Show resolved Hide resolved
lib/x11test.pm Show resolved Hide resolved
Test also handles loading a user's email
@okurz
Copy link
Member

okurz commented Jul 30, 2024

s/evoultion/evolution/

@DeepthiYV DeepthiYV changed the title Fix evoultion smoke test Fix evolution smoke test Jul 30, 2024
@DeepthiYV DeepthiYV merged commit e6593eb into os-autoinst:master Jul 31, 2024
10 checks passed
@okurz
Copy link
Member

okurz commented Jul 31, 2024

@DeepthiYV you didn't fix the commit message, just the PR description.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
qe-core Tag used to filter PR's created by QE-Core's members or are assigned to them
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants