Commit 63981b5

authored

Allow a bit more time for mocked tests to correctly cleanup. (#3060)

This should avoid some sporadic CI test failures.

1 parent 50561aa commit 63981b5Copy full SHA for 63981b5

File tree

+18

-2

lines changed

+18

-2

lines changed

Lines changed: 18 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -17,6 +17,7 @@ package daemon`
`17`	`17`
`18`	`18`	`import (`
`19`	`19`	`"fmt"`
	`20`	`+ "os"`
`20`	`21`	`"testing"`
`21`	`22`	`"time"`
`22`	`23`
`@@ -66,8 +67,15 @@ func TestBoardListMock(t *testing.T) {`
`66`	`67`	`}))`
`67`	`68`
`68`	`69`	`// Run a BoardList`
`69`		`- resp, err := grpcInst.BoardList(time.Second)`
`70`		`- require.NoError(t, err)`
	`70`	`+ var resp *commands.BoardListResponse`
	`71`	`+ for range 5 { // wait up to 5 seconds`
	`72`	`+ resp, err = grpcInst.BoardList(time.Second)`
	`73`	`+ require.NoError(t, err)`
	`74`	`+ if len(resp.Ports) > 0 {`
	`75`	`+ break`
	`76`	`+ }`
	`77`	`+ time.Sleep(time.Second)`
	`78`	`+ }`
`71`	`79`	`require.NotEmpty(t, resp.Ports)`
`72`	`80`	`for _, port := range resp.Ports {`
`73`	`81`	`if port.GetPort().GetProtocol() == "serial" {`
`@@ -81,6 +89,14 @@ func TestBoardListMock(t *testing.T) {`
`81`	`89`	`}`
`82`	`90`
`83`	`91`	`// Check if the discoveries have been successfully close`
	`92`	`+ for range 5 { // wait up to 5 seconds`
	`93`	`+ _, err1 := os.Lstat(tmp1)`
	`94`	`+ _, err2 := os.Lstat(tmp2)`
	`95`	`+ if err1 != nil && err2 != nil {`
	`96`	`+ break`
	`97`	`+ }`
	`98`	`+ time.Sleep(time.Second)`
	`99`	`+ }`
`84`	`100`	`require.NoFileExists(t, tmp1, "discovery has not been closed")`
`85`	`101`	`require.NoFileExists(t, tmp2, "discovery has not been closed")`
`86`	`102`	`}`

Comments

(0)