I don't believe this was ever required, awaiting writeInTerminal should make sure the
data event is triggered in xterm, the following enter should just trigger another data
event. This doesn't guarantee it was accepted by the process yet, just that xterm.js
emitted the event.
Part of #137847
* smoke - move data migration tests into one and align
* fix app starting
* `createWorkspaceFile` is not async
* 💄
* support screenshot on failure even for stable app
* smoke - try to remove timeout (#137847)
* improve exit call
* Add a bunch of tests
* tweak tests
* add profile tests, now breaking (no profiles showing up)
* uncomment tests
* update expected
* fix test
* merge
* all passing
* bunch of improvements
* improve readability
* all should be passing and cleaned up
* all passing
* fix#137243 and clean-up
* remove .only
* fix#137247
* get rid of enum
* see if this fixes timing issue
* use keybinding for show terminal instead
* remove delay, implement better accept function
* get rid of unneeded index
* try something else
* wait for empty editor to be active
* remove .only
* decrease redundancy in names
* re-add xterm selector
* try something else
* undo change that broke things worse
* add terminal-tabs test
* all tabs tests passing locally
* kill all
* await before running each
* fix tests
* remove .only
* try waiting for focused xterm
* remove .only
* clean up
* 🧹
* fix profiles tests by using this.app.props
* revert to use default shell
* re-add conditional
* Get tests passing on WebKit
On WebKit the smoke tests were failing because the quick pick was
triggering a focus event on the terminal after the blur event had
already happened, this caused the view service to think the terminal
was still focused when it wasn't. The fix was to reset the context key
also in IViewsService.closeView.
* Safari -> WebKit
* Prefer arrow functions
* remove beforeEach timeout
* add timeouts to contributed profile tests
* add terminal editor tests, fix#137377
* add a bunch of tests
* refactor
* get rid of profile specific command
* more polish
* refactor getTabs -> assertTerminalGroups
* fix failing test
* more polish
* remove .only
* add assertSingleTab
* remove unused import
* fix error
* fix more failures
* more changes
* fix almost all except for plus button
* fix failing plus button test
* all passing
* modify error that gets thrown
* modify error message again
* remove unused wildcard/ ANY_NAME, fix icon code
* fix icon code
* large refactor, improvements
* finish polish
* 1 failing
* get test to pass
Co-authored-by: Daniel Imms <2193314+Tyriar@users.noreply.github.com>
* 🆙 product
* 🆙 parcel/watcher
* 🆙 parcel/watcher
* tests - separate remote logs from others
* store server logs too
* more tweaks
* fix name
* error when 10s passed
* first cut upload log files from integration test runs
* always publish log files
* move logs into integration-tests folder
* also produce logs for smoke tests
* store remote log files
* wire in logspath arg properly
* always publish logs
* enable logs for browser based smoke tests
* log only in verbose
* Initial commit for isolated smoke tests
* Do not wait for the explorer view when restarting
* Debug failed localization test
* User data path suffix length
* Kill server when application exits
* Do not run yarn install
* Refactor application start/stop
* Reverted some changes
* Missed one
* One more change to revert
* Restored one more file
* Pull request feedback
* tests - disable getting started in integration tests and smoke tests
* --skip-getting-started => --skip-welcome
Co-authored-by: Jackson Kearl <jakearl@microsoft.com>
