* Update id of second sources mock to be unique
Both were originally set to ‘123’
* Remove unused types from interface
These were never used and were only uncovered while testing. These both exist on `group` but not at the top level.
* Add group mock
* Fix typo in reducer name
* Add tests for group_logic
* Remove redundant messages check
This is not needed with global flash messages, as the component will not render with no messages.
* Add tests for group_router
* Add group mock
* Add tests for groups_logic
* Convert groups_router to use children
This allow for testing the component visiblity and aligns with other usage of React Router
* Add tests for groups_router
* Refactor pagination logic
This commit removes the useDidUpdateEffect custom hook and moves the logic to fetch the search results to a listener inside of Kea.
* dd tests for main groups container
* Lint fixes
* Fix broken test and remove comment
The issue was that the delay needed to be in the catch block to properly execute.
* Add comments and make test explicit
In other places in the tests, we explicitly test for an error string in the catch block. Changing this to match.
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
* convert pdf.js to TS
* more typescript
* simplify caller
* more typescript
* more typescript
* fix the code to match the expected interface
* very cool comment
* interface correction
* remove unused class method
* add unit test for PdfMaker
* file rename for typo correction
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
* Change an example in embeddable to use controls instead of knobs
* Add controls to SyncBadge APM story
* Convert both to [CSF](https://storybook.js.org/docs/react/api/csf)
* Remove the Knobs addon from the default Storybook configuration
Do not remove the Knobs addon package, since Canvas is still using it and this does not change anything in Canvas.
Fixes the `extract-mitre-attacks` script and generates `v6.3` tactics and techniques. Will move to `7.x` as part of the structural changes to support sub-technique in https://github.com/elastic/kibana/issues/75771.
This is being used in other click* helpers, but was missing here.
Resulting in failures in #80399
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
* [Discover] Deangularize context.app
* Add a unit test
* Addressing PR comments; fixin I18n provider; replacing kui with EUI code
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
* Add code to display/hide UX section when appropriate.
* Suppress errors with link to dedicated issue.
* Fix typo in call to action.
* Remove type error suppression.
* deletes the created rule in the after hook
* fixes failing test
* changes rule deletion order
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
- Fixes a race condition where searches for data grid results with different parameters would return in different order with the wrong results on display. Fix uses a pattern to cancel useEffect callback for getIndexData().
- Fixes identifying pre 7.10 feature influence format for outlier detection and will display a callout on the results page with information for a workaround.
- To fix identifying the legacy format, some cleanup of other code relating to the old format had to be done. The ml results object field is no longer treated as a "special" field for outlier detection and is treated and retrieved in the same way as other fields.
- Adds an error callout if no Kibana index pattern is available for source/dest index.