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

[Maps] fix point to point source regression #60930

Merged
merged 2 commits into from
Mar 23, 2020

Conversation

nreese
Copy link
Contributor

@nreese nreese commented Mar 23, 2020

Fixes regression caused by #57879.

#57879 changed AbstractESSource._getGeoField from a method function to an arrow function. By making the function an arrow function, it moved the function from prototype to the instance. This means that even though ESPewPewSource overrides the method on the prototype, AbstractESSource will use the version on the instance instead of looking at the protoype change.

Here is the error in action
Screen Shot 2020-03-23 at 10 03 07 AM

Now pew-pew works as expected
Screen Shot 2020-03-23 at 10 08 57 AM

@nreese nreese added [Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.7.0 labels Mar 23, 2020
@nreese nreese requested a review from a team as a code owner March 23, 2020 16:20
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-gis (Team:Geo)

Copy link
Contributor

@thomasneirynck thomasneirynck left a comment

Choose a reason for hiding this comment

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

code review

@nreese
Copy link
Contributor Author

nreese commented Mar 23, 2020

If you are wondering why there was no functional test to catch this regression, point to point source was added when percy visual testing was all the rage. So the functional testing for this is using percy and those tests have been disabled and no longer fail builds.

I have added a functional test to ensure that pew-pew source works even if percy tests are disabled.

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@nreese nreese merged commit d32c4c8 into elastic:master Mar 23, 2020
nreese added a commit to nreese/kibana that referenced this pull request Mar 23, 2020
* [Maps] fix pew pew regression

* add functional test for pew pew source
nreese added a commit that referenced this pull request Mar 23, 2020
* [Maps] fix pew pew regression

* add functional test for pew pew source
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation release_note:skip Skip the PR/issue when compiling release notes v7.7.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants