test(editor): add tests for multi workspace folder setup #10904
Merged
Conversation
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
github-actions
bot
added
A-editor
Member
Author
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
Sysix
force-pushed
the
05-08-test_editor_add_tests_for_multi_workspace_folder_setup
branch
from
47cce94 to
0c7d29e
Compare
Sysix
force-pushed
the
05-07-feat_language_server_editor_support_multi_workspace_folders
branch
from
478c735 to
2528d6a
Compare
Sysix
force-pushed
the
05-08-test_editor_add_tests_for_multi_workspace_folder_setup
branch
3 times, most recently
from
311596d to
342798c
Compare
Sysix
force-pushed
the
05-07-feat_language_server_editor_support_multi_workspace_folders
branch
from
2528d6a to
b2b53b9
Compare
Sysix
force-pushed
the
05-08-test_editor_add_tests_for_multi_workspace_folder_setup
branch
from
342798c to
6eac7c3
Compare
Sysix
force-pushed
the
05-07-feat_language_server_editor_support_multi_workspace_folders
branch
from
b2b53b9 to
4c069fe
Compare
Sysix
force-pushed
the
05-08-test_editor_add_tests_for_multi_workspace_folder_setup
branch
2 times, most recently
from
ad0f62e to
ca4fb84
Compare
Sysix
force-pushed
the
05-07-feat_language_server_editor_support_multi_workspace_folders
branch
from
4c069fe to
100bea0
Compare
graphite-app
bot
force-pushed
the
05-07-feat_language_server_editor_support_multi_workspace_folders
branch
2 times, most recently
from
e4493ff to
2113390
Compare
graphite-app
bot
force-pushed
the
05-08-test_editor_add_tests_for_multi_workspace_folder_setup
branch
from
ca4fb84 to
2a17146
Compare
Sysix
force-pushed
the
05-07-feat_language_server_editor_support_multi_workspace_folders
branch
from
2113390 to
bfadfd6
Compare
Sysix
force-pushed
the
05-08-test_editor_add_tests_for_multi_workspace_folder_setup
branch
from
2a17146 to
a6ca961
Compare
Sysix
force-pushed
the
05-07-feat_language_server_editor_support_multi_workspace_folders
branch
from
bfadfd6 to
0725f19
Compare
Sysix
force-pushed
the
05-08-test_editor_add_tests_for_multi_workspace_folder_setup
branch
from
a6ca961 to
67ce1cb
Compare
graphite-app
bot
changed the base branch from
05-07-feat_language_server_editor_support_multi_workspace_folders
to
graphite-base/10904
camc314
approved these changes
May 11, 2025
graphite-app
bot
force-pushed
the
05-08-test_editor_add_tests_for_multi_workspace_folder_setup
branch
from
67ce1cb to
c196c76
Compare
graphite-app
bot
force-pushed
the
graphite-base/10904
branch
from
0725f19 to
be7f7e1
Compare
graphite-app
bot
force-pushed
the
05-08-test_editor_add_tests_for_multi_workspace_folder_setup
branch
from
c196c76 to
d12e9cb
Compare
Sysix
force-pushed
the
05-08-test_editor_add_tests_for_multi_workspace_folder_setup
branch
from
d12e9cb to
3dbfcdb
Compare
Member
Author
Merge activity
|
![graphite-app[bot]](https://avatars.githubusercontent.com/in/158384?s=60&v=4){kind=small}
We have now 3 different test setups: ### `test()` Will be executed in both modes. Should be used to make sure this works for both ways. ### `testSingleFolderMode()` Will only be executed for single folders. Testing with multi folder is redundant. ### `testMultiFolderMode()` Will only be executed in multi folder mode. These are special tests which are only designed for multi folders and makes no sense that it will work in single folder. --- `loadFixture()` and `getDiagnostics()` will use the second folder when the multi folder mode is activated
graphite-app
bot
force-pushed
the
05-08-test_editor_add_tests_for_multi_workspace_folder_setup
branch
from
3dbfcdb to
76b6b33
Compare
graphite-app
bot
deleted the
05-08-test_editor_add_tests_for_multi_workspace_folder_setup
branch
camc314
added a commit
that referenced
this pull request
May 16, 2025
## [0.16.11] - 2025-05-16 ### Features - 078bf0b language_server: Better fallback handling when passing invalid `Options` values (#10930) (Sysix) - be7f7e1 language_server/editor: Support multi workspace folders (#10875) (Sysix) - eef93b4 linter: Add import/no-unassigned-import (#10970) (yefan) - cc0112f linter: No-unused-vars add setting for `reportVarsOnlyUsedAsTypes` (#11009) (camc314) - 17e49c3 linter: Implement configuration and checking loops for `eslint/no_constant_condition` (#10949) (Ulrich Stark) - 21117ac linter: Implement react/forbid-elements (#10928) (Thomas BOCQUEZ) - 466c24a linter: Add gitlab reporter output format (#10927) (Connor Pearson) - a064082 linter: Add import/consistent-type-specifier-style rule (#10858) (yefan) - 4733b52 linter/no-extraneous-class: Add conditional fixer (#10798) (DonIsaac) ### Bug Fixes - 87bf2a8 editor: Send only `workspace/didChangeConfiguration` when some workspace configuration is effected (#11017) (Sysix) - ed5708d editor: Detect all workspaces config path changes (#11016) (Sysix) - 89cc21b language_server: Normalize oxlintrc config path (#10982) (Sysix) - c52a9ba linter: Fix plugins inside overrides not being applied (#11057) (camc314) - b12bd48 linter: Fix rule config not being correctly applied (#11055) (camc314) - 9a368be linter: False negative in no-restriced-imports with `patterns` and side effects (#11027) (camc314) - 8c2cfbc linter: False negative in no-restricted-imports (#11026) (camc314) - 8956870 linter: False positive in no-unused-vars (#11002) (camc314) - 33a60d2 linter: Skip eslint/no-redeclare when running on modules (#11004) (camc314) - 39063ce linter: Reword diagnostic message for no-control-regex (#10993) (camc314) - 9eedb58 linter: False positive with negative matches in no-restricted-imports (#10976) (camc314) - 10e77d7 linter: Improve diagnostics for no-control-regex (#10959) (camc314) - 0961296 linter: Add `gitlab` to linter `--help` docs (#10932) (camc314) - 82889ae linter/no-extraneous-class: Improve docs, reporting and code refactor (#10797) (DonIsaac) - 11c34e7 linter/no-img-element: Improve diagnostic and docs (#10908) (DonIsaac) - 584d8b9 napi: Enable mimalloc `no_opt_arch` feature on linux aarch64 (#11053) (Boshen) - 126ae75 semantic: Distinguish class private elements (#11044) (magic-akari) - 773d0de semantic: Correctly handle nested brackets in jsdoc parsing (#10922) (camc314) - b215b6c semantic: Dont parse `@` as jsdoc tags inside `[`/`]` (#10919) (camc314) ### Documentation - db6afb9 linter: Improve docs of no-debugger (#11033) (camc314) - 16541de linter: Improve docs of default-param-last (#11032) (camc314) - 2c2f3c4 linter: Improve docs of default-case-last (#11031) (camc314) - 56bb9ce linter: Improve docs of array-callback-return (#11030) (camc314) - 13dbcc6 linter: Correct docs for default config for no-redeclare (#10995) (camc314) - a86cbb3 linter: Fix incorrect backticks of fenced code blocks (#10947) (Ulrich Stark) ### Refactor - 3cc1466 language_server: New configuration structure for `initialize` and `workspace/didChangeConfiguration` (#10890) (Sysix) - bd2ef7d language_server: Use `Arc` for `diagnostic_report_map` (#10940) (Sysix) - bb999a3 language_server: Avoid cloning linter by taking reference in LintService (#10907) (Ulrich Stark) - d1b0c83 linter: Remove overrides index vec (#11058) (camc314) - 7ad6cf8 linter: Store severity separately, remove `RuleWithSeverity` (#11051) (camchenry) - e31c361 linter: Remove nested match statements in no-restricted-imports (#10975) (camc314) - 6ad9d4f linter: Tidy `eslint/func-names` (#10923) (camc314) - faf0a95 syntax: Rename `NameSpaceModule` to `NamespaceModule` (#10917) (Dunqing) ### Testing - 76b6b33 editor: Add tests for multi workspace folder setup (#10904) (Sysix) Co-authored-by: camc314 <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
We have now 3 different test setups:
test()Will be executed in both modes. Should be used to make sure this works for both ways.
testSingleFolderMode()Will only be executed for single folders. Testing with multi folder is redundant.
testMultiFolderMode()Will only be executed in multi folder mode. These are special tests which are only designed for multi folders and makes no sense that it will work in single folder.
loadFixture()andgetDiagnostics()will use the second folder when the multi folder mode is activated