Skip to content

Commit

Permalink
Merge pull request #15792 from hpidcock/fix-test-watchrawk8sspec
Browse files Browse the repository at this point in the history
#15792

Since juju 3.0, we have used the changestream from mongo which doesn't give us coalescing. This test was still expecting it.

## QA steps

```
$ cd state
$ go test -c
# run the test a bunch
$ while ./state.test -check.f=TestWatchRawK8sSpec; do :; done
```

## Documentation changes

N/A

## Bug reference

https://jenkins.juju.canonical.com/job/unit-tests-race-amd64/1270/consoleFull
  • Loading branch information
jujubot committed Jun 22, 2023
2 parents 68596da + b36219e commit 9d02e3e
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions state/podspec_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -259,6 +259,8 @@ func (s *PodSpecSuite) TestWatchRawK8sSpec(c *gc.C) {
// Multiple changes coalesced.
err = s.applySetRawK8sSpecOperation(nil, s.application.ApplicationTag(), strPtr("raw spec 1"))
c.Assert(err, jc.ErrorIsNil)
// TODO(quiescence): these two changes should be one event.
wc.AssertOneChange()
err = s.applySetRawK8sSpecOperation(nil, s.application.ApplicationTag(), strPtr("raw spec 2"))
c.Assert(err, jc.ErrorIsNil)
wc.AssertOneChange()
Expand Down

0 comments on commit 9d02e3e

Please sign in to comment.