VienDesu/schemars
3
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Remove irrelevant comments

Browse source 
I'm reasonably satisfied that the current behaviour of enum variants with `with`/`schema_with` attributes is correct
This commit is contained in:
Graham Esau 2024-08-04 17:26:08 +01:00
parent ef9c8dc56b
commit 71b45a8ba3
3 changed files with 0 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

4
schemars/tests/enum.rs
View file

@ -31,7 +31,6 @@ enum External {
},
UnitTwo,
Tuple(i32, bool),
// FIXME this should probably only replace the "payload" of the enum
#[schemars(with = "i32")]
WithInt,
}
@ -54,7 +53,6 @@ enum Internal {
bar: bool,
},
UnitTwo,
// FIXME this should probably only replace the "payload" of the enum
#[schemars(with = "i32")]
WithInt,
}
@ -77,7 +75,6 @@ enum Untagged {
bar: bool,
},
Tuple(i32, bool),
// FIXME this should probably only replace the "payload" of the enum
#[schemars(with = "i32")]
WithInt,
}
@ -101,7 +98,6 @@ enum Adjacent {
},
Tuple(i32, bool),
UnitTwo,
// FIXME this should probably only replace the "payload" of the enum
#[schemars(with = "i32")]
WithInt,
}

4
schemars/tests/enum_deny_unknown_fields.rs
View file

@ -33,7 +33,6 @@ enum External {
},
UnitTwo,
Tuple(i32, bool),
// FIXME this should probably only replace the "payload" of the enum
#[schemars(with = "i32")]
WithInt,
}
@ -57,7 +56,6 @@ enum Internal {
bar: bool,
},
UnitTwo,
// FIXME this should only replace the "payload" of the enum (which doesn't even make sense for unit enums!)
#[schemars(with = "i32")]
WithInt,
}
@ -81,7 +79,6 @@ enum Untagged {
bar: bool,
},
Tuple(i32, bool),
// FIXME this should probably only replace the "payload" of the enum
#[schemars(with = "i32")]
WithInt,
}
@ -106,7 +103,6 @@ enum Adjacent {
},
Tuple(i32, bool),
UnitTwo,
// FIXME this should probably only replace the "payload" of the enum
#[schemars(with = "i32")]
WithInt,
}

4
schemars/tests/schema_with_enum.rs
View file

@ -21,7 +21,6 @@ pub enum External {
#[schemars(schema_with = "schema_fn")] DoesntImplementJsonSchema,
i32,
),
// FIXME this should probably only replace the "payload" of the enum
#[schemars(schema_with = "schema_fn")]
Unit,
}
@ -39,7 +38,6 @@ pub enum Internal {
foo: DoesntImplementJsonSchema,
},
NewType(#[schemars(schema_with = "schema_fn")] DoesntImplementJsonSchema),
// FIXME this should probably only replace the "payload" of the enum
#[schemars(schema_with = "schema_fn")]
Unit,
}
@ -61,7 +59,6 @@ pub enum Untagged {
#[schemars(schema_with = "schema_fn")] DoesntImplementJsonSchema,
i32,
),
// FIXME this should probably only replace the "payload" of the enum
#[schemars(schema_with = "schema_fn")]
Unit,
}
@ -83,7 +80,6 @@ pub enum Adjacent {
#[schemars(schema_with = "schema_fn")] DoesntImplementJsonSchema,
i32,
),
// FIXME this should probably only replace the "payload" of the enum
#[schemars(schema_with = "schema_fn")]
Unit,
}