Skip to content

Commit

Permalink
fix race in internal integration spec
Browse files Browse the repository at this point in the history
  • Loading branch information
onsi committed Dec 10, 2022
1 parent 1869212 commit 366815d
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions internal/internal_integration/report_suite_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -369,7 +369,7 @@ var _ = Describe("Sending reports to ReportBeforeSuite and ReportAfterSuite node

Context("the happy path", func() {
BeforeEach(func() {
By("proc1 signals that its ReportBeforeSuites succeeded")
// proc1 signals that its ReportBeforeSuites succeeded
client.PostReportBeforeSuiteCompleted(types.SpecStatePassed)
Eventually(done).Should(BeClosed())
})
Expand All @@ -385,7 +385,7 @@ var _ = Describe("Sending reports to ReportBeforeSuite and ReportAfterSuite node

Context("when the ReportBeforeSuite node fails", func() {
BeforeEach(func() {
By("proc1 signals that its ReportBeforeSuites failed")
// proc1 signals that its ReportBeforeSuites failed
client.PostReportBeforeSuiteCompleted(types.SpecStateFailed)
Eventually(done).Should(BeClosed())
})
Expand All @@ -397,7 +397,7 @@ var _ = Describe("Sending reports to ReportBeforeSuite and ReportAfterSuite node

Context("when proc1 exits before reporting", func() {
BeforeEach(func() {
By("proc1 signals that its ReportBeforeSuites failed")
// proc1 signals that its ReportBeforeSuites failed
close(exitChannels[1])
Eventually(done).Should(BeClosed())
})
Expand Down

0 comments on commit 366815d

Please sign in to comment.