Bump serde from 1.0.133 to 1.0.136 (#260)

* Bump serde from 1.0.133 to 1.0.136

Bumps [serde](https://github.com/serde-rs/serde) from 1.0.133 to 1.0.136.
- [Release notes](https://github.com/serde-rs/serde/releases)
- [Commits](serde-rs/serde@v1.0.133...v1.0.136)

---
updated-dependencies:
- dependency-name: serde
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* fix tests for serde message changes

* update changelog

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: David Pacheco <[email protected]>

Author: dependabot[bot]

CHANGELOG.adoc

@@ -24,6 +24,7 @@ https://github.com/oxidecomputer/dropshot/compare/v0.6.0\...HEAD[Full list of co
 
 * https://github.com/oxidecomputer/dropshot/pull/198[#198] Responses that used `()` (the unit type) as their `Body` type parameter previously (and inaccurately) were represented in OpenAPI as an empty `responseBody`. They are now more accurately represented as a body whose value is `null` (4 bytes). We encourage those use cases to instead use either `HttpResponseUpdatedNoContent` or `HttpResponseDeleted` both of which have empty response bodies. If there are other situations where you would like a response type with no body, please file an issue.
 * https://github.com/oxidecomputer/dropshot/pull/252[#252] Endpoints specified with the `#[endpoint ..]` attribute macro now use the first line of a doc comment as the OpenAPI `summary` and subsequent lines as the `description`. Previously all lines were used as the `description`.
+* https://github.com/oxidecomputer/dropshot/pull/260[#260] Pulls in a newer serde that changes error messages around parsing NonZeroU32.
 
 == 0.6.0 (released 2021-11-18)
 

Cargo.lock

@@ -48,7 +48,7 @@ version = "0.14"
 features = [ "full" ]
 
 [dependencies.serde]
-version = "1.0.133"
+version = "1.0.136"
 features = [ "derive" ]
 
 [dependencies.slog]

dropshot/Cargo.toml

@@ -227,7 +227,8 @@ async fn test_paginate_errors() {
     let test_cases = vec![
         ErrorTestCase {
             path: "/intapi?limit=0".to_string(),
-            message: "unable to parse query string: expected a non-zero value",
+            message: "unable to parse query string: invalid value: \
+                integer `0`, expected a nonzero u32",
         },
         ErrorTestCase {
             path: "/intapi?limit=-3".to_string(),
@@ -427,7 +428,8 @@ async fn test_paginate_empty() {
     assert_error(
         &client,
         "/empty?limit=0",
-        "unable to parse query string: expected a non-zero value",
+        "unable to parse query string: invalid value: integer `0`, \
+        expected a nonzero u32",
     )
     .await;