Skip to content

Commit

Permalink
Update rabbitmq version CI
Browse files Browse the repository at this point in the history
The test StreamShouldReadTheAmqp10PropertiesMessages now works correctly due of
rabbitmq/rabbitmq-server#7001 so we removed the workaround
for the issue

Signed-off-by: Gabriele Santomaggio <[email protected]>
  • Loading branch information
Gsantomaggio committed Feb 14, 2023
1 parent 2fdb57a commit f0ea2a7
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 6 deletions.
2 changes: 1 addition & 1 deletion .ci/versions.json
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
{
"erlang": "25.2",
"rabbitmq": "3.11.7"
"rabbitmq": "3.11.9"
}
6 changes: 1 addition & 5 deletions Tests/FromToAMQPTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -351,11 +351,7 @@ public async void StreamShouldReadTheAmqp10PropertiesMessages()

new Utils<Message>(_testOutputHelper).WaitUntilTaskCompletes(tcs);
var result = tcs.Task.Result;
// Why do we need result.Data.Contents.ToArray()[5..]?
// Because of https://github.com/rabbitmq/rabbitmq-server/issues/6937
// When it will be fixed we can remove the [5..]
// For the moment we leave it as it is because it is not a problem for the client
Assert.Equal("msg from amqp 1.0", Encoding.UTF8.GetString(result.Data.Contents.ToArray()[5..]));
Assert.Equal("msg from amqp 1.0", result.AmqpValue);
await consumer.Close();
await system.DeleteStream(stream);
await system.Close();
Expand Down

0 comments on commit f0ea2a7

Please sign in to comment.