Skip to content

Commit

Permalink
gexec: allow more headroom for SIGABRT-related unit tests (#581)
Browse files Browse the repository at this point in the history
  • Loading branch information
thediveo committed Aug 29, 2022
1 parent 061fd26 commit 5b78f40
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions gexec/session_test.go
Expand Up @@ -130,7 +130,7 @@ var _ = Describe("Session", func() {
Expect(err).ShouldNot(HaveOccurred())

session.Signal(syscall.SIGABRT)
Eventually(session).Should(Exit(128 + 6))
Eventually(session).WithTimeout(5 * time.Second).Should(Exit(128 + 6))
})

It("should ignore sending a signal if the command did not start", func() {
Expand Down Expand Up @@ -251,9 +251,9 @@ var _ = Describe("Session", func() {

Signal(syscall.SIGABRT)

Eventually(session1).Should(Exit(128 + 6))
Eventually(session2).Should(Exit(128 + 6))
Eventually(session3).Should(Exit(128 + 6))
Eventually(session1).WithTimeout(5 * time.Second).Should(Exit(128 + 6))
Eventually(session2).WithTimeout(5 * time.Second).Should(Exit(128 + 6))
Eventually(session3).WithTimeout(5 * time.Second).Should(Exit(128 + 6))
})
})

Expand Down

0 comments on commit 5b78f40

Please sign in to comment.