Skip to content

fix: improve pty and ptytest #5327

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.

Sign up for GitHub

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

Merged
merged 17 commits into from
Dec 7, 2022
Merged

fix: improve pty and ptytest #5327

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
f823b02
Fix: improve ptytest
mtojek Dec 6, 2022
8ab62d4
Merge branch 'main' into fix-pty-1
mtojek Dec 6, 2022
a97a7ad
Disable skip
mtojek Dec 6, 2022
593a891
WIP
mtojek Dec 7, 2022
8a3f6e2
WIP
mtojek Dec 7, 2022
1a7070f
WIP
mtojek Dec 7, 2022
3afeca9
WIP
mtojek Dec 7, 2022
7ea26b6
WIP
mtojek Dec 7, 2022
14a79d1
WIP
mtojek Dec 7, 2022
04fce69
WIP
mtojek Dec 7, 2022
dd2cfd0
WIP
mtojek Dec 7, 2022
ea3b170
Fix
mtojek Dec 7, 2022
a042bfc
WIP
mtojek Dec 7, 2022
dc86c9a
Fix
mtojek Dec 7, 2022
e48d4ab
fix: pty close on Windows
mtojek Dec 7, 2022
eff5966
Revert changes around workflow
mtojek Dec 7, 2022
d88b552
fix
mtojek Dec 7, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
WIP
  • Loading branch information
@mtojek
mtojek committed Dec 7, 2022
commit 593a891e9523ced6e3dd4f082871b398c0549c96
8 changes: 8 additions & 0 deletions pty/pty_other.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -114,9 +114,17 @@ func (p *otherPty) Resize(height uint16, width uint16) error {
}

func (p *otherPty) Close() error {
if p.opts.logger != nil {
p.opts.logger.Println("Will close...")
}

p.mutex.Lock()
defer p.mutex.Unlock()

if p.opts.logger != nil {
p.opts.logger.Println("Closing...")
}

if p.closed {
return p.err
}
Expand Down
4 changes: 2 additions & 2 deletions pty/ptytest/ptytest.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -152,7 +152,7 @@ func (p *PTY) ExpectMatch(str string) string {
p.logf("matched %q = %q", str, buffer.String())
return buffer.String()
case <-timeout.Done():
// Ensure gorouine is cleaned up before test exit.
// Ensure goroutine is cleaned up before test exit.
_ = p.out.closeErr(p.Close())
<-match

Expand Down Expand Up @@ -202,7 +202,7 @@ func logf(t *testing.T, name, format string, args ...interface{}) {
func fatalf(t *testing.T, name, reason, format string, args ...interface{}) {
t.Helper()

panic(fmt.Sprintf("%s: %s %s", name, reason, fmt.Sprintf(format, args...)))
t.Fatal(fmt.Sprintf("%s: %s %s", name, reason, fmt.Sprintf(format, args...)))
}

// stdbuf is like a buffered stdout, it buffers writes until read.
Expand Down