run dotnet format (#537)

run `dotnet format`

JustSaying.IntegrationTests/JustSayingFluently/MultiRegion/WithSqsPointToPointSubscriber/WhenOneBusIsSubscribedToTwoRegions.cs

@@ -61,7 +61,7 @@ private void GivenASubscriptionToAQueueInTwoRegions(string primaryRegion, string
             handler.Handle(Arg.Any<SimpleMessage>()).Returns(true);
             handler
                 .When(x => x.Handle(Arg.Any<SimpleMessage>()))
-                .Do(async x => await _handler.Complete((SimpleMessage) x.Args()[0]));
+                .Do(async x => await _handler.Complete((SimpleMessage)x.Args()[0]));
 
             _subscriber = CreateMeABus
                 .WithLogging(LoggerFactory)

JustSaying.IntegrationTests/JustSayingFluently/MultiRegion/WithSqsTopicSubscriber/WhenOneBusIsSubscribedToTwoRegions.cs

@@ -61,7 +61,7 @@ private void GivenASubscriptionToAQueueInTwoRegions(string primaryRegion, string
             handler.Handle(Arg.Any<SimpleMessage>()).Returns(true);
             handler
                 .When(x => x.Handle(Arg.Any<SimpleMessage>()))
-                .Do(async x => await _handler.Complete((SimpleMessage) x.Args()[0]));
+                .Do(async x => await _handler.Complete((SimpleMessage)x.Args()[0]));
 
             _subscriber = CreateMeABus
                 .WithLogging(LoggerFactory)

JustSaying.Models/Message.cs

@@ -13,7 +13,7 @@ protected Message()
         public Guid Id { get; set; }
         public DateTime TimeStamp { get; set; }
         public string RaisingComponent { get; set; }
-        public string Version{ get; private set; }
+        public string Version { get; private set; }
         public string SourceIp { get; private set; }
         public string Tenant { get; set; }
         public string Conversation { get; set; }

JustSaying.Tools/CommandParser.cs

@@ -26,18 +26,18 @@ await CommandLine
         private static void InitializeCommandLineParser(ICommandLineElementParser<ICommand> x)
         {
             x.Add((from arg in x.Argument("exit")
-                   select (ICommand) new ExitCommand())
+                   select (ICommand)new ExitCommand())
                 .Or(from arg in x.Argument("quit")
-                    select (ICommand) new ExitCommand())
+                    select (ICommand)new ExitCommand())
                 .Or(from arg in x.Argument("help")
-                    select (ICommand) new HelpCommand())
+                    select (ICommand)new HelpCommand())
 
                 .Or(from arg in x.Argument("move")
                     from sourceQueueName in x.Definition("from")
                     from destinationQueueName in x.Definition("to")
                     from region in x.Definition("in")
                     from count in (from d in x.Definition("count") select d).Optional("count", "1")
-                    select (ICommand) new MoveCommand(sourceQueueName.Value, destinationQueueName.Value, region.Value,
+                    select (ICommand)new MoveCommand(sourceQueueName.Value, destinationQueueName.Value, region.Value,
                         int.Parse(count.Value, CultureInfo.InvariantCulture)))
                 );
         }

JustSaying.UnitTests/AwsTools/MessageHandling/ExactlyOnceReaderTests.cs

@@ -9,7 +9,7 @@ public class ExactlyOnceReaderTests
         [Fact]
         public void ObjectTypeDoesNotHaveExactlyOnce()
         {
-            var reader = new ExactlyOnceReader(typeof (object));
+            var reader = new ExactlyOnceReader(typeof(object));
 
             reader.Enabled.ShouldBeFalse();
         }
@@ -25,7 +25,7 @@ public void UnadornedHandlerType_DoesNotHaveExactlyOnce()
         [Fact]
         public void OnceTestHandlerAsyncType_HasExactlyOnce()
         {
-            var reader = new ExactlyOnceReader(typeof (OnceTestHandlerAsync));
+            var reader = new ExactlyOnceReader(typeof(OnceTestHandlerAsync));
 
             reader.Enabled.ShouldBeTrue();
         }

JustSaying.UnitTests/AwsTools/MessageHandling/HandlerMapTests.cs

@@ -31,7 +31,7 @@ public void EmptyMapReturnsNullHandlers()
         public void HandlerIsReturnedForMatchingType()
         {
             var map = new HandlerMap();
-            map.Add(typeof(SimpleMessage), m => Task.FromResult(true) );
+            map.Add(typeof(SimpleMessage), m => Task.FromResult(true));
 
             var handler = map.Get(typeof(SimpleMessage));
 
@@ -72,7 +72,7 @@ public void CorrectHandlerIsReturnedForType()
             var handler1 = map.Get(typeof(SimpleMessage));
 
             handler1.ShouldBe(fn1);
-            
+
             var handler2 = map.Get(typeof(AnotherSimpleMessage));
 
             handler2.ShouldBe(fn2);

JustSaying.UnitTests/AwsTools/MessageHandling/MessageContextAccessorTests.cs

@@ -49,7 +49,7 @@ public async Task DifferentThreadsHaveDifferentContexts()
             var data2 = MakeUniqueMessageContext();
 
             var t1 = Task.Run(async () => await ThreadLocalDataRemainsTheSame(data1));
-            var t2 =  Task.Run(async () => await ThreadLocalDataRemainsTheSame(data2));
+            var t2 = Task.Run(async () => await ThreadLocalDataRemainsTheSame(data2));
 
             await Task.WhenAll(t1, t2);
         }

JustSaying.UnitTests/AwsTools/MessageHandling/MessageDispatcherTests/WhenDispatchingMessage.cs

@@ -86,7 +86,7 @@ protected virtual void Given()
             _serializationRegister.DeserializeMessage(Arg.Any<string>()).Returns(_typedMessage);
         }
 
