Skip to content

Commit 2cfb3c9

Browse files
committed
test(stackable-versioned): Simplify gender defaulting
1 parent b73d475 commit 2cfb3c9

File tree

1 file changed

+3
-9
lines changed

1 file changed

+3
-9
lines changed

crates/stackable-versioned/tests/person.rs

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -63,10 +63,7 @@ pub mod versioned {
6363

6464
// We started out with a enum. As we *need* to provide a default, we have a Unknown variant.
6565
// Afterwards we figured let's be more flexible and accept any arbitrary String.
66-
#[versioned(
67-
added(since = "v2", default = "default_gender"),
68-
changed(since = "v3", from_type = "Gender")
69-
)]
66+
#[versioned(added(since = "v2"), changed(since = "v3", from_type = "Gender"))]
7067
gender: String,
7168

7269
#[versioned(nested)]
@@ -93,18 +90,15 @@ impl Default for PersonStatus {
9390
}
9491
}
9592

96-
#[derive(Clone, Debug, Deserialize, Serialize, JsonSchema)]
93+
#[derive(Clone, Debug, Default, Deserialize, Serialize, JsonSchema)]
9794
#[serde(rename_all = "PascalCase")]
9895
pub enum Gender {
96+
#[default]
9997
Unknown,
10098
Male,
10199
Female,
102100
}
103101

104-
fn default_gender() -> Gender {
105-
Gender::Unknown
106-
}
107-
108102
impl From<Gender> for String {
109103
fn from(value: Gender) -> Self {
110104
match value {

0 commit comments

Comments
 (0)