8000 Scripted `pause` command timed out · Issue #6671 · sbt/sbt · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Scripted pause command timed out #6671

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
kxbmap opened this issue Sep 24, 2021 · 2 comments · Fixed by #7336
Closed

Scripted pause command timed out #6671

kxbmap opened this issue Sep 24, 2021 · 2 comments · Fixed by #7336
Labels

Comments

@kxbmap
Copy link
Contributor
kxbmap commented Sep 24, 2021

steps

scripted test file:

> doSomething
$ pause

Then run scripted test.

problem

[info] [success] Total time: 9 s, completed 2021/09/24 9:53:36
Pausing in ...\sbt_e5996dbf
Press enter to continue. [error] x group/name
sbt.internal.scripted.TestException: {line 4}  Command timed out
  | => cat sbt.scriptedtest.BatchScriptRunner.processStatement(BatchScriptRunner.scala:72)
...
Caused by: java.util.concurrent.TimeoutException
        at java.base/java.util.concurrent.FutureTask.get(FutureTask.java:204)
        at sbt.scriptedtest.BatchScriptRunner.processStatement(BatchScriptRunner.scala:55)
...

The test directory deletes while investigating.

expectation

pause command should not timeout.

notes

@kxbmap kxbmap added the Bug label Sep 24, 2021
@steinybot
Copy link
Contributor
steinybot commented Jul 11, 2023

This is really annoying. It is hard coded to 5 minutes https://github.com/sbt/sbt/blob/1.9.x/scripted-sbt-redux/src/main/scala/sbt/scriptedtest/BatchScriptRunner.scala#L47.

The only workaround I can think of is to run sbt --jvm-debug 5005 attach the debugger to sbt and then pause the whole JVM after it has started the scripted test.

@eed3si9n
Copy link
Member

The timeout was added in #5151. We can revert the entire PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants
0