Skip to content

Commit

Permalink
Fix a Shovel test
Browse files Browse the repository at this point in the history
1. There is no need to test Shovel deletion in
   this specific test
2. When a Shovel is starting, it cannot be deleted
   with several RabbitMQ versions due to [A]

A. rabbitmq/rabbitmq-server#11324
  • Loading branch information
michaelklishin committed Jun 15, 2024
1 parent f103983 commit 9f5bac1
Showing 1 changed file with 2 additions and 9 deletions.
11 changes: 2 additions & 9 deletions rabbithole_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -3842,8 +3842,8 @@ var _ = Describe("RabbitMQ HTTP API client", func() {
vh := "rabbit/hole"
sn := "temporary"

ssu := URISet([]string{"amqp://127.0.0.1/%2f"})
sdu := URISet([]string{"amqp://127.0.0.1/%2f"})
ssu := URISet([]string{"amqp://localhost/%2f"})
sdu := URISet([]string{"amqp://localhost/%2f"})

shovelDefinition := ShovelDefinition{
SourceURI: ssu,
Expand All @@ -3870,13 +3870,6 @@ var _ = Describe("RabbitMQ HTTP API client", func() {

_, err = rmqc.DeleteShovel(vh, sn)
Ω(err).Should(BeNil())

Eventually(func(g Gomega) []ShovelStatus {
xs, err := rmqc.ListShovelStatus(vh)
Ω(err).Should(BeNil())

return xs
}).Should(BeEmpty())
})
})

Expand Down

0 comments on commit 9f5bac1

Please sign in to comment.