Flaky Test: Terminal - Terminal Persistence - detach/attach - should persist buffer content

[hediet]

1. VSCode Smoke Tests (Electron)
   Terminal
   Terminal Persistence
   detach/attach
   should persist buffer content:
   Error: Timeout: get element '.quick-input-widget' after 20 seconds.
   at Code.poll (/Users/runner/work/1/s/test/automation/out/code.js:204:23)
   at async Code.waitForElement (/Users/runner/work/1/s/test/automation/out/code.js:163:16)
   at async QuickInput.waitForQuickInputClosed (/Users/runner/work/1/s/test/automation/out/quickinput.js:32:9)
   at async Terminal.runCommandWithValue (/Users/runner/work/1/s/test/automation/out/terminal.js:109:9)
   at async Context. (out/areas/terminal/terminal-persistence.test.js:64:17)

https://dev.azure.com/monacotools/Monaco/_build/results?buildId=278997&view=logs&j=260a512b-e5ca-5cf1-3f8e-20c0fa41efc9&t=dc7a7a05-ae89-521b-55db-0cb2ded34668

[meganrogge]

1) VSCode Smoke Tests (Electron)
       Terminal
         Terminal Persistence
           detach/attach
             should persist buffer content:
     Error: Timeout: get terminal buffer '#terminal .terminal-wrapper' after 20 seconds.
      at Code.poll (D:\a\_work\1\s\test\automation\out\code.js:204:23)
      at async Code.waitForTerminalBuffer (D:\a\_work\1\s\test\automation\out\code.js:175:9)
      at async Terminal.waitForTerminalText (D:\a\_work\1\s\test\automation\out\terminal.js:262:13)
      at async Context.<anonymous> (out\areas\terminal\terminal-persistence.test.js:68:17)

https://dev.azure.com/monacotools/Monaco/_build/results?buildId=286027&view=results

[meganrogge]

Fails bc of line wrapping differences

Before detaching:

After attaching:

[meganrogge]

i think we should change the test so that it removes \n from the resultant string

[vs-code-engineering]

		Issue marked as unreleased but unable to locate closing commit in issue timeline. You can manually reference a commit by commenting `\closedWith someCommitSha`, or directly add the `insiders-released` label if you know this has already been releaased

[meganrogge]

still seeing a failure 🤔

[meganrogge]

wondering if we also need to match \r for windows

[alexr00]

Looks like it's still failing. Skipping to unblock the build.

[meganrogge]

Thanks @alexr00

[meganrogge]

I'm out of ideas for how to fix this, think it should remain skipped to prevent build failures