support serde skip_serializing

[samchouse]

No description provided.

[juhaku]

This is good, but there is a test missing, and docs as well.

Docs should be updated here: https://github.com/juhaku/utoipa/blob/master/utoipa-gen/src/lib.rs#L179

And here:
https://github.com/juhaku/utoipa/blob/master/utoipa-gen/src/lib.rs#L1569

I suppose this skip_serializing should work in both IntoParams and in ToSchema

[samchouse]

I couldn't find where the tests are for the original skip and I couldn't find anything for the original skip related to IntoParams. Can you point them out for me please?

[juhaku]

Sure here is the example test for skip for ToSchema

utoipa/utoipa-gen/tests/schema_derive_test.rs

Lines 2645 to 2682 in 13a3aae

	fn derive_parse_serde_field_attributes() {
	struct S;
	let post = api_doc! {
	#[derive(Serialize)]
	#[serde(rename_all = "camelCase")]
	struct Post<S> {
	#[serde(rename = "uuid")]
	id: String,
	#[serde(skip)]
	_p: PhantomData<S>,
	long_field_num: i64,
	}
	};
	assert_value! {post=>
	"properties.uuid.type" = r#""string""#, "Post id type"
	"properties.longFieldNum.type" = r#""integer""#, "Post long_field_num type"
	"properties.longFieldNum.format" = r#""int64""#, "Post logn_field_num format"
	}
	}
	#[test]
	fn derive_parse_serde_simple_enum_attributes() {
	let value = api_doc! {
	#[derive(Serialize)]
	#[serde(rename_all = "camelCase")]
	enum Value {
	A,
	B,
	#[serde(skip)]
	C,
	}
	};
	assert_value! {value=>
	"enum" = r#"["a","b"]"#, "Value enum variants"
	}
	}

Seems like the skip support is not implemented for IntoParams. I remembered wrong. So we can ignore it now. That can be another PR because it needs more changes for IntoParams than just updating docs and adding tests.

So just creating a test for for skip_serializing attribute is enough in tests/schema_derive_test.rs

[juhaku]

Also instead of using assert_value! macro you can use assert_json_eq! as well.

[juhaku]

It's good, since you added the skip_serializing to the same test, we can remove the assert_value!