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
Fixes #8217 regression, make sure temporary metrics are available to UserCountryMap for cities layer #8228
Conversation
…ary metrics in this case.
…ed manually in JS.
… IPs + PHP geoip provider.
60335ae
to
a6bc74c
Compare
// for proper geolocation | ||
LocationProvider::setCurrentProvider(LocationProvider\GeoIp\Php::ID); | ||
IPAnonymizer::deactivate(); | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it make sense to make a more specific fixture? This change + adding self::downloadGeoIpDbs()
makes the base fixture more complex (and maybe slower, not sure about that), that's why I'm asking.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Base fixture?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
default fixture? Isn't that the default fixture used for ui tests?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I don't see how it will slow the tests down, the GeoIP database will only be downloaded if it doesn't exist already. So it will happen once per test run (on travis). Is that what you're worried about?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK no worries :)
Fixes #8217 regression, make sure temporary metrics are available to UserCountryMap for cities layer + add regions/cities UI tests.
As title, also includes UI tests for cities/regions layer, plus change that allows UI tests to send mouse events to specific coordinates.
In order to get some visits to show up on the city map, I had to set the IPs in UITestFixture visits to IPs that have locations. And force the use of GeoIP there.