all: prefer AddCleanup to TearDown #5025

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
Contributor

davecheney commented Apr 7, 2016

A common failure mode of our tests is when an issue occurs during
the SetUp phase of the suite or test, not only does the test fail, but
the Tear down phase blows up because it is rarely written to handle the
case where the setup phase did not complete.

(Review request: http://reviews.vapour.ws/r/4467/)

Contributor

davecheney commented Apr 8, 2016

$$merge$$

Contributor

jujubot commented Apr 8, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

jujubot commented Apr 8, 2016

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/7247

Contributor

jujubot commented Apr 9, 2016

Build failed: Does not match ['fixes-1568312']
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/7275

Contributor

davecheney commented May 16, 2016

$$merge$$

Contributor

jujubot commented May 16, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

jujubot commented May 17, 2016

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/7766

Contributor

davecheney commented May 17, 2016

$$merge$$

Contributor

jujubot commented May 17, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

jujubot commented May 17, 2016

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/7768

Contributor

davecheney commented May 17, 2016

$$merge$$

Contributor

jujubot commented May 17, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

jujubot commented May 17, 2016

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/7772

Contributor

davecheney commented May 18, 2016

$$merge$$

Contributor

jujubot commented May 18, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

jujubot commented May 18, 2016

Build failed: Generating tarball failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/7787

all: prefer AddCleanup to TearDown
A common failure mode of our tests is when an issue occurs during
the SetUp phase of the suite or test, not only does the test fail, but
the Tear down phase blows up because it is rarely written to handle the
case where the setup phase did not complete.
Contributor

davecheney commented May 18, 2016

$$merge$$

Contributor

jujubot commented May 18, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

jujubot commented May 18, 2016

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/7789

Owner

howbazaar commented Aug 9, 2016

Too much churn underneath to attempt to make good to land.

@howbazaar howbazaar closed this Aug 9, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment