Skip to content

[8.19] [dashboard] remove url_field_formatter functional test (#223448)#223917

Merged
kibanamachine merged 1 commit intoelastic:8.19from
kibanamachine:backport/8.19/pr-223448
Jun 13, 2025
Merged

[8.19] [dashboard] remove url_field_formatter functional test (#223448)#223917
kibanamachine merged 1 commit intoelastic:8.19from
kibanamachine:backport/8.19/pr-223448

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

Part of elastic#192910

url_field_formatter functional test was
[skipped](elastic#193293) in Sept 2024.
Instead of removing `.skip`, the entire test should be removed for the
following reasons
1) It does not test dashboard code. The test verifies rendering of
legacy visualize table on a dashboard and discover table (in the
discover app). Both of these are not our area of responsibility.
2) the test case is better covered by unit tests.

(cherry picked from commit 03af622)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Jun 13, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) June 13, 2025 13:36
@kibanamachine kibanamachine merged commit 2eb5c30 into elastic:8.19 Jun 13, 2025
11 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

cc @nreese

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants