dont write default version

2024-03-17 20:56:27 +11:00 · 2024-03-17 20:56:27 +11:00 · 090a89264a
--- a/readme.md
+++ b/readme.md
@ -55,7 +55,6 @@ public async Task HttpResponse()
 <a id='snippet-Tests.HttpResponse.verified.txt'></a>
 ```txt
 {
-  Version: 1.1,
  Status: 200 OK,
  Headers: {
    Accept-Ranges: bytes,
@ -104,7 +103,7 @@ SOFTWARE.
  Request: https://raw.githubusercontent.com/VerifyTests/Verify/main/license.txt
 }
 ```
-<sup><a href='/src/Tests/Tests.HttpResponse.verified.txt#L1-L49' title='Snippet source file'>snippet source</a> | <a href='#snippet-Tests.HttpResponse.verified.txt' title='Start of snippet'>anchor</a></sup>
+<sup><a href='/src/Tests/Tests.HttpResponse.verified.txt#L1-L48' title='Snippet source file'>snippet source</a> | <a href='#snippet-Tests.HttpResponse.verified.txt' title='Start of snippet'>anchor</a></sup>
 <!-- endSnippet -->


@ -418,7 +417,7 @@ The requests/response pairs will be appended to the verified file.
          Connection: keep-alive,
          Content-Security-Policy: default-src 'none'; style-src 'unsafe-inline'; sandbox,
          Cross-Origin-Resource-Policy: cross-origin,
-          ETag: "529be0ead7b278443390ecd828509337d4a46effa2699c3d5aac62bfb4cd266c",
+          ETag: "f0a2c3f56177936f3b5b4c75fa1b1d54b3a553996faede4ede667dc668e3668e",
          Strict-Transport-Security: max-age=31536000,
          Vary: Authorization,Accept-Encoding,Origin,
          Via: 1.1 varnish,
@ -433,7 +432,7 @@ The requests/response pairs will be appended to the verified file.
        ContentString:
 {
  "sdk": {
-    "version": "8.0.201",
+    "version": "8.0.202",
    "allowPrerelease": true,
    "rollForward": "latestFeature"
  }
@ -694,7 +693,6 @@ public async Task ExplicitContent()
 <a id='snippet-MockHttpClientTests.ExplicitContent.verified.txt'></a>
 ```txt
 {
-  Version: 1.1,
  Status: 200 OK,
  Content: {
    Headers: {
@ -706,7 +704,7 @@ public async Task ExplicitContent()
  }
 }
 ```
-<sup><a href='/src/Tests/MockHttpClientTests.ExplicitContent.verified.txt#L1-L12' title='Snippet source file'>snippet source</a> | <a href='#snippet-MockHttpClientTests.ExplicitContent.verified.txt' title='Start of snippet'>anchor</a></sup>
+<sup><a href='/src/Tests/MockHttpClientTests.ExplicitContent.verified.txt#L1-L11' title='Snippet source file'>snippet source</a> | <a href='#snippet-MockHttpClientTests.ExplicitContent.verified.txt' title='Start of snippet'>anchor</a></sup>
 <!-- endSnippet -->


@ -776,7 +774,6 @@ public async Task ExplicitResponse()
 <a id='snippet-MockHttpClientTests.ExplicitResponse.verified.txt'></a>
 ```txt
 {
-  Version: 1.1,
  Status: 200 OK,
  Content: {
    Headers: {
@ -786,7 +783,7 @@ public async Task ExplicitResponse()
  }
 }
 ```
-<sup><a href='/src/Tests/MockHttpClientTests.ExplicitResponse.verified.txt#L1-L10' title='Snippet source file'>snippet source</a> | <a href='#snippet-MockHttpClientTests.ExplicitResponse.verified.txt' title='Start of snippet'>anchor</a></sup>
+<sup><a href='/src/Tests/MockHttpClientTests.ExplicitResponse.verified.txt#L1-L9' title='Snippet source file'>snippet source</a> | <a href='#snippet-MockHttpClientTests.ExplicitResponse.verified.txt' title='Start of snippet'>anchor</a></sup>
 <!-- endSnippet -->


@ -832,7 +829,6 @@ public async Task ResponseBuilder()
 ```txt
 {
  result1: {
-    Version: 1.1,
    Status: 200 OK,
    Content: {
      Headers: {
@ -842,7 +838,6 @@ public async Task ResponseBuilder()
    }
  },
  result2: {
-    Version: 1.1,
    Status: 200 OK,
    Content: {
      Headers: {
@ -853,7 +848,7 @@ public async Task ResponseBuilder()
  }
 }
 ```
-<sup><a href='/src/Tests/MockHttpClientTests.ResponseBuilder.verified.txt#L1-L22' title='Snippet source file'>snippet source</a> | <a href='#snippet-MockHttpClientTests.ResponseBuilder.verified.txt' title='Start of snippet'>anchor</a></sup>
+<sup><a href='/src/Tests/MockHttpClientTests.ResponseBuilder.verified.txt#L1-L20' title='Snippet source file'>snippet source</a> | <a href='#snippet-MockHttpClientTests.ResponseBuilder.verified.txt' title='Start of snippet'>anchor</a></sup>
 <!-- endSnippet -->


@ -898,7 +893,6 @@ public async Task EnumerableResponses()
 ```txt
 {
  result1: {
-    Version: 1.1,
    Status: 200 OK,
    Content: {
      Headers: {
@ -908,7 +902,6 @@ public async Task EnumerableResponses()
    }
  },
  result2: {
-    Version: 1.1,
    Status: 200 OK,
    Content: {
      Headers: {
@ -919,7 +912,7 @@ public async Task EnumerableResponses()
  }
 }
 ```
-<sup><a href='/src/Tests/MockHttpClientTests.EnumerableResponses.verified.txt#L1-L22' title='Snippet source file'>snippet source</a> | <a href='#snippet-MockHttpClientTests.EnumerableResponses.verified.txt' title='Start of snippet'>anchor</a></sup>
+<sup><a href='/src/Tests/MockHttpClientTests.EnumerableResponses.verified.txt#L1-L20' title='Snippet source file'>snippet source</a> | <a href='#snippet-MockHttpClientTests.EnumerableResponses.verified.txt' title='Start of snippet'>anchor</a></sup>
 <!-- endSnippet -->


--- a/src/Tests/MockHttpClientTests.EnumerableResponses.verified.txt
+++ b/src/Tests/MockHttpClientTests.EnumerableResponses.verified.txt
@ -1,6 +1,5 @@
 {
  result1: {
-    Version: 1.1,
    Status: 200 OK,
    Content: {
      Headers: {
@ -10,7 +9,6 @@
    }
  },
  result2: {
-    Version: 1.1,
    Status: 200 OK,
    Content: {
      Headers: {
--- a/src/Tests/MockHttpClientTests.ExplicitContent.verified.txt
+++ b/src/Tests/MockHttpClientTests.ExplicitContent.verified.txt
@ -1,5 +1,4 @@
 {
-  Version: 1.1,
  Status: 200 OK,
  Content: {
    Headers: {
--- a/src/Tests/MockHttpClientTests.ExplicitResponse.verified.txt
+++ b/src/Tests/MockHttpClientTests.ExplicitResponse.verified.txt
@ -1,5 +1,4 @@
 {
-  Version: 1.1,
  Status: 200 OK,
  Content: {
    Headers: {
--- a/src/Tests/MockHttpClientTests.GetJsonContent_mediaType=application-foo+json.DotNet8_0.verified.txt
+++ b/src/Tests/MockHttpClientTests.GetJsonContent_mediaType=application-foo+json.DotNet8_0.verified.txt
@ -1,5 +1,4 @@
 {
-  Version: 1.1,
  Status: 200 OK,
  Content: {
    Headers: {
--- a/src/Tests/MockHttpClientTests.GetJsonContent_mediaType=application-json.DotNet8_0.verified.txt
+++ b/src/Tests/MockHttpClientTests.GetJsonContent_mediaType=application-json.DotNet8_0.verified.txt
@ -1,5 +1,4 @@
 {
-  Version: 1.1,
  Status: 200 OK,
  Content: {
    Headers: {
--- a/src/Tests/MockHttpClientTests.PostFormContent.DotNet8_0.verified.txt
+++ b/src/Tests/MockHttpClientTests.PostFormContent.DotNet8_0.verified.txt
@ -1,6 +1,5 @@
 {
  result: {
-    Version: 1.1,
    Status: 200 OK,
    Content: {
      Headers: {}
--- a/src/Tests/MockHttpClientTests.PostJsonContent_mediaType=application-foo+json.DotNet8_0.verified.txt
+++ b/src/Tests/MockHttpClientTests.PostJsonContent_mediaType=application-foo+json.DotNet8_0.verified.txt
@ -1,6 +1,5 @@
 {
  result: {
-    Version: 1.1,
    Status: 200 OK,
    Content: {
      Headers: {}
--- a/src/Tests/MockHttpClientTests.PostJsonContent_mediaType=application-json.DotNet8_0.verified.txt
+++ b/src/Tests/MockHttpClientTests.PostJsonContent_mediaType=application-json.DotNet8_0.verified.txt
@ -1,6 +1,5 @@
 {
  result: {
-    Version: 1.1,
    Status: 200 OK,
    Content: {
      Headers: {}
--- a/src/Tests/MockHttpClientTests.PostJsonStringContent_mediaType=application-foo+json.DotNet8_0.verified.txt
+++ b/src/Tests/MockHttpClientTests.PostJsonStringContent_mediaType=application-foo+json.DotNet8_0.verified.txt
@ -1,6 +1,5 @@
 {
  result: {
-    Version: 1.1,
    Status: 200 OK,
    Content: {
      Headers: {}
--- a/src/Tests/MockHttpClientTests.PostJsonStringContent_mediaType=application-json.DotNet8_0.verified.txt
+++ b/src/Tests/MockHttpClientTests.PostJsonStringContent_mediaType=application-json.DotNet8_0.verified.txt
@ -1,6 +1,5 @@
 {
  result: {
-    Version: 1.1,
    Status: 200 OK,
    Content: {
      Headers: {}
--- a/src/Tests/MockHttpClientTests.PostStreamContent.DotNet8_0.verified.txt
+++ b/src/Tests/MockHttpClientTests.PostStreamContent.DotNet8_0.verified.txt
@ -1,6 +1,5 @@
 {
  result: {
-    Version: 1.1,
    Status: 200 OK,
    Content: {
      Headers: {}
--- a/src/Tests/MockHttpClientTests.PostStreamContentWithContentType.DotNet8_0.verified.txt
+++ b/src/Tests/MockHttpClientTests.PostStreamContentWithContentType.DotNet8_0.verified.txt
@ -1,6 +1,5 @@
 {
  result: {
-    Version: 1.1,
    Status: 200 OK,
    Content: {
      Headers: {}
--- a/src/Tests/MockHttpClientTests.PostStringContent.DotNet8_0.verified.txt
+++ b/src/Tests/MockHttpClientTests.PostStringContent.DotNet8_0.verified.txt
@ -1,6 +1,5 @@
 {
  result: {
-    Version: 1.1,
    Status: 200 OK,
    Content: {
      Headers: {}
--- a/src/Tests/MockHttpClientTests.ResponseBuilder.verified.txt
+++ b/src/Tests/MockHttpClientTests.ResponseBuilder.verified.txt
@ -1,6 +1,5 @@
 {
  result1: {
-    Version: 1.1,
    Status: 200 OK,
    Content: {
      Headers: {
@ -10,7 +9,6 @@
    }
  },
  result2: {
-    Version: 1.1,
    Status: 200 OK,
    Content: {
      Headers: {
--- a/src/Tests/Tests.HttpResponse.verified.txt
+++ b/src/Tests/Tests.HttpResponse.verified.txt
@ -1,5 +1,4 @@
 {
-  Version: 1.1,
  Status: 200 OK,
  Headers: {
    Accept-Ranges: bytes,
--- a/src/Tests/Tests.HttpResponseNested.verified.txt
+++ b/src/Tests/Tests.HttpResponseNested.verified.txt
@ -1,6 +1,5 @@
 {
  result: {
-    Version: 1.1,
    Status: 200 OK,
    Headers: {
      Accept-Ranges: bytes,
--- a/src/Tests/Tests.IgnoreHeader.verified.txt
+++ b/src/Tests/Tests.IgnoreHeader.verified.txt
@ -1,5 +1,4 @@
 {
-  Version: 1.1,
  Status: 200 OK,
  Headers: {
    Accept-Ranges: bytes,
--- a/src/Tests/Tests.JsonGet.verified.txt
+++ b/src/Tests/Tests.JsonGet.verified.txt
@ -1,7 +1,7 @@
 {
  target: {
    sdk: {
-      version: 8.0.201,
+      version: 8.0.202,
      allowPrerelease: true,
      rollForward: latestFeature
    }
@ -48,7 +48,7 @@
          Connection: keep-alive,
          Content-Security-Policy: default-src 'none'; style-src 'unsafe-inline'; sandbox,
          Cross-Origin-Resource-Policy: cross-origin,
-          ETag: "529be0ead7b278443390ecd828509337d4a46effa2699c3d5aac62bfb4cd266c",
+          ETag: "f0a2c3f56177936f3b5b4c75fa1b1d54b3a553996faede4ede667dc668e3668e",
          Strict-Transport-Security: max-age=31536000,
          Vary: Authorization,Accept-Encoding,Origin,
          Via: 1.1 varnish,
@ -63,7 +63,7 @@
        ContentString:
 {
  "sdk": {
-    "version": "8.0.201",
+    "version": "8.0.202",
    "allowPrerelease": true,
    "rollForward": "latestFeature"
  }
--- a/src/Tests/Tests.TestHttpRecording.verified.txt
+++ b/src/Tests/Tests.TestHttpRecording.verified.txt
@ -44,7 +44,7 @@
          Connection: keep-alive,
          Content-Security-Policy: default-src 'none'; style-src 'unsafe-inline'; sandbox,
          Cross-Origin-Resource-Policy: cross-origin,
-          ETag: "529be0ead7b278443390ecd828509337d4a46effa2699c3d5aac62bfb4cd266c",
+          ETag: "f0a2c3f56177936f3b5b4c75fa1b1d54b3a553996faede4ede667dc668e3668e",
          Strict-Transport-Security: max-age=31536000,
          Vary: Authorization,Accept-Encoding,Origin,
          Via: 1.1 varnish,
@ -59,7 +59,7 @@
        ContentString:
 {
  "sdk": {
-    "version": "8.0.201",
+    "version": "8.0.202",
    "allowPrerelease": true,
    "rollForward": "latestFeature"
  }
--- a/src/Verify.Http/Converters/HttpResponseMessageConverter.cs
+++ b/src/Verify.Http/Converters/HttpResponseMessageConverter.cs
@ -5,7 +5,11 @@
    {
        writer.WriteStartObject();

-        writer.WriteMember(response, response.Version, "Version");
+        if (!response.IsDefaultVersion())
+        {
+            writer.WriteMember(response, response.Version, "Version");
+        }
+
        writer.WriteMember(response, response.StatusText(), "Status");

        WriteHeaders(writer, response);
--- a/src/Verify.Http/HttpExtensions.cs
+++ b/src/Verify.Http/HttpExtensions.cs
@ -19,6 +19,9 @@ static class HttpExtensions
    public static bool IsDefaultVersion(this HttpRequestMessage request) =>
        request.Version == defaultRequestVersion;

+    public static bool IsDefaultVersion(this HttpResponseMessage request) =>
+        request.Version == defaultRequestVersion;
+
    public static bool IsDefaultVersionPolicy(this HttpRequest request) =>
        request.VersionPolicy == defaultRequestVersionPolicy;