Fix Integration Tests in Builds (#224)

* Tweak condition in outputdatareceived

* Wait for new message

* Go to 4.0.4 for Storage package

* redo 5.0

* Add 60 second wait, add logging

* Always start Azurite

* Remove unnecessary logging

* Look for api url instead of functions string

Author: chlafreniere

test/Integration/IntegrationTestBase.cs

@@ -60,8 +60,8 @@ public class IntegrationTestBase : IDisposable
         public IntegrationTestBase(ITestOutputHelper output)
         {
             this.TestOutput = output;
-
             this.SetupDatabase();
+            this.StartAzurite();
         }
 
         /// <summary>
@@ -193,13 +193,13 @@ protected void StartFunctionHost(string functionName, bool useTestFolder = false
             {
                 // This string is printed after the function host is started up - use this to ensure that we wait long enough
                 // since sometimes the host can take a little while to fully start up
-                if (e.Data.Contains("Job host started"))
+                if (e != null && !string.IsNullOrEmpty(e.Data) && e.Data.Contains($"http://localhost:{this.Port}/api"))
                 {
                     taskCompletionSource.SetResult(true);
                 }
             };
             this.TestOutput.WriteLine($"Waiting for Azure Function host to start...");
-            taskCompletionSource.Task.Wait();
+            taskCompletionSource.Task.Wait(60000);
             this.TestOutput.WriteLine($"Azure Function host started!");
         }
 

test/Integration/SqlOutputBindingIntegrationTests.cs

@@ -102,8 +102,6 @@ public void QueueTriggerProductsTest()
         [Fact]
         public void TimerTriggerProductsTest()
         {
-            // Timer trigger function requires local storage to be running
-            this.StartAzurite();
             this.StartFunctionHost(nameof(TimerTriggerProducts));
 
             // Since this function runs on a schedule (every 5 seconds), we don't need to invoke it.