This would include:

* set the variables that enable user country and language selection as needed

* visit profile edit page

* ensure the country select shows and has appropriate values

* select user country

* ensure that the language select now reflects only languages for that country

* ensure session country_code variable is set to the new country

CommentFileSizeAuthor
#3 user_preferences.patch3.12 KBcatch
#1 user_default_test.patch3.33 KBcatch
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

catch’s picture

Status: Active » Needs review
FileSize
3.33 KB

Here's a test. I'm attaching as a patch rather than committing directly, because it seems to be causing failures in the country switcher block test (and I had some other side-effects earlier with it although those are either eliminated or commented out). The test itself passes fine.

With this version of the patch, I'm getting *363 passes, 4 fails, 6 exceptions - the fails are all in testCountrySwitcherBlock() - if someone could patch, run tests, and verify whether they get the same failures, that'd be great.

nedjo’s picture

Status: Needs review » Postponed

Thanks catch.

I think we need to put this on hold a bit while we sort out #326726: Refactor to set global langauge to country-specific version?. That patch will require a lot of changes including ones that impact this test.

catch’s picture

FileSize
3.12 KB

The previous failures now pass, presumably due to fixes elsewhere. I'll not commit this until the other issue is resolved, but here's a slight update. Have completely left session checks for now since that's the most likely thing to change. Since it's working, we could probably go ahead and commit this, then fix it when refactoring breaks it, but I'll leave that up for discussion

catch’s picture

Status: Postponed » Fixed

Test still passes. Committed. Noticed a failure for the country switcher block test, so will open an issue for that if there's not one already.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.