added localization test

[stupchiy]

Selecting a different locale/language should go to that page, and have /[$locale/] in the URL

[stephendonner]

When I ran this through Jenkins, I got:

raise AssertionError('%s is not found in %s' % (needle, haystack))
E AssertionError: en-us is not found in https://firefoxflicks.allizom.org/en-US/

Do we know where the lower-case "en-us" is coming from?

[stephendonner]

Ah, that comes from the app itself:

We're discussing in #webdev as to whether we should do a case-insensitive string-compare in this test.

[stephendonner]

From @jsocol:

"stephend: well, the behavior of Mozilla sites is to have one canonical URL and to capitalize the US, so en-us -> en-US is correct, and that's what the test should assert
I'd argue that that's correct in general"

[stupchiy]

Fixed. Done it fast before going to sleep, so please check if it works for you now @stephendonner . Thanks

[stephendonner]

Thx; ran it through the proverbial grinder (Jenkins), and it's fine now :-)

[davehunt]

We should try to avoid splitting the locator tuple in this way. You could use return len(self.selenium.find_element(*self._language_locator).find_elements(By.TAG_NAME, 'option')), or create a new locator for languages.