Strip out BELL/ALERT sequence

Author: Naatan

helpers_windows.go

@@ -2,12 +2,8 @@ package termtest
 
 import (
 	"bytes"
-
-	"golang.org/x/sys/windows"
 )
 
-var ERR_ACCESS_DENIED = windows.ERROR_ACCESS_DENIED
-
 const UnicodeEscapeRune = '\u001B'
 const UnicodeBellRune = '\u0007'
 const UnicodeBackspaceRune = '\u0008' // Note in the docs this is \u007f, but in actual use we're seeing \u0008. Possibly badly documented.
@@ -51,6 +47,11 @@ func cleanPtySnapshot(snapshot []byte, cursorPos int, isPosix bool) ([]byte, int
 		switch {
 		// SEQUENCE START
 
+		// Delete alert / bell sequence
+		case !inEscapeSequence && r == '\a':
+			dropPos(pos)
+			continue
+
 		// Detect start of escape sequence
 		case !inEscapeSequence && r == UnicodeEscapeRune:
 			inEscapeSequence = true

helpers_windows_test.go

@@ -116,6 +116,13 @@ func Test_cleanPtySequences(t *testing.T) {
 			[]byte("foo"),
 			-1,
 		},
+		{
+			"Alert / bell character",
+			[]byte("\aP\x1b[?25lython 3.9.5"),
+			0,
+			[]byte("Python 3.9.5"),
+			-1,
+		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {

termtest_windows.go

@@ -7,8 +7,11 @@ import (
 	"time"
 
 	gopsutil "github.com/shirou/gopsutil/v3/process"
+	"golang.org/x/sys/windows"
 )
 
+var ERR_ACCESS_DENIED = windows.ERROR_ACCESS_DENIED
+
 func syscallErrorCode(err error) int {
 	if errv, ok := err.(syscall.Errno); ok {
 		return int(errv)