-        private async Task When() =>  await SystemUnderTest.DispatchMessage(_sqsMessage, CancellationToken.None);
+        private async Task When() => await SystemUnderTest.DispatchMessage(_sqsMessage, CancellationToken.None);
 
         private MessageDispatcher CreateSystemUnderTestAsync()
         {
@@ -144,7 +144,7 @@ public void ShouldInvokeMessageBackoffStrategyWithNumberOfReceives()
             [Fact]
             public void ShouldUpdateMessageVisibility()
             {
-                _amazonSqsClient.Received(1).ChangeMessageVisibilityAsync(Arg.Is<ChangeMessageVisibilityRequest>(x => x.QueueUrl == ExpectedQueueUrl && x.ReceiptHandle == _sqsMessage.ReceiptHandle && x.VisibilityTimeout == (int) _expectedBackoffTimeSpan.TotalSeconds));
+                _amazonSqsClient.Received(1).ChangeMessageVisibilityAsync(Arg.Is<ChangeMessageVisibilityRequest>(x => x.QueueUrl == ExpectedQueueUrl && x.ReceiptHandle == _sqsMessage.ReceiptHandle && x.VisibilityTimeout == (int)_expectedBackoffTimeSpan.TotalSeconds));
             }
         }
 

JustSaying.UnitTests/AwsTools/MessageHandling/MessageHandlerWrapperTests.cs

@@ -33,7 +33,7 @@ public async Task ReturnedFunctionIsCallable()
             mockHandler.Handle(Arg.Any<SimpleMessage>()).Returns(true);
 
             // act
-             var wrapped = handlerWrapper.WrapMessageHandler(() => mockHandler);
+            var wrapped = handlerWrapper.WrapMessageHandler(() => mockHandler);
 
             var result = await wrapped(new SimpleMessage());
 

JustSaying.UnitTests/AwsTools/MessageHandling/Sns/TopicByName/WhenPublishingTestBase.cs

@@ -35,6 +35,6 @@ public virtual Task DisposeAsync()
         protected abstract void Given();
         protected abstract Task<SnsTopicByName> CreateSystemUnderTestAsync();
 
-       protected abstract Task WhenAsync();
+        protected abstract Task WhenAsync();
     }
 }

JustSaying.UnitTests/AwsTools/MessageHandling/Sqs/WhenPublishing.cs

