Skip to content

[7.x] Plugin mocks (#40123)#40362

Merged
streamich merged 4 commits intoelastic:7.xfrom
streamich:backport/7.x/pr-40123
Jul 15, 2019
Merged

[7.x] Plugin mocks (#40123)#40362
streamich merged 4 commits intoelastic:7.xfrom
streamich:backport/7.x/pr-40123

Conversation

@streamich
Copy link
Contributor

Backports the following commits to 7.x:

* test: 💍 add ability to mock plugins in ui/new_platform

* chore: 🤖 delete olds NP backdoor mock

* test: 💍 normalize plugin contract naming in line with Core

* test: 💍 use mock from ui/new_platform instead of core

* test: 💍 remove ui/new_platform mocking from Core

* chore: 🤖 leave only one mocking strategy for ui/new_platform

* style: 💄 format according to Prettier

* chore: 🤖 remove /src/plugins/__mocks__ folder
@streamich streamich added the backport This PR is a backport of another PR label Jul 4, 2019
@elasticmachine
Copy link
Contributor

💔 Build Failed

@streamich streamich added the release_note:skip Skip the PR/issue when compiling release notes label Jul 4, 2019
@stacey-gammon
Copy link

still planning to backport?

…40123

# Conflicts:
#	src/legacy/core_plugins/kbn_vislib_vis_types/public/pie_fn.test.js
#	src/legacy/core_plugins/metric_vis/public/metric_vis_fn.test.js
#	src/legacy/core_plugins/region_map/public/region_map_fn.test.js
#	src/legacy/core_plugins/table_vis/public/table_vis_fn.test.js
#	src/legacy/core_plugins/tagcloud/public/tag_cloud_fn.test.js
#	src/legacy/core_plugins/tile_map/public/tilemap_fn.test.js
#	src/legacy/core_plugins/vis_type_markdown/public/markdown_fn.test.ts
#	src/legacy/ui/public/visualize/loader/embedded_visualize_handler.test.ts
@streamich
Copy link
Contributor Author

@stacey-gammon yes, thx, forgot about this

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@streamich streamich merged commit 76572d4 into elastic:7.x Jul 15, 2019
@streamich streamich deleted the backport/7.x/pr-40123 branch July 15, 2019 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR release_note:skip Skip the PR/issue when compiling release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants