-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Fix failing to start shell being hidden from user #5626
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
Closed
Closed
Changes from 3 commits
Commits
Show all changes
86 commits
Select commit
Hold shift + click to select a range
ed6f522
Fix failing to start shell being hidden from user
Nokel81 87dc14c
Cleanup
Nokel81 dba4f59
Add enough code to get first couple terminal tests to pass
Nokel81 8136e23
Fix type errors from testing in dev mode
Nokel81 f111450
Introduce and use ApplicationBuilder.quit in afterEach() hooks
Nokel81 c9f1594
Upgrade XTermJs to v5 to get snapshot testing working
Nokel81 2de3068
Use webgl when available, add search and better weblinks
Nokel81 d293554
Get tests passing
Nokel81 988307e
Add script to download the correct version of canvas
Nokel81 071ba8a
Fix installing canvas on all platforms
Nokel81 ea603c7
Fix the script
Nokel81 78b05ba
Remove console logs
Nokel81 aa41117
Fix lint
Nokel81 19466f1
Temporarily upgrade to beta version of xterm to verify fix
Nokel81 d1acca4
Fix post install script because of temp version of xterm
Nokel81 f2d19e7
Fix some more lint
Nokel81 2a96eb8
Fix usage of new asyncComputed type
Nokel81 862cd2a
Update (mostly just) snapshots and a few other tests
Nokel81 3c9791e
Fix type error
Nokel81 c340256
Replace last use of customRequestPromise with di
Nokel81 f522d23
Replace usage of request in setupLensProxy
Nokel81 37955f0
Replace usage of request in k8sRequest and cluster detectors
Nokel81 b86bc6d
Remove direct dependency on request et al
Nokel81 4c1d076
Add process of cleaning up lens proxy startup if down quickly
Nokel81 c6a9d56
Fix bad vscode rename
Nokel81 e49b0f6
Remove calls to show error popup on stopping
Nokel81 d173e45
Fix starting of lens proxy
Nokel81 b96e115
Fix type errors
Nokel81 6ce833c
Make ApplicationBuilder.quit more realistic
Nokel81 0dabbb6
Replace override of catalogSyncToRenderer with a global override
Nokel81 a82a120
Fix many tests by changing injectables
Nokel81 e90a181
Fix listing-active-helm-repositories-in-preferences.test.ts
Nokel81 9c75199
Update snapshot
Nokel81 97dfe4d
Fix new tests
Nokel81 e8bc887
Fix postinstall script
Nokel81 58cbf5f
Fix bad imports of node-fetch
Nokel81 8761aa1
Fix type errors
Nokel81 8fd732b
Fix unit tests
Nokel81 e9e248e
Fix tests not stopping by forcing cleanup
Nokel81 71ad8e1
Generate tray icons before installing canvas
Nokel81 3a7ac10
Add updated icons
Nokel81 c11d8c6
Fix lint
Nokel81 0088add
Fix code after rebase
Nokel81 6321b10
Fix test failures
Nokel81 9683a7c
Fix tests and lint
Nokel81 a271c83
Update snapshots
Nokel81 3b72137
Remove unused (but newly added) dep
Nokel81 19a8691
Update snapshots
Nokel81 f6b5214
Fixup new test file after rebase
Nokel81 78614af
Update snapshots
Nokel81 56703cd
Update snapshots
Nokel81 a1d2fa0
Fix lint
Nokel81 3fbbf02
Fix errors blocking compilation
Nokel81 8b18e58
Update snapshots
Nokel81 d9a7a48
Fix lint
Nokel81 ea49929
Fix types
Nokel81 0e1b2c0
Fix test failures
Nokel81 12bd506
Fix lint
Nokel81 2f43d24
Fix build
Nokel81 14f50a3
Fix unit tests
Nokel81 6d0e527
Fix build type errors
Nokel81 d22af76
Fix most tests
Nokel81 3492741
Fix kubeconfig-manager tests
Nokel81 6dd926a
Fix tests not using ApplicationBuilder
Nokel81 cfad2c0
Fix expected URL
Nokel81 186d7f9
Fix tests not using ApplicationBuilder
Nokel81 9adddaf
Fix top bar tests
Nokel81 e88743d
Add suspend plugin
Nokel81 4885461
More work to get tests to pass in a more realistic way
Nokel81 120a5f7
Fix tests by faking the version detector
Nokel81 91cd2c8
Remove debug logging
Nokel81 7971755
Cleanup uses and overrides of requestAnimationFrame
Nokel81 c658a0b
Get terminal tests to pass again
Nokel81 9b7e77d
Remove unneccessary causesSideEffects
Nokel81 4815510
Update cluster-frame test snapshots
Nokel81 3471397
Fix edit-namespace-from-new-tab tests
Nokel81 e77411a
Fix tests failing due to allow resources not updateing
Nokel81 d1eb149
Fix tooltip tests
Nokel81 6880d52
Update install-helm-chart-from-new-tab snapshots
Nokel81 d2ee54f
Update showing-details-for-helm-release snapshots
Nokel81 8704067
Update sidebar-and-tab-navigation-for-extensions snapshots
Nokel81 9539c1c
Fix cluster-store tests
Nokel81 5f30b81
Fix kube-auth-proxy tests
Nokel81 4edfa40
Update more snapshots
Nokel81 29a7e2a
Fix kube-api-version-detection tests
Nokel81 51b8c91
Fix build
Nokel81 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
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
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
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
15 changes: 0 additions & 15 deletions
15
src/common/utils/channel/request-from-channel-injection-token.ts
This file was deleted.
Oops, something went wrong.
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
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
Binary file added
BIN
+593 Bytes
...uster-frame-when-new-terminal-tab-is-opened-shows-connecting-message-1-snap.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
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.
Uh oh!
There was an error while loading. Please reload this page.
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.
We shouldn't be removing the injection token.
requestFromChannelis contract which is implemented separately in renderer and main but the contract is same.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.
Important thing is that channel is something that can be called from anywhere and it doesn't know who answers. That being said, the "request" or "message" doesn't even have to go over IPC, the listener and caller can be located in same environment. (It doesn't work like this yet due YAGNI but conceptually that's what Channel is.)
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.
The contract isn't the same. A request channel being requested on has no meaning on
mainand thusmainshouldn't even know about it.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.
maincan request stuff from channel. It just doesn't know who answers it, might be main or renderer.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.
No it cannot, where is that possible?