Update TestRecoveringConsumerHandlerOnConnection_EventArgumentsArePassedDown

Author: Saibamen

projects/Unit/TestConnectionRecovery.cs

@@ -744,10 +744,17 @@ public void TestRecoveringConsumerHandlerOnConnection()
         [Test]
         public void TestRecoveringConsumerHandlerOnConnection_EventArgumentsArePassedDown()
         {
-            var myArgs = new Dictionary<string, object> { { "first-argument", "some-value" } };
+            const string key = "first-argument";
+            const string value = "some-value";
+
+            IDictionary<string, object> arguments = new Dictionary<string, object>
+            {
+                { key, value }
+            };
+
             string q = Model.QueueDeclare(GenerateQueueName(), false, false, false, null).QueueName;
             var cons = new EventingBasicConsumer(Model);
-            string expectedCTag = Model.BasicConsume(cons, q, arguments: myArgs);
+            string expectedCTag = Model.BasicConsume(cons, q, arguments: arguments);
 
             bool ctagMatches = false;
             bool consumerArgumentMatches = false;
@@ -757,14 +764,15 @@ public void TestRecoveringConsumerHandlerOnConnection_EventArgumentsArePassedDow
                 // passed to a CallbackExceptionHandler, instead of failing the test. Instead, we have to do this trick
                 // and assert in the test function.
                 ctagMatches = args.ConsumerTag == expectedCTag;
-                consumerArgumentMatches = (string)args.ConsumerArguments["first-argument"] == "some-value";
-                args.ConsumerArguments["first-argument"] = "event-handler-set-this-value";
+                consumerArgumentMatches = (string)args.ConsumerArguments[key] == value;
             };
 
             CloseAndWaitForRecovery();
             Assert.That(ctagMatches, Is.True, "expected consumer tag to match");
             Assert.That(consumerArgumentMatches, Is.True, "expected consumer arguments to match");
-            Assert.That(myArgs, Does.ContainKey("first-argument").WithValue("event-handler-set-this-value"));
+            Assert.That(arguments.ContainsKey(key), Is.True);
+            string actualVal = arguments[key];
+            Assert.That(actualVal, Is.EqualTo(value));
         }
 
         [Test]