OpenAPI Flaky fix (#20383)

Don't use network in tests unless we have to

Author: ryanbrandenburg

src/Tools/Microsoft.dotnet-openapi/test/OpenApiAddURLTests.cs

@@ -424,7 +424,7 @@ public async Task OpenAPi_Add_URL_InvalidUrl()
         {
             var project = CreateBasicProject(withOpenApi: false);
 
-            var app = GetApplication(realHttp: true);
+            var app = GetApplication();
             var url = BrokenUrl;
             var run = app.Execute(new[] { "add", "url", url });
 

src/Tools/Microsoft.dotnet-openapi/test/OpenApiTestBase.cs

@@ -106,7 +106,8 @@ private IDictionary<string, Tuple<string, ContentDispositionHeaderValue>> Downlo
                 { PackageUrl, Tuple.Create<string, ContentDispositionHeaderValue>(PackageUrlContent, null) },
                 { NoDispositionUrl, Tuple.Create<string, ContentDispositionHeaderValue>(Content, null) },
                 { NoExtensionUrl, Tuple.Create(Content, noExtension) },
-                { NoSegmentUrl, Tuple.Create(Content, justAttachments) }
+                { NoSegmentUrl, Tuple.Create(Content, justAttachments) },
+                { BrokenUrl, null }
             };
         }
 
@@ -139,10 +140,14 @@ public void Dispose()
         public Task<IHttpResponseMessageWrapper> GetResponseAsync(string url)
         {
             var result = _results[url];
-            byte[] byteArray = Encoding.ASCII.GetBytes(result.Item1);
-            var stream = new MemoryStream(byteArray);
+            MemoryStream stream = null;
+            if(result != null)
+            {
+                byte[] byteArray = Encoding.ASCII.GetBytes(result.Item1);
+                stream = new MemoryStream(byteArray);
+            }
 
-            return Task.FromResult<IHttpResponseMessageWrapper>(new TestHttpResponseMessageWrapper(stream, result.Item2));
+            return Task.FromResult<IHttpResponseMessageWrapper>(new TestHttpResponseMessageWrapper(stream, result?.Item2));
         }
     }
 
@@ -154,7 +159,17 @@ public class TestHttpResponseMessageWrapper : IHttpResponseMessageWrapper
 
         public bool IsSuccessCode()
         {
-            return true;
+            switch(StatusCode)
+            {
+                case HttpStatusCode.OK:
+                case HttpStatusCode.Created:
+                case HttpStatusCode.NoContent:
+                case HttpStatusCode.Accepted:
+                    return true;
+                case HttpStatusCode.NotFound:
+                default:
+                    return false;
+            }
         }
 
         private readonly ContentDispositionHeaderValue _contentDisposition;
@@ -164,6 +179,10 @@ public TestHttpResponseMessageWrapper(
             ContentDispositionHeaderValue header)
         {
             Stream = Task.FromResult<Stream>(stream);
+            if (header is null && stream is null)
+            {
+                StatusCode = HttpStatusCode.NotFound;
+            }
             _contentDisposition = header;
         }