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

Add new timezone selection to Evolution #4067

Merged
merged 1 commit into from Dec 13, 2017
Merged

Conversation

czerw
Copy link
Contributor

@czerw czerw commented Dec 13, 2017

@@ -38,8 +39,15 @@ sub run {
send_key "alt-s";
wait_still_screen 3;
send_key "ret";
send_key_until_needlematch("timezone-asia-shanghai", "up")
|| send_key_until_needlematch("timezone-asia-shanghai", "down");
if (sle_version_at_least('12-SP3')) {
Copy link
Contributor

Choose a reason for hiding this comment

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

hmm. I would prefer if you did this without hardcoding versions - but by means of a check_screen. Because leap will get this update too I assume - and then you end up with strange version checks.

Fix poo#29363: Evolution changed timezone selection dialog in SLE12-SP3.
@czerw
Copy link
Contributor Author

czerw commented Dec 13, 2017

@coolo test was updated as suggested, links to verification runs were updated in description

@coolo coolo merged commit f35b885 into os-autoinst:master Dec 13, 2017
@czerw
Copy link
Contributor Author

czerw commented Dec 13, 2017

@coolo could you please merge needles too?

@coolo
Copy link
Contributor

coolo commented Dec 13, 2017

makes sense, no? 😺

rwx788 pushed a commit to rwx788/os-autoinst-distri-opensuse that referenced this pull request Dec 14, 2017
Fix poo#29363: Evolution changed timezone selection dialog in SLE12-SP3.
@czerw czerw deleted the poo#29363 branch August 24, 2018 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants