synchronize terminal test on prompt, avoid panic (#7717)

Signed-off-by: Alex Suraci <alex@dagger.io>
dagger · Jun 21, 2024 · 4788020 · 4788020
1 parent 3828ad6
commit 4788020
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/core/integration/module_terminal_test.go b/core/integration/module_terminal_test.go
@@ -143,6 +143,9 @@ type Test struct {
 		err = cmd.Start()
 		require.NoError(t, err)
 
+		_, err = console.ExpectString(" $ ")
+		require.NoError(t, err)
+
 		_, err = console.SendLine("pwd")
 		require.NoError(t, err)
 

diff --git a/dagql/idtui/frontend_pretty.go b/dagql/idtui/frontend_pretty.go
@@ -69,11 +69,15 @@ func New() Frontend {
 	profile := ColorProfile()
 	view := new(strings.Builder)
 	return &frontendPretty{
-		db:   db,
-		logs: newPrettyLogs(),
-
+		db:        db,
+		logs:      newPrettyLogs(),
 		autoFocus: true,
 
+		// set empty initial row state to avoid nil checks
+		rowsView: &RowsView{},
+		rows:     &Rows{BySpan: map[trace.SpanID]*TraceRow{}},
+
+		// initial TUI state
 		window:  tea.WindowSizeMsg{Width: -1, Height: -1}, // be clear that it's not set
 		fps:     30,                                       // sane default, fine-tune if needed
 		profile: profile,