Improve assertions

Author: andreasohlund

src/ServiceControl.AcceptanceTests/Recoverability/MessageFailures/When_ingesting_failed_message_with_missing_headers.cs

@@ -45,6 +45,8 @@ public async Task Should_be_ingested_when_minimal_required_headers_is_present()
         {
             FailedMessageView failure = null;
 
+            var testStartTime = DateTime.UtcNow;
+
             await Define<MyContext>(c =>
                 {
                     c.AddMinimalRequiredHeaders();
@@ -60,15 +62,19 @@ await Define<MyContext>(c =>
 
             Assert.That(failure, Is.Not.Null);
             Assert.That(failure.TimeSent, Is.Null);
+
+            //No failure time will result in utc now being used
+            Assert.That(failure.TimeOfFailure, Is.GreaterThan(testStartTime));
+
+            // Both host and endpoint name is currently needed so this will be null since no host can be detected from the failed q header5
+            Assert.That(failure.ReceivingEndpoint, Is.Null);
         }
 
         [Test]
         public async Task Should_include_headers_required_by_ServicePulse()
         {
             FailedMessageView failure = null;
 
-            var testStartTime = DateTime.UtcNow;
-
             var context = await Define<MyContext>(c =>
                 {
                     c.AddMinimalRequiredHeaders();
@@ -91,9 +97,6 @@ public async Task Should_include_headers_required_by_ServicePulse()
 
             Assert.That(failure, Is.Not.Null);
 
-            //No failure time will result in utc now being used
-            Assert.That(failure.TimeOfFailure, Is.GreaterThan(testStartTime));
-
             // ServicePulse assumes that the receiving endpoint name is present
             Assert.That(failure.ReceivingEndpoint, Is.Not.Null);
             Assert.That(failure.ReceivingEndpoint.Name, Is.EqualTo(context.EndpointNameOfReceivingEndpoint));