contains the following manual changes:
update [bazel]: modify test repository URL and commit hash in `BazelTestCases` for `ProtobufResolve`
refactor [bazel]: replace `findAndKillProcesses` with `ProcessKiller` API in integration tests for improved process management
commits from master:
[bazel][Platform UI] IJPL-196093 Search Everywhere: UI tests for LabelSearchEverywhereContributor
(cherry picked from commit e5ee0dd7d05b93b3184b2a99212e56640fef887d)
refactor [performancePlugin]: change screenshots folder creation logic;
When any screenshot folder is created, they get assigned an incrementing counter in front of it. This will allow sorting all the screenshots folders in order they were made.
Merge-request: IJ-MR-182352
Merged-by: Danil Tereshchenko <Danil.Tereshchenko@jetbrains.com>
(cherry picked from commit 803ca46defce9304a9d0e661504e36049decf0db)
refactor [bazel]: remove "build and sync" parameter in `syncBazelProject`ImportBazelSyncTest
(cherry picked from commit 6c1d7b46787edf2b54be948abca97c4013f8a953)
refactor [bazel]: wrap `MoveKotlinFileTest` commands in `ideFrame` to improve test stability and avoid race conditions
(cherry picked from commit c224c58fb339fd804e93c0fe2f36d803f1b85553)
maintenance [bazel]: add ImportBazelSyncTest to verify project import and sync on Hirschgarten
(cherry picked from commit 35de9d9b1ee7e74813b80d697190f66de8a1fd2d)
refactor [bazel] refactor integration tests to use `syncBazelProject` and `waitForIndicators` with explicit 10-minute timeout
(cherry picked from commit 31f67ef862badeb5bf76902ffc4f788dac92ae3d)
[bazel] refactor GolandSyncTest to use `syncBazelProject`; replace `waitForBazelSync` and add explicit indicator timeout
(cherry picked from commit f7ba2f28232d727910c8f2cb5d2b6dc0b2a305fd)
[bazel] refactor BazelVersionUpdateTest to use `syncBazelProject` and wrap it in ideFrame to avoid race conditions
(cherry picked from commit bc248c82da8d46c9f8527e00367abc7883194000)
BAZEL-2658 [bazel] update PyCharm test to use `syncBazelProjectCloseDialog`; improve Git dialog handling
Merge-request: IJ-MR-182198
Merged-by: Danil Tereshchenko <Danil.Tereshchenko@jetbrains.com>
(cherry picked from commit 760c95e81324fe215914f5df8b8839f621b60102)
[bazel] split pycharm test into three to avoid bug with "add file to VCS"
Created by Danil Tereshchenko (Danil.Tereshchenko) via Patronus
Merge-request: IJ-MR-182195
Merged-by: Danil Tereshchenko <Danil.Tereshchenko@jetbrains.com>
(cherry picked from commit a8ba87da73f00db45ee8f451e48d1f518db87bd3)
[bazel] fix performance test race condition;
move commandChain commands into explicit execution inside ideFrame to avoid race condition, where test incorrectly detects import finish on IDE opening.
Merge-request: IJ-MR-182194
Merged-by: Danil Tereshchenko <Danil.Tereshchenko@jetbrains.com>
(cherry picked from commit ef75aa0d34f2a7508fcbc054cc90fb003a08c87d)
BAZEL-2636 [bazel] update pycharm e2e tests with action to close "add file" dialog
(cherry picked from commit d3d8b65cb4d89b6beec1d55c02e8463255e8998d)
[bazel][maintenance] Add `tearDown` method to clean up Bazel server processes in IDE tests
Merge-request: IJ-MR-180571
Merged-by: Danil Tereshchenko <Danil.Tereshchenko@jetbrains.com>
(cherry picked from commit b47af046008dfe6dd8ff81ba2cbe824135256950)
Co-authored-by: Tatiana Elfimova <Tatiana.Elfimova@jetbrains.com>
Merge-request: IJ-MR-184432
Merged-by: Danil Tereshchenko <Danil.Tereshchenko@jetbrains.com>
GitOrigin-RevId: c82378fa28d635f49cf7495b8e859f8e24948b8f
We use 'name' attribute in XML for it, and now the module ID is composed of the name and the namespace, so it would be confusing to use 'id' for 'name'.
(cherry picked from commit dafa24422dbde094a63a57fd307721f6848ad26b)
IJ-CR-178875
GitOrigin-RevId: 08280f7e484c97c8e3bc790f1ff22be66ed0e84f
These modules and their classes don't have external usages, so they shouldn't be made 'public' at least for now. The 'namespace' is also set to 'jetbrains' for plugins which contain such modules or modules which use them to allow 'internal' visibility to work.
GitOrigin-RevId: 198007e49320075dc27faadde6963e98332296a4
The original find usages performance test is actually testing the Show Usages action. Find Usages and Show Usages have some important differences, namely, Find Usages groups the usages and is designed to show a larger number; Show Usages is intended to show only a subset.
This can result in skewed performance results, as the Show Usages action will spend a lot more time sorting and comparing usages for its popup than the Find Usages tool window would.
GitOrigin-RevId: 3af1c016769ec174bfc47608644c173c4a7d8dce
- use find usages as warmup to avoid publishing fus statistics from rename refactoring during warmup stage
- use average time for search
- perform undo to ensure there is something to search for
^KTIJ-35267 fixed
GitOrigin-RevId: 1cbd6d421aaddc80a6e924c57b537dec9e98dd6d
Introduce separate overloads for the switcher and recent files,
to hide the implementation detail parameters,
and to make the expectation on other parameters clearer
(for example, that event is null for the recent files,
and onlyEditedFiles is null for the switcher).
This refactoring will help adding a key released handler
needed to fix the issue with events being missed
while suspending code is running.
Because this is only an issue for the switcher,
not for the recent files popup.
GitOrigin-RevId: 3a9b40a2191c563b01721c65767384ab0906a92b
