Skip to content

Commit ae2d781

Browse files
committed
fix(tests): fix ReservePort/ReleasePort ordering for NATS runner in 5 test files
1 parent 5a68a68 commit ae2d781

5 files changed

Lines changed: 5 additions & 5 deletions

File tree

src/code.cloudfoundry.org/gorouter/common/component_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -168,8 +168,8 @@ var _ = Describe("Component", func() {
168168

169169
BeforeEach(func() {
170170
natsPort := test_util.ReservePort()
171-
test_util.ReleasePort(natsPort)
172171
natsRunner = test_util.NewNATSRunner(int(natsPort))
172+
test_util.ReleasePort(natsPort)
173173
natsRunner.Start()
174174
mbusClient = natsRunner.MessageBus
175175
mbusClient.Opts.SkipSubjectValidation = true

src/code.cloudfoundry.org/gorouter/integration/main_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,8 +73,8 @@ var _ = Describe("Router Integration", func() {
7373
sslPort = test_util.ReservePort()
7474
routeServiceServerPort = test_util.ReservePort()
7575

76-
test_util.ReleasePort(natsPort)
7776
natsRunner = test_util.NewNATSRunner(int(natsPort))
77+
test_util.ReleasePort(natsPort)
7878
natsRunner.Start()
7979
oauthServerURL = oauthServer.Addr()
8080
})

src/code.cloudfoundry.org/gorouter/integration/nats_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,8 +43,8 @@ var _ = Describe("NATS Integration", func() {
4343
natsPort = test_util.ReservePort()
4444
routeServiceServerPort = test_util.ReservePort()
4545

46-
test_util.ReleasePort(natsPort)
4746
natsRunner = test_util.NewNATSRunner(int(natsPort))
47+
test_util.ReleasePort(natsPort)
4848
natsRunner.Start()
4949
})
5050

src/code.cloudfoundry.org/gorouter/mbus/subscriber_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,8 +39,8 @@ var _ = Describe("Subscriber", func() {
3939

4040
BeforeEach(func() {
4141
natsPort = test_util.ReservePort()
42-
test_util.ReleasePort(natsPort)
4342
natsRunner = test_util.NewNATSRunner(int(natsPort))
43+
test_util.ReleasePort(natsPort)
4444
natsRunner.Start()
4545
natsClient = natsRunner.MessageBus
4646

src/code.cloudfoundry.org/gorouter/router/router_drain_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -152,8 +152,8 @@ var _ = Describe("Router", func() {
152152
BeforeEach(func() {
153153
logger = test_util.NewTestLogger("test")
154154
natsPort = test_util.ReservePort()
155-
test_util.ReleasePort(natsPort)
156155
natsRunner = test_util.NewNATSRunner(int(natsPort))
156+
test_util.ReleasePort(natsPort)
157157
natsRunner.Start()
158158

159159
proxyPort := test_util.NextAvailPort()

0 commit comments

Comments
 (0)