@@ -29,7 +29,7 @@ protected override async Task<SqsPublisher> CreateSystemUnderTestAsync()
         protected override void Given()
         {
             Sqs.GetQueueUrlAsync(Arg.Any<string>())
-                .Returns(new GetQueueUrlResponse {QueueUrl = Url});
+                .Returns(new GetQueueUrlResponse { QueueUrl = Url });
 
             Sqs.GetQueueAttributesAsync(Arg.Any<GetQueueAttributesRequest>())
                 .Returns(new GetQueueAttributesResponse());

JustSaying.UnitTests/AwsTools/MessageHandling/Sqs/WhenPublishingAsync.cs

@@ -28,11 +28,11 @@ protected override async Task<SqsPublisher> CreateSystemUnderTestAsync()
 
         protected override void Given()
         {
-           Sqs.GetQueueUrlAsync(Arg.Any<string>())
-                .Returns(new GetQueueUrlResponse { QueueUrl = Url });
+            Sqs.GetQueueUrlAsync(Arg.Any<string>())
+                 .Returns(new GetQueueUrlResponse { QueueUrl = Url });
 
-           Sqs.GetQueueAttributesAsync(Arg.Any<GetQueueAttributesRequest>())
-                .Returns(new GetQueueAttributesResponse());
+            Sqs.GetQueueAttributesAsync(Arg.Any<GetQueueAttributesRequest>())
+                 .Returns(new GetQueueAttributesResponse());
 
             _serializationRegister.Serialize(_message, false)
                 .Returns("serialized_contents");

JustSaying.UnitTests/AwsTools/MessageHandling/Sqs/WhenPublishingTestBase.cs

@@ -34,6 +34,6 @@ public virtual Task DisposeAsync()
         protected abstract void Given();
         protected abstract Task<SqsPublisher> CreateSystemUnderTestAsync();
 
-       protected abstract Task WhenAsync();
+        protected abstract Task WhenAsync();
     }
 }

JustSaying.UnitTests/AwsTools/MessageHandling/SqsNotificationListener/Support/SignallingHandler.cs

@@ -21,7 +21,7 @@ public async Task<bool> Handle(T message)
             {
                 return await _inner.Handle(message);
             }
-            finally 
+            finally
             {
                 TaskHelpers.DelaySendDone(_doneSignal);
             }

JustSaying.UnitTests/AwsTools/MessageHandling/SqsNotificationListener/WhenAttemptingToInterrogateASubscriber.cs

@@ -11,7 +11,7 @@ public class WhenAttemptingToInterrogateASubscriber : BaseQueuePollingTest
         public void SubscribedMessagesAreAddedToTheInterrogationDetails()
         {
             SystemUnderTest.Subscribers.Count.ShouldBe(1);
-            SystemUnderTest.Subscribers.First(x => x.MessageType == typeof (SimpleMessage)).ShouldNotBe(null);
+            SystemUnderTest.Subscribers.First(x => x.MessageType == typeof(SimpleMessage)).ShouldNotBe(null);
         }
     }
 }

JustSaying.UnitTests/AwsTools/MessageHandling/SqsNotificationListener/WhenExactlyOnceIsAppliedToHandler.cs

@@ -22,9 +22,9 @@ protected override void Given()
             _expectedTimeout = 5;
 
             var messageLockResponse = new MessageLockResponse
-                {
-                    DoIHaveExclusiveLock = true
-                };
+            {
+                DoIHaveExclusiveLock = true
+            };
 
             MessageLock = Substitute.For<IMessageLockAsync>();
             MessageLock.TryAquireLockAsync(Arg.Any<string>(), Arg.Any<TimeSpan>())

JustSaying.UnitTests/AwsTools/MessageHandling/SqsNotificationListener/WhenThereAreExceptionsInSqsCalling.cs

@@ -33,7 +33,7 @@ protected override void Given()
             Sqs.ReceiveMessageAsync(
                     Arg.Any<ReceiveMessageRequest>(),
                     Arg.Any<CancellationToken>())
-                .Returns(_ =>  ExceptionOnFirstCall());
+                .Returns(_ => ExceptionOnFirstCall());
         }
 
         private Task ExceptionOnFirstCall()

JustSaying.UnitTests/AwsTools/MessageHandling/SqsNotificationListener/WhenThereAreNoMessagesToProcess.cs

@@ -59,7 +59,7 @@ private void Given()
                     Arg.Any<CancellationToken>())
                 .Returns(x => Task.FromResult(GenerateEmptyMessage()));
 
