-
Notifications
You must be signed in to change notification settings - Fork 8.6k
SKA: Relocate /test to /src/platform/test #210956
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
Merged
gsoldevila
merged 41 commits into
elastic:main
from
gsoldevila:kbn-team-1309-relocate-20250213090121
Mar 14, 2025
Merged
Changes from all commits
Commits
Show all changes
41 commits
Select commit
Hold shift + click to select a range
3345db0
Relocating module `@kbn/test-suites-src`
gsoldevila 00a1c81
Manually update broken references to /test folder
gsoldevila 3e8df5c
Update .eslint rules
gsoldevila 0ac87e3
[CI] Auto-commit changed files from 'node scripts/capture_oas_snapsho…
kibanamachine c62d40f
Generate and manually update codeowners
gsoldevila 6cdd231
[CI] Auto-commit changed files from 'node scripts/check_mappings_upda…
kibanamachine 5cfd9f5
[CI] Auto-commit changed files from 'node scripts/eslint --no-cache -…
kibanamachine 17cec52
Fix broken paths
gsoldevila a462813
Update stateful configs paths
gsoldevila 13cc384
Update getPluginCategories
gsoldevila f859de0
Sorry CI, but current_mappings should not include test SO types
gsoldevila 95aab39
Consider src/platform/test platform/shared by default
gsoldevila 5220f45
Manually update CodeQL config path
gsoldevila 7fb954e
Manually replace references to test subfolders
gsoldevila f99097f
[CI] Auto-commit changed files from 'node scripts/eslint --no-cache -…
kibanamachine 45db38d
Fix outdated path
gsoldevila 9ded61b
Merge branch 'main' into kbn-team-1309-relocate-20250213090121
gsoldevila 40fc72f
Merge branch 'main' into kbn-team-1309-relocate-20250213090121
gsoldevila 4c40b8e
Fix outdated path to tests (thanks dima)
gsoldevila 91a9d6b
Merge branch 'main' into kbn-team-1309-relocate-20250213090121
gsoldevila 4ca8723
Remove SO types defined by test plugins
gsoldevila ed52662
Merge branch 'main' into kbn-team-1309-relocate-20250213090121
gsoldevila 28374ee
[CI] Auto-commit changed files from 'node scripts/capture_oas_snapsho…
kibanamachine 1492503
Merge branch 'main' into kbn-team-1309-relocate-20250213090121
gsoldevila 992c5c6
Fix incorrect esArchiver path
gsoldevila 27290f5
[CI] Auto-commit changed files from 'node scripts/eslint --no-cache -…
kibanamachine 12127c2
Merge branch 'main' into kbn-team-1309-relocate-20250213090121
gsoldevila 756576a
[CI] Auto-commit changed files from 'node scripts/capture_oas_snapsho…
kibanamachine df84fe6
Merge branch 'main' into kbn-team-1309-relocate-20250213090121
gsoldevila 138d82f
Merge branch 'main' into kbn-team-1309-relocate-20250213090121
gsoldevila 450af8b
Add missing encryptedSavedObjects.encryptionKey
gsoldevila dc68ad6
Merge branch 'main' into kbn-team-1309-relocate-20250213090121
gsoldevila 279c2a6
Merge branch 'main' into kbn-team-1309-relocate-20250213090121
gsoldevila 634e307
Update outdated archive paths
gsoldevila b8bb7a0
Merge branch 'main' into kbn-team-1309-relocate-20250213090121
gsoldevila 705b89f
Update functional tests ES archives paths
gsoldevila b98b9a3
Move orphan test files
gsoldevila 8702332
[CI] Auto-commit changed files from 'node scripts/eslint --no-cache -…
kibanamachine b7346c3
Merge branch 'main' into kbn-team-1309-relocate-20250213090121
gsoldevila 2aa0712
[CI] Auto-commit changed files from 'node scripts/generate codeowners'
kibanamachine 6e291ac
Merge branch 'main' into kbn-team-1309-relocate-20250213090121
gsoldevila File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
@gsoldevila I remember you mentioned that the automation you use would automatically handle updates in the CodeQL "ignore" config, but I don’t see any changes in the CodeQL config in this PR.... Wouldn't this change break this (i.e. relocated tests will no longer be excluded by CodeQL)?
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.
I had to disable reference replacement for this particular relocation.
:onedoesnotsimply: replace all occurrences of
teststring.Thanks for the heads up, I'll update this one manually