bindings tests: need better error detection

[edsantiago]

The logging test does not seem to detect a client crash; this manifests as a hang in the tests.

podman/pkg/bindings/test/containers_test.go

Lines 354 to 374 in a245da1

	It("logging", func() {
	stdoutChan := make(chan string, 10)
	s := specgen.NewSpecGenerator(alpine.name, false)
	s.Terminal = true
	s.Command = []string{"date", "-R"}
	r, err := containers.CreateWithSpec(bt.conn, s, nil)
	Expect(err).To(BeNil())
	err = containers.Start(bt.conn, r.ID, nil)
	Expect(err).To(BeNil())
	_, err = containers.Wait(bt.conn, r.ID, nil)
	Expect(err).To(BeNil())
	opts := new(containers.LogOptions).WithStdout(true).WithFollow(true)
	go func() {
	containers.Logs(bt.conn, r.ID, opts, stdoutChan, nil)
	}()
	o := <-stdoutChan
	o = strings.TrimSpace(o)
	_, err = time.Parse(time.RFC1123Z, o)
	Expect(err).ShouldNot(HaveOccurred())

The crash in this instance is a bug (#12175) but even after that's fixed, there will be others.

[Luap99]

#12177 should fix this