-            _sqs.When(x =>  x.ReceiveMessageAsync(
+            _sqs.When(x => x.ReceiveMessageAsync(
                     Arg.Any<ReceiveMessageRequest>(),
                     Arg.Any<CancellationToken>()))
                 .Do(x => _callCount++);

JustSaying.UnitTests/AwsTools/QueueCreation/WhenSerializingRedrivePolicy.cs

@@ -4,7 +4,7 @@
 
 namespace JustSaying.UnitTests.AwsTools.QueueCreation
 {
-    public class WhenSerializingRedrivePolicy 
+    public class WhenSerializingRedrivePolicy
     {
         [Fact]
         public void CanDeserializeIntoRedrivePolicy()

JustSaying.UnitTests/JustSayingBus/WhenPublishingMessageWithoutMonitor.cs

@@ -11,7 +11,7 @@ namespace JustSaying.UnitTests.JustSayingBus
     public class WhenPublishingMessageWithoutMonitor : GivenAServiceBusWithoutMonitoring
     {
         private readonly IMessagePublisher _publisher = Substitute.For<IMessagePublisher>();
-        
+
         protected override async Task WhenAsync()
         {
             SystemUnderTest.AddMessagePublisher<SimpleMessage>(_publisher, string.Empty);

JustSaying.UnitTests/JustSayingBus/WhenRegisteringMessageHandlers.cs

@@ -16,7 +16,7 @@ public class WhenRegisteringMessageHandlers : GivenAServiceBus
         private string _region;
         private Func<IHandlerAsync<Message>> _futureHandler1;
         private Func<IHandlerAsync<Message2>> _futureHandler2;
-        
+
         protected override void Given()
         {
             base.Given();

JustSaying.UnitTests/JustSayingBus/WhenRegisteringSubscribers.cs

@@ -27,7 +27,7 @@ protected override void Given()
             });
             _subscriber2 = Substitute.For<INotificationSubscriber>();
             _subscriber2.Queue.Returns("queue2");
-            _subscriber2.Subscribers.Returns(new Collection<ISubscriber> {new Subscriber(typeof (SimpleMessage))});
+            _subscriber2.Subscribers.Returns(new Collection<ISubscriber> { new Subscriber(typeof(SimpleMessage)) });
         }
 
         protected override Task WhenAsync()

JustSaying.UnitTests/JustSayingBus/WhenSubscribingAndNotPassingATopic.cs

@@ -22,7 +22,7 @@ protected override Task WhenAsync()
         [Fact]
         public void ArgExceptionThrown()
         {
-            ((ArgumentException) ThrownException).ParamName.ShouldBe("region");
+            ((ArgumentException)ThrownException).ParamName.ShouldBe("region");
         }
     }
 }

JustSaying.UnitTests/JustSayingBus/WhenUsingMultipleRegions.cs

@@ -12,7 +12,7 @@ public class WhenUsingMultipleRegions : GivenAServiceBus
         protected override void Given()
         {
             base.Given();
-            Config.Regions.Returns(new List<string>{"region1", "region2"});
+            Config.Regions.Returns(new List<string> { "region1", "region2" });
         }
 
         protected override Task WhenAsync()

JustSaying.UnitTests/Messaging/MessageHandling/WhenEnsuringMessageIsOnlyHandledExactlyOnce.cs

@@ -14,7 +14,7 @@ public class WhenEnsuringMessageIsOnlyHandledExactlyOnce
         public async Task WhenMessageIsLockedByAnotherHandler_MessageWillBeLeftInTheQueue()
         {
             var messageLock = Substitute.For<IMessageLockAsync>();
-            messageLock.TryAquireLockAsync(Arg.Any<string>(), Arg.Any<TimeSpan>()).Returns(new MessageLockResponse {DoIHaveExclusiveLock = false});
+            messageLock.TryAquireLockAsync(Arg.Any<string>(), Arg.Any<TimeSpan>()).Returns(new MessageLockResponse { DoIHaveExclusiveLock = false });
             var sut = new ExactlyOnceHandler<OrderAccepted>(Substitute.For<IHandlerAsync<OrderAccepted>>(), messageLock, 1, "handlerName");
 
             var result = await sut.Handle(new OrderAccepted());

JustSaying.UnitTests/Messaging/Monitoring/StopwatchHandlerTests.cs

@@ -21,7 +21,7 @@ public async Task WhenHandlerIsWrappedinStopWatch_InnerHandlerIsCalled()
 
             var result = await stopWatchHandler.Handle(new OrderAccepted());
             result.ShouldBeTrue();
-            
+
             await handler.Received(1).Handle(Arg.Any<OrderAccepted>());
         }
 
@@ -35,7 +35,7 @@ public async Task WhenHandlerIsWrappedinStopWatch_MonitoringIsCalled()
 
             await stopWatchHandler.Handle(new OrderAccepted());
 
-            monitoring.Received(1). HandlerExecutionTime(
+            monitoring.Received(1).HandlerExecutionTime(
                 Arg.Any<Type>(), Arg.Any<Type>(), Arg.Any<TimeSpan>());
         }
 
@@ -66,7 +66,7 @@ public async Task WhenHandlerIsWrappedinStopWatch_MonitoringIsCalledWithTiming()
             await stopWatchHandler.Handle(new OrderAccepted());
 
             monitoring.Received(1).HandlerExecutionTime(
-                Arg.Any<Type>(), Arg.Any<Type>(), 
+                Arg.Any<Type>(), Arg.Any<Type>(),
                 Arg.Is<TimeSpan>(ts => ts > TimeSpan.Zero));
         }
 

JustSaying.UnitTests/Messaging/Serialization/Newtonsoft/WhenAskingForANewSerializer.cs

@@ -11,7 +11,7 @@ public class WhenAskingForANewSerializer : XBehaviourTest<NewtonsoftSerializatio
 
         protected override void Given()
         {
-            
+
         }
 
         protected override void WhenAction()

JustSaying.UnitTests/Messaging/Serialization/Newtonsoft/WhenSerializingAndDeserializing.cs

@@ -34,7 +34,7 @@ public void MessagesContainSameDetails()
             _messageOut.RaisingComponent.ShouldBe(_messageIn.RaisingComponent);
             _messageOut.TimeStamp.ShouldBe(_messageIn.TimeStamp);
         }
-        
+
         [Fact]
         public void EnumsAreRepresentedAsStrings()
         {

JustSaying.UnitTests/Messaging/Serialization/Newtonsoft/WhenUsingCustomSettings.cs

@@ -42,6 +42,7 @@ public void MessageHasBeenCreated()
         public void EnumsAreNotRepresentedAsStrings()
         {
             _jsonMessage.ShouldContain("EnumVal");
-            _jsonMessage.ShouldNotContain("Two");        }
+            _jsonMessage.ShouldNotContain("Two");
+        }
     }
 }

JustSaying.UnitTests/Messaging/Serialization/SerializationRegister/WhenDeserializingMessage.cs

@@ -26,7 +26,7 @@ protected override void WhenAction()
         {
             var messageSerializer = Substitute.For<IMessageSerializer>();
             messageSerializer.GetMessageSubject(messageBody).Returns(typeof(CustomMessage).Name);
-            messageSerializer.Deserialize(messageBody, typeof (CustomMessage)).Returns(new CustomMessage());
+            messageSerializer.Deserialize(messageBody, typeof(CustomMessage)).Returns(new CustomMessage());
             SystemUnderTest.AddSerializer<CustomMessage>(messageSerializer);
         }
 

JustSaying.UnitTests/Messaging/Serialization/SubjectProviders/GenericMessageSubjectProviderTests.cs

@@ -6,7 +6,7 @@ namespace JustSaying.UnitTests.Messaging.Serialization.SubjectProviders
 {
     public class GenericMessageSubjectProviderTests
     {
-// these classes are never instantiated, but the types are used in tests
+        // these classes are never instantiated, but the types are used in tests
 #pragma warning disable CA1812
         class Foo { }
 

JustSaying.UnitTests/XBehaviourTest.cs

@@ -14,7 +14,7 @@ protected XBehaviourTest()
 
         protected TSystemUnderTest SystemUnderTest { get; private set; }
         protected Exception ThrownException { get; private set; }
- 
+
         protected virtual TSystemUnderTest CreateSystemUnderTest()
         {
             var fixture = new Fixture();
@@ -30,7 +30,7 @@ protected void Execute()
                 SystemUnderTest = CreateSystemUnderTest();
                 WhenAction();
             }
-            catch (Exception ex) when(_recordThrownExceptions)
+            catch (Exception ex) when (_recordThrownExceptions)
             {
                 ThrownException = ex;
             }

JustSaying/AwsTools/DefaultAwsClientFactory.cs

@@ -22,7 +22,7 @@ public DefaultAwsClientFactory(AWSCredentials customCredentials)
 
         public Uri ServiceUri { get; set; }
 
-        public IAmazonSimpleNotificationService GetSnsClient(RegionEndpoint region) 
+        public IAmazonSimpleNotificationService GetSnsClient(RegionEndpoint region)
             => new AmazonSimpleNotificationServiceClient(_credentials, CreateSNSConfig(region));
 
         public IAmazonSQS GetSqsClient(RegionEndpoint region)

JustSaying/AwsTools/JustSayingConstants.cs

@@ -17,7 +17,7 @@ public static class JustSayingConstants
         /// Default visibility timeout for message
         /// </summary>
         public static TimeSpan DefaultVisibilityTimeout => TimeSpan.FromSeconds(30);
-        
+
         /// <summary>
         /// Number of times a handler will retry a message until a message 
         /// is sent to error queue
@@ -49,7 +49,7 @@ public static class JustSayingConstants
         /// Maximum message retention period on a queue.
         /// </summary>
         public static TimeSpan MaximumRetentionPeriod => TimeSpan.FromDays(14);
-        
+
         /// <summary>
         /// Minimum delay in message delivery for SQS. This is also the default.
         /// </summary>