Hi all, I've recently added the Actions bit mentioned by Andrew to FilterSuite. I'll defintely have a look over that. We're chatting about why that might be happening. Have you seen this more than once?
Bodie > From: Andrew Wilkins <[email protected]> > Date: Thu, Jul 10, 2014 at 3:19 PM > Subject: Re: New flakey juju tests > To: Martin Packman <[email protected]> > Cc: "[email protected]" <[email protected]> > > > Here's another, that I've started seeing today: > > FAIL: filter_test.go:508: FilterSuite.TestActionEvents > ... > filter_test.go:531: > assertChange(testIds) > filter_test.go:490: > c.Assert(event.ActionId, gc.Equals, id) > ... obtained string = "wordpress/0_a_5" > ... expected string = "wordpress/0_a_1" > > > On Wed, Jul 9, 2014 at 5:26 PM, Martin Packman < > [email protected]> wrote: > >> We've done a lot of work recently improving the reliability of our >> test suite. Unfortunately, feature work has been introducing new tests >> that are intermittently failing. >> >> <http://juju-ci.vapour.ws:8080/job/github-merge-juju/1/console> >> >> FAIL: machine_test.go:1750: MachineSuite.TestWatchInterfaces >> ... >> machine_test.go:1808: >> wc.AssertOneChange() >> testing/watcher.go:76: >> c.Fatalf("watcher sent unexpected change: (_, %v)", ok) >> ... Error: watcher sent unexpected change: (_, true) >> >> ... >> FAIL github.com/juju/juju/state 123.265s >> >> This test was added in pr 207. >> >> <https://github.com/juju/juju/pull/207> >> >> >> <http://juju-ci.vapour.ws:8080/job/github-merge-juju/3/console> >> >> FAIL: server_test.go:96: >> serverSuite.TestAPIServerCanListenOnBothIPv4AndIPv6 >> ... >> server_test.go:104: >> c.Assert(err, gc.IsNil) >> ... value *net.OpError = &net.OpError{Op:"listen", Net:"tcp", >> Addr:(*net.TCPAddr)(0xc2107e9f00), >> Err:(*os.SyscallError)(0xc21027f560)} ("listen tcp :54321: bind: >> address already in use") >> >> ... >> FAIL github.com/juju/juju/state/apiserver 30.369s >> >> This test was added in pr 224. >> >> <https://github.com/juju/juju/pull/224> >> >> >> Can we have another look at these tests and fix them up to be properly >> robust? I don't want to back out changes that have been in trunk for a >> while, but we can't leave unreliable tests on trunk. Thanks, >> >> Martin >> >> -- >> Juju-dev mailing list >> [email protected] >> Modify settings or unsubscribe at: >> https://lists.ubuntu.com/mailman/listinfo/juju-dev >> > > > -- > Juju-dev mailing list > [email protected] > Modify settings or unsubscribe at: > https://lists.ubuntu.com/mailman/listinfo/juju-dev > > >
-- Juju-dev mailing list [email protected] Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/juju-dev
