Skip to content

Commit e803620

Browse files
committed
fix(schema)!: Make Event non-exhaustive
1 parent 3f6eb77 commit e803620

4 files changed

Lines changed: 13 additions & 0 deletions

File tree

crates/libtest-json/src/event.rs

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
44
#[cfg_attr(feature = "serde", serde(rename_all = "snake_case"))]
55
#[cfg_attr(feature = "serde", serde(tag = "event"))]
6+
#[non_exhaustive]
67
pub enum Event {
78
DiscoverStart(DiscoverStart),
89
DiscoverCase(DiscoverCase),
@@ -82,6 +83,7 @@ impl From<RunComplete> for Event {
8283
#[cfg_attr(feature = "unstable-schema", derive(schemars::JsonSchema))]
8384
#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
8485
#[cfg_attr(feature = "serde", serde(rename_all = "snake_case"))]
86+
#[non_exhaustive]
8587
pub struct DiscoverStart {
8688
#[cfg_attr(
8789
feature = "serde",
@@ -122,6 +124,7 @@ impl DiscoverStart {
122124
#[cfg_attr(feature = "unstable-schema", derive(schemars::JsonSchema))]
123125
#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
124126
#[cfg_attr(feature = "serde", serde(rename_all = "snake_case"))]
127+
#[non_exhaustive]
125128
pub struct DiscoverCase {
126129
/// An identifier that is unique across the entire run
127130
pub name: String,
@@ -191,6 +194,7 @@ impl DiscoverCase {
191194
#[cfg_attr(feature = "unstable-schema", derive(schemars::JsonSchema))]
192195
#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
193196
#[cfg_attr(feature = "serde", serde(rename_all = "snake_case"))]
197+
#[non_exhaustive]
194198
pub struct DiscoverComplete {
195199
#[cfg_attr(
196200
feature = "serde",
@@ -228,6 +232,7 @@ impl DiscoverComplete {
228232
#[cfg_attr(feature = "unstable-schema", derive(schemars::JsonSchema))]
229233
#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
230234
#[cfg_attr(feature = "serde", serde(rename_all = "snake_case"))]
235+
#[non_exhaustive]
231236
pub struct RunStart {
232237
#[cfg_attr(
233238
feature = "serde",
@@ -265,6 +270,7 @@ impl RunStart {
265270
#[cfg_attr(feature = "unstable-schema", derive(schemars::JsonSchema))]
266271
#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
267272
#[cfg_attr(feature = "serde", serde(rename_all = "snake_case"))]
273+
#[non_exhaustive]
268274
pub struct CaseStart {
269275
/// An identifier that is unique across the entire run
270276
pub name: String,
@@ -309,6 +315,7 @@ impl CaseStart {
309315
#[cfg_attr(feature = "unstable-schema", derive(schemars::JsonSchema))]
310316
#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
311317
#[cfg_attr(feature = "serde", serde(rename_all = "snake_case"))]
318+
#[non_exhaustive]
312319
pub struct CaseMessage {
313320
/// An identifier that is unique across the entire run
314321
pub name: String,
@@ -371,6 +378,7 @@ impl CaseMessage {
371378
#[cfg_attr(feature = "unstable-schema", derive(schemars::JsonSchema))]
372379
#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
373380
#[cfg_attr(feature = "serde", serde(rename_all = "snake_case"))]
381+
#[non_exhaustive]
374382
pub struct CaseComplete {
375383
/// An identifier that is unique across the entire run
376384
pub name: String,
@@ -415,6 +423,7 @@ impl CaseComplete {
415423
#[cfg_attr(feature = "unstable-schema", derive(schemars::JsonSchema))]
416424
#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))]
417425
#[cfg_attr(feature = "serde", serde(rename_all = "snake_case"))]
426+
#[non_exhaustive]
418427
pub struct RunComplete {
419428
#[cfg_attr(
420429
feature = "serde",

crates/libtest2-harness/src/notify/pretty.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,7 @@ impl<W: std::io::Write> super::Notifier for PrettyRunNotifier<W> {
7272
Event::RunComplete(_) => {
7373
self.summary.write_complete(&mut self.writer)?;
7474
}
75+
_ => {}
7576
}
7677
Ok(())
7778
}

crates/libtest2-harness/src/notify/summary.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -137,6 +137,7 @@ impl super::Notifier for Summary {
137137
Event::RunComplete(inner) => {
138138
self.elapsed_s = inner.elapsed_s;
139139
}
140+
_ => {}
140141
}
141142
Ok(())
142143
}

crates/libtest2-harness/src/notify/terse.rs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ impl<W: std::io::Write> super::Notifier for TerseListNotifier<W> {
3838
Event::CaseMessage(_) => {}
3939
Event::CaseComplete(_) => {}
4040
Event::RunComplete(_) => {}
41+
_ => {}
4142
}
4243
Ok(())
4344
}
@@ -83,6 +84,7 @@ impl<W: std::io::Write> super::Notifier for TerseRunNotifier<W> {
8384
Event::RunComplete(_) => {
8485
self.summary.write_complete(&mut self.writer)?;
8586
}
87+
_ => {}
8688
}
8789
Ok(())
8890
}

0 commit comments

Comments
 